Changes the integration test target so it's a test instead of a binary
This will provide us with some of the builtin instrumentation and out-of-the-box parallelization of tests
It refactors the execute_tx_and_sync function so it waits until the tx gets committed before continuing
doing this we found that we had a bug where we'd be storing the note ids instead of the nullifiers in the transactions table. While it's fixed here, I also openned a separate PR on #235 to take care of the bug.
@igamigo also found an issue in miden-node which caused some notes to not be included in blocks (mentioned here), for now we'll keep the tests serialized but once that gets fixed we can uncomment the #[tokio::test] and remove the main test.
The idea for tests from now on is:
all tests will use the same node
If we want to test account importing/exporting we shall make a specific test for that one
each test will create it's own accounts
if we want to test the interaction between accounts on different clients, we make a specific test for that
closes #215
This PR does a few things:
execute_tx_and_syncfunction so it waits until the tx gets committed before continuing#[tokio::test]and remove themaintest.The idea for tests from now on is:
(CI currently fails until #205 gets merged)