Adding a conditional & assertion for a diff() case

avoidwork · Oct 5, 2023 · 147438b · 147438b
1 parent bc45b96
commit 147438b
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 6 deletions.
diff --git a/dist/precise.cjs b/dist/precise.cjs
@@ -3,7 +3,7 @@
  *
  * @copyright 2023 Jason Mulligan <jason.mulligan@avoidwork.com>
  * @license BSD-3-Clause
- * @version 4.0.0
+ * @version 4.0.1
  */
 'use strict';
 
@@ -22,6 +22,10 @@ class Precise {
 	}
 
 	diff (ms = false) {
+		if (this.started === BIG_INT_NEG_1) {
+			throw new Error(notStarted);
+		}
+
 		if (this.stopped === BIG_INT_NEG_1) {
 			throw new Error(notStopped);
 		}

diff --git a/dist/precise.js b/dist/precise.js
@@ -3,7 +3,7 @@
  *
  * @copyright 2023 Jason Mulligan <jason.mulligan@avoidwork.com>
  * @license BSD-3-Clause
- * @version 4.0.0
+ * @version 4.0.1
  */
 import {hrtime}from'node:process';const hasStarted = "Timer has been started";
 const hasStopped = "Timer has been stopped";
@@ -16,6 +16,10 @@ const BIG_INT_NEG_1 = BigInt(-1);class Precise {
 	}
 
 	diff (ms = false) {
+		if (this.started === BIG_INT_NEG_1) {
+			throw new Error(notStarted);
+		}
+
 		if (this.stopped === BIG_INT_NEG_1) {
 			throw new Error(notStopped);
 		}

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "precise",
   "description": "Precision timer for nanosecond differences",
-  "version": "4.0.0",
+  "version": "4.0.1",
   "homepage": "http://avoidwork.github.io/precise",
   "author": "Jason Mulligan <jason.mulligan@avoidwork.com>",
   "repository": {

diff --git a/src/precise.js b/src/precise.js
@@ -8,6 +8,10 @@ export class Precise {
 	}
 
 	diff (ms = false) {
+		if (this.started === BIG_INT_NEG_1) {
+			throw new Error(notStarted);
+		}
+
 		if (this.stopped === BIG_INT_NEG_1) {
 			throw new Error(notStopped);
 		}

diff --git a/test/precise_test.js b/test/precise_test.js
@@ -29,7 +29,9 @@ describe("Testing stopping state", function () {
 		const timer = this.timer;
 		assert.throws(function () { timer.diff(); }, Error, "Should be an 'Error'");
 		assert.throws(function () { timer.stop(); }, Error, "Should be an 'Error'");
-		this.timer.start().stop();
+		this.timer.start();
+		assert.throws(function () { timer.diff(); }, Error, "Should be an 'Error'");
+		this.timer.stop();
 		assert.notEqual(this.timer.started, BIG_INT_NEG_1, "Shouldn't be '-1n'");
 		assert.notEqual(this.timer.started, BIG_INT_NEG_1, "Shouldn't be '-1n'");
 		assert.throws(function () { timer.stop(); }, Error, "Should be an 'Error'");