[scanner] fix: mixed test infrastructure failures#20564
[scanner] fix: mixed test infrastructure failures#20564kubestellar-hive[bot] wants to merge 2 commits into
Conversation
…, dynamic-cards, networking) Fixes #20556 Root causes addressed: - analytics-errors.ts: recentlyReportedErrors not cleared in _resetErrorThrottles() - alertStorage.ts: non-quota errors logged via console.warn instead of console.error - notifications.ts: sendSingleNotification logged via console.warn instead of console.error - agentConnectivity.test.tsx: test checked result.current.type but hook returns status - StackContext.test.tsx: mock factory hardcoded isDemoMode:false; wired via vi.hoisted() - api.test.ts / api-methods.test.ts: global setup.ts mock overrode real module; add vi.unmock() - compiler.ts: blob URL import() unsupported in jsdom/node; add injectable _setCardImportFn - compiler tests: use Function-based evaluator via _setCardImportFn/afterEach reset Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: scanner-bot <scanner-bot@kubestellar.io>
kubestellar-prow
Bot
added
the
dco-signoff: yes
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 Hey @kubestellar-hive[bot] — thanks for opening this PR!
This is an automated message. |
kubestellar-prow
Bot
added
the
size/L
✅ Deploy Preview for kubestellarconsole ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Test Coverage CheckAll new source files in this PR have corresponding test files. Checked |
♿ 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. |
… PR #20564 Signed-off-by: kubestellar-hive[bot] <kubestellar-hive[bot]@users.noreply.github.com>
Fixes #20556
Root causes fixed across 5 test groups:
analytics-errors.ts:recentlyReportedErrorsnot cleared in_resetErrorThrottles()alertStorage.ts: non-quota errors logged viaconsole.warninstead ofconsole.errornotifications.ts:sendSingleNotificationusedconsole.warninstead ofconsole.erroragentConnectivity.test.tsx: test checkedresult.current.typebut hook returnsstatusStackContext.test.tsx: mock factory hardcodedisDemoMode: false; fixed viavi.hoisted()api.test.ts/api-methods.test.ts: global setup.ts mock overrode real module; addedvi.unmock()compiler.ts: blob URLimport()not supported in jsdom/node; added injectable_setCardImportFnand_resetCardImportFnfor tests🤖 Generated with Claude Code