Skip to content

[scanner] fix: split DynamicCard test coverage - #22901

Open
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
scanner/fix-22885
Open

[scanner] fix: split DynamicCard test coverage#22901
kubestellar-hive[bot] wants to merge 1 commit into
mainfrom
scanner/fix-22885

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

Fixes #22885, Fixes #22826

Splits the monolithic DynamicCard.test.tsx (890 lines) into focused modules, all under 500 lines:

  • DynamicCard.test.shared.tsx — shared fixtures, mocks, and vi.mock calls
  • DynamicCard.top-level.test.tsx — top-level DynamicCard config validation and error handling (7 tests)
  • DynamicCard.tier1.test.tsxTier1CardRuntime: validation, rendering, search filters, stats layouts, badge formats, pagination, and API data fetching (20 tests)
  • DynamicCard.tier2.test.tsxTier2CardRuntime: compilation, component rendering, error handling, cleanup, and Coverage Gap: DynamicCard Tier 2 Runtime Failure #5282 failure paths (16 tests)

No test coverage was lost.

— hive: agent=scanner backend=copilot model=auto

Splits the monolithic DynamicCard.test.tsx (890 lines) into focused modules:

- DynamicCard.test.shared.tsx  — shared fixtures, mocks, and vi.mock calls
- DynamicCard.top-level.test.tsx — top-level DynamicCard config validation (7 tests)
- DynamicCard.tier1.test.tsx   — Tier1CardRuntime: validation, rendering, search,
                                  stats, badges, pagination, API fetching (20 tests)
- DynamicCard.tier2.test.tsx   — Tier2CardRuntime: compilation, error handling,
                                  cleanup, and #5282 failure paths (16 tests)

All files are under 500 lines. No test coverage was lost.

Fixes #22885
Fixes #22826

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: GitHub Copilot <223556219+Copilot@users.noreply.github.com>
@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Aug 29, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@netlify

netlify Bot commented Aug 29, 2026

Copy link
Copy Markdown

Deploy Preview for kubestellarconsole ready!

Name Link
🔨 Latest commit 3aee040
🔍 Latest deploy log https://app.netlify.com/projects/kubestellarconsole/deploys/6a9293498bf4c300082e3557
😎 Deploy Preview https://deploy-preview-22901.console-deploy-preview.kubestellar.io
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

Copy link
Copy Markdown
Contributor

👋 Hey @kubestellar-hive[bot] — thanks for opening this PR!

🤖 This project is developed exclusively using AI coding assistants.

Please do not attempt to code anything for this project manually.
All contributions should be authored using an AI coding tool such as:

This ensures consistency in code style, architecture patterns, test coverage,
and commit quality across the entire codebase.


This is an automated message.

@github-actions github-actions Bot added the ai-generated Pull request generated by AI label Aug 29, 2026
@kubestellar-prow kubestellar-prow Bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Aug 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Test Coverage Check

All new source files in this PR have corresponding test files.

Checked web/src/hooks/ and web/src/components/ against origin/main.

@github-actions

Copy link
Copy Markdown
Contributor

♿ Accessibility Audit (WCAG 2.1 AA)

✅ No WCAG 2.1 AA violations detected in audited routes.


Powered by axe-core. Target: WCAG 2.1 AA compliance.

@kubestellar-hive

Copy link
Copy Markdown
Contributor Author

🛑 Fix loop escalated — human attention needed

This PR has failed CI on 1 distinct fix attempts (new commits, still red). The hive has stopped dispatching further automated fixes for it.

Failing checks: All Cards TTFI (Hard Gate), App Visual Regression, Build Frontend, build (linux/amd64, ubuntu-24.04, amd64), build (linux/arm64, ubuntu-24.04-arm, arm64), build-gate, fullstack-smoke

Raw failure evidence (from check-run annotations):

All Cards TTFI (Hard Gate): No files were found with the provided path: web/e2e/test-results/ttfi-report.json
All Cards TTFI (Hard Gate): Process completed with exit code 1.
All Cards TTFI (Hard Gate): Untyped function calls may not accept type arguments.

Remove the needs-human label after addressing the root cause to return the PR to the automated fix lane.

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

Labels

ai-generated Pull request generated by AI dco-signoff: yes Indicates the PR's author has signed the DCO. needs-human size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. tier/1-lightweight

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant