Skip to content

Conversation

@lokesh-bitgo
Copy link
Contributor

Ticket: WP-6330

@lokesh-bitgo lokesh-bitgo force-pushed the WP-6330-express-add-supertest-for-api-v-2-coin-wallet-id-recovertoken-typed-routes branch from ea7b9be to ed27f52 Compare October 17, 2025 09:52
@lokesh-bitgo lokesh-bitgo requested a review from Copilot October 17, 2025 09:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a comprehensive supertest + sinon-based test suite for the walletRecoverToken typed route, covering success paths, error handling, invalid inputs, and edge-case response validation.

  • Introduces end-to-end style tests hitting /api/v2/:coin/wallet/:id/recovertoken with varied payloads
  • Mocks BitGo coin and wallet interactions using sinon to simulate diverse success and failure scenarios
  • Expands response validation via RecoverTokenResponse codec, including malformed and type-mismatch cases

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@lokesh-bitgo lokesh-bitgo marked this pull request as ready for review October 17, 2025 10:04
@lokesh-bitgo lokesh-bitgo requested a review from a team as a code owner October 17, 2025 10:04
@lokesh-bitgo lokesh-bitgo merged commit 7313851 into master Oct 17, 2025
13 checks passed
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.

4 participants