Add BDD

FIAP-3SOAT-G15 · May 21, 2024 · bdbc0f6 · bdbc0f6
1 parent 6490c59
commit bdbc0f6
Show file tree

Hide file tree

Showing 27 changed files with 376 additions and 101 deletions.
diff --git a/README.md b/README.md
@@ -1,5 +1,13 @@
 # Microsserviço de estoque
 
+### BDD
+
+Este microsserviço contém testes de integração implementados com Cucumber.
+
+### Análise estática
+
+Projeto no SonarCloud: [https://sonarcloud.io/project/overview?id=FIAP-3SOAT-G15_stock-api](https://sonarcloud.io/project/overview?id=FIAP-3SOAT-G15_stock-api)
+
 [![Quality Gate Status](https://sonarcloud.io/api/project_badges/measure?project=FIAP-3SOAT-G15_stock-api&metric=alert_status)](https://sonarcloud.io/summary/new_code?id=FIAP-3SOAT-G15_stock-api)
 [![Coverage](https://sonarcloud.io/api/project_badges/measure?project=FIAP-3SOAT-G15_stock-api&metric=coverage)](https://sonarcloud.io/summary/new_code?id=FIAP-3SOAT-G15_stock-api)
 

diff --git a/pom.xml b/pom.xml
@@ -19,6 +19,7 @@
         <kotlin.compiler.jvmTarget>17</kotlin.compiler.jvmTarget>
         <org.mapstruct.version>1.5.3.Final</org.mapstruct.version>
         <testcontainers.version>1.19.1</testcontainers.version>
+        <skipUnitTests>false</skipUnitTests>
         <skipITs>true</skipITs>
         <skipOpenAPIGen>true</skipOpenAPIGen>
         <sonar.organization>fiap-3soat-g15</sonar.organization>
@@ -92,6 +93,36 @@
             <artifactId>spring-boot-starter-test</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>io.cucumber</groupId>
+            <artifactId>cucumber-java</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>io.cucumber</groupId>
+            <artifactId>cucumber-junit</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>io.cucumber</groupId>
+            <artifactId>cucumber-spring</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.junit.platform</groupId>
+            <artifactId>junit-platform-suite</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>io.cucumber</groupId>
+            <artifactId>cucumber-junit-platform-engine</artifactId>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>org.springframework.boot</groupId>
             <artifactId>spring-boot-starter-data-jpa</artifactId>
@@ -143,6 +174,20 @@
                 <type>pom</type>
                 <scope>import</scope>
             </dependency>
+            <dependency>
+                <groupId>io.cucumber</groupId>
+                <artifactId>cucumber-bom</artifactId>
+                <version>7.15.0</version>
+                <type>pom</type>
+                <scope>import</scope>
+            </dependency>
+            <dependency>
+                <groupId>org.junit</groupId>
+                <artifactId>junit-bom</artifactId>
+                <version>5.10.1</version>
+                <type>pom</type>
+                <scope>import</scope>
+            </dependency>
         </dependencies>
     </dependencyManagement>
     <build>
@@ -337,26 +382,32 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-surefire-plugin</artifactId>
+                <version>3.2.5</version>
                 <configuration>
-                    <excludedGroups>IntegrationTest</excludedGroups>
+                    <skipTests>${skipUnitTests}</skipTests>
                 </configuration>
             </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-failsafe-plugin</artifactId>
+                <version>3.1.2</version>
                 <configuration>
-                    <includes>
-                        <include>**/*</include>
-                    </includes>
-                    <groups>IntegrationTest</groups>
-                    <skipITs>${skipITs}</skipITs>
+                    <properties>
+                        <configurationParameters>
+                            cucumber.junit-platform.naming-strategy=long
+                        </configurationParameters>
+                    </properties>
                 </configuration>
                 <executions>
                     <execution>
+                        <id>integration-tests</id>
                         <goals>
                             <goal>integration-test</goal>
                             <goal>verify</goal>
                         </goals>
+                        <configuration>
+                            <skipTests>${skipITs}</skipTests>
+                        </configuration>
                     </execution>
                 </executions>
             </plugin>

diff --git a/src/main/kotlin/com/fiap/stock/application/adapter/gateway/ComponentGateway.kt b/src/main/kotlin/com/fiap/stock/application/adapter/gateway/ComponentGateway.kt
@@ -14,6 +14,4 @@ interface ComponentGateway {
     fun update(component: Component): Component
 
     fun delete(component: Component): Component
-
-    fun deleteAll()
 }
diff --git a/src/main/kotlin/com/fiap/stock/application/adapter/gateway/ProductGateway.kt b/src/main/kotlin/com/fiap/stock/application/adapter/gateway/ProductGateway.kt
@@ -19,6 +19,4 @@ interface ProductGateway {
     fun update(product: Product): Product
 
     fun delete(productNumber: Long): Product
-
-    fun deleteAll()
 }
diff --git a/src/main/kotlin/com/fiap/stock/application/adapter/gateway/impl/ComponentGatewayImpl.kt b/src/main/kotlin/com/fiap/stock/application/adapter/gateway/impl/ComponentGatewayImpl.kt
@@ -67,10 +67,6 @@ class ComponentGatewayImpl(
         return component
     }
 
-    override fun deleteAll() {
-        componentJpaRepository.deleteAll()
-    }
-
     private fun persist(component: Component): Component =
         component
             .let(mapper::toEntity)

diff --git a/src/main/kotlin/com/fiap/stock/application/adapter/gateway/impl/ProductGatewayImpl.kt b/src/main/kotlin/com/fiap/stock/application/adapter/gateway/impl/ProductGatewayImpl.kt
@@ -58,10 +58,6 @@ class ProductGatewayImpl(
             )
     }
 
-    override fun deleteAll() {
-        productJpaRepository.deleteAll()
-    }
-
     private fun persist(product: Product): Product =
         product
             .let(mapper::toEntity)

diff --git a/src/test/kotlin/IntegrationTestFixtures.kt b/src/test/kotlin/IntegrationTestFixtures.kt
diff --git a/src/test/kotlin/TestAnnotations.kt → ...fiap/stock/application/TestAnnotations.kt b/src/test/kotlin/TestAnnotations.kt → ...fiap/stock/application/TestAnnotations.kt
@@ -1,13 +1,9 @@
-import org.junit.jupiter.api.Tag
+package com.fiap.stock.application
+
+import com.fiap.stock.application.it.PostgreSQLContainerInitializer
 import org.springframework.boot.test.autoconfigure.jdbc.AutoConfigureTestDatabase
-import org.springframework.test.context.ActiveProfiles
 import org.springframework.test.context.ContextConfiguration
 
-@Tag("IntegrationTest")
-@ActiveProfiles("test")
-@Target(AnnotationTarget.CLASS, AnnotationTarget.FILE)
-annotation class IntegrationTest
-
 @ContextConfiguration(initializers = [PostgreSQLContainerInitializer::class])
 @AutoConfigureTestDatabase(replace = AutoConfigureTestDatabase.Replace.NONE)
 @Target(AnnotationTarget.CLASS, AnnotationTarget.FILE)

diff --git a/src/test/kotlin/TestFixtures.kt → ...om/fiap/stock/application/TestFixtures.kt b/src/test/kotlin/TestFixtures.kt → ...om/fiap/stock/application/TestFixtures.kt
@@ -1,3 +1,5 @@
+package com.fiap.stock.application
+
 import com.fiap.stock.application.domain.entities.Component
 import com.fiap.stock.application.domain.entities.Product
 import com.fiap.stock.application.domain.entities.Stock

diff --git a/src/test/kotlin/com/fiap/stock/application/adapter/controller/ComponentControllerTest.kt b/src/test/kotlin/com/fiap/stock/application/adapter/controller/ComponentControllerTest.kt
@@ -4,7 +4,7 @@ import com.fiap.stock.application.driver.web.request.ComponentRequest
 import com.fiap.stock.application.usecases.CreateComponentUseCase
 import com.fiap.stock.application.usecases.LoadComponentUseCase
 import com.fiap.stock.application.usecases.SearchComponentUseCase
-import createComponent
+import com.fiap.stock.application.createComponent
 import io.mockk.every
 import io.mockk.mockk
 import org.assertj.core.api.Assertions.assertThat
@@ -101,4 +101,4 @@ class ComponentControllerTest {
 
 
 
-}
+}
diff --git a/src/test/kotlin/com/fiap/stock/application/adapter/controller/MenuControllerTest.kt b/src/test/kotlin/com/fiap/stock/application/adapter/controller/MenuControllerTest.kt
@@ -3,7 +3,7 @@ package com.fiap.stock.application.adapter.controller
 import com.fiap.stock.application.domain.valueobjects.ProductCategory
 import com.fiap.stock.application.usecases.LoadProductUseCase
 import com.fiap.stock.application.usecases.SearchProductUseCase
-import createProduct
+import com.fiap.stock.application.createProduct
 import io.mockk.every
 import io.mockk.mockk
 import io.mockk.unmockkAll
@@ -106,4 +106,4 @@ class MenuControllerTest {
     }
 
 
-}
+}
diff --git a/src/test/kotlin/com/fiap/stock/application/adapter/controller/ProductControllerTest.kt b/src/test/kotlin/com/fiap/stock/application/adapter/controller/ProductControllerTest.kt
@@ -7,8 +7,8 @@ import com.fiap.stock.application.usecases.AdjustStockUseCase
 import com.fiap.stock.application.usecases.AssembleProductsUseCase
 import com.fiap.stock.application.usecases.LoadProductUseCase
 import com.fiap.stock.application.usecases.SearchProductUseCase
-import createProduct
-import createProductRequest
+import com.fiap.stock.application.createProduct
+import com.fiap.stock.application.it.createProductRequest
 import io.mockk.every
 import io.mockk.mockk
 import org.assertj.core.api.Assertions.assertThat

diff --git a/src/test/kotlin/com/fiap/stock/application/adapter/gateway/ComponentGatewayTest.kt b/src/test/kotlin/com/fiap/stock/application/adapter/gateway/ComponentGatewayTest.kt
@@ -4,7 +4,7 @@ import com.fiap.stock.application.adapter.gateway.impl.ComponentGatewayImpl
 import com.fiap.stock.application.domain.errors.SelfOrderManagementException
 import com.fiap.stock.application.driver.database.persistence.jpa.ComponentJpaRepository
 import com.fiap.stock.application.driver.database.persistence.mapper.ComponentMapper
-import createComponent
+import com.fiap.stock.application.createComponent
 import io.mockk.every
 import io.mockk.mockk
 import org.assertj.core.api.Assertions.assertThat
@@ -106,17 +106,6 @@ class ComponentGatewayTest {
 
             assertThat(result).isNotNull()
         }
-
-        @Test
-        fun `deleteAll should return nothing`() {
-
-            every { componentJpaRepository.deleteAll() } returns Unit
-
-            gateway.deleteAll()
-        }
-
-
-
     }
 
     @Nested
@@ -170,4 +159,4 @@ class ComponentGatewayTest {
     }
 
 
-}
+}
diff --git a/src/test/kotlin/com/fiap/stock/application/adapter/gateway/ProductGatewayTest.kt b/src/test/kotlin/com/fiap/stock/application/adapter/gateway/ProductGatewayTest.kt
@@ -5,7 +5,7 @@ import com.fiap.stock.application.domain.errors.SelfOrderManagementException
 import com.fiap.stock.application.domain.valueobjects.ProductCategory
 import com.fiap.stock.application.driver.database.persistence.jpa.ProductJpaRepository
 import com.fiap.stock.application.driver.database.persistence.mapper.ProductMapper
-import createProduct
+import com.fiap.stock.application.createProduct
 import io.mockk.every
 import io.mockk.mockk
 import org.assertj.core.api.Assertions.assertThat
@@ -118,15 +118,6 @@ class ProductGatewayTest {
 
             assertThat(response).isNotNull()
         }
-
-        @Test
-        fun `deleteAll should return nothing`() {
-
-            every { productJpaRepository.deleteAll() } returns Unit
-
-            gateway.deleteAll()
-        }
-
     }
 
     @Nested
@@ -191,4 +182,4 @@ class ProductGatewayTest {
     }
 
 
-}
+}
diff --git a/src/test/kotlin/com/fiap/stock/application/adapter/gateway/StockGatewayTest.kt b/src/test/kotlin/com/fiap/stock/application/adapter/gateway/StockGatewayTest.kt
@@ -4,7 +4,7 @@ import com.fiap.stock.application.adapter.gateway.impl.StockGatewayImpl
 import com.fiap.stock.application.domain.errors.SelfOrderManagementException
 import com.fiap.stock.application.driver.database.persistence.jpa.StockJpaRepository
 import com.fiap.stock.application.driver.database.persistence.mapper.StockMapper
-import createStock
+import com.fiap.stock.application.createStock
 import io.mockk.every
 import io.mockk.mockk
 import org.assertj.core.api.Assertions.assertThat
@@ -106,4 +106,4 @@ class StockGatewayTest {
     }
 
 
-}
+}
diff --git a/src/test/kotlin/com/fiap/stock/application/it/CommonSteps.kt b/src/test/kotlin/com/fiap/stock/application/it/CommonSteps.kt
@@ -0,0 +1,32 @@
+package com.fiap.stock.application.it
+
+import io.cucumber.java.Before
+import io.restassured.RestAssured
+import org.springframework.beans.factory.annotation.Autowired
+import org.springframework.boot.test.web.server.LocalServerPort
+import org.springframework.jdbc.core.JdbcTemplate
+
+class CommonSteps: IntegrationTest() {
+
+    @Autowired
+    lateinit var jdbcTemplate: JdbcTemplate
+
+    @LocalServerPort
+    private val port: Int? = null
+
+    @Before
+    fun setup() {
+        RestAssured.baseURI = "http://localhost:$port"
+        RestAssured.enableLoggingOfRequestAndResponseIfValidationFails()
+    }
+
+    @Before("@database")
+    fun setupDatabase() {
+        jdbcTemplate.execute("""
+            TRUNCATE TABLE product RESTART IDENTITY CASCADE;
+            TRUNCATE TABLE stock RESTART IDENTITY;
+            TRUNCATE TABLE component RESTART IDENTITY CASCADE;
+        """.trimIndent()
+        )
+    }
+}