Skip to content

Commit 8386749

Browse files
authored
fix(e2e-tests): run in unit test suite MONGOSH-1716 (#1823)
3f55e4c did not add a test-ci-coverage task to the e2e-test package, but changed the definition of the unit test tasks in CI to run the test-ci-coverage task, so that e2e tests wouldn't run as part of the unit test suite.
1 parent 459b0a7 commit 8386749

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

packages/e2e-tests/package.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313
"scripts": {
1414
"test": "cross-env TS_NODE_PROJECT=../../configs/tsconfig-mongosh/tsconfig.test.json mocha -r \"../../scripts/import-expansions.js\" --timeout 15000 --colors -r ts-node/register \"./test/e2e*.spec.ts\"",
1515
"test-ci": "node ../../scripts/run-if-package-requested.js npm test",
16+
"test-coverage": "nyc --no-clean --cwd ../.. --reporter=none npm run test",
17+
"test-ci-coverage": "nyc --no-clean --cwd ../.. --reporter=none npm run test-ci",
1618
"eslint": "eslint",
1719
"lint": "npm run eslint . && npm run prettier -- --check .",
1820
"check": "npm run lint && npm run depcheck",

packages/e2e-tests/test/e2e-auth.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -282,7 +282,7 @@ describe('Auth e2e', function () {
282282
shell.assertContainsOutput('{ n: 2, ok: 1 }');
283283
} catch {
284284
// The 5.0+ server responds with a Long.
285-
shell.assertContainsOutput('{ n: Long("2"), ok: 1 }');
285+
shell.assertContainsOutput("{ n: Long('2'), ok: 1 }");
286286
}
287287
});
288288
const result = await db.command({ usersInfo: 1 });
@@ -492,7 +492,7 @@ describe('Auth e2e', function () {
492492
shell.assertContainsOutput('{ n: 2, ok: 1 }');
493493
} catch {
494494
// The 5.0+ server responds with a Long.
495-
shell.assertContainsOutput('{ n: Long("2"), ok: 1 }');
495+
shell.assertContainsOutput("{ n: Long('2'), ok: 1 }");
496496
}
497497
});
498498
const result = await db.command({ rolesInfo: 1 });

0 commit comments

Comments
 (0)