From 1f5a37c1e29e5012cfa008010eaf0cc247307b87 Mon Sep 17 00:00:00 2001 From: Muthu Kumar Date: Tue, 18 Sep 2018 17:24:16 +0530 Subject: [PATCH] [duration] Replace performance with less resolution Date --- package.json | 3 --- sample/sample7.test.js | 2 +- src/lib/caller.js | 12 +++++------- src/lib/testrunner.js | 4 ++-- src/util/perf_hooks.js | 2 -- 5 files changed, 8 insertions(+), 15 deletions(-) delete mode 100644 src/util/perf_hooks.js diff --git a/package.json b/package.json index fb77c4f..acc6bc7 100644 --- a/package.json +++ b/package.json @@ -23,9 +23,6 @@ "bugs": { "url": "https://github.com/klenty/gunner/issues" }, - "browser": { - "perf_utils": "src/util/perf_hooks.js" - }, "homepage": "https://github.com/klenty/gunner#readme", "dependencies": { "@codefeathers/iseq": "^1.2.1", diff --git a/sample/sample7.test.js b/sample/sample7.test.js index 4005d69..2b897de 100644 --- a/sample/sample7.test.js +++ b/sample/sample7.test.js @@ -99,4 +99,4 @@ gunner.test('(should fail) should not resolve to 5', () => const trace = process.argv.slice(2).indexOf('--trace') !== -1; const reporter = process.argv.slice(2).indexOf('--log') !== -1; -gunner.run({ trace, reporter }); +gunner.run({ trace, reporter }).then(console.log); diff --git a/src/lib/caller.js b/src/lib/caller.js index de71a0f..72a7324 100644 --- a/src/lib/caller.js +++ b/src/lib/caller.js @@ -1,5 +1,3 @@ -const { performance } = require('perf_hooks'); - const { isPromise } = require('../util'); const caller = (test, state) => { @@ -8,11 +6,11 @@ const caller = (test, state) => { let value, error, errored; try { - perf.start = performance.now(); + perf.start = Date.now(); value = test(state); - perf.end = performance.now(); + perf.end = Date.now(); } catch (e) { - perf.end = performance.now(); + perf.end = Date.now(); errored = true; error = e; } @@ -22,13 +20,13 @@ const caller = (test, state) => { if (promise) { return value .then(res => ({ - duration: performance.now() - perf.start, + duration: Date.now() - perf.start, status: 'ok', resolve: res, promise: true })) .catch(rej => ({ - duration: performance.now() - perf.start, + duration: Date.now() - perf.start, status: 'notOk', rejection: rej, promise: true diff --git a/src/lib/testrunner.js b/src/lib/testrunner.js index f1f980a..00e161d 100644 --- a/src/lib/testrunner.js +++ b/src/lib/testrunner.js @@ -116,14 +116,14 @@ const reduceQueue = */ const testrunner = (instance) => { - const perf = { start: performance.now() }; + const perf = { start: Date.now() }; return Promise.object(pipe( buildTestQueue, reduceQueue, pick('results'), )(instance)).then(results => { - results.duration = performance.now() - perf.start; + results.duration = Date.now() - perf.start; return results; }); diff --git a/src/util/perf_hooks.js b/src/util/perf_hooks.js deleted file mode 100644 index d6c08b1..0000000 --- a/src/util/perf_hooks.js +++ /dev/null @@ -1,2 +0,0 @@ -/* global performance */ -module.exports = { performance };