test: bouncer script to init swaps and send sol #5579
Conversation
dandanlen
force-pushed
the
feat/1.7.7-fixes
branch
from
36496a2 to
e613dd6
Compare
| const depositAddresses = await requestBatchOfSwaps(batchSize); | ||
| console.log(`Sent funds to ${depositAddresses.length} deposit channels`); | ||
| for (const depositAddress of depositAddresses) { | ||
| await sendSol(depositAddress, solAmount); |
There was a problem hiding this comment.
As mentioned IRL, we should add a small amount of SOL as we go so each amount is different, otherwise the RPC will dedupe them.
You can just copy what's in spam_sol.ts:
solAmount = solAmount.plus(new BigNumber(1).div(10 ** decimals));| // Opens 1000 deposit channels | ||
| // Execute: ./commands/run_test.ts spam_solana_deposit_channels.ts | ||
| async function main() { | ||
| const batches = 200; |
There was a problem hiding this comment.
This is too many batches for a bouncer run, will take a long time to complete. We can drop this to around 10 and get most of the benefit, in particular because it'll be running alongside all the other tests.
For manual testing, we can always increase this value.
There was a problem hiding this comment.
Yeah, definitely. I think it doesn't really make senses to run this automatic at all in my opinion, since it is pretty hard to verify the expected result. The idea was to have this more as a tool to debug a network.
There was a problem hiding this comment.
Not sure if we even want to have this permanently in our code base though :/
There was a problem hiding this comment.
Yeah, we can probably just leave it as a manual script, or delete it, I don't mind either way. Deleting is probably better for maintainability
Pull Request
Closes: PRO-xxx
Checklist
Please conduct a thorough self-review before opening the PR.
Summary
Please include a succinct description of the purpose and content of the PR. What problem does it solve, and how? Link issues, discussions, other PRs, and anything else that will help the reviewer.