Skip to content

Commit

Permalink
HBASE-27285: Fix sonar report paths (#103)
Browse files Browse the repository at this point in the history
Signed-off-by: Peter Somogyi <psomogyi@apache.org>
Reviewed-by: Mate Szalay-Beko <symat@apache.com>
  • Loading branch information
horvathdora committed Aug 15, 2022
1 parent 25d60ab commit 496cd58
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions dev-support/code-coverage/run-coverage.sh
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,8 @@ MAIN_POM="${SCRIPT_DIR}/../../pom.xml"

# If the required parameters are given, the code coverage results are uploaded to the SonarQube Server
if [ -n "$SONAR_LOGIN" ] && [ -n "$SONAR_PROJECT_KEY" ] && [ -n "$SONAR_URL" ]; then
mvn -B -e -Pcoverage sonar:sonar -Dsonar.clover.reportPath=./target/clover/clover.xml \
-Dsonar.host.url="$SONAR_URL" -Dsonar.login="$SONAR_LOGIN" -Dsonar.projectKey="$SONAR_PROJECT_KEY" -Dsonar.projectName="$SONAR_PROJECT_NAME"
mvn -B -e -Pcoverage sonar:sonar -Dsonar.host.url="$SONAR_URL" -Dsonar.login="$SONAR_LOGIN" \
-Dsonar.projectKey="$SONAR_PROJECT_KEY" -Dsonar.projectName="$SONAR_PROJECT_NAME"
fi
}

Expand Down
4 changes: 2 additions & 2 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -733,10 +733,10 @@
<sonar.java.binaries>**/target/classes</sonar.java.binaries>
<sonar.surefire.reportsPath>${project.build.directory}/surefire-reports</sonar.surefire.reportsPath>
<sonar.coverage.jacoco.xmlReportPaths>
${sonar.projectBaseDir}/test-reporting/target/site/jacoco-aggregate/jacoco.xml
${sonar.projectBaseDir}/test-reporting/target/code-coverage/jacoco-reports/jacoco.xml
</sonar.coverage.jacoco.xmlReportPaths>
<sonar.scala.coverage.reportPaths>
${sonar.projectBaseDir}/spark/hbase-spark/target/scoverage.xml
${sonar.projectBaseDir}/test-reporting/target/code-coverage/scoverage-reports/scoverage.xml
</sonar.scala.coverage.reportPaths>
<argLine />
<main.basedir>${project.basedir}</main.basedir>
Expand Down

0 comments on commit 496cd58

Please sign in to comment.