Closed gianfra-t closed 11 months ago
Hey @ebma, could you please convert this into a draft? It seems like I don't have team access to do it.
I actually can't because it's not available for private repos with our current Github plan. 🫥
Ohh then no problem, it is not at all important!
Sure!, I'm okay with those changes. As for the errors, actually I removed them because everything was going to slack, but we can add a log of whatever would be sent to slack so we can debug without creating/installing the slack app.
Did you use prettier for the formatting or another tool? We should add some command to the package.json that handles formatting/linting the code and maybe also configure a pre-commit hook
I am using prettier now! (for the last commit). We could use the same hook as in the squids repo probably.
Yes, let's add the same hook 👍 IIRC I used option 1) described here to add it to the squid repo.
Merging this now as all features we wanted are ready.
Implementation
High Level Overview:
TimeOut
errors when sending multiple concurrent transactions with the same account.polkadot/api
access for a given test network and a mutex to ensure that multiple transactions for a givenaccountId
are not executed at the same time. This prevent a nonce error. Nevertheless, the transactions are executed on the same block.completion_window_minutes
is exceeded, terminating the test.test_issuance
andtest_redeem
for each vault/network, and provides errors handling raised during tests by sending to slack. See hereThings that will probably be left for a future issue