Skip to content

test: cover verifier reject history flow#610

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

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.

@1nonlypiece

Copy link
Copy Markdown
Contributor

@zhangquanshen123-droid this PR needs to be linked to an issue you're actually assigned to before it can be merged through the campaign — closing for now, happy to take another look once that's in place.

@1nonlypiece 1nonlypiece reopened this Jul 24, 2026
@1nonlypiece

Copy link
Copy Markdown
Contributor

@zhangquanshen123-droid this PR needs to be linked to an issue you're actually assigned to before it can be merged through the campaign — closing for now, happy to take another look once that's in place.

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

2 participants