Skip to content

Fix [PubDownloads] test - #12101

Merged
PyvesB merged 2 commits into
badges:masterfrom
hatefrad:fix-pub-downloads-test
Aug 9, 2026
Merged

Fix [PubDownloads] test#12101
PyvesB merged 2 commits into
badges:masterfrom
hatefrad:fix-pub-downloads-test

Conversation

@hatefrad

@hatefrad hatefrad commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

The PubDownloads live test has been failing in the daily test runs.

The monthly download count for analysis_options has grown past 1000, which is the top threshold in downloadCount:

// services/color-formatters.js
function downloadCount(downloads) {
  return floorCount(downloads, 10, 100, 1000)
}

So the badge now renders brightgreen while the test still expected green. This updates the expected color to match. As brightgreen is the top bucket, this shouldn't drift again.

Verified locally with npm run test:services -- --only=pub (13 passing).

The monthly download count for `analysis_options` has grown past 1000,
which is the top threshold in `downloadCount` (`floorCount(downloads,
10, 100, 1000)`), so the badge now renders `brightgreen` rather than
`green`. Update the expected color to match.
@hatefrad
hatefrad force-pushed the fix-pub-downloads-test branch from d92129e to d04889e Compare August 6, 2026 22:38

@PyvesB PyvesB left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for fixing this, @hatefrad! 🚀

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor
Messages
📖 ✨ Thanks for your contribution to Shields, @hatefrad!

Generated by 🚫 dangerJS against 72115c9

@PyvesB
PyvesB added this pull request to the merge queue Aug 9, 2026
Merged via the queue into badges:master with commit 2a1c370 Aug 9, 2026
19 checks passed
@PyvesB PyvesB added the keep-service-tests-green Related to fixing failing tests of the services label Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

keep-service-tests-green Related to fixing failing tests of the services

Development

Successfully merging this pull request may close these issues.

2 participants