Skip to content

test: add analytics charts and notification retention tests - #1021

Merged
Austinaminu2 merged 1 commit into
FlowwStar:mainfrom
Folex1275:wave10
Oct 18, 2026
Merged

Austinaminu2 merged 1 commit into
FlowwStar:mainfrom
Folex1275:wave10

Conversation

@Folex1275

Copy link
Copy Markdown
Contributor

Add test coverage for untested components and hooks

This PR adds test coverage for four previously untested areas identified in the Vitest coverage report.

Changes

#922 - DashboardStatsSkeleton tests

  • Added __tests__/components/dashboard-stats-skeleton.test.tsx
  • Tests verify: expected number of placeholder cards (4), loading animation classes, proper structure, no text content

#923 - Notification polling retention error recovery tests

  • Added __tests__/hooks/use-notifications-retention.test.ts
  • Tests verify: retention error + getHealth OK → cursor reset to oldestLedger - 1, retention error + getHealth fails → fallback to latest ledger, both fail → cursor unchanged
  • Includes test for all retention error message variations

#920 - ConnectWalletButton copy-address tests

  • Extended __tests__/components/connect-wallet-button.test.tsx
  • Tests verify: copies full address to clipboard, shows confirmation feedback after copying

#921 - AnalyticsCharts render tests

  • Added __tests__/components/analytics-charts.test.tsx
  • Tests verify: renders with empty snapshot, renders with populated data, ChartTooltip formats values, ChartSkeleton renders placeholders
  • Covers token amount formatting, unlock progress calculation, recipient display logic, empty states

All tests follow existing patterns in the codebase using Vitest + Testing Library.

Closes #920
Closes #921
Closes #922
Closes #923

@drips-wave

drips-wave Bot commented Oct 17, 2026

Copy link
Copy Markdown

@Folex1275 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Austinaminu2
Austinaminu2 merged commit e9f58a2 into FlowwStar:main Oct 18, 2026
1 of 7 checks passed
@grantfox-oss grantfox-oss Bot mentioned this pull request Oct 18, 2026
3 tasks

This branch had an error being deployed

1 failed deployment
staging — d61fb960 Deployed Oct 17, 2026 by Folex1275 via deploy-staging #342
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants