chore: Add a melos script for the FDv2 contract tests - #342
Merged
Conversation
kinyoklion
marked this pull request as ready for review
September 3, 2026 23:12
tanderson-ld
approved these changes
Sep 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CI runs the client-side contract tests twice: the FDv1 suite with sdk-test-harness v2 and the FDv2 suite with sdk-test-harness v3 (the "Contract Tests (FDv2)" step in the shared CI action). Locally,
melos run client-contract-testsonly covered the v2 run, so the FDv2 suite had no one-command local equivalent.melos run client-contract-tests-fdv2, which starts the contract test service and runs the v3 harness with the same parameters and suppressions file as CI.Note
Overview
Adds
melos run client-contract-tests-fdv2so developers can run the FDv2 client-side contract suite locally with sdk-test-harness v3, matching the CI “Contract Tests (FDv2)” step. The existingclient-contract-testsscript is unchanged in behavior but now has a description labeling it as FDv1 / harness v2.CONTRIBUTING.md is updated to document both commands and note that CI runs both suites.
Reviewed by Cursor Bugbot for commit d360030. Bugbot is set up for automated code reviews on this repo. Configure here.