ci(task-nr): test nested needs outputs (#19)

Co-authored-by: André Caiado da Silva <[email protected]>

.github/workflows/CI-CD.yml

@@ -2,13 +2,11 @@ name: CI/CD
 
 on:
   pull_request:
-    types: [opened, synchronize, reopened, closed]
+    types: [opened, synchronize, reopened]
     branches:
       - main
   release:
     types: [published]
-    branches:
-      - main
 
 concurrency:
   group: ${{ github.ref }}
@@ -45,7 +43,7 @@ jobs:
         id: set-push-image
         shell: bash
         run: |
-          if [[ "${{ github.event_name }}" == "pull_request" && "${{ github.event.action }}" == "closed" && "${{ github.event.pull_request.merged }}" == "true" ]] || 
+          if [[ "${{ github.event_name }}" == "pull_request" && "${{ github.event.pull_request.merged }}" == "true" ]] || 
               [[ "${{ github.event_name }}" == "release" && "${{ github.event.action }}" == "published" ]]; then
             echo "Setting PUSH_IMAGE to true"
             echo "PUSH_IMAGE=true" >> "$GITHUB_OUTPUT"
@@ -57,7 +55,7 @@ jobs:
         id: set-deploy
         shell: bash
         run: |
-          if [[ "${{ github.event_name }}" == "pull_request" && "${{ github.event.action }}" == "closed" && "${{ github.event.pull_request.merged }}" == "true" ]]; then
+          if [[ "${{ github.event_name }}" == "pull_request" && "${{ github.event.pull_request.merged }}" == "true" ]]; then
             echo "Setting DEPLOY to true"
             echo "Setting ENVIRONMENT to staging"
             echo "DEPLOY=true" >> "$GITHUB_OUTPUT"
@@ -72,6 +70,7 @@ jobs:
             echo "DEPLOY=false" >> "$GITHUB_OUTPUT"
           fi
   build-and-test:
+    if: ${{ github.event_name }} != 'release'
     runs-on: ubuntu-latest
     needs: prepare
     steps:
@@ -90,6 +89,18 @@ jobs:
         run: |
           echo "Running the tests"
           ./mvnw test
+      - name: Add coverage to PR
+        id: jacoco
+        uses: madrapps/[email protected]
+        with:
+          paths: |
+            ${{ github.workspace }}/target/site/jacoco/jacoco.xml
+          token: ${{ secrets.GITHUB_TOKEN }}
+          min-coverage-overall: 40
+          min-coverage-changed-files: 60
+          title: Coverage report
+          comment: true
+          only-changed-files: true
   build-image-push:
     runs-on: ubuntu-latest
     needs: [prepare, build-and-test]
@@ -116,7 +127,7 @@ jobs:
           tags: ${{ secrets.DOCKER_USERNAME }}/${{ github.event.repository.name }}:${{ needs.prepare.outputs.tag_version }}
   deploy:
     runs-on: ubuntu-latest
-    needs: [prepare, build-and-test, build-image-push]
+    needs: [prepare, build-image-push]
     if: ${{ needs.prepare.outputs.deploy == 'true' }}
     steps:
       - name: Deploy

README.md

@@ -4,6 +4,8 @@ The main purpose of this repository is to demonstrate how to set up a CI/CD work
 
 The application used is a simple Spring Boot application with a REST controller and a test class.
 
+Test
+
 ## Workflows
 
 There are two workflows in this repository:

pom.xml

@@ -46,6 +46,25 @@
 				<groupId>org.springframework.boot</groupId>
 				<artifactId>spring-boot-maven-plugin</artifactId>
 			</plugin>
+			<plugin>
+				<groupId>org.jacoco</groupId>
+				<artifactId>jacoco-maven-plugin</artifactId>
+				<version>0.8.11</version>
+				<executions>
+					<execution>
+						<goals>
+							<goal>prepare-agent</goal>
+						</goals>
+					</execution>
+					<execution>
+						<id>report</id>
+						<phase>test</phase>
+						<goals>
+							<goal>report</goal>
+						</goals>
+					</execution>
+				</executions>
+			</plugin>
 		</plugins>
 	</build>
 

src/main/java/com/example/spring/test/controller/TutorialController.java

@@ -29,6 +29,11 @@ public class TutorialController {
   @Autowired
   TutorialRepository tutorialRepository;
 
+  @GetMapping("/cenas")
+  public ResponseEntity<Void> getCenas() {
+    return new ResponseEntity<>(HttpStatus.OK);
+  }
+
   @GetMapping("/tutorials")
   public ResponseEntity<List<Tutorial>> getAllTutorials(@RequestParam(required = false) String title) {
     try {