Skip to content

fix: Added test.exclusions parameter and set default values#8

Merged
pierantoniomerlino merged 2 commits intodevelopfrom
add_sonar_exclusions_property
Jan 13, 2026
Merged

fix: Added test.exclusions parameter and set default values#8
pierantoniomerlino merged 2 commits intodevelopfrom
add_sonar_exclusions_property

Conversation

@pierantoniomerlino
Copy link
Copy Markdown
Member

This pull request updates the continuous integration pipeline to improve SonarQube analysis configuration, specifically by introducing a new parameter for test exclusions and enhancing validation and documentation. The main changes focus on allowing separate configuration of source and test file exclusions for Sonar analysis, improving security checks, and updating documentation.

SonarQube Analysis Configuration:

  • Added a new testExclusions parameter to the Sonar configuration, allowing users to specify test paths to be excluded from Sonar analysis. The default value is set to **/*.java,*.xml. [1] [2]
  • Updated the pipeline to pass both sonar.exclusions and sonar.test.exclusions to the SonarQube analysis step, ensuring both source and test exclusions are respected.

Validation and Security:

  • Enhanced input validation to check that testExclusions is a string and does not contain potentially dangerous characters, similar to the existing check for exclusions.

Documentation:

  • Updated the documentation in continuousIntegrationPipeline.md to describe the new testExclusions parameter, its default value, and its purpose.

@mattdibi mattdibi self-requested a review January 12, 2026 11:01
@pierantoniomerlino pierantoniomerlino force-pushed the add_sonar_exclusions_property branch from fbd84bb to 6ec2de9 Compare January 12, 2026 11:26
mattdibi
mattdibi previously approved these changes Jan 12, 2026
Signed-off-by: pierantoniomerlino <pierantonio.merlino@eurotech.com>
Signed-off-by: pierantoniomerlino <pierantonio.merlino@eurotech.com>
@pierantoniomerlino pierantoniomerlino merged commit fedfac7 into develop Jan 13, 2026
2 checks passed
@pierantoniomerlino pierantoniomerlino deleted the add_sonar_exclusions_property branch January 13, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants