Skip to content

SLCORE-1555 Run medium tests in parallel #1438

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

SLCORE-1555 Run medium tests in parallel #1438

wants to merge 1 commit into from

Conversation

@damien-urruty-sonarsource
Copy link
Contributor

@damien-urruty-sonarsource damien-urruty-sonarsource commented Jul 23, 2025 •
edited by atlassian bot
Loading

@damien-urruty-sonarsource damien-urruty-sonarsource force-pushed the task/dam/SLCORE-1555-parallel-tests branch 2 times, most recently from af5870d to 96f72cd Compare July 23, 2025 10:10
@sonarqube-next
Copy link

sonarqube-next bot commented Jul 23, 2025

Quality Gate failed Quality Gate failed

Failed conditions
3 New issues
81.4% Coverage on New Code (required ≥ 85%)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE SonarQube for IDE

@damien-urruty-sonarsource damien-urruty-sonarsource force-pushed the task/dam/SLCORE-1555-parallel-tests branch 2 times, most recently from f706167 to ac777c7 Compare July 23, 2025 14:15
@damien-urruty-sonarsource damien-urruty-sonarsource force-pushed the task/dam/SLCORE-1555-parallel-tests branch from ac777c7 to 343fa95 Compare July 23, 2025 15:07
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 22, 2025

🤖 Pull Request summary

Updates test infrastructure for better parallelization and dynamic port allocation.

• Maven configuration: Added surefire plugin with forkCount=1C and reuseForks=true for parallel test execution
• Test timeout adjustment: Increased assertion timeout from 1 to 2 seconds in EffectiveRulesMediumTests
• WebSocket URI handling: Changed from String to URI type for WebSocket endpoints and switched to dynamic port allocation
• New NetworkUtils utility: Added comprehensive port allocation logic with blocked port filtering and collision avoidance
• Test cleanup: Removed unused EMBEDDED_SERVER capability imports

Review focus: The NetworkUtils port allocation logic is complex - verify the blocked ports list and collision detection work correctly across parallel test execution.

💬 Please send your feedback

Quality Gate Failed Quality Gate failed

Failed conditions
5 New issues

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@damien-urruty-sonarsource