redhat-developer / redhat-developer/rhdh-plugin-export-overlays
Update stale dynamic-home-page references in scorecard E2E test configs
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 9
- Forks
- 72
- Avg merge
- 3d 11h
- Merged PRs (30d)
- 143
Description
What happened
PR #3202 removed the stale plugins/dynamic-home-page: entry from workspaces/homepage/plugins-list.yaml, completing a rename to plugins/homepage that started in PR #2644. The review agent (workflow run 31201381924) identified during its analysis that dynamic-home-page is still referenced in two scorecard E2E config files, but this finding was never posted because the PR was merged before the review completed. The scorecard workspace was subsequently upgraded in PR #3101 without cleaning up these references.
What could go better
The scorecard E2E test configs at workspaces/scorecard/e2e-tests/tests/config/dynamic-plugins.yaml (lines 133-154) and workspaces/scorecard/e2e-tests/tests/config/filecheck/dynamic-plugins.yaml (lines 59-80) still reference dynamic-home-page with OCI tag bs_1.49.4__1.13.1. These are the only two files in the entire repository that still reference the old plugin name. Currently this works because the old OCI image is still published and accessible, but if that image is removed or the scorecard tests are updated to use a newer Backstage version, these references will break nightly E2E runs. Confidence is high that these are stale references — there is no metadata YAML or plugins-list entry for dynamic-home-page anywhere in the repo.
Proposed change
Update the two scorecard E2E config files to reference the renamed homepage plugin instead of dynamic-home-page. In workspaces/scorecard/e2e-tests/tests/config/dynamic-plugins.yaml and workspaces/scorecard/e2e-tests/tests/config/filecheck/dynamic-plugins.yaml, replace the dynamic-home-page package name and OCI reference with the corresponding homepage package name and current OCI tag from workspaces/homepage/metadata/. The scorecard tests use this plugin to provide the home.page/cards mount point where scorecard homepage cards are rendered, so the replacement plugin must serve the same mount point.
Validation criteria
After the update: (1) grep -r 'dynamic-home-page' workspaces/scorecard/ returns zero results, (2) scorecard E2E nightly tests pass with the updated plugin reference (verifiable via the next nightly run or /test e2e-tests PR command), (3) the scorecard homepage cards still render correctly in the E2E test assertions.
Generated by retro agent from https://github.com/redhat-developer/rhdh-plugin-export-overlays/pull/3202
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with workspaces/scorecard/e2e-tests/tests/config/dynamic-plugins.yaml and workspaces/scorecard/e2e-tests/tests/config/filecheck/dynamic-plugins.yaml, then check the current OCI tag in workspaces/homepage/metadata/. Confirm that the replacement provides the home.page/cards mount point, remove all dynamic-home-page references under workspaces/scorecard/, and run the scorecard E2E validation or /test e2e-tests command.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- yaml
- Domain
- testing-qa
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 84/100