From 5dba0fca5f696a5e8d428f7657a3174cdb763c67 Mon Sep 17 00:00:00 2001 From: pahor167 <47992132+pahor167@users.noreply.github.com> Date: Fri, 1 Sep 2023 10:59:52 +0200 Subject: [PATCH] tests stability fixes --- .github/workflows/circleci.yml | 4 ++-- packages/protocol/test/common/feehandlerseller.ts | 2 +- .../protocol/test/governance/validators/downtimeslasher.ts | 6 +++--- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/circleci.yml b/.github/workflows/circleci.yml index ecc0dea746d..2a57e8de931 100644 --- a/.github/workflows/circleci.yml +++ b/.github/workflows/circleci.yml @@ -354,7 +354,7 @@ jobs: - name: Execute matrix command for test uses: nick-fields/retry@v2 with: - timeout_minutes: 30 + timeout_minutes: 60 max_attempts: 3 command: | ${{ matrix.command }} @@ -669,7 +669,7 @@ jobs: - name: Execute matrix command for test uses: nick-fields/retry@v2 with: - timeout_minutes: 30 + timeout_minutes: 60 max_attempts: 3 command: | ${{ matrix.command }} diff --git a/packages/protocol/test/common/feehandlerseller.ts b/packages/protocol/test/common/feehandlerseller.ts index df8c13f6faf..f0f47aa6e64 100644 --- a/packages/protocol/test/common/feehandlerseller.ts +++ b/packages/protocol/test/common/feehandlerseller.ts @@ -46,7 +46,7 @@ contract('FeeHandlerSeller', (accounts: string[]) => { mentoFeeHandlerSeller = await MentoFeeHandlerSeller.new(true) contractsToTest = [mentoFeeHandlerSeller, uniswapFeeHandlerSeller] for (const contract of contractsToTest) { - contract.initialize(registry.address, [], []) + await contract.initialize(registry.address, [], []) } }) diff --git a/packages/protocol/test/governance/validators/downtimeslasher.ts b/packages/protocol/test/governance/validators/downtimeslasher.ts index fd8763a22c8..a7319efcc2f 100644 --- a/packages/protocol/test/governance/validators/downtimeslasher.ts +++ b/packages/protocol/test/governance/validators/downtimeslasher.ts @@ -109,9 +109,9 @@ contract('DowntimeSlasher', (accounts: string[]) => { await validators.affiliate(groups[0], { from: validatorList[1] }) await validators.affiliate(groups[1], { from: validatorList[2] }) await slasher.initialize(registry.address, slashingPenalty, slashingReward, slashableDowntime) - await Promise.all( - accounts.map((account) => mockLockedGold.setAccountTotalLockedGold(account, 50000)) - ) + for (const account of accounts) { + await mockLockedGold.setAccountTotalLockedGold(account, 50000) + } }) describe('#initialize()', () => {