From efa078493fe24550c924391d65d28c369f0c6074 Mon Sep 17 00:00:00 2001
From: Krinkle <krinkle@users.mediawiki.org>
Date: Fri, 12 Aug 2011 16:02:03 +0000
Subject: [PATCH] Renaming qunit test files to end in ".test.js" (finally!)

* There shouldn't be two files with the same name in core, especially not the module and the test suite. Previously postponed due to compatibility with our TestSwarm script, that has been fixed now.

* Had to modify the files in the same commit since the module name is referenced inside the test suite in the module() call, which is the hint for QUnit when filtering is done through the ?filter= parameter.

* As has been added to the conventions, there must be only one module() call per test suite file and it MUST match "filename without .test.js". Otherwise the module will not be submitted to TestSwarm (which glob()'s at the /suites/ directory and extracts test-suite-module-names from the filenames).
---
 tests/qunit/index.html                        | 30 +++++++++----------
 ...llipsis.js => jquery.autoEllipsis.test.js} |  2 +-
 ...yteLength.js => jquery.byteLength.test.js} |  2 +-
 ....byteLimit.js => jquery.byteLimit.test.js} |  2 +-
 ...jquery.client.js => jquery.client.test.js} |  2 +-
 ....colorUtil.js => jquery.colorUtil.test.js} |  2 +-
 ...ry.getAttrs.js => jquery.getAttrs.test.js} |  2 +-
 ...ry.localize.js => jquery.localize.test.js} |  2 +-
 ...xtension.js => jquery.mwExtension.test.js} |  2 +-
 ...ry.tabIndex.js => jquery.tabIndex.test.js} |  2 +-
 .../jquery/jquery.tablesorter.test.js         |  2 +-
 ...ection.js => jquery.textSelection.test.js} |  2 +-
 ...> mediawiki.special.recentchanges.test.js} |  2 +-
 .../{mediawiki.js => mediawiki.test.js}       |  2 +-
 ...awiki.title.js => mediawiki.title.test.js} |  2 +-
 ...diawiki.user.js => mediawiki.user.test.js} |  2 +-
 ...diawiki.util.js => mediawiki.util.test.js} |  2 +-
 17 files changed, 31 insertions(+), 31 deletions(-)
 rename tests/qunit/suites/resources/jquery/{jquery.autoEllipsis.js => jquery.autoEllipsis.test.js} (98%)
 rename tests/qunit/suites/resources/jquery/{jquery.byteLength.js => jquery.byteLength.test.js} (97%)
 rename tests/qunit/suites/resources/jquery/{jquery.byteLimit.js => jquery.byteLimit.test.js} (99%)
 rename tests/qunit/suites/resources/jquery/{jquery.client.js => jquery.client.test.js} (99%)
 rename tests/qunit/suites/resources/jquery/{jquery.colorUtil.js => jquery.colorUtil.test.js} (98%)
 rename tests/qunit/suites/resources/jquery/{jquery.getAttrs.js => jquery.getAttrs.test.js} (91%)
 rename tests/qunit/suites/resources/jquery/{jquery.localize.js => jquery.localize.test.js} (99%)
 rename tests/qunit/suites/resources/jquery/{jquery.mwExtension.js => jquery.mwExtension.test.js} (98%)
 rename tests/qunit/suites/resources/jquery/{jquery.tabIndex.js => jquery.tabIndex.test.js} (97%)
 rename tests/qunit/suites/resources/jquery/{jquery.textSelection.js => jquery.textSelection.test.js} (99%)
 rename tests/qunit/suites/resources/mediawiki.special/{mediawiki.special.recentchanges.js => mediawiki.special.recentchanges.test.js} (98%)
 rename tests/qunit/suites/resources/mediawiki/{mediawiki.js => mediawiki.test.js} (99%)
 rename tests/qunit/suites/resources/mediawiki/{mediawiki.title.js => mediawiki.title.test.js} (99%)
 rename tests/qunit/suites/resources/mediawiki/{mediawiki.user.js => mediawiki.user.test.js} (95%)
 rename tests/qunit/suites/resources/mediawiki/{mediawiki.util.js => mediawiki.util.test.js} (99%)

diff --git a/tests/qunit/index.html b/tests/qunit/index.html
index 7eb8fba59a..e8bb99a7d6 100644
--- a/tests/qunit/index.html
+++ b/tests/qunit/index.html
@@ -59,24 +59,24 @@
 	<script src="data/testrunner.js"></script>
 
 	<!-- QUnit: Load test suites (maintain the same order as above please) -->
-	<script src="suites/resources/mediawiki/mediawiki.js"></script>
-	<script src="suites/resources/mediawiki/mediawiki.user.js"></script>
+	<script src="suites/resources/mediawiki/mediawiki.test.js"></script>
+	<script src="suites/resources/mediawiki/mediawiki.user.test.js"></script>
 
-	<script src="suites/resources/jquery/jquery.client.js"></script>
-	<script src="suites/resources/jquery/jquery.mwExtension.js"></script>
-	<script src="suites/resources/mediawiki/mediawiki.util.js"></script>
+	<script src="suites/resources/jquery/jquery.client.test.js"></script>
+	<script src="suites/resources/jquery/jquery.mwExtension.test.js"></script>
+	<script src="suites/resources/mediawiki/mediawiki.util.test.js"></script>
 
-	<script src="suites/resources/jquery/jquery.autoEllipsis.js"></script>
-	<script src="suites/resources/jquery/jquery.byteLength.js"></script>
-	<script src="suites/resources/jquery/jquery.byteLimit.js"></script>
-	<script src="suites/resources/jquery/jquery.colorUtil.js"></script>
-	<script src="suites/resources/jquery/jquery.getAttrs.js"></script>
-	<script src="suites/resources/jquery/jquery.localize.js"></script>
-	<script src="suites/resources/jquery/jquery.tabIndex.js"></script>
+	<script src="suites/resources/jquery/jquery.autoEllipsis.test.js"></script>
+	<script src="suites/resources/jquery/jquery.byteLength.test.js"></script>
+	<script src="suites/resources/jquery/jquery.byteLimit.test.js"></script>
+	<script src="suites/resources/jquery/jquery.colorUtil.test.js"></script>
+	<script src="suites/resources/jquery/jquery.getAttrs.test.js"></script>
+	<script src="suites/resources/jquery/jquery.localize.test.js"></script>
+	<script src="suites/resources/jquery/jquery.tabIndex.test.js"></script>
 	<script src="suites/resources/jquery/jquery.tablesorter.test.js" charset="UTF-8"></script>
-	<script src="suites/resources/jquery/jquery.textSelection.js" charset="UTF-8"></script>
-	<script src="suites/resources/mediawiki/mediawiki.title.js"></script>
-	<script src="suites/resources/mediawiki.special/mediawiki.special.recentchanges.js"></script>
+	<script src="suites/resources/jquery/jquery.textSelection.test.js" charset="UTF-8"></script>
+	<script src="suites/resources/mediawiki/mediawiki.title.test.js"></script>
+	<script src="suites/resources/mediawiki.special/mediawiki.special.recentchanges.test.js"></script>
 </head>
 <body>
 	<h1 id="qunit-header">MediaWiki JavaScript Test Suite</h1>
diff --git a/tests/qunit/suites/resources/jquery/jquery.autoEllipsis.js b/tests/qunit/suites/resources/jquery/jquery.autoEllipsis.test.js
similarity index 98%
rename from tests/qunit/suites/resources/jquery/jquery.autoEllipsis.js
rename to tests/qunit/suites/resources/jquery/jquery.autoEllipsis.test.js
index caf5a6f1f1..690ffb21e8 100644
--- a/tests/qunit/suites/resources/jquery/jquery.autoEllipsis.js
+++ b/tests/qunit/suites/resources/jquery/jquery.autoEllipsis.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.autoEllipsis.js' );
+module( 'jquery.autoEllipsis' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/jquery/jquery.byteLength.js b/tests/qunit/suites/resources/jquery/jquery.byteLength.test.js
similarity index 97%
rename from tests/qunit/suites/resources/jquery/jquery.byteLength.js
rename to tests/qunit/suites/resources/jquery/jquery.byteLength.test.js
index f82fda27b3..27913c6396 100644
--- a/tests/qunit/suites/resources/jquery/jquery.byteLength.js
+++ b/tests/qunit/suites/resources/jquery/jquery.byteLength.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.byteLength.js' );
+module( 'jquery.byteLength' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/jquery/jquery.byteLimit.js b/tests/qunit/suites/resources/jquery/jquery.byteLimit.test.js
similarity index 99%
rename from tests/qunit/suites/resources/jquery/jquery.byteLimit.js
rename to tests/qunit/suites/resources/jquery/jquery.byteLimit.test.js
index 1d8ca9ba26..70343fd67a 100644
--- a/tests/qunit/suites/resources/jquery/jquery.byteLimit.js
+++ b/tests/qunit/suites/resources/jquery/jquery.byteLimit.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.byteLimit.js' );
+module( 'jquery.byteLimit' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/jquery/jquery.client.js b/tests/qunit/suites/resources/jquery/jquery.client.test.js
similarity index 99%
rename from tests/qunit/suites/resources/jquery/jquery.client.js
rename to tests/qunit/suites/resources/jquery/jquery.client.test.js
index 50df2928cd..116a1f0b11 100644
--- a/tests/qunit/suites/resources/jquery/jquery.client.js
+++ b/tests/qunit/suites/resources/jquery/jquery.client.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.client.js' );
+module( 'jquery.client' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/jquery/jquery.colorUtil.js b/tests/qunit/suites/resources/jquery/jquery.colorUtil.test.js
similarity index 98%
rename from tests/qunit/suites/resources/jquery/jquery.colorUtil.js
rename to tests/qunit/suites/resources/jquery/jquery.colorUtil.test.js
index 93f12b8295..88791cada0 100644
--- a/tests/qunit/suites/resources/jquery/jquery.colorUtil.js
+++ b/tests/qunit/suites/resources/jquery/jquery.colorUtil.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.colorUtil.js' );
+module( 'jquery.colorUtil' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/jquery/jquery.getAttrs.js b/tests/qunit/suites/resources/jquery/jquery.getAttrs.test.js
similarity index 91%
rename from tests/qunit/suites/resources/jquery/jquery.getAttrs.js
rename to tests/qunit/suites/resources/jquery/jquery.getAttrs.test.js
index 3d3d01e137..5685756f3d 100644
--- a/tests/qunit/suites/resources/jquery/jquery.getAttrs.js
+++ b/tests/qunit/suites/resources/jquery/jquery.getAttrs.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.getAttrs.js' );
+module( 'jquery.getAttrs' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/jquery/jquery.localize.js b/tests/qunit/suites/resources/jquery/jquery.localize.test.js
similarity index 99%
rename from tests/qunit/suites/resources/jquery/jquery.localize.js
rename to tests/qunit/suites/resources/jquery/jquery.localize.test.js
index 40b5868713..c5d8c4ddda 100644
--- a/tests/qunit/suites/resources/jquery/jquery.localize.js
+++ b/tests/qunit/suites/resources/jquery/jquery.localize.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.localize.js' );
+module( 'jquery.localize' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/jquery/jquery.mwExtension.js b/tests/qunit/suites/resources/jquery/jquery.mwExtension.test.js
similarity index 98%
rename from tests/qunit/suites/resources/jquery/jquery.mwExtension.js
rename to tests/qunit/suites/resources/jquery/jquery.mwExtension.test.js
index f68bd582e3..09e981e4b3 100644
--- a/tests/qunit/suites/resources/jquery/jquery.mwExtension.js
+++ b/tests/qunit/suites/resources/jquery/jquery.mwExtension.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.mwExtension.js' );
+module( 'jquery.mwExtension' );
 
 test( 'String functions', function() {
 
diff --git a/tests/qunit/suites/resources/jquery/jquery.tabIndex.js b/tests/qunit/suites/resources/jquery/jquery.tabIndex.test.js
similarity index 97%
rename from tests/qunit/suites/resources/jquery/jquery.tabIndex.js
rename to tests/qunit/suites/resources/jquery/jquery.tabIndex.test.js
index 1ff81e58e5..7d014f38e4 100644
--- a/tests/qunit/suites/resources/jquery/jquery.tabIndex.js
+++ b/tests/qunit/suites/resources/jquery/jquery.tabIndex.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.tabIndex.js' );
+module( 'jquery.tabIndex' );
 
 test( '-- Initial check', function() {
 	expect(2);
diff --git a/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js b/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js
index b79843290c..2efd6cd9a9 100644
--- a/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js
+++ b/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js
@@ -1,6 +1,6 @@
 (function() {
 
-module( 'jquery.tablesorter.test.js' );
+module( 'jquery.tablesorter' );
 
 // setup hack
 mw.config.set('wgMonthNames', window.wgMonthNames = ['', 'January', 'February', 'March', 'April', 'May', 'June', 'July', 'August', 'September', 'October', 'November', 'December']);
diff --git a/tests/qunit/suites/resources/jquery/jquery.textSelection.js b/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js
similarity index 99%
rename from tests/qunit/suites/resources/jquery/jquery.textSelection.js
rename to tests/qunit/suites/resources/jquery/jquery.textSelection.test.js
index c6848863f8..cea91c85c2 100644
--- a/tests/qunit/suites/resources/jquery/jquery.textSelection.js
+++ b/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js
@@ -1,4 +1,4 @@
-module( 'jquery.textSelection.js' );
+module( 'jquery.textSelection' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.js b/tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.test.js
similarity index 98%
rename from tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.js
rename to tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.test.js
index bcc9b96bbf..fca2a80b11 100644
--- a/tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.js
+++ b/tests/qunit/suites/resources/mediawiki.special/mediawiki.special.recentchanges.test.js
@@ -1,4 +1,4 @@
-module( 'mediawiki.special.recentchanges.js' );
+module( 'mediawiki.special.recentchanges' );
 
 test( '-- Initial check', function() {
 	expect( 2 );
diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.js b/tests/qunit/suites/resources/mediawiki/mediawiki.test.js
similarity index 99%
rename from tests/qunit/suites/resources/mediawiki/mediawiki.js
rename to tests/qunit/suites/resources/mediawiki/mediawiki.test.js
index 15b8377d05..4a8778295b 100644
--- a/tests/qunit/suites/resources/mediawiki/mediawiki.js
+++ b/tests/qunit/suites/resources/mediawiki/mediawiki.test.js
@@ -1,4 +1,4 @@
-module( 'mediawiki.js' );
+module( 'mediawiki' );
 
 test( '-- Initial check', function() {
 	expect(8);
diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.title.js b/tests/qunit/suites/resources/mediawiki/mediawiki.title.test.js
similarity index 99%
rename from tests/qunit/suites/resources/mediawiki/mediawiki.title.js
rename to tests/qunit/suites/resources/mediawiki/mediawiki.title.test.js
index c0c0dc0b9e..9d1c517402 100644
--- a/tests/qunit/suites/resources/mediawiki/mediawiki.title.js
+++ b/tests/qunit/suites/resources/mediawiki/mediawiki.title.test.js
@@ -1,4 +1,4 @@
-module( 'mediawiki.title.js' );
+module( 'mediawiki.title' );
 
 // mw.Title relies on these three config vars
 // Restore them after each test run
diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.user.js b/tests/qunit/suites/resources/mediawiki/mediawiki.user.test.js
similarity index 95%
rename from tests/qunit/suites/resources/mediawiki/mediawiki.user.js
rename to tests/qunit/suites/resources/mediawiki/mediawiki.user.test.js
index d5c6baad3c..1aa46989fd 100644
--- a/tests/qunit/suites/resources/mediawiki/mediawiki.user.js
+++ b/tests/qunit/suites/resources/mediawiki/mediawiki.user.test.js
@@ -1,4 +1,4 @@
-module( 'mediawiki.user.js' );
+module( 'mediawiki.user' );
 
 test( '-- Initial check', function() {
 	expect(1);
diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.util.js b/tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js
similarity index 99%
rename from tests/qunit/suites/resources/mediawiki/mediawiki.util.js
rename to tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js
index 4dfb0dcb35..db78f7c5d9 100644
--- a/tests/qunit/suites/resources/mediawiki/mediawiki.util.js
+++ b/tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js
@@ -1,4 +1,4 @@
-module( 'mediawiki.util.js' );
+module( 'mediawiki.util' );
 
 test( '-- Initial check', function() {
 	expect(1);
-- 
2.20.1