Skip to content

test: scenario 2 - long tests (16 min) before sonar scanner - #42

Open
CristianAmbrosini wants to merge 1 commit into
mainfrom
test/scenario2-long-tests
Open

test: scenario 2 - long tests (16 min) before sonar scanner#42
CristianAmbrosini wants to merge 1 commit into
mainfrom
test/scenario2-long-tests

Conversation

@CristianAmbrosini

Copy link
Copy Markdown
Owner

Testing check-run watching: tests take 16 min before sonar scanner step. Bot should wait for the check run to complete (status: in_progress), NOT abandon early. After tests pass and scanner runs, bot should get SQC analysis.

@sonar-review-dev18

sonar-review-dev18 Bot commented Apr 30, 2026

Copy link
Copy Markdown

Summary

This PR adds a 16-minute delay step to the SonarCloud workflow to test an external bot's behavior around check-run status watching. The new "Run long tests" step adds a 960-second sleep before the SonarQube Scan step, simulating a long-running test suite. The test validates that the bot correctly monitors check-run status while in_progress and doesn't abandon early, then proceeds to collect SonarQube analysis results after the tests complete.

What reviewers should know

What changed: A single step was added to .github/workflows/sonarcloud.yml between checkout and the SonarQube Scan step. The step runs a 16-minute sleep to simulate a long test run.

Why this matters: This is an integration test scenario designed to verify that bot automation (external to this repo) correctly handles long-running GitHub check-runs. The sleep ensures the workflow stays in progress for 16 minutes, allowing the bot to demonstrate it won't time out or abandon the check-run prematurely.

What to check: The workflow change is minimal and intentional—the goal is to observe the bot's behavior during execution, not to review the workflow logic itself. The SonarQube Scan step remains unchanged and will run after the tests complete, at which point the bot should collect the analysis results.


  • Generate Walkthrough
  • Generate Diagram

🗣️ Give feedback

@sonarqube-cloud-dev18

Copy link
Copy Markdown

@sonar-review-dev18 sonar-review-dev18 Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM! ✅

🗣️ Give feedback

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