Refactor VeiculoController and test classes to use JUnit 5 assertions and improve code readability

Author: lazaromx

pom.xml

@@ -38,11 +38,6 @@
             <version>5.3.1</version>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>com.h2database</groupId>
-            <artifactId>h2</artifactId>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
 </project>

src/main/java/br/lazaro/controllers/VeiculoController.java

@@ -6,15 +6,12 @@
 import java.util.List;
 
 public class VeiculoController {
-    private VeiculoRepository repository = new VeiculoRepository();
+    private final VeiculoRepository repository;
 
     public VeiculoController(VeiculoRepository repository){
         this.repository = repository;
     }
-//    public static void comprarVeiculo(String marca, String modelo, String cor, int anoFabricacao, double preco){
-//        Veiculo veiculo = new Veiculo(marca, modelo, cor, anoFabricacao, preco);
-//        repository.estoque.add(veiculo);
-//    }
+
     public void comprarVeiculo(Veiculo veiculo){
         //TODO: Melhorar mensagem de erro
         if (veiculo.getPreco() <= 0) {

src/test/java/br/lazaro/VeiculoIntegrationTest.java

@@ -3,6 +3,7 @@
 import br.lazaro.controllers.VeiculoController;
 import br.lazaro.repositories.VeiculoRepository;
 import br.lazaro.models.Veiculo;
+import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import static org.junit.Assert.*;
@@ -22,10 +23,12 @@ void setUp() {
     void compraEVendaVeiculos(){
         Veiculo veiculo = new Veiculo("Fiat", "Uno", "Branco", 2015, 15000.0);
         controller.comprarVeiculo(veiculo);
-        assertTrue(repository.obterVeiculosPorStatus("estoque").contains(veiculo));
+        Assertions.assertTrue(repository.obterVeiculosPorStatus("estoque").contains(veiculo));
+
         controller.venderVeiculo(0);
-        assertTrue(repository.obterVeiculosPorStatus("vendido").contains(veiculo));
-        assertEquals("vendido", veiculo.getStatus());
-        assertFalse(repository.obterVeiculosPorStatus("estoque").contains(veiculo));
+        Assertions.assertTrue(repository.obterVeiculosPorStatus("vendido").contains(veiculo));
+
+        Assertions.assertEquals("vendido", veiculo.getStatus());
+        Assertions.assertFalse(repository.obterVeiculosPorStatus("estoque").contains(veiculo));
     }
 }

src/test/java/br/lazaro/VeiculoRepositoryTest.java

@@ -5,6 +5,7 @@
 
 import java.util.List;
 
+import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -35,19 +36,19 @@ public void comprarVeiculo() {
         Veiculo veiculo = new Veiculo("Fiat", "Uno", "Branco", 2015, 15000.0);
         repository.adicionarVeiculo(veiculo);
         List<Veiculo> estoque = repository.obterVeiculosPorStatus("estoque");
-        assertEquals(2, estoque.size());
-        assertEquals(veiculo, estoque.get(1));
+        Assertions.assertEquals(2, estoque.size());
+        Assertions.assertEquals(veiculo, estoque.get(1));
     }
 
     @Test
     public void test_estoque() {
         List<Veiculo> estoque = repository.obterVeiculosPorStatus("estoque");
-        assertEquals(1, estoque.size());
-        assertEquals("Renault", estoque.get(0).getMarca());
-        assertEquals("Duster", estoque.get(0).getModelo());
-        assertEquals("Preto", estoque.get(0).getCor());
-        assertEquals(2010, estoque.get(0).getAnoFabricacao());
-        assertEquals(200, estoque.get(0).getPreco(), 0.001);
+        Assertions.assertEquals(1, estoque.size());
+        Assertions.assertEquals("Renault", estoque.get(0).getMarca());
+        Assertions.assertEquals("Duster", estoque.get(0).getModelo());
+        Assertions.assertEquals("Preto", estoque.get(0).getCor());
+        Assertions.assertEquals(2010, estoque.get(0).getAnoFabricacao());
+        Assertions.assertEquals(200, estoque.get(0).getPreco(), 0.001);
     }
 
     @Test
@@ -66,20 +67,20 @@ public void test_vendidos() {
         veiculoMock.setStatus("vendido");
         repository.atualizarVeiculo(veiculoMock);
         List<Veiculo> vendidos = repository.obterVeiculosPorStatus("vendido");
-        assertEquals(1, vendidos.size());
-        assertEquals(veiculoMock, vendidos.get(0));
+        Assertions.assertEquals(1, vendidos.size());
+        Assertions.assertEquals(veiculoMock, vendidos.get(0));
     }
 
 
     @Test
     public void test_get_veiculo_with_valid_id() {
         veiculoMock = repository.obterVeiculoPorId(0);
-        assertNotNull(veiculoMock);
-        assertEquals("Renault", veiculoMock.getMarca());
-        assertEquals("Duster", veiculoMock.getModelo());
-        assertEquals("Preto", veiculoMock.getCor());
-        assertEquals(2010, veiculoMock.getAnoFabricacao());
-        assertEquals(200, veiculoMock.getPreco(), 0.001);
+        Assertions.assertNotNull(veiculoMock);
+        Assertions.assertEquals("Renault", veiculoMock.getMarca());
+        Assertions.assertEquals("Duster", veiculoMock.getModelo());
+        Assertions.assertEquals("Preto", veiculoMock.getCor());
+        Assertions.assertEquals(2010, veiculoMock.getAnoFabricacao());
+        Assertions.assertEquals(200, veiculoMock.getPreco(), 0.001);
     }
 
     @Test
@@ -95,7 +96,7 @@ public void test_comprar_veiculo_with_vendido_status() {
         repository.adicionarVeiculo(veiculo);
 
         // Validando que o estoque adicionou +1 e vendidos manteve a mesma quantidade
-        assertEquals(estoqueAtual + 1, repository.obterVeiculosPorStatus("estoque").size());
-        assertEquals(vendidosAtual, repository.obterVeiculosPorStatus("vendido").size());
+        Assertions.assertEquals(estoqueAtual + 1, repository.obterVeiculosPorStatus("estoque").size());
+        Assertions.assertEquals(vendidosAtual, repository.obterVeiculosPorStatus("vendido").size());
     }
 }