Skip to content

SONARJAVA-6397 Add Mall spring project to ruling test#5630

Open
leonardo-pilastri-sonarsource wants to merge 1 commit into
masterfrom
lp/add-ruling-mall
Open

SONARJAVA-6397 Add Mall spring project to ruling test#5630
leonardo-pilastri-sonarsource wants to merge 1 commit into
masterfrom
lp/add-ruling-mall

Conversation

@leonardo-pilastri-sonarsource
Copy link
Copy Markdown
Contributor

@leonardo-pilastri-sonarsource leonardo-pilastri-sonarsource commented May 26, 2026


Summary by Gitar

  • Test infrastructure:
    • Added mall spring project to the integration ruling tests.
    • Updated its/sources to track lp/add-spring-ruling branch.
    • Added extensive rule violation baselines (.json files) for various SonarJava rules in its/ruling/src/test/resources/mall/.

This will update automatically on new commits.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Add Mall spring project to ruling test SONARJAVA-6397 Add Mall spring project to ruling test May 26, 2026
@hashicorp-vault-sonar-prod
Copy link
Copy Markdown
Contributor

hashicorp-vault-sonar-prod Bot commented May 26, 2026

SONARJAVA-6397

@gitar-bot
Copy link
Copy Markdown

gitar-bot Bot commented May 26, 2026

CI failed: The integration tests failed because the newly added 'mall' project is missing a required 'pom.xml' file, causing the Maven build to crash during test execution.

Overview

The integration testing suite is failing across environments because the 'mall' project directory, introduced in this PR, lacks the necessary 'pom.xml' file required for the build orchestrator to execute the tests.

Failures

Missing 'mall' Project Configuration (confidence: high)

  • Type: build
  • Affected jobs: 77866403099
  • Related to change: yes
  • Root cause: The build orchestrator throws an IllegalStateException because it cannot locate its/sources/mall/pom.xml during the JavaRulingTest.spring_mall test execution.
  • Suggested fix: Ensure that the its/sources/mall/ directory contains a valid Maven pom.xml and that it is correctly committed to the repository. Verify that no directory structure or path naming issues are preventing the file from being discovered.

Missing Test Output Artifacts (confidence: high)

  • Type: build
  • Affected jobs: Multiple (Ruling QA pipelines)
  • Related to change: yes
  • Root cause: Because the primary test execution step fails to generate results for the 'mall' project, subsequent pipeline steps attempting to generate diffs of the test output fail due to missing target/actual directories.
  • Suggested fix: This is a downstream effect of the first failure. Addressing the missing pom.xml should resolve this artifact error.

Summary

  • Change-related failures: 2 (The addition of the 'mall' project has introduced a missing-file error in the test suite and a subsequent failure in report generation).
  • Infrastructure/flaky failures: 0
  • Recommended action: Verify the contents of the its/sources/mall/ directory in the PR and ensure the pom.xml is present. Re-run the CI pipeline after adding the missing file.
Code Review ✅ Approved

Integrates the mall spring project into integration ruling tests and adds the corresponding rule violation baselines. No issues found.

Tip

Comment Gitar fix CI or enable auto-apply: gitar auto-apply:on

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqube-next
Copy link
Copy Markdown

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.

1 participant