GitHub labels: Stellar Wave
Drips complexity: Medium (100 points)
Area: Testing
Description
AcquirePositionDialog replaced alert() on secondary acquire, but component-level tests for disconnected wallet and KYC gate are missing.
Requirements and Context
Regression risk is high on the P2P flow. Mock useWallet and KYC state to assert dialog blocks submit appropriately.
Suggested Execution
git checkout -b test/acquire-position-dialog
- Create branch test/acquire-position-dialog
- Render dialog with mock listing
- Assert connect prompt when wallet disconnected
- Assert KYC block when required
- Mock useTransaction execute path
Example commit message
test(secondary): add component tests for AcquirePositionDialog gating
File Paths
components/invoice/AcquirePositionDialog.tsx
app/secondary/page.tsx
hooks/useTransaction.ts
Guidelines
- Assignment required before starting
- PR description must include:
Closes #<issue_number>
- Run
npm run lint, npm run type-check, npm run test, and npm run build before opening PR
- Add screenshots or recordings for UI changes
- Complexity: Medium (100 points) — focused test coverage work
Acceptance Criteria
GitHub labels:
Stellar WaveDrips complexity: Medium (100 points)
Area: Testing
Description
AcquirePositionDialog replaced alert() on secondary acquire, but component-level tests for disconnected wallet and KYC gate are missing.
Requirements and Context
Regression risk is high on the P2P flow. Mock useWallet and KYC state to assert dialog blocks submit appropriately.
Suggested Execution
Example commit message
File Paths
components/invoice/AcquirePositionDialog.tsxapp/secondary/page.tsxhooks/useTransaction.tsGuidelines
Closes #<issue_number>npm run lint,npm run type-check,npm run test, andnpm run buildbefore opening PRAcceptance Criteria