✨ Deep test coverage: 0% → 53% (60 agents, ~7,000 test cases)

[clubanderson]

Summary

• Fixed 110 broken test files across all coverage suite shards
• Wrote ~7,000 deep regression-preventing test cases via 60 parallel agents
• Narrowed coverage scope to hooks+lib+contexts for meaningful metrics
• Increased shards from 4 → 12 to prevent timeout issues
• Fixed gist badge update with GIST_TOKEN secret

Coverage progression

0% → 23% → 37% → 45% → 48% → 51% → 53%

Test plan

• All 12 coverage suite shards pass
• Badge updates correctly
• No production code changes (test-only)

[kubestellar-prow]

[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 mikespreitzer 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:

• OWNERS

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

[netlify]

✅ Deploy Preview for kubestellarconsole ready!

Name	Link
🔨 Latest commit	5fbb940
🔍 Latest deploy log	https://app.netlify.com/projects/kubestellarconsole/deploys/69ce63561fbae60008b0f955
😎 Deploy Preview	https://deploy-preview-4222.console-deploy-preview.kubestellar.io
📱 Preview on mobile	Toggle 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]

👋 Hey @clubanderson — 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:

• Claude Code (Opus 4.5 / 4.6) — recommended
• GitHub Copilot
• Cursor
• Other AI coding assistants

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

---

This is an automated message.

[Copilot]

Pull request overview

This PR focuses on significantly increasing automated test coverage (targeting hooks/libs/contexts) and stabilizing CI coverage runs by sharding, without changing production behavior.

Changes:

• Added/expanded deep branch-coverage tests across many hooks and a handful of components, with more robust mocking patterns.
• Updated multiple tests to preserve real constants/modules via importOriginal() while overriding only specific values.
• Adjusted the hourly coverage workflow to run 12 shards and switched gist badge auth to GIST_TOKEN.

Reviewed changes

Copilot reviewed 43 out of 126 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
web/src/hooks/tests/useKyverno.test.ts	Adds extensive branch-coverage tests for kyverno hook behavior, caching, polling, and errors.
web/src/hooks/tests/useKubectl.test.ts	Updates constants mocking and aligns assertions with the hook’s returned API.
web/src/hooks/tests/useKagentBackend.test.ts	Uses importOriginal() for constants mocks and updates shape assertion.
web/src/hooks/tests/useHelmActions.test.ts	Reworks tests with explicit fetch stubbing and adds deeper state/error-path coverage.
web/src/hooks/tests/useGadget.test.ts	Adds comprehensive tests around cache keys, demo data, and trace fetchers with authFetch mocking.
web/src/hooks/tests/useFeatureRequests.test.ts	Expands coverage for request/notification flows and adds table-driven checks for exported mappings.
web/src/hooks/tests/useFeatureHints.test.ts	Adds more lifecycle/analytics/storage coverage and improves constants mocking.
web/src/hooks/tests/useExecSession.test.ts	Adds regression tests for exit codes, reconnect/error clearing, and message handling.
web/src/hooks/tests/useDiagnoseRepairLoop.test.ts	Updates constants mocks and aligns hook invocation with required params.
web/src/hooks/tests/useDependencies.test.ts	Adds deep tests across demo/REST/agent fallback paths and URL encoding, with improved mocks.
web/src/hooks/tests/useDataCompliance.test.ts	Adds extensive tests for posture aggregation, scoring, caching, polling, and error paths.
web/src/hooks/tests/useConsoleCRs.test.ts	Adds broad CRUD/fetch coverage for console CR hooks, including persistence toggles and status updates.
web/src/hooks/tests/useClusterProgress.test.ts	Adds regression tests for websocket error/close/reconnect and payload edge cases.
web/src/hooks/tests/useCardRecommendations.test.ts	Adds boundary and deduplication/capping tests for recommendation logic.
web/src/hooks/tests/useCachedProw.test.ts	Adapts tests to updated hook interface and mocks additional dependencies.
web/src/hooks/tests/useCRDs.test.ts	Adds deeper tests for demo fallback, caching, failures, headers, and refetch behavior.
web/src/hooks/tests/useBranding.test.tsx	Expands branding/provider tests with analytics ID updating and cancellation behavior.
web/src/hooks/tests/useBenchmarkData.test.ts	Adds regression coverage for benchmark cache/stream edge cases and passthrough flags.
web/src/hooks/tests/useBackendHealth.test.ts	Adds polling/failure-threshold and fallback-path tests, plus shape assertions.
web/src/hooks/tests/useAlerts.test.ts	Adds extensive coverage for context/no-context paths and Slack webhook persistence/notification behavior.
web/src/hooks/tests/useAdmissionWebhooks.test.ts	Updates constants mocks to preserve original exports while overriding needed keys.
web/src/hooks/tests/useActiveUsers.test.ts	Adds deeper polling/circuit-breaker/refetch tests and expands shape assertions.
web/src/hooks/tests/useAIPredictions.test.ts	Adds regression tests around settings-driven filtering, shapes, and stable callbacks.
web/src/components/updates/UpdateProgressBanner.test.tsx	Updates network constants mock to preserve original exports.
web/src/components/ui/tests/RotatingTip.test.tsx	Extends analytics mock to include newly used emitTipShown.
web/src/components/settings/tests/Settings.test.tsx	Updates constants mock pattern to spread original network constants.
web/src/components/layout/tests/Layout.test.tsx	Updates constants mock pattern to spread original network constants.
web/src/components/dashboard/tests/GettingStartedBanner.test.tsx	Updates storage constants mock pattern to spread original exports.
web/src/components/dashboard/tests/CreateDashboardModal.test.tsx	Updates network constants mock pattern to spread original exports.
web/src/components/dashboard/tests/AddCardModal.test.tsx	Updates network constants mock pattern to spread original exports.
web/src/components/dashboard/CreateDashboardModal.test.tsx	Updates network constants mock pattern to spread original exports.
web/src/components/clusters/tests/Clusters.test.tsx	Updates constants mock to preserve original exports while overriding needed values.
web/src/components/cards/llmd/tests/EPPRouting.test.tsx	Adds missing mocks and timer setup to avoid runtime crashes during render.
.github/workflows/coverage-hourly.yml	Increases coverage shards to 12 and switches gist update auth to GIST_TOKEN.

[Copilot]

This mock fully replaces ../../lib/constants with only STORAGE_KEY_TOKEN. If useCRDs (or its imports) relies on any other exports from that module, they will be undefined in tests (and behavior will diverge from production). Consider switching to the async (importOriginal) => ({ ...actual, STORAGE_KEY_TOKEN: 'token' }) pattern used elsewhere in this PR to preserve other constants.

Suggested change

	vi.mock('../../lib/constants', () => ({
	STORAGE_KEY_TOKEN: 'token',
	}))
	vi.mock('../../lib/constants', async (importOriginal) => {
	const actual = await importOriginal<typeof import('../../lib/constants')>()
	return {
	...actual,
	STORAGE_KEY_TOKEN: 'token',
	}
	})

[Copilot]

The workflow comment still says 'Run tests in 4 shards' but the matrix is now 12 shards. Please update the comment to match the new sharding strategy (and, if still relevant, the memory rationale) to avoid confusion when maintaining CI.

Suggested change

	# Run tests in 4 shards to stay within 7GB runner memory
	# Run tests in 12 shards to stay within 7GB runner memory

[Copilot]

The workflow comment still says 'Run tests in 4 shards' but the matrix is now 12 shards. Please update the comment to match the new sharding strategy (and, if still relevant, the memory rationale) to avoid confusion when maintaining CI.

[Copilot]

mockContextValue is declared and reset but never used to drive the mocked context (the tests instead pass values via Ctx.Provider). This is dead state that can be removed, or (if intended) wired into the mock so it actually controls the context value.

Suggested change

	// We keep a mutable variable that vi.mock reads from via closure.
	let mockContextValue: Record<string, unknown> | null = null
	vi.mock('../../contexts/AlertsContext', async (importOriginal) => {
	const actual = (await importOriginal()) as Record<string, unknown>
	const { createContext } = await import('react')
	// Create a real context whose value we control via mockContextValue
	vi.mock('../../contexts/AlertsContext', async (importOriginal) => {
	const actual = (await importOriginal()) as Record<string, unknown>
	const { createContext } = await import('react')
	// Create a real context whose value tests can control via the Provider

[Copilot]

mockContextValue is declared and reset but never used to drive the mocked context (the tests instead pass values via Ctx.Provider). This is dead state that can be removed, or (if intended) wired into the mock so it actually controls the context value.

[Copilot]

The test resolves an async fetch promise outside of React's act(). Even if the implementation prevents state updates after unmount, resolving async work outside act() can produce warnings/flakiness in some environments. Wrapping the promise resolution in an act(async () => ...) (or awaiting a microtask inside act) will make the test more robust.

[clubanderson]

This PR has merge conflicts and cannot be merged as-is. Please rebase onto main and resolve conflicts. The changes look good (test-only + CI workflow tweak) — will merge once conflicts are resolved.

[github-actions]

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

Check out what's new:

• KubeStellar Console — Live multi-cluster dashboard
• Marketplace — Community extensions
• Knowledge Base — Troubleshooting and how-tos

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