test: update invariant_NoPriceChangesBeforeStart, add comment

clemlak · clemlak · commit 0f8d2e1c48fe · 2024-10-24T15:21:09.000+04:00
diff --git a/test/invariant/DopplerInvariants.sol b/test/invariant/DopplerInvariants.sol
@@ -122,8 +122,10 @@ contract DopplerInvariantsTest is BaseTest {
         }
     }
 
+    /// forge-config: default.invariant.fail-on-revert = true
     function invariant_NoPriceChangesBeforeStart() public {
         vm.warp(DEFAULT_STARTING_TIME - 1);
-        assertEq(hook.getCurrentTick(poolId), DEFAULT_START_TICK);
+        // TODO: I think this test is broken because we don't set the tick in the constructor.
+        assertEq(hook.getCurrentTick(poolId), hook.getStartingTick());
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -122,8 +122,10 @@ contract DopplerInvariantsTest is BaseTest {`
`122`	`122`	`}`
`123`	`123`	`}`
`124`	`124`
	`125`	`+ /// forge-config: default.invariant.fail-on-revert = true`
`125`	`126`	`function invariant_NoPriceChangesBeforeStart() public {`
`126`	`127`	`vm.warp(DEFAULT_STARTING_TIME - 1);`
`127`		`- assertEq(hook.getCurrentTick(poolId), DEFAULT_START_TICK);`
	`128`	`+ // TODO: I think this test is broken because we don't set the tick in the constructor.`
	`129`	`+ assertEq(hook.getCurrentTick(poolId), hook.getStartingTick());`
`128`	`130`	`}`
`129`	`131`	`}`