Skip to content

Commit

Permalink
test: refactor public cross chain tests for speed (#6082)
Browse files Browse the repository at this point in the history
Fixing #6055, time goes from 400 seconds -> 150 seconds.

Minor issues when dealing with the snapshots across instances as the
cross chain harness is taking the pxe and node as args for its
constructor so if going directly from snapshot could run into issue
where it is connected to a different instance or the like.

Also ran into a deserialization issue for the aztec address, unclear why
as it seems like it have been registered 🤷
  • Loading branch information
LHerskind authored Apr 29, 2024
1 parent e1f3e32 commit 6065a6c
Show file tree
Hide file tree
Showing 8 changed files with 693 additions and 415 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ This fetches the wallets from the sandbox and deploys our cross chain harness on
## Public flow test
#include_code e2e_public_cross_chain /yarn-project/end-to-end/src/e2e_public_cross_chain_messaging.test.ts typescript
#include_code e2e_public_cross_chain /yarn-project/end-to-end/src/e2e_public_cross_chain_messaging/deposits.test.ts typescript
## Running the test
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,10 +89,8 @@ export class BlacklistTokenContractTest {
this.admin = this.wallets[0];
this.other = this.wallets[1];
this.blacklisted = this.wallets[2];
// this.accounts = this.wallets.map(a => a.getCompleteAddress());
this.accounts = await pxe.getRegisteredAccounts();
this.wallets.forEach((w, i) => this.logger.verbose(`Wallet ${i} address: ${w.getAddress()}`));
this.accounts.forEach((w, i) => this.logger.verbose(`Account ${i} address: ${w.address}`));
});

await this.snapshotManager.snapshot(
Expand Down
Loading

0 comments on commit 6065a6c

Please sign in to comment.