From f812e66252dec692d43e43eb8d9f1fe9453e39e2 Mon Sep 17 00:00:00 2001 From: Jos de Jong Date: Wed, 3 Feb 2021 17:58:37 +0100 Subject: [PATCH] Configure default timeout for mocha tests as 10 seconds --- package.json | 8 ++++---- test/node-tests/esm.test.js | 2 -- test/unit-tests/expression/Help.test.js | 1 - test/unit-tests/function/algebra/rationalize.test.js | 2 -- test/unit-tests/function/algebra/simplify.test.js | 1 - test/unit-tests/function/arithmetic/pow.test.js | 2 -- test/unit-tests/type/unit/Unit.test.js | 2 -- 7 files changed, 4 insertions(+), 14 deletions(-) diff --git a/package.json b/package.json index c638573e2..7fad9a1a3 100644 --- a/package.json +++ b/package.json @@ -122,13 +122,13 @@ "lint": "standard --env=mocha --env=worker", "validate:ascii": "gulp --gulpfile gulpfile.cjs validate:ascii", "test": "npm run test:src && npm run lint", - "test:src": "mocha test/unit-tests --recursive --forbid-only", - "test:generated": "mocha test/generated-code-tests --recursive --forbid-only", - "test:node": "mocha test/node-tests/*.test.js test/node-tests/**/*.test.js --recursive --forbid-only", + "test:src": "mocha test/unit-tests --recursive --forbid-only --timeout 10000", + "test:generated": "mocha test/generated-code-tests --recursive --forbid-only --timeout 10000", + "test:node": "mocha test/node-tests/*.test.js test/node-tests/**/*.test.js --recursive --forbid-only --timeout 10000", "test:all": "npm run test:src && npm run test:generated && npm run test:node", "test:browser": "karma start test/browser-test-config/local-karma.js", "test:browserstack": "karma start test/browser-test-config/browserstack-karma.js", - "coverage": "nyc --reporter=lcov --reporter=text-summary mocha test/unit-tests --recursive && echo \"\nDetailed coverage report is available at ./coverage/lcov-report/index.html\"", + "coverage": "nyc --reporter=lcov --reporter=text-summary mocha test/unit-tests --recursive --forbid-only --timeout 10000 && echo \"\nDetailed coverage report is available at ./coverage/lcov-report/index.html\"", "prepublishOnly": "npm run test:all && npm run lint", "prepare": "npm run build", "update-authors": "node ./tools/update-authors.js" diff --git a/test/node-tests/esm.test.js b/test/node-tests/esm.test.js index c8a2e2aa6..bf7bc88db 100644 --- a/test/node-tests/esm.test.js +++ b/test/node-tests/esm.test.js @@ -4,8 +4,6 @@ const cp = require('child_process') const path = require('path') describe('lib/esm', function () { - this.timeout(10000) - it('should load via mjs', function (done) { const filename = path.join(__dirname, 'esmApp.mjs') cp.exec('node ' + filename, function (error, result) { diff --git a/test/unit-tests/expression/Help.test.js b/test/unit-tests/expression/Help.test.js index 4a90e7a13..fa3625c20 100644 --- a/test/unit-tests/expression/Help.test.js +++ b/test/unit-tests/expression/Help.test.js @@ -32,7 +32,6 @@ describe('help', function () { }) it('should throw an error when constructed without new operator', function () { - this.timeout(10000) assert.throws(function () { console.log(Help(embeddedDocs.sin)) }, /Constructor must be called with the new operator/) diff --git a/test/unit-tests/function/algebra/rationalize.test.js b/test/unit-tests/function/algebra/rationalize.test.js index 279701202..ab401727b 100644 --- a/test/unit-tests/function/algebra/rationalize.test.js +++ b/test/unit-tests/function/algebra/rationalize.test.js @@ -34,8 +34,6 @@ function objToStrings (obj) { /// ////////////////// rationalize /////////////////////// describe('rationalize', function () { - this.timeout(10000) // For IE/Edge - it('invalid expression', function () { assert.throws(function () { math.rationalize('(x*/2)') }, /Value expected \(char 4\)/) }) diff --git a/test/unit-tests/function/algebra/simplify.test.js b/test/unit-tests/function/algebra/simplify.test.js index a46d51de1..f05f2b38f 100644 --- a/test/unit-tests/function/algebra/simplify.test.js +++ b/test/unit-tests/function/algebra/simplify.test.js @@ -174,7 +174,6 @@ describe('simplify', function () { }) it('should preserve the value of BigNumbers', function () { - this.timeout(10000) const bigmath = math.create({ number: 'BigNumber', precision: 64 }) assert.deepStrictEqual(bigmath.simplify('111111111111111111 + 111111111111111111').evaluate(), bigmath.evaluate('222222222222222222')) assert.deepStrictEqual(bigmath.simplify('1 + 111111111111111111').evaluate(), bigmath.evaluate('111111111111111112')) diff --git a/test/unit-tests/function/arithmetic/pow.test.js b/test/unit-tests/function/arithmetic/pow.test.js index b32b5e58d..6f753a762 100644 --- a/test/unit-tests/function/arithmetic/pow.test.js +++ b/test/unit-tests/function/arithmetic/pow.test.js @@ -246,8 +246,6 @@ describe('pow', function () { }) it('should compute large size of square matrix', function () { - this.timeout(10000) - const a = math.identity(30).valueOf() approx.deepEqual(pow(a, 1000), a) approx.deepEqual(pow(matrix(a), 1000), matrix(a)) diff --git a/test/unit-tests/type/unit/Unit.test.js b/test/unit-tests/type/unit/Unit.test.js index e99c740a3..486878170 100644 --- a/test/unit-tests/type/unit/Unit.test.js +++ b/test/unit-tests/type/unit/Unit.test.js @@ -916,8 +916,6 @@ describe('Unit', function () { }) it('should parse the value of the unit as Fraction or BigNumber when math.js is configured so', function () { - this.timeout(10000) // For IE tests - const math2 = math.create({ number: 'Fraction' }) const unit2 = math2.Unit.parse('5kg') assert(isFraction(unit2.value))