Skip to content

test: add e2e coverage for marketplace status cards - #24434

Merged
hivecommons-hive[bot] merged 1 commit into
mainfrom
scanner/fix-24433
Oct 18, 2026
Merged

hivecommons-hive[bot] merged 1 commit into
mainfrom
scanner/fix-24433

Conversation

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Closes #24433

Adds Playwright e2e coverage in web/e2e/marketplace-status-cards.spec.ts for marketplace status cards (buildpacks_status, coredns_status, openkruise_status, openyurt_status): render with data, loading, and error states, following cluster-admin-cards.spec.ts patterns.

Note: kubeflow_status and notary_status are not registered in this repo's card registry, so they cannot be rendered here; they remain uncovered.

— hive: agent=scanner backend=copilot model=claude-fable-5 copilot=1.0.88

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com>
@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Oct 18, 2026
@netlify

netlify Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for kubestellarconsole ready!

Name Link
🔨 Latest commit bb34871
🔍 Latest deploy log https://app.netlify.com/projects/kubestellarconsole/deploys/6ad4bd645b800a01d35f98e4
😎 Deploy Preview https://deploy-preview-24434.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.

@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

@github-actions

Copy link
Copy Markdown
Contributor

👋 Hey @hivecommons-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 Oct 18, 2026
@kubestellar-prow kubestellar-prow Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 18, 2026
@hivecommons-hive hivecommons-hive Bot added author/hive This repository's own hive (the App or one of its agent lanes) authored this PR dco All commits are DCO signed-off role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) stage/050-review-pending Review swarm requested; awaiting a verdict for the current head stage/070-fixing Hive-authored PR: CI red, changes requested or needs rebase; a hive agent is fixing it and removed dco-signoff: yes Indicates the PR's author has signed the DCO. stage/050-review-pending Review swarm requested; awaiting a verdict for the current head labels Oct 18, 2026
@hivecommons-hive
hivecommons-hive Bot merged commit 1f1b039 into main Oct 18, 2026
44 of 46 checks passed
@hivecommons-hive

Copy link
Copy Markdown
Contributor Author
Evidence bundle `#24434bb34871` sha256:10e7e7beb28d12e136d4feec17a467de5324720df85fd8dfe346c87162082456 — download from the dashboard or `hivectl review evidence #24434`.

@kubestellar-prow
kubestellar-prow Bot deleted the scanner/fix-24433 branch October 18, 2026 12:53
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Your PR has been merged.

Check out what's new:

Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey

@hivecommons-hive hivecommons-hive Bot removed role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) stage/070-fixing Hive-authored PR: CI red, changes requested or needs rebase; a hive agent is fixing it labels Oct 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Post-merge build verification passed ✅

Both Go and frontend builds compiled successfully against merge commit 1f1b039abe476aaa57194cd2ff0df7da52c8f713.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Post-Merge Verification: passed

Commit: 1f1b039abe476aaa57194cd2ff0df7da52c8f713
Specs run: smoke.spec.ts
Report: https://github.com/kubestellar/console/actions/runs/39129149551

hivecommons-hive Bot added a commit that referenced this pull request Oct 18, 2026
…24435)

The marketplace-status-cards spec added in #24434 seeds
kubestellar-cluster-admin-cards localStorage entries without a config
field. loadDashboardCardsFromStorage validates every stored entry with
isValidDashboardCard, which requires config to be a plain object; the
seeded array fails validation, storage is cleared, and the default
cluster-admin card set renders instead — so every
[data-card-type="<type>_status"] locator times out. All 7 tests in the
spec fail deterministically on main shard 2.

Adding config: {} to each seeded card makes the entries pass validation
so the cards under test actually mount.

Signed-off-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com>
Co-authored-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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 author/hive This repository's own hive (the App or one of its agent lanes) authored this PR dco All commits are DCO signed-off size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tier/1-lightweight

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[quality] Marketplace status cards have unit coverage but no e2e coverage

0 participants