libretic/etherpad-lite
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

tests: Use window.foo instead of var foo

Browse source
This commit is contained in:
Richard Hansen 2021-04-22 13:20:01 -04:00
parent aea2fb7448
commit 4fa9f9e9d8
2 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/node/hooks/express/tests.js
View file

@ -26,7 +26,7 @@ exports.expressCreateServer = (hookName, args, cb) => {
console.debug('Sent browser the following test specs:', files);
res.setHeader('content-type', 'application/javascript');
res.end(`var frontendTestSpecs = ${JSON.stringify(files)};\n`);
res.end(`window.frontendTestSpecs = ${JSON.stringify(files)};\n`);
});
const rootTestFolder = path.join(settings.root, 'src/tests/frontend/');

4
src/tests/frontend/runner.js
View file

@ -1,7 +1,5 @@
'use strict';
/* global frontendTestSpecs */
$(() => {
const stringifyException = (exception) => {
let err = exception.stack || exception.toString();
@ -144,7 +142,7 @@ $(() => {
require.setGlobalKeyPath('require');
const $body = $('body');
for (const spec of frontendTestSpecs.map((spec) => encodeURI(spec))) {
for (const spec of window.frontendTestSpecs.map((spec) => encodeURI(spec))) {
$body.append($('<script>').attr('src', spec.startsWith('/') ? spec : `specs/${spec}`));
}