Skip to content

Commit

Permalink
Merge pull request #18 from FIAP-3SOAT-G15/improve-coverage
Browse files Browse the repository at this point in the history
Improve coverage
  • Loading branch information
wellyfrs authored May 20, 2024
2 parents 31f1bc6 + ed68f64 commit 2b3976e
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 22 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ jobs:
uses: aws-actions/amazon-ecr-login@v2

- name: Build, tag, and push docker image to Amazon ECR
#if: github.ref == 'refs/heads/main' && github.event_name == 'push'
if: github.ref == 'refs/heads/main' && github.event_name == 'push'
env:
ECR_REGISTRY: ${{ steps.login-ecr.outputs.registry }}
ECR_REPOSITORY: ${{ vars.AWS_ECR_REPO_NAME }}
Expand Down
4 changes: 1 addition & 3 deletions .github/workflows/provision.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,12 @@ on:
paths:
- .github/workflows/provision.yml
- 'terraform/**'
- 'src/**'
pull_request:
branches:
- main
paths:
- .github/workflows/provision.yml
- 'terraform/**'
- 'src/**'
workflow_dispatch:

jobs:
Expand Down Expand Up @@ -65,5 +63,5 @@ jobs:
run: exit 1

- name: Terraform Apply
#if: github.ref == 'refs/heads/main' && github.event_name == 'push'
if: github.ref == 'refs/heads/main' && github.event_name == 'push'
run: terraform apply -auto-approve -input=false
19 changes: 4 additions & 15 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,10 @@
**/com/fiap/stock/application/driver/web/MenuAPI*,
**/com/fiap/stock/application/driver/web/ProductAPI*
</sonar.exclusions>

<sonar.coverage.exclusions>
**/com/fiap/stock/application/**/config/**/*,
**/com/fiap/stock/application/driver/web/*API*.*,
</sonar.coverage.exclusions>
</properties>
<dependencies>
<dependency>
Expand Down Expand Up @@ -377,20 +380,6 @@
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>0.8.12</version>
<configuration>
<excludes>
<exclude>**/com/fiap/stock/application/driver/database/configuration/GatewayConfig.kt</exclude>
<exclude>**/com/fiap/stock/application/config/JWTSecurityConfig.kt</exclude>
<exclude>**/com/fiap/stock/application/config/RestTemplateConfig.kt</exclude>
<exclude>**/com/fiap/stock/application/adapter/controller/configuration/ServiceConfig.kt</exclude>
<exclude>**/com/fiap/stock/application/**/configuration/**</exclude>
<exclude>**/com/fiap/stock/application/driver/web/*API</exclude>
<exclude>**/com/fiap/stock/application/driver/web/ComponentAPI*</exclude>
<exclude>**/com/fiap/stock/application/driver/web/ProductAPI*</exclude>
<exclude>**/com/fiap/stock/application/driver/web/MenuAPI*</exclude>
<exclude>**/com/fiap/stock/application/driver/web/StockAPI.kt</exclude>
</excludes>
</configuration>
<executions>
<execution>
<id>prepare-agent</id>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.fiap.stock.application.adapter.controller.configuration
package com.fiap.stock.application.adapter.controller.config

import com.fiap.stock.application.domain.errors.ErrorType
import com.fiap.stock.application.domain.errors.SelfOrderManagementException
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.fiap.stock.application.adapter.controller.configuration
package com.fiap.stock.application.adapter.controller.config

import com.fiap.stock.application.StockApiApp
import com.fiap.stock.application.adapter.gateway.ComponentGateway
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package com.fiap.stock.application.driver.database.configuration
package com.fiap.stock.application.driver.database.config

import com.fiap.stock.application.StockApiApp
import com.fiap.stock.application.adapter.gateway.ComponentGateway
Expand Down

0 comments on commit 2b3976e

Please sign in to comment.