From 5405d47945e95295d411a01b76254849b1e4b9e8 Mon Sep 17 00:00:00 2001 From: Chris Hibbert Date: Tue, 8 Oct 2024 14:33:05 -0700 Subject: [PATCH] chore: update comment & lint fixes --- .../test/bootstrapTests/updateGovernedParams.test.ts | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/packages/boot/test/bootstrapTests/updateGovernedParams.test.ts b/packages/boot/test/bootstrapTests/updateGovernedParams.test.ts index 140e4ae70bc..7fe5c25f582 100644 --- a/packages/boot/test/bootstrapTests/updateGovernedParams.test.ts +++ b/packages/boot/test/bootstrapTests/updateGovernedParams.test.ts @@ -1,7 +1,7 @@ /** - * @file The goal of this test is to see that the - * upgrade scripts re-wire all the contracts so new auctions and - * price feeds are connected to vaults correctly. + * @file The goal of this test is to ensure that governance can update params + * after an upgrade. There was a point when the contractGovernor kept trying to + * use the old paramManager which was ephemeral and no longer useable. * * 1. enter a bid * 2. force prices to drop so a vault liquidates @@ -21,7 +21,7 @@ import { import { updateVaultDirectorParams, updateVaultManagerParams, -} from '../tools/changeVaultParams'; +} from '../tools/changeVaultParams.js'; const makeDefaultTestContext = async t => { console.time('DefaultTestContext'); @@ -76,10 +76,6 @@ test.after.always(t => { return t.context.shutdown && t.context.shutdown(); }); -const outcome = { - bids: [{ payouts: { Bid: 0, Collateral: 1.800828 } }], -}; - test('modify manager & director params; update vats, check', async t => { const { agoricNamesRemotes, evalProposal, buildProposal, gd } = t.context;