Skip to content

test: cover ConfirmationModal approve/reject flow#150

Closed
go165 wants to merge 1 commit into
Disciplr-Org:mainfrom
go165:go165/test-confirmation-modal
Closed

test: cover ConfirmationModal approve/reject flow#150
go165 wants to merge 1 commit into
Disciplr-Org:mainfrom
go165:go165/test-confirmation-modal

Conversation

@go165

@go165 go165 commented Jun 17, 2026

Copy link
Copy Markdown

Closes #133.

Summary

  • add RTL coverage for ConfirmationModal open/closed rendering, approve/reject decisions, notes passthrough, cancel/close behavior, evidence link rendering, backdrop behavior, and reopen reset behavior
  • assert focus-trap wiring for outside-click allowance, Escape deactivation, and onDeactivate close handling
  • keep this PR test-only; no production component changes

Verification

  • npm test -- --run src/components/__tests__/ConfirmationModal.test.tsx
  • npm test -- --run src/components/__tests__/ConfirmationModal.test.tsx src/pages/__tests__/ValidationDetail.test.tsx
  • git diff --check

Targeted coverage for src/components/ConfirmationModal.tsx is 100% statements / branches / functions / lines in the focused run. The existing duplicate test script warning in package.json is unchanged by this PR.

Payout wallet if reward eligible: 0x1f0130669ca6fd02e025a984cc038f139df19a2f

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 the ConfirmationModal approve/reject flow

2 participants