From 43f89ebdb459cf12d4edfc548207286b12c98724 Mon Sep 17 00:00:00 2001 From: Timo Tijhof Date: Sat, 25 Apr 2015 02:09:07 +0100 Subject: [PATCH] qunit: Remove mwlogenv feature I added this in the JSTesting branch back in SVN (r107521, merged in r107919 for MediaWiki 1.19.0). It was mostly for debugging the setup/teardown in the branch. It never had much practical use. Change-Id: I8d6eb9e06ad070c23cb9c9041b5cf4143ee6ac0b --- tests/qunit/data/testrunner.js | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/tests/qunit/data/testrunner.js b/tests/qunit/data/testrunner.js index 3dd2af69bc..7c3d6991a6 100644 --- a/tests/qunit/data/testrunner.js +++ b/tests/qunit/data/testrunner.js @@ -164,13 +164,6 @@ mwTester = new CompletenessTest( mw, mwTestIgnore ); } - /** - * Test environment recommended for all QUnit test modules - * - * Whether to log environment changes to the console - */ - QUnit.config.urlConfig.push( 'mwlogenv' ); - /** * Reset mw.config and others to a fresh copy of the live config for each test(), * and restore it back to the live one afterwards. @@ -179,7 +172,7 @@ * */ QUnit.newMwEnvironment = ( function () { - var warn, error, log, liveConfig, liveMessages, + var warn, error, liveConfig, liveMessages, ajaxRequests = []; liveConfig = mw.config.values; @@ -230,8 +223,6 @@ ajaxRequests.push( { xhr: jqXHR, options: ajaxOptions } ); } - log = QUnit.urlParams.mwlogenv ? mw.log : function () {}; - return function ( localEnv ) { localEnv = $.extend( { // QUnit @@ -244,8 +235,6 @@ return { setup: function () { - log( 'MwEnvironment> SETUP for "' + QUnit.config.current.module - + ': ' + QUnit.config.current.testName + '"' ); // Greetings, mock environment! mw.config.values = freshConfigCopy( localEnv.config ); @@ -261,8 +250,6 @@ teardown: function () { var timers, active; - log( 'MwEnvironment> TEARDOWN for "' + QUnit.config.current.module - + ': ' + QUnit.config.current.testName + '"' ); localEnv.teardown.call( this ); -- 2.20.1