improve timestamp comparison idioms

Agoric · May 2, 2023 · d6989a4 · d6989a4
1 parent 61b0fde
commit d6989a4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/packages/solo/test/test-home.js b/packages/solo/test/test-home.js
@@ -147,7 +147,7 @@ test.serial('home.localTimerService makeNotifier', async t => {
 
   // Tests gets an actual localTimerService, which returns actual times. We
   // can't verify the actual time, so we compare to make sure it's increasing.
-  t.truthy(TimeMath.absValue(update2.value) > TimeMath.absValue(update1.value));
+  t.truthy(TimeMath.compareAbs(update2.value, update1.value) > 0);
 });
 
 function makeHandler() {
@@ -178,9 +178,7 @@ test.serial('home.localTimerService makeRepeater', async t => {
   await E(notifier).getUpdateSince();
 
   t.truthy(handler.getCalls() >= 1);
-  t.truthy(
-    TimeMath.absValue(handler.getArgs()[0]) > TimeMath.absValue(timestamp),
-  );
+  t.truthy(TimeMath.compareAbs(handler.getArgs()[0], timestamp) > 0);
 });
 
 // =========================================

diff --git a/packages/zoe/test/unitTests/test-manualTimer.js b/packages/zoe/test/unitTests/test-manualTimer.js
@@ -28,7 +28,7 @@ test('manualTimer makeNotifier', async t => {
   const update2 = await p2;
   // @ts-expect-error updateCount could be undefined
   t.truthy(BigInt(update2.updateCount) > BigInt(update1.updateCount));
-  t.truthy(TimeMath.absValue(update2.value) > TimeMath.absValue(update1.value));
+  t.truthy(TimeMath.compareAbs(update2.value, update1.value) > 0);
 });
 
 const makeHandler = () => {
@@ -58,7 +58,7 @@ test('manualTimer makeRepeater', async t => {
   await manualTimer.tick();
 
   t.is(1n, handler.getCalls());
-  t.is(TimeMath.compareAbs(handler.getArgs()[0], timestamp), 1);
+  t.truthy(TimeMath.compareAbs(handler.getArgs()[0], timestamp) > 0);
 });
 
 const stallLots = async () => {