Update to deploy 9 #429
Update to deploy 9 #429
Conversation
| export const BALANCER_COMPOSITE_LIQUIDITY_ROUTER: Record<number, Address> = { | ||
| [ChainId.SEPOLIA]: '0x89cA59Bc46c00D90C496Fc99f16668b00Dd6B5CC', |
There was a problem hiding this comment.
added this to help prepare for the upcoming boosting / nesting
| @@ -24,7 +24,7 @@ export default defineConfig(({ mode }) => { | |||
| }, | |||
| test: { | |||
| testTimeout: 20_000, | |||
| hookTimeout: 30_000, | |||
| hookTimeout: 60_000, | |||
There was a problem hiding this comment.
As you can see by the commits on this PR, the create pool tests I just refactored will timeout on ~half the attempts. I think because I moved pool creation into the beforeAll hook. Thought it made sense to re-use same pool when testing deployment and initialization, but let me know if this is bad practice or if it is important to keep hook timeout at 30s
b-sdk/test/v3/createPool/stable/stable.integration.test.ts
Lines 84 to 88 in 9768175
There was a problem hiding this comment.
These tests have always been a pain! Not sure if we're missing something but haven't had time to check yet. Lets see how it goes for now.
You know what it is 🤣