Mirror
/
zulip
mirror of https://github.com/zulip/zulip.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

zjsunit: Extract complain_about_unused_mocks. 

This is prep toward allowing individual tests
to fail while continuing to run the test suite.

Most of the steps in namespace.finish() could
be put in a `finally` block, but once a test
fails, there's no reason to complain about
unused mocks, since there are bigger things
to address.
This commit is contained in:
Steve Howell 2021-03-13 13:14:40 +00:00 committed by Steve Howell
parent e0f0b7f4a7
commit e5669f9087
2 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
frontend_tests/zjsunit/index.js
View File

@ -74,6 +74,7 @@ function run_one_module(file) {
console.info("running test " + path.basename(file, ".js"));
test.set_current_file_name(file);
require(file);
namespace.complain_about_unused_mocks();
}
test.set_verbose(files.length === 1);

17
frontend_tests/zjsunit/namespace.js
View File

@ -136,6 +136,16 @@ exports.zrequire = function (short_fn) {
const staticPath = path.resolve(__dirname, "../../static") + path.sep;
const templatesPath = staticPath + "templates" + path.sep;
exports.complain_about_unused_mocks = function () {
for (const filename of module_mocks.keys()) {
if (!used_module_mocks.has(filename)) {
throw new Error(
`You asked to mock ${filename} but we never saw it during compilation.`,
);
}
}
};
exports.finish = function () {
/*
Handle cleanup tasks after we've run one module.
@ -151,13 +161,6 @@ exports.finish = function () {
Module._load = actual_load;
actual_load = undefined;
for (const filename of module_mocks.keys()) {
if (!used_module_mocks.has(filename)) {
throw new Error(
`You asked to mock ${filename} but we never saw it during compilation.`,
);
}
}
module_mocks.clear();
used_module_mocks.clear();