Skip to content

Add RecentAttestationsPanel test coverage#663

Open
Mayreck wants to merge 1 commit into
Commitlabs-Org:masterfrom
Mayreck:codex/recent-attestations-panel-tests
Open

Add RecentAttestationsPanel test coverage#663
Mayreck wants to merge 1 commit into
Commitlabs-Org:masterfrom
Mayreck:codex/recent-attestations-panel-tests

Conversation

@Mayreck

@Mayreck Mayreck commented Jun 23, 2026

Copy link
Copy Markdown

Summary

  • Add Vitest/RTL coverage for RecentAttestationsPanel empty and populated states
  • Cover severity labels/classes, timestamp text, row order, compact tx hashes, and callbacks
  • Document the coverage in docs/testing/RECENT_ATTESTATIONS_TESTS.md

Validation

  • npx vitest run src/components/RecentAttestationsPanel/RecentAttestationsPanel.test.tsx
  • npx vitest run src/components/RecentAttestationsPanel/RecentAttestationsPanel.test.tsx src/components/tests/ErrorLayout.test.tsx
  • npx eslint src/components/RecentAttestationsPanel/RecentAttestationsPanel.test.tsx

Baseline notes

  • npm run lint still fails on existing unrelated repo lint/syntax issues outside this patch, including src/app/api/commitments/[id]/settle/route.ts and src/components/modals/CommitmentDetailsModal.tsx
  • npx tsc --noEmit is blocked by the same pre-existing syntax errors

Closes #643.

@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

@Mayreck is attempting to deploy a commit to the 1nonly's projects Team on Vercel.

A member of the Team first needs to authorize it.

@GBOYEE

GBOYEE commented Jun 24, 2026

Copy link
Copy Markdown

I'd like to work on this.

Approach:

  • I'll write the test cases and ensure they pass with full coverage
  • Verify with existing tests + add new ones if needed

Estimated effort: ~1-2 hours. PR incoming shortly.

1 similar comment
@GBOYEE

GBOYEE commented Jun 24, 2026

Copy link
Copy Markdown

I'd like to work on this.

Approach:

  • I'll write the test cases and ensure they pass with full coverage
  • Verify with existing tests + add new ones if needed

Estimated effort: ~1-2 hours. PR incoming shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add RTL tests for RecentAttestationsPanel severity coloring, density, and empty state

2 participants