chore(ci): update Maven CI workflow for SonarQube compatibility

- Adjusted artifact handling to include binary class files required by SonarQube.
- Set `fetch-depth` to 0 for full Git history to improve Sonar analysis.
- Updated JaCoCo XML report path in `pom.xml` for accurate coverage reporting (Sonar needs the full path to the XML file).

Author: poikilotherm

.github/workflows/maven_unit_test.yml

@@ -147,7 +147,7 @@ jobs:
           # Get the build output from the integration test job
           - uses: actions/download-artifact@v8
             with:
-                name: java-reportdir
+              name: java-reportdir
           - run: tar -xvf java-reportdir.tar
 
           # Deposit Code Coverage
@@ -162,28 +162,37 @@ jobs:
                 -DrepoToken=${COVERALLS_SECRET} -DpullRequest=${{ github.event.number }}
                 jacoco:report coveralls:report
 
-    sonarqube:
+    sonarcloud:
       runs-on: ubuntu-latest
       needs: integration-test
       name: SonarQube Analysis and Coverage
       steps:
         # TODO: As part of #10618 change to setup-maven custom action
         # Basic setup chores
         - uses: actions/checkout@v7
+          with:
+            # Sonarcube benefits from a full Git history in it's analysis (assign issues, etc)
+            fetch-depth: 0
         - uses: actions/setup-java@v5
           with:
             java-version: '21'
             distribution: temurin
             cache: maven
 
-        # Get the build output from the integration test job
+        # Get the coverage output from the integration test job
         - uses: actions/download-artifact@v8
           with:
             name: java-reportdir
-        - run: tar -xvf java-reportdir.tar
+        # Get the binary class files as Sonar needs them for analysis
+        - uses: actions/download-artifact@v8
+          with:
+            name: java-artifacts
+        - run: |
+            tar -xvf java-builddir.tar
+            tar -xvf java-reportdir.tar
 
         # Run Sonarcube analysis
-        - name: Commit Data to SonarCloud
+        - name: Analyze and commit to SonarCloud
           env:
             GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}  # Needed to get PR information, if any
             SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

pom.xml

@@ -27,7 +27,7 @@
         <failsafe.jacoco.args>-Ddummy.jacoco.property=true</failsafe.jacoco.args>
 
         <jacoco.merged.outputdir>${project.reporting.outputDirectory}/jacoco-merged-test-coverage-report</jacoco.merged.outputdir>
-        <sonar.coverage.jacoco.xmlReportPaths>${jacoco.merged.outputdir}</sonar.coverage.jacoco.xmlReportPaths>
+        <sonar.coverage.jacoco.xmlReportPaths>${jacoco.merged.outputdir}/jacoco.xml</sonar.coverage.jacoco.xmlReportPaths>
 
         <!-- By default, this module will produce a WAR file. -->
         <!-- This will be switched within the container profile! -->