Skip to content

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

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

feat(scorecard): add unit tests for scorecard-openssf#2940
alizard0 wants to merge 7 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

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

@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 (46a666e) to head (942091e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2940   +/-   ##
=======================================
  Coverage   60.75%   60.75%           
=======================================
  Files        2067     2067           
  Lines       63769    63769           
  Branches    16541    16541           
=======================================
+ Hits        38742    38746    +4     
- Misses      24160    24172   +12     
+ Partials      867      851   -16     
Flag Coverage Δ *Carryforward flag
adoption-insights 83.58% <ø> (ø) Carriedforward from 46a666e
ai-integrations 70.03% <ø> (ø) Carriedforward from 46a666e
app-defaults 69.60% <ø> (ø) Carriedforward from 46a666e
augment 69.36% <ø> (ø) Carriedforward from 46a666e
bulk-import 72.43% <ø> (ø) Carriedforward from 46a666e
cost-management 16.49% <ø> (ø) Carriedforward from 46a666e
dcm 32.85% <ø> (ø) Carriedforward from 46a666e
extensions 61.42% <ø> (ø) Carriedforward from 46a666e
global-floating-action-button 73.75% <ø> (ø) Carriedforward from 46a666e
global-header 61.62% <ø> (ø) Carriedforward from 46a666e
homepage 50.84% <ø> (ø) Carriedforward from 46a666e
konflux 91.01% <ø> (ø) Carriedforward from 46a666e
lightspeed 69.64% <ø> (ø) Carriedforward from 46a666e
mcp-integrations 81.59% <ø> (ø) Carriedforward from 46a666e
orchestrator 33.07% <ø> (ø) Carriedforward from 46a666e
quickstart 62.64% <ø> (ø) Carriedforward from 46a666e
sandbox 79.56% <ø> (ø) Carriedforward from 46a666e
scorecard 83.74% <ø> (+0.12%) ⬆️
theme 64.54% <ø> (ø) Carriedforward from 46a666e
translations 8.49% <ø> (ø) Carriedforward from 46a666e
x2a 82.09% <ø> (ø) Carriedforward from 46a666e

*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 46a666e...942091e. 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.

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