From 53661b3d8373c984a0635697654e4843b59fc599 Mon Sep 17 00:00:00 2001 From: Fomafix Date: Sun, 24 Feb 2019 10:08:49 +0100 Subject: [PATCH] resourceloader: Avoid global window.mwTestFooJsCounter in unit tests This change avoids a failure when running Special:JavaScriptTest with the option noglobals. Change-Id: Id61fca5ccedbd793f7786ad95681eaebb9feafb6 --- .../suites/resources/mediawiki/mediawiki.loader.test.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js index 8b3427f93f..ed1288b676 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js @@ -562,7 +562,8 @@ QUnit.test( '.implement( package files )', function ( assert ) { var done = assert.async(), - initJsRan = false; + initJsRan = false, + counter = 41; mw.loader.implement( 'test.implement.packageFiles', { @@ -570,9 +571,8 @@ files: { 'resources/src/foo/data/hello.json': { hello: 'world' }, 'resources/src/foo/foo.js': function ( require, module ) { - window.mwTestFooJsCounter = window.mwTestFooJsCounter || 41; - window.mwTestFooJsCounter++; - module.exports = { answer: window.mwTestFooJsCounter }; + counter++; + module.exports = { answer: counter }; }, 'resources/src/bar/bar.js': function ( require, module ) { var core = require( './core.js' ); -- 2.20.1