Skip to content

Conversation

@NexionisJake
Copy link

@NexionisJake NexionisJake commented Dec 24, 2025

  • Fix variable shadowing: Rename loop variable 'test' to 'test_item' in get_info_for_pr_comment
  • Add uniqueness tracking: Use sets to prevent duplicate RegressionTest entries in result lists
  • Add database refresh: Call g.db.refresh(test) before processing to ensure fresh data

Fixes issue where tests in multiple categories were listed multiple times.

In raising this pull request, I confirm the following :

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows :

  • I have never used the project.
  • I have used the project briefly.
  • I have used the project extensively, but have not contributed previously.
  • I am an active contributor to the project.

- Fix variable shadowing: Rename loop variable 'test' to 'test_item' in get_info_for_pr_comment
- Add uniqueness tracking: Use sets to prevent duplicate RegressionTest entries in result lists
- Add database refresh: Call g.db.refresh(test) before processing to ensure fresh data

This ensures PR comment test counts match the web UI results page exactly.
Fixes issue where tests in multiple categories were listed multiple times.
@sonarqubecloud
Copy link

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