Skip to content

IDE-325: Configure SonarQube Code Coverage Job#5183

Closed
harshsomankar123-tech wants to merge 9 commits intoCatrobat:developfrom
harshsomankar123-tech:IDE-325-sonarqube-coverage-job
Closed

IDE-325: Configure SonarQube Code Coverage Job#5183
harshsomankar123-tech wants to merge 9 commits intoCatrobat:developfrom
harshsomankar123-tech:IDE-325-sonarqube-coverage-job

Conversation

@harshsomankar123-tech
Copy link
Copy Markdown
Member

@harshsomankar123-tech harshsomankar123-tech commented Apr 1, 2026

Reference:
Jira Ticket: (https://catrobat.atlassian.net/browse/IDE-325)

Description:

This pull request introduces a dedicated SonarQube code coverage job to the CI pipeline. The goal is to automate the reporting of test coverage and code quality metrics, ensuring better visibility into the codebase health and maintaining high development standards within the IDE module.

Changes:

  • Configured the workflow to trigger SonarQube analysis upon pushes and pull requests to the develop branch.
  • Integrated Jacoco/Sonar plugins to track and upload coverage data.
  • Ensured the job properly identifies the project key and organization within the SonarQube environment.

Checklist for this pull request

Please review the contributing guidelines and wiki pages of this repository.

  • Include the name of the Jira ticket in the PR’s title
  • Include a summary of the changes plus the relevant context
  • Choose the proper base branch (develop)
  • Confirm that the changes follow the project’s coding guidelines
  • Verify that the changes generate no compiler or linter warnings
  • Perform a self-review of the changes
  • Verify to commit no other files than the intentionally changed ones
  • Include reasonable and readable tests verifying the added or changed behavior
  • Confirm that new and existing unit tests pass locally
  • Check that the commits’ message style matches the project’s guideline
  • Stick to the project’s gitflow workflow
  • Verify that your changes do not have any conflicts with the base branch
  • After the PR, verify that all CI checks have passed
  • Post a message in the catroid-stage or catroid-ide Slack channel and ask for a code reviewer

@harshsomankar123-tech harshsomankar123-tech marked this pull request as draft April 1, 2026 18:34
@harshsomankar123-tech harshsomankar123-tech added the Active Member Tickets that are assigned to members that are still currently active label Apr 1, 2026
@harshsomankar123-tech harshsomankar123-tech marked this pull request as ready for review April 2, 2026 17:01
@wslany wslany requested a review from Copilot April 3, 2026 10:12
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds SonarCloud/SonarQube analysis with coverage reporting to the CI pipeline for the Catroid IDE module by integrating the Gradle Sonar plugin and wiring CI artifacts into a dedicated analysis job.

Changes:

  • Adds the org.sonarqube Gradle plugin and configures Sonar project/org/host properties.
  • Uploads JaCoCo XML from unit tests and Android test coverage XML from emulator-based workflows as CI artifacts.
  • Introduces a sonarqube GitHub Actions job that downloads coverage artifacts and runs ./gradlew sonar with sonar.coverage.jacoco.xmlReportPaths.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 6 comments.

File Description
build.gradle Adds Sonar Gradle plugin + root Sonar properties for SonarCloud.
.github/workflows/main.yml Uploads unit-test coverage XML, adds a Sonar analysis job that downloads coverage artifacts and runs Sonar.
.github/workflows/emulated_tests.yml Uploads Android test coverage XML as an artifact for later Sonar ingestion.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/emulated_tests.yml Outdated
Comment thread .github/workflows/emulated_tests.yml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/main.yml
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml
Comment thread .github/workflows/main.yml
Comment thread .github/workflows/main.yml Outdated
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 4, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Active Member Tickets that are assigned to members that are still currently active

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants