Skip to content

Commit c954188

Browse files
author
Andrey Fel
committed
Fix exam command acceptance tests
Actualise number of tests. Remove `ember-test` from the list of tests which doesn't actually exist.
1 parent 32fce8d commit c954188

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

node-tests/acceptance/exam-test.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ function getNumberOfTests(str) {
2121
return match && parseInt(match[1], 10);
2222
}
2323

24-
const TOTAL_NUM_TESTS = 65; // Total Number of tests without the global 'Ember.onerror validation tests'
24+
const TOTAL_NUM_TESTS = 67; // Total Number of tests without the global 'Ember.onerror validation tests'
2525

2626
function getTotalNumberOfTests(output) {
2727
// In ember-qunit 3.4.0, this new check was added: https://github.com/emberjs/ember-qunit/commit/a7e93c4b4b535dae62fed992b46c00b62bfc83f4
@@ -513,7 +513,6 @@ describe('Acceptance | Exam Command', function () {
513513
failedBrowsers: [],
514514
executionMapping: {
515515
1: [
516-
'ember-test',
517516
'dummy/tests/unit/test-loader-test',
518517
'dummy/tests/unit/multiple-edge-cases-test',
519518
'dummy/tests/unit/multiple-ember-tests-test',
@@ -559,7 +558,7 @@ describe('Acceptance | Exam Command', function () {
559558
assertOutput(output, 'Browser Id', [1]);
560559
assert.strictEqual(
561560
getNumberOfTests(output),
562-
27,
561+
25,
563562
'ran all of the tests for browser one',
564563
);
565564
});
@@ -617,7 +616,7 @@ describe('Acceptance | Exam Command', function () {
617616
assertOutput(output, 'Browser Id', ['2']);
618617
assert.strictEqual(
619618
getNumberOfTests(output),
620-
40,
619+
44,
621620
'ran all of the tests for browser two',
622621
);
623622
});

0 commit comments

Comments
 (0)