Skip to content

feat(scorecard): add unit tests for scorecard-openssf#2940

Open
alizard0 wants to merge 8 commits intomainfrom
RHDHPLAN-1200
Open

feat(scorecard): add unit tests for scorecard-openssf#2940
alizard0 wants to merge 8 commits intomainfrom
RHDHPLAN-1200

Conversation

@alizard0
Copy link
Copy Markdown
Member

@alizard0 alizard0 commented Apr 28, 2026

Implements

  1. add test for client annotation validation
  2. add unit test for error not 200 response
  3. add normalize test
  4. add boundary check for scorecard value 0,10
  5. add propagate error unit test
  6. add unit test for metric providers

RHIDP-13338
RHIDP-13337
RHIDP-13340
RHIDP-13324
RHIDP-13322
RHIDP-13327

@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.75%. Comparing base (3d2854b) to head (8d21145).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2940      +/-   ##
==========================================
- Coverage   60.96%   60.75%   -0.22%     
==========================================
  Files        2067     2067              
  Lines       63977    63692     -285     
  Branches    16606    16509      -97     
==========================================
- Hits        39005    38696     -309     
+ Misses      24729    24147     -582     
- Partials      243      849     +606     
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 942091e
ai-integrations 70.03% <ø> (ø) Carriedforward from 942091e
app-defaults 69.60% <ø> (ø) Carriedforward from 942091e
augment 69.36% <ø> (ø) Carriedforward from 942091e
bulk-import 72.42% <ø> (-0.03%) ⬇️ Carriedforward from 942091e
cost-management 16.49% <ø> (ø) Carriedforward from 942091e
dcm 32.85% <ø> (ø) Carriedforward from 942091e
extensions 61.51% <ø> (-0.20%) ⬇️ Carriedforward from 942091e
global-floating-action-button 73.75% <ø> (ø) Carriedforward from 942091e
global-header 61.62% <ø> (-0.06%) ⬇️ Carriedforward from 942091e
homepage 50.84% <ø> (ø) Carriedforward from 942091e
konflux 91.01% <ø> (ø) Carriedforward from 942091e
lightspeed 69.60% <ø> (-0.47%) ⬇️ Carriedforward from 942091e
mcp-integrations 81.59% <ø> (ø) Carriedforward from 942091e
orchestrator 33.08% <ø> (-1.75%) ⬇️ Carriedforward from 942091e
quickstart 62.64% <ø> (ø) Carriedforward from 942091e
sandbox 79.56% <ø> (ø) Carriedforward from 942091e
scorecard 83.74% <ø> (+0.12%) ⬆️
theme 64.54% <ø> (ø) Carriedforward from 942091e
translations 8.49% <ø> (ø) Carriedforward from 942091e
x2a 82.09% <ø> (ø) Carriedforward from 942091e

*This pull request uses carry forward flags. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d2854b...8d21145. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@alizard0 alizard0 requested a review from a team as a code owner May 8, 2026 10:41
@alizard0 alizard0 requested a review from dzemanov May 8, 2026 10:41
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant