Skip to content

Commit

Permalink
DEV: Set QUnit per-test timeout (discourse#21102)
Browse files Browse the repository at this point in the history
60s is a lot, and I'd prefer 15s, but the first test in a suite can take up to 40s. 60s is still better than `Infinity` 😌
  • Loading branch information
CvX authored Apr 15, 2023
1 parent ba11cf4 commit b8cf798
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 1 addition & 0 deletions app/assets/javascripts/discourse/tests/setup-tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -202,6 +202,7 @@ export default function setupTests(config) {
setupDeprecationCounter(QUnit);

QUnit.config.hidepassed = true;
QUnit.config.testTimeout = 60_000;

sinon.config = {
injectIntoThis: false,
Expand Down
2 changes: 0 additions & 2 deletions test/run-qunit.js
Original file line number Diff line number Diff line change
Expand Up @@ -213,8 +213,6 @@ function logQUnit() {

console.log("\nRunning: " + JSON.stringify(QUnit.urlParams) + "\n");

QUnit.config.testTimeout = 10000;

let durations = {};

let inTest = false;
Expand Down

0 comments on commit b8cf798

Please sign in to comment.