Skip to content

Fix remote debugging of Ruling tests#628

Merged
jonas-wielage-sonarsource merged 1 commit intomasterfrom
margo/fix_remote_debug
Apr 10, 2026
Merged

Fix remote debugging of Ruling tests#628
jonas-wielage-sonarsource merged 1 commit intomasterfrom
margo/fix_remote_debug

Conversation

@leveretka
Copy link
Copy Markdown
Contributor

Part of

@sonarqube-next
Copy link
Copy Markdown

sonarqube-next Bot commented Jul 7, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 3, 2025

🤖 Pull Request summary

Updates debug environment variable name in SlangRulingTest.

• Changed SONAR_SCANNER_DEBUG_OPTS to SONAR_SCANNER_JAVA_OPTS for debug configuration

Review focus: Verify this aligns with SonarScanner's expected environment variable naming.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Godin Godin self-assigned this Jan 7, 2026
@Godin Godin marked this pull request as draft January 7, 2026 09:32
@jonas-wielage-sonarsource jonas-wielage-sonarsource marked this pull request as ready for review April 10, 2026 12:32
@jonas-wielage-sonarsource jonas-wielage-sonarsource enabled auto-merge (squash) April 10, 2026 12:32
@sonar-review-alpha
Copy link
Copy Markdown
Contributor

sonar-review-alpha Bot commented Apr 10, 2026

Summary

Changes the environment variable used to pass debug options to the Sonar Scanner during ruling test execution. The fix switches from SONAR_SCANNER_DEBUG_OPTS to SONAR_SCANNER_JAVA_OPTS, which is the correct variable name recognized by the scanner for passing JVM debug arguments when remote debugging is enabled.

What reviewers should know

Single-line change in its/ruling/src/test/java/org/sonarsource/slang/SlangRulingTest.java:254

The change is in the debug port configuration block (lines 251-256) that runs when sonar.rulingDebugPort system property is set. The fix ensures debug options are properly forwarded to the scanner's JVM when developers enable remote debugging.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@sonarqube-next
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@jonas-wielage-sonarsource jonas-wielage-sonarsource merged commit 70d8a09 into master Apr 10, 2026
7 of 8 checks passed
@jonas-wielage-sonarsource jonas-wielage-sonarsource deleted the margo/fix_remote_debug branch April 10, 2026 12:50
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.

3 participants