fix(ci): improve test report handling and fix Failsafe HTML generation

- Ensure Failsafe plugin generates an HTML report like Surefire
- Remove redundant Surefire report generation step
- Improve artifact storage for easier access to test results

Author: Piotr Kubicki

.github/workflows/ci.yml

@@ -25,36 +25,16 @@ jobs:
           java-version: '21'
           cache: 'maven'
 
-      - name: Build and Run Tests with Allure
-        run: mvn clean verify -DtrimStackTrace=false -Dorg.slf4j.simpleLogger.defaultLogLevel=debug
+      - name: Build and Run Tests
+        run: mvn clean verify
 
-      - name: Generate Surefire and Allure Reports
-        run: mvn site allure:report
+      - name: Generate Test Reports (Surefire & Failsafe)
+        run: mvn surefire-report:report-only
 
-      - name: Upload Test Reports (Surefire + Allure)
+      - name: Upload HTML Test Reports (Surefire + Failsafe)
         if: always()
         uses: actions/upload-artifact@v4
         with:
-          name: test-reports
-          path: |
-            target/surefire-reports/
-            target/failsafe-reports/
-            target/site/
-            target/allure-report/
+          name: surefire-failsafe-html-report
+          path: target/site/surefire-report.html
           retention-days: 90
-
-      - name: Publish Allure Report
-        if: always()
-        uses: actions/upload-pages-artifact@v3
-        with:
-          path: target/allure-report/
-
-  deploy-allure:
-    needs: build
-    runs-on: ubuntu-latest
-    permissions:
-      pages: write
-      id-token: write
-    steps:
-      - name: Deploy Allure Report to GitHub Pages
-        uses: actions/deploy-pages@v4

pom.xml

@@ -76,11 +76,6 @@
 			<artifactId>spring-security-test</artifactId>
 			<scope>test</scope>
 		</dependency>
-		<dependency>
-			<groupId>io.qameta.allure</groupId>
-			<artifactId>allure-junit5</artifactId>
-			<version>2.29.1</version>
-		</dependency>
 	</dependencies>
 
 	<build>
@@ -110,26 +105,38 @@
 				</configuration>
 			</plugin>
 			<plugin>
-				<groupId>io.qameta.allure</groupId>
-				<artifactId>allure-maven</artifactId>
-				<version>2.15.2</version>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-surefire-plugin</artifactId>
+				<version>3.5.2</version>
+			</plugin>
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-failsafe-plugin</artifactId>
+				<version>3.5.2</version>
 				<executions>
 					<execution>
 						<goals>
-							<goal>report</goal>
-							<goal>serve</goal>
+							<goal>integration-test</goal>
+							<goal>verify</goal>
 						</goals>
 					</execution>
 				</executions>
 			</plugin>
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
-				<artifactId>maven-surefire-plugin</artifactId>
+				<artifactId>maven-surefire-report-plugin</artifactId>
 				<version>3.5.2</version>
+				<executions>
+					<execution>
+						<phase>verify</phase>
+						<goals>
+							<goal>report-only</goal>
+						</goals>
+					</execution>
+				</executions>
 				<configuration>
-					<systemPropertyVariables>
-						<allure.results.directory>${project.build.directory}/allure-results</allure.results.directory>
-					</systemPropertyVariables>
+					<reportName>Test Report</reportName>
+					<aggregate>true</aggregate>
 				</configuration>
 			</plugin>
 		</plugins>

…pointmentBookingAppApplicationTests.java …/AppointmentBookingAppApplicationIT.javasrc/test/java/com/capgemini/training/appointment_booking_app/AppointmentBookingAppApplicationTests.java renamed to src/test/java/com/capgemini/training/appointment_booking_app/AppointmentBookingAppApplicationIT.java src/test/java/com/capgemini/training/appointment_booking_app/AppointmentBookingAppApplicationTests.java renamed to src/test/java/com/capgemini/training/appointment_booking_app/AppointmentBookingAppApplicationIT.java

@@ -4,7 +4,7 @@
 import org.springframework.boot.test.context.SpringBootTest;
 
 @SpringBootTest
-class AppointmentBookingAppApplicationTests {
+class AppointmentBookingAppApplicationIT {
 
 	@Test
 	void contextLoads() {