Skip to content

test: cover verifier reject history flow#610

Open
zhangquanshen123-droid wants to merge 1 commit into
Disciplr-Org:mainfrom
zhangquanshen123-droid:test/verifier-reject-flow
Open

test: cover verifier reject history flow#610
zhangquanshen123-droid wants to merge 1 commit into
Disciplr-Org:mainfrom
zhangquanshen123-droid:test/verifier-reject-flow

Conversation

@zhangquanshen123-droid

Copy link
Copy Markdown

Summary

  • Adds stronger integration coverage for the verifier reject path from queue detail through history.
  • Verifies rejection notes are required before confirmation.
  • Verifies the rejected task is removed from the pending queue and persisted in validation history with rejected status and notes.
  • Adds a global window.matchMedia test stub so components using Tooltip/StatusChip render reliably in jsdom.

Closes #491.

Tests

  • npx vitest run src/pages/__tests__/verifierFlow.integration.test.tsx --coverage=false
  • npx eslint src/pages/__tests__/verifierFlow.integration.test.tsx src/setupTests.ts

Notes

  • npm run build currently fails on existing repository-wide TypeScript issues outside this patch, including missing Vitest globals in unrelated tests, existing unused imports, and existing type errors in Analytics/VaultTransactions/VerifierDashboard.
  • npm run lint currently fails on existing repository-wide lint issues outside this patch; the two changed files pass targeted ESLint.

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 an integration test for the verifier reject flow updating queue and history

1 participant