Skip to content

🌱 test(web): add unit coverage for GlobeErrorBoundary - #24688

Merged
hivecommons-hive[bot] merged 1 commit into
mainfrom
quality/test-globe-error-boundary
Oct 29, 2026
Merged

hivecommons-hive[bot] merged 1 commit into
mainfrom
quality/test-globe-error-boundary

Conversation

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Test Improvement

Adds unit test coverage for GlobeErrorBoundary.tsx, the error
boundary that keeps the login page usable when the 3D globe
animation (WebGL/Three.js) crashes. The component had 0% coverage and
no test file.

Tests cover: rendering children on the happy path, rendering nothing
(not propagating) when a child throws, the analytics reporting call
(emitError/markErrorReported), and that a crash in the globe
subtree leaves sibling content mounted. Verified locally with
npx vitest run src/components/animations/globe/GlobeErrorBoundary.test.tsx
(4 passed).

Fixes #24687


Filed by quality agent (ACMM L4/L6 — full mode)

— hive: agent=quality backend=copilot model=claude-opus-4.7 copilot=1.0.88

GlobeErrorBoundary wraps the login page's 3D globe animation and is
the fallback that keeps the login flow usable if WebGL/Three.js
crashes, but it shipped with 0% test coverage and no test file.

Add tests covering the happy path (renders children), the crash path
(renders nothing instead of propagating the error), the analytics
reporting call (emitError/markErrorReported), and that a crash in the
globe subtree does not unmount unrelated sibling content.

Signed-off-by: quality <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 29, 2026
@netlify

netlify Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for kubestellarconsole ready!

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

@kubestellar-prow kubestellar-prow Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 29, 2026
@github-actions github-actions Bot added ai-generated Pull request generated by AI tier/1-lightweight labels Oct 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.

@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 labels Oct 29, 2026
@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.

@hivecommons-hive
hivecommons-hive Bot merged commit 2deeccd into main Oct 29, 2026
56 of 57 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the quality/test-globe-error-boundary branch October 29, 2026 09:55
@hivecommons-hive

Copy link
Copy Markdown
Contributor Author
Evidence bundle `#24688ba00918` sha256:f02fdced252063eaa214e89608a4f77bbed4de911f1c748cadc825a9af69b097 — download from the dashboard or `hivectl review evidence #24688`.

@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/050-review-pending Review swarm requested; awaiting a verdict for the current head labels Oct 29, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Post-merge build verification passed ✅

Both Go and frontend builds compiled successfully against merge commit 2deeccd17e37ea86127786417d01b5585b8a1390.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Post-Merge Verification: passed

Commit: 2deeccd17e37ea86127786417d01b5585b8a1390
Specs run: smoke.spec.ts
Report: https://github.com/kubestellar/console/actions/runs/40853491431

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 dco-signoff: yes Indicates the PR's author has signed the DCO. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. tier/1-lightweight

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[quality] GlobeErrorBoundary has zero test coverage — login-page WebGL fallback untested

0 participants