Skip to content

feat(#4267): improve scorecard code-coverage module - #4271

Open
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/4267-improve-code-coverage-module
Open

feat(#4267): improve scorecard code-coverage module#4271
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/4267-improve-code-coverage-module

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Add documentation, demo data, and e2e tests for the scorecard code-coverage module.

  • Add "Importing data" section to the module README with curl
    examples for Cobertura XML and LCOV formats
  • Create examples/code-coverage/ with a demo Cobertura XML report
    and an import.sh script for one-command data seeding
  • Add Playwright e2e tests verifying all 8 code-coverage metrics
    (line/branch percentage, available, covered, missed) display
    correctly, metric values match mocked data, and empty state
    renders when no metrics are available
  • Add code-coverage metric provider schedule to app-config.yaml

Backend-to-backend authentication was already implemented in #3476 using AuthService.getOwnServiceCredentials() and getPluginRequestToken(), verified by existing unit tests.

E2e tests use route-intercepted mock responses (the standard pattern in this workspace) rather than live data import. The demo data in examples/code-coverage/ is available for manual import into a running Backstage instance.


Closes #4267

Post-script verification

  • Branch is not main/master (agent/4267-improve-code-coverage-module)
  • Secret scan passed (gitleaks — 71104e89659ff72714a434d7728f9e053c33885f..HEAD)
  • Pre-commit hooks passed (authoritative run on runner)
  • Tests ran inside sandbox

@rhdh-gh-app

rhdh-gh-app Bot commented Aug 12, 2026

Copy link
Copy Markdown

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-scorecard-backend-module-code-coverage

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
app-legacy workspaces/scorecard/packages/app-legacy none v0.0.0
@red-hat-developer-hub/backstage-plugin-scorecard-backend-module-code-coverage workspaces/scorecard/plugins/scorecard-backend-module-code-coverage none v0.0.0

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.38%. Comparing base (ee6947c) to head (7828d90).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4271   +/-   ##
=======================================
  Coverage   61.38%   61.38%           
=======================================
  Files        2520     2520           
  Lines      100936   100936           
  Branches    28304    28304           
=======================================
  Hits        61958    61958           
  Misses      38427    38427           
  Partials      551      551           
Flag Coverage Δ *Carryforward flag
adoption-insights 84.55% <ø> (ø) Carriedforward from ee6947c
ai-integrations 73.13% <ø> (ø) Carriedforward from ee6947c
app-defaults 69.79% <ø> (ø) Carriedforward from ee6947c
augment 46.67% <ø> (ø) Carriedforward from ee6947c
boost 79.00% <ø> (ø) Carriedforward from ee6947c
bulk-import 72.79% <ø> (ø) Carriedforward from ee6947c
cost-management 13.55% <ø> (ø) Carriedforward from ee6947c
dcm 67.21% <ø> (ø) Carriedforward from ee6947c
e2e-adoption-insights 60.00% <ø> (ø) Carriedforward from ee6947c
e2e-extensions 62.13% <ø> (ø) Carriedforward from ee6947c
e2e-global-header 49.45% <ø> (ø) Carriedforward from ee6947c
e2e-homepage 43.49% <ø> (ø) Carriedforward from ee6947c
e2e-intelligent-assistant 46.68% <ø> (ø) Carriedforward from ee6947c
e2e-orchestrator 50.42% <ø> (ø) Carriedforward from ee6947c
e2e-quickstart 55.21% <ø> (ø) Carriedforward from ee6947c
e2e-scorecard 50.21% <ø> (ø) Carriedforward from ee6947c
e2e-theme 16.36% <ø> (ø) Carriedforward from ee6947c
extensions 56.59% <ø> (ø) Carriedforward from ee6947c
global-floating-action-button 71.18% <ø> (ø) Carriedforward from ee6947c
global-header 66.50% <ø> (ø) Carriedforward from ee6947c
homepage 47.50% <ø> (ø) Carriedforward from ee6947c
install-dynamic-plugins 59.95% <ø> (ø) Carriedforward from ee6947c
intelligent-assistant 75.42% <ø> (ø) Carriedforward from ee6947c
konflux 91.98% <ø> (ø) Carriedforward from ee6947c
lightspeed 69.02% <ø> (ø) Carriedforward from ee6947c
mcp-integrations 83.40% <ø> (ø) Carriedforward from ee6947c
orchestrator 70.87% <ø> (ø) Carriedforward from ee6947c
quickstart 63.74% <ø> (ø) Carriedforward from ee6947c
sandbox 79.56% <ø> (ø) Carriedforward from ee6947c
scorecard 87.36% <ø> (ø) Carriedforward from ee6947c
theme 88.91% <ø> (ø) Carriedforward from ee6947c
translations 5.12% <ø> (ø) Carriedforward from ee6947c
x2a 79.20% <ø> (ø) Carriedforward from ee6947c

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


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee6947c...7828d90. 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.

@christoph-jerolimov

Copy link
Copy Markdown
Member

/fs-fix rebase and resolve conflicts

@fullsend-ai-coder

fullsend-ai-coder Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Finished Fix · ✅ Success · Started 6:37 PM UTC · Completed 6:42 PM UTC

Commit: 929203b · View workflow run →

Add documentation, demo data, and e2e tests for the scorecard
code-coverage module.

- Add "Importing data" section to the module README with curl
  examples for Cobertura XML and LCOV formats
- Create examples/code-coverage/ with a demo Cobertura XML report
  and an import.sh script for one-command data seeding
- Add Playwright e2e tests verifying all 8 code-coverage metrics
  (line/branch percentage, available, covered, missed) display
  correctly, metric values match mocked data, and empty state
  renders when no metrics are available
- Add code-coverage metric provider schedule to app-config.yaml

Backend-to-backend authentication was already implemented in #3476
using AuthService.getOwnServiceCredentials() and
getPluginRequestToken(), verified by existing unit tests.

E2e tests use route-intercepted mock responses (the standard
pattern in this workspace) rather than live data import. The
demo data in examples/code-coverage/ is available for manual
import into a running Backstage instance.

Closes #4267
@fullsend-ai-coder

Copy link
Copy Markdown
Contributor Author

🔧 Fix agent — iteration 1 (human-triggered)

Rebased branch onto origin/main (46 new commits from main). Resolved one merge conflict in workspaces/scorecard/app-config.yaml by keeping both the catalog.requiredAttributes provider from main and the codeCoverage.coverageReport provider from this PR.

Fixed (1):

  1. rebase and resolve conflicts (workspaces/scorecard/app-config.yaml): Rebased branch onto origin/main and resolved merge conflict in app-config.yaml. The conflict was between the new catalog.requiredAttributes provider (added to main in feat(scorecard): add catalog module to validate fields (starting with a required attributes provider) #4258) and the codeCoverage.coverageReport provider (added by this PR). Resolution: kept both providers, each with their own schedule block.

Tests: passed

Updated by fullsend fix agent

@fullsend-ai-coder
fullsend-ai-coder Bot force-pushed the agent/4267-improve-code-coverage-module branch from 929203b to 7828d90 Compare August 16, 2026 18:42
@sonarqubecloud

Copy link
Copy Markdown

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

Labels

ready-for-review Agent PR ready for human review workspace/scorecard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve scorecard code-coverage module

1 participant