Skip to content

e2e: Testing different public API providers#479

Open
blenbot wants to merge 1 commit intocaravan-bitcoin:mainfrom
blenbot:testing_different_public_api_providers
Open

e2e: Testing different public API providers#479
blenbot wants to merge 1 commit intocaravan-bitcoin:mainfrom
blenbot:testing_different_public_api_providers

Conversation

@blenbot
Copy link

@blenbot blenbot commented Feb 15, 2026

What kind of change does this PR introduce?

Test improvement / E2E test coverage
Issue Number:
#442

If relevant, did you update the documentation?

Not applicable (test-only change)

Summary

This PR adds end-to-end tests to cover public API provider selection and response validation in the Caravan coordinator.

Previously, selecting and switching between public Bitcoin client providers (Mempool.space, Blockstream.info) was not covered by E2E tests, and live API response shapes were only validated against mocked data in unit tests. This change ensures that:

  • Public provider radio buttons are enabled and Mempool.space is the default on mainnet
  • Selecting Mempool or Blockstream correctly updates the UI and hides private client settings
  • Switching between Mempool, Blockstream, and Private providers behaves correctly across transitions
  • Live fee estimate responses from Mempool.space and Blockstream.info match the expected shapes that BlockchainClient depends on
  • Live address data from Mempool.space returns the chain_stats/mempool_stats structure used by getAddressStatus

Does this PR introduce a breaking change?

No

Checklist

  • I have tested my changes thoroughly.
  • I have added or updated tests to cover my changes (if applicable).
  • I have verified that test coverage meets or exceeds 95% (if applicable).
  • I have run the test suite locally, and all tests pass.
  • I have written tests for all new changes/features
  • I have followed the project's coding style and conventions.
  • I have created a changeset to document my changes (npm run changeset)

Other information

nothing much

Have you read the contributing guide?

Yes

For information on creating and using changesets, please refer to our documentation on changesets.

Signed-off-by: blenbot <harshitiszz@gmail.com>
@changeset-bot
Copy link

changeset-bot bot commented Feb 15, 2026

⚠️ No Changeset found

Latest commit: 0ca796b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Feb 15, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
caravan-coordinator Ready Ready Preview, Comment Feb 15, 2026 7:01am

Request Review

@blenbot
Copy link
Author

blenbot commented Feb 15, 2026

@Legend101Zz @bucko13 please review, thanks!

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.

1 participant