From 4799cab3c49b27ddd4348eaea32ca4f550d798ed Mon Sep 17 00:00:00 2001
From: Scott Silver <scott@compound.finance>
Date: Wed, 20 Apr 2022 15:39:51 -0700
Subject: [PATCH] add BigNumber comment; reduce scale of negative comet base
 balance test

---
 scenario/ConstraintScenario.ts                 | 2 +-
 scenario/constraints/CometBalanceConstraint.ts | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/scenario/ConstraintScenario.ts b/scenario/ConstraintScenario.ts
index 68e292eb9..4838c8c7b 100644
--- a/scenario/ConstraintScenario.ts
+++ b/scenario/ConstraintScenario.ts
@@ -66,7 +66,7 @@ scenario(
     expectApproximately(
       await albert.getCometBaseBalance(),
       -100n * scale,
-      (scale / 1000n) // .001 units of asset
+      (scale / 1000000n) // .000001 units of asset
     );
   }
 );
diff --git a/scenario/constraints/CometBalanceConstraint.ts b/scenario/constraints/CometBalanceConstraint.ts
index 6a215d447..0cc67a15f 100644
--- a/scenario/constraints/CometBalanceConstraint.ts
+++ b/scenario/constraints/CometBalanceConstraint.ts
@@ -137,6 +137,8 @@ export class CometBalanceConstraint<T extends CometContext, R extends Requiremen
           const amount = parseAmount(rawAmount);
           const decimals = await asset.token.decimals();
           const baseToken = await comet.baseToken();
+          // Chai matchers (like `.to.be.at.most()`) only work for numbers and
+          // BigNumbers, so we convert from BigInt to BigNumber
           let actualBalance: BigNumber;
           let expectedBalance: BigNumber;
           if (asset.address === baseToken) {