Skip to content

Conversation

@sanket1729
Copy link
Member

When moving from integration tests repo to cargo test, I forgot to
add the test file for miniscript tests from cpp.

When moving from integration tests repo to `cargo test`, I forgot to
add the test file for miniscript tests from cpp.
Initially setup tests were run once per each integration test. This
makes it run only once
Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 39c9c83

@apoelstra
Copy link
Member

Lol, oops, good catch.

@sanket1729 sanket1729 merged commit 44c0e4b into rust-bitcoin:master Jun 10, 2022
@sanket1729
Copy link
Member Author

You Acked a different commit, but I took the liberty to merge this as it was only adding test code.

heap-coder added a commit to heap-coder/rust-miniscript that referenced this pull request Sep 27, 2025
…ing miniscripts

1ab637f86d16cfb0708c263858b243d8e8eb91b7 Move setup tests to a single test (sanket1729)
67c38e8dd06af1bf1f720d346f004ccfc1eed4c8 Add the missing file for testing miniscripts (sanket1729)

Pull request description:

  When moving from integration tests repo to `cargo test`, I forgot to
  add the test file for miniscript tests from cpp.

Top commit has no ACKs.

Tree-SHA512: 9e762c3b208a2fd2ae35ebc743641e8cbda534e4e30ef8d20050a8db594f102355eb189940902e9b47be2879c3ea5dee769de1720f9443cc23abe55ae1aa92bc
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.

2 participants