From 1e9c361960587b889911f6239026f6358c880845 Mon Sep 17 00:00:00 2001 From: "James D. Forrester" Date: Mon, 20 Feb 2017 15:45:58 -0800 Subject: [PATCH] tests: Replace implicit Bugzilla bug numbers with Phab ones It's unreasonable to expect newbies to know that "bug 12345" means "Task T14345" except where it doesn't, so let's just standardise on the real numbers. Change-Id: I46261416f7603558dceb76ebe695a5cac274e417 --- tests/common/TestSetup.php | 2 +- tests/parser/ParserTestRunner.php | 6 +-- tests/parser/README | 2 +- tests/phpunit/includes/BlockTest.php | 4 +- tests/phpunit/includes/EditPageTest.php | 4 +- tests/phpunit/includes/ExtraParserTest.php | 2 +- .../GlobalFunctions/wfTimestampTest.php | 4 +- tests/phpunit/includes/MessageTest.php | 2 +- tests/phpunit/includes/PreferencesTest.php | 6 +-- tests/phpunit/includes/PrefixSearchTest.php | 4 +- tests/phpunit/includes/SanitizerTest.php | 4 +- .../includes/SanitizerValidateEmailTest.php | 2 +- tests/phpunit/includes/TitleTest.php | 2 +- tests/phpunit/includes/XmlTest.php | 2 +- .../phpunit/includes/api/ApiEditPageTest.php | 4 +- .../includes/api/ApiQueryAllPagesTest.php | 2 +- .../includes/api/query/ApiQueryBasicTest.php | 2 +- .../includes/cache/LocalisationCacheTest.php | 2 +- .../includes/cache/MessageCacheTest.php | 2 +- .../includes/db/DatabaseMysqlBaseTest.php | 2 +- .../includes/deferred/SearchUpdateTest.php | 4 +- tests/phpunit/includes/http/HttpTest.php | 6 +-- tests/phpunit/includes/libs/CSSMinTest.php | 8 ++-- .../includes/libs/JavaScriptMinifierTest.php | 6 +-- .../includes/libs/mime/MimeAnalyzerTest.php | 2 +- tests/phpunit/includes/libs/xmp/XMPTest.php | 2 +- .../includes/logging/LogFormatterTest.php | 2 +- .../includes/media/FormatMetadataTest.php | 2 +- .../media/SVGMetadataExtractorTest.php | 2 +- .../includes/page/ArticleTablesTest.php | 8 ++-- .../includes/parser/ParserPreloadTest.php | 4 +- .../includes/parser/PreprocessorTest.php | 2 +- .../search/SearchEnginePrefixTest.php | 4 +- .../specials/SpecialBooksourcesTest.php | 2 +- .../specials/SpecialRecentchangesTest.php | 4 +- .../includes/specials/SpecialSearchTest.php | 4 +- .../includes/upload/UploadBaseTest.php | 18 ++++----- .../includes/upload/UploadStashTest.php | 4 +- tests/phpunit/includes/user/UserTest.php | 2 +- tests/phpunit/languages/LanguageTest.php | 2 +- .../classes/LanguageBe_taraskTest.php | 8 ++-- .../languages/classes/LanguageMlTest.php | 2 +- .../languages/classes/LanguageTrTest.php | 4 +- .../maintenance/backupPrefetchTest.php | 2 +- .../phpunit/mocks/filebackend/MockFSFile.php | 2 +- tests/phpunit/skins/SideBarTest.php | 2 +- .../resources/jquery/jquery.byteLimit.test.js | 6 +-- .../jquery/jquery.tablesorter.parsers.test.js | 2 +- .../jquery/jquery.tablesorter.test.js | 38 +++++++++---------- .../jquery/jquery.textSelection.test.js | 4 +- .../mediawiki.api/mediawiki.api.test.js | 2 +- .../mediawiki/mediawiki.Title.test.js | 2 +- .../resources/mediawiki/mediawiki.Uri.test.js | 4 +- .../mediawiki/mediawiki.jscompat.test.js | 2 +- .../mediawiki/mediawiki.loader.test.js | 14 +++---- .../mediawiki/mediawiki.util.test.js | 8 ++-- 56 files changed, 124 insertions(+), 126 deletions(-) diff --git a/tests/common/TestSetup.php b/tests/common/TestSetup.php index 38f56d20a1..3733e60f2b 100644 --- a/tests/common/TestSetup.php +++ b/tests/common/TestSetup.php @@ -89,7 +89,7 @@ class TestSetup { ]; $wgAuth = new MediaWiki\Auth\AuthManagerAuthPlugin(); - // Bug 44192 Do not attempt to send a real e-mail + // T46192 Do not attempt to send a real e-mail Hooks::clear( 'AlternateUserMailer' ); Hooks::register( 'AlternateUserMailer', diff --git a/tests/parser/ParserTestRunner.php b/tests/parser/ParserTestRunner.php index 97e24b65f9..7edde2ae4b 100644 --- a/tests/parser/ParserTestRunner.php +++ b/tests/parser/ParserTestRunner.php @@ -295,7 +295,7 @@ class ParserTestRunner { MediaWikiServices::getInstance()->resetServiceForTesting( 'MediaHandlerFactory' ); }; - // SqlBagOStuff broke when using temporary tables on r40209 (bug 15892). + // SqlBagOStuff broke when using temporary tables on r40209 (T17892). // It seems to have been fixed since (r55079?), but regressed at some point before r85701. // This works around it for now... global $wgObjectCaches; @@ -316,7 +316,7 @@ class ParserTestRunner { private function appendNamespaceSetup( &$setup, &$teardown ) { // Add a namespace shadowing a interwiki link, to test - // proper precedence when resolving links. (bug 51680) + // proper precedence when resolving links. (T53680) $setup['wgExtraNamespaces'] = [ 100 => 'MemoryAlpha', 101 => 'MemoryAlpha_talk' @@ -1321,7 +1321,7 @@ class ParserTestRunner { if ( $this->useTemporaryTables ) { if ( $this->db->getType() == 'sqlite' ) { # Under SQLite the searchindex table is virtual and need - # to be explicitly destroyed. See bug 29912 + # to be explicitly destroyed. See T31912 # See also MediaWikiTestCase::destroyDB() wfDebug( __METHOD__ . " explicitly destroying sqlite virtual table parsertest_searchindex\n" ); $this->db->query( "DROP TABLE `parsertest_searchindex`" ); diff --git a/tests/parser/README b/tests/parser/README index f1a82ee645..a62db5ac01 100644 --- a/tests/parser/README +++ b/tests/parser/README @@ -9,4 +9,4 @@ To run parser tests via PHPUnit: You can optionally filter by title using --filter, e.g. - ./phpunit.php --testsuite parsertests --filter="Bug 6200" + ./phpunit.php --testsuite parsertests --filter="T6400" diff --git a/tests/phpunit/includes/BlockTest.php b/tests/phpunit/includes/BlockTest.php index 12b277e94f..63d05a0653 100644 --- a/tests/phpunit/includes/BlockTest.php +++ b/tests/phpunit/includes/BlockTest.php @@ -84,7 +84,7 @@ class BlockTest extends MediaWikiLangTestCase { } /** - * per bug 26425 + * per T28425 */ public function testBug26425BlockTimestampDefaultsToTime() { // delta to stop one-off errors when things happen to go over a second mark. @@ -99,7 +99,7 @@ class BlockTest extends MediaWikiLangTestCase { /** * CheckUser since being changed to use Block::newFromTarget started failing * because the new function didn't accept empty strings like Block::load() - * had. Regression bug 29116. + * had. Regression T31116. * * @dataProvider provideBug29116Data * @covers Block::newFromTarget diff --git a/tests/phpunit/includes/EditPageTest.php b/tests/phpunit/includes/EditPageTest.php index 5a01dc006f..9507811ec2 100644 --- a/tests/phpunit/includes/EditPageTest.php +++ b/tests/phpunit/includes/EditPageTest.php @@ -53,7 +53,7 @@ class EditPageTest extends MediaWikiLangTestCase { false ], [ - "An initial section with a fake heder (bug 32617)\n\n== Test == ??\nwtf", + "An initial section with a fake heder (T34617)\n\n== Test == ??\nwtf", false ], [ @@ -61,7 +61,7 @@ class EditPageTest extends MediaWikiLangTestCase { "Section" ], [ - "== Section== \t\r\n followed by whitespace (bug 35051)", + "== Section== \t\r\n followed by whitespace (T37051)", 'Section', ], ]; diff --git a/tests/phpunit/includes/ExtraParserTest.php b/tests/phpunit/includes/ExtraParserTest.php index 5dd4f7a07c..4e95a30e09 100644 --- a/tests/phpunit/includes/ExtraParserTest.php +++ b/tests/phpunit/includes/ExtraParserTest.php @@ -32,7 +32,7 @@ class ExtraParserTest extends MediaWikiTestCase { } /** - * @see Bug 8689 + * @see T10689 * @covers Parser::parse */ public function testLongNumericLinesDontKillTheParser() { diff --git a/tests/phpunit/includes/GlobalFunctions/wfTimestampTest.php b/tests/phpunit/includes/GlobalFunctions/wfTimestampTest.php index 5ca373d19f..b6682f7274 100644 --- a/tests/phpunit/includes/GlobalFunctions/wfTimestampTest.php +++ b/tests/phpunit/includes/GlobalFunctions/wfTimestampTest.php @@ -87,13 +87,13 @@ class WfTimestampTest extends MediaWikiTestCase { /** * This test checks wfTimestamp() with values outside. * It needs PHP 64 bits or PHP > 5.1. - * See r74778 and bug 25451 + * See r74778 and T27451 * @dataProvider provideOldTimestamps */ public function testOldTimestamps( $input, $outputType, $output, $message ) { $timestamp = wfTimestamp( $outputType, $input ); if ( substr( $output, 0, 1 ) === '/' ) { - // Bug 64946: Day of the week calculations for very old + // T66946: Day of the week calculations for very old // timestamps varies from system to system. $this->assertRegExp( $output, $timestamp, $message ); } else { diff --git a/tests/phpunit/includes/MessageTest.php b/tests/phpunit/includes/MessageTest.php index 1de9c23a36..2694b850a0 100644 --- a/tests/phpunit/includes/MessageTest.php +++ b/tests/phpunit/includes/MessageTest.php @@ -448,7 +448,7 @@ class MessageTest extends MediaWikiLangTestCase { } /** - * FIXME: This should not need database, but Language#formatExpiry does (bug 55912) + * FIXME: This should not need database, but Language#formatExpiry does (T57912) * @group Database * @covers Message::expiryParam * @covers Message::expiryParams diff --git a/tests/phpunit/includes/PreferencesTest.php b/tests/phpunit/includes/PreferencesTest.php index 289853de57..90b6396e28 100644 --- a/tests/phpunit/includes/PreferencesTest.php +++ b/tests/phpunit/includes/PreferencesTest.php @@ -42,7 +42,7 @@ class PreferencesTest extends MediaWikiTestCase { } /** - * Placeholder to verify bug 34302 + * Placeholder to verify T36302 * @covers Preferences::profilePreferences */ public function testEmailAuthenticationFieldWhenUserHasNoEmail() { @@ -54,7 +54,7 @@ class PreferencesTest extends MediaWikiTestCase { } /** - * Placeholder to verify bug 34302 + * Placeholder to verify T36302 * @covers Preferences::profilePreferences */ public function testEmailAuthenticationFieldWhenUserEmailNotAuthenticated() { @@ -66,7 +66,7 @@ class PreferencesTest extends MediaWikiTestCase { } /** - * Placeholder to verify bug 34302 + * Placeholder to verify T36302 * @covers Preferences::profilePreferences */ public function testEmailAuthenticationFieldWhenUserEmailIsAuthenticated() { diff --git a/tests/phpunit/includes/PrefixSearchTest.php b/tests/phpunit/includes/PrefixSearchTest.php index c5a7e04e30..cf2e1205be 100644 --- a/tests/phpunit/includes/PrefixSearchTest.php +++ b/tests/phpunit/includes/PrefixSearchTest.php @@ -260,7 +260,7 @@ class PrefixSearchTest extends MediaWikiLangTestCase { ], ] ], [ [ - 'Exact match not on top (bug 70958)', + 'Exact match not on top (T72958)', 'provision' => [ 'Barcelona', 'Bar', @@ -274,7 +274,7 @@ class PrefixSearchTest extends MediaWikiLangTestCase { ], ] ], [ [ - 'Exact match missing (bug 70958)', + 'Exact match missing (T72958)', 'provision' => [ 'Barcelona', 'Barbara', diff --git a/tests/phpunit/includes/SanitizerTest.php b/tests/phpunit/includes/SanitizerTest.php index 12db1a198f..862b7d07d5 100644 --- a/tests/phpunit/includes/SanitizerTest.php +++ b/tests/phpunit/includes/SanitizerTest.php @@ -211,7 +211,7 @@ class SanitizerTest extends MediaWikiTestCase { [ [ 'foo6' => 'baz' ], 'foo6=baz', 'Numbers are allowed' ], # This bit is more relaxed than XML rules, but some extensions use - # it, like ProofreadPage (see bug 27539) + # it, like ProofreadPage (see T29539) [ [ '1foo' => 'baz' ], '1foo=baz', 'Leading numbers are allowed' ], [ [], 'foo$=baz', 'Symbols are not allowed' ], [ [], 'foo@=baz', 'Symbols are not allowed' ], @@ -286,7 +286,7 @@ class SanitizerTest extends MediaWikiTestCase { [ ' ', '/* /* */' ], [ 'display: block;', "display:/* foo */block;" ], [ 'display: block;', "display:\\2f\\2a foo \\2a\\2f block;", - 'Backslash-escaped comments must be stripped (bug 28450)' ], + 'Backslash-escaped comments must be stripped (T30450)' ], [ '', '/* unfinished comment structure', 'Remove anything after a comment-start token' ], [ '', "\\2f\\2a unifinished comment'", diff --git a/tests/phpunit/includes/SanitizerValidateEmailTest.php b/tests/phpunit/includes/SanitizerValidateEmailTest.php index f47e74e287..2448513345 100644 --- a/tests/phpunit/includes/SanitizerValidateEmailTest.php +++ b/tests/phpunit/includes/SanitizerValidateEmailTest.php @@ -64,7 +64,7 @@ class SanitizerValidateEmailTest extends PHPUnit_Framework_TestCase { } /** - * bug 26948 : comma were matched by an incorrect regexp range + * T28948 : comma were matched by an incorrect regexp range */ public function testEmailWithCommasAreInvalids() { $this->invalid( "user,foo@example.org" ); diff --git a/tests/phpunit/includes/TitleTest.php b/tests/phpunit/includes/TitleTest.php index 7925c6f8f9..238b65f429 100644 --- a/tests/phpunit/includes/TitleTest.php +++ b/tests/phpunit/includes/TitleTest.php @@ -267,7 +267,7 @@ class TitleTest extends MediaWikiTestCase { $this->assertEquals( $expectedParam, $par, - "Bug 31100 regression check: Title->fixSpecialName() should preserve parameter" + "T33100 regression check: Title->fixSpecialName() should preserve parameter" ); } diff --git a/tests/phpunit/includes/XmlTest.php b/tests/phpunit/includes/XmlTest.php index 18ff1f4b11..184dd438d2 100644 --- a/tests/phpunit/includes/XmlTest.php +++ b/tests/phpunit/includes/XmlTest.php @@ -83,7 +83,7 @@ class XmlTest extends MediaWikiTestCase { $this->assertEquals( '', Xml::input( 'name', false, 0 ), - 'Input with a value of 0 (bug 23797)' + 'Input with a value of 0 (T25797)' ); } diff --git a/tests/phpunit/includes/api/ApiEditPageTest.php b/tests/phpunit/includes/api/ApiEditPageTest.php index 0ffcbca762..e091153281 100644 --- a/tests/phpunit/includes/api/ApiEditPageTest.php +++ b/tests/phpunit/includes/api/ApiEditPageTest.php @@ -204,7 +204,7 @@ class ApiEditPageTest extends ApiTestCase { /** * Test action=edit§ion=new * Run it twice so we test adding a new section on a - * page that doesn't exist (bug 52830) and one that + * page that doesn't exist (T54830) and one that * does exist */ public function testEditNewSection() { @@ -416,7 +416,7 @@ class ApiEditPageTest extends ApiTestCase { $count++; /* - * bug 41990: if the target page has a newer revision than the redirect, then editing the + * T43990: if the target page has a newer revision than the redirect, then editing the * redirect while specifying 'redirect' and *not* specifying 'basetimestamp' erroneously * caused an edit conflict to be detected. */ diff --git a/tests/phpunit/includes/api/ApiQueryAllPagesTest.php b/tests/phpunit/includes/api/ApiQueryAllPagesTest.php index 76872362cd..9f28aaf56e 100644 --- a/tests/phpunit/includes/api/ApiQueryAllPagesTest.php +++ b/tests/phpunit/includes/api/ApiQueryAllPagesTest.php @@ -13,7 +13,7 @@ class ApiQueryAllPagesTest extends ApiTestCase { } /** - *Test bug 25702 + *Test T27702 *Prefixes of API search requests are not handled with case sensitivity and may result *in wrong search results */ diff --git a/tests/phpunit/includes/api/query/ApiQueryBasicTest.php b/tests/phpunit/includes/api/query/ApiQueryBasicTest.php index e5971b4418..fbc794ed12 100644 --- a/tests/phpunit/includes/api/query/ApiQueryBasicTest.php +++ b/tests/phpunit/includes/api/query/ApiQueryBasicTest.php @@ -325,7 +325,7 @@ class ApiQueryBasicTest extends ApiQueryTestBase { } /** - * Test bug 51821 + * Test T53821 */ public function testGeneratorRedirects() { $this->editPage( 'AQBT-Target', 'test' ); diff --git a/tests/phpunit/includes/cache/LocalisationCacheTest.php b/tests/phpunit/includes/cache/LocalisationCacheTest.php index ed821530a2..5eed01cb2d 100644 --- a/tests/phpunit/includes/cache/LocalisationCacheTest.php +++ b/tests/phpunit/includes/cache/LocalisationCacheTest.php @@ -75,7 +75,7 @@ class LocalisationCacheTest extends MediaWikiTestCase { public function testRecacheFallbacksWithHooks() { // Use hook to provide updates for messages. This is what the - // LocalisationUpdate extension does. See bug 68781. + // LocalisationUpdate extension does. See T70781. $this->mergeMwGlobalArrayValue( 'wgHooks', [ 'LocalisationCacheRecacheFallback' => [ function ( diff --git a/tests/phpunit/includes/cache/MessageCacheTest.php b/tests/phpunit/includes/cache/MessageCacheTest.php index a12c8b2d09..b03eeba295 100644 --- a/tests/phpunit/includes/cache/MessageCacheTest.php +++ b/tests/phpunit/includes/cache/MessageCacheTest.php @@ -92,7 +92,7 @@ class MessageCacheTest extends MediaWikiLangTestCase { // Existing message with customizations on the fallbacks [ 'sunday', 'ab', 'амҽыш' ], - // bug 46579 + // T48579 [ 'FallbackLanguageTest-NoDervContLang', 'de', 'de/none' ], // UI language different from content language should only use de/none as last option [ 'FallbackLanguageTest-NoDervContLang', 'fit', 'de/none' ], diff --git a/tests/phpunit/includes/db/DatabaseMysqlBaseTest.php b/tests/phpunit/includes/db/DatabaseMysqlBaseTest.php index f54170bd4d..b95462abfe 100644 --- a/tests/phpunit/includes/db/DatabaseMysqlBaseTest.php +++ b/tests/phpunit/includes/db/DatabaseMysqlBaseTest.php @@ -120,7 +120,7 @@ class DatabaseMysqlBaseTest extends MediaWikiTestCase { /** * Feeds testAddIdentifierQuotes * - * Named per bug 20281 convention. + * Named per T22281 convention. */ function provideDiapers() { return [ diff --git a/tests/phpunit/includes/deferred/SearchUpdateTest.php b/tests/phpunit/includes/deferred/SearchUpdateTest.php index 602a1757ec..9e4dbea221 100644 --- a/tests/phpunit/includes/deferred/SearchUpdateTest.php +++ b/tests/phpunit/includes/deferred/SearchUpdateTest.php @@ -66,13 +66,13 @@ EOT $this->assertNotEquals( '', $this->updateText( $text ), - 'Bug 18609' + 'T20609' ); } /** * @covers SearchUpdate::updateText - * Test bug 32712 + * Test T34712 * Test if unicode quotes in article links make its search index empty */ public function testUnicodeLinkSearchIndexError() { diff --git a/tests/phpunit/includes/http/HttpTest.php b/tests/phpunit/includes/http/HttpTest.php index 7e98d1c069..a598d13ea9 100644 --- a/tests/phpunit/includes/http/HttpTest.php +++ b/tests/phpunit/includes/http/HttpTest.php @@ -50,7 +50,7 @@ class HttpTest extends MediaWikiTestCase { /** * Test Http::isValidURI() - * @bug 27854 : Http::isValidURI is too lax + * T29854 : Http::isValidURI is too lax * @dataProvider provideURI * @covers Http::isValidURI */ @@ -145,7 +145,7 @@ class HttpTest extends MediaWikiTestCase { * * These tests are for code that makes use of an artifact of how CURL * handles header reporting on redirect pages, and will need to be - * rewritten when bug 29232 is taken care of (high-level handling of + * rewritten when T31232 is taken care of (high-level handling of * HTTP redirects). */ public function testRelativeRedirections() { @@ -483,7 +483,7 @@ class HttpTest extends MediaWikiTestCase { * Added this test based on an issue experienced with HHVM 3.3.0-dev * where it did not define a cURL constant. * - * @bug 70570 + * @T72570 * @dataProvider provideCurlConstants */ public function testCurlConstants( $value ) { diff --git a/tests/phpunit/includes/libs/CSSMinTest.php b/tests/phpunit/includes/libs/CSSMinTest.php index 366714b193..2e5c0bb429 100644 --- a/tests/phpunit/includes/libs/CSSMinTest.php +++ b/tests/phpunit/includes/libs/CSSMinTest.php @@ -54,7 +54,7 @@ class CSSMinTest extends MediaWikiTestCase { [ "foo, bar {\n\tprop: value; /* comment */\n}", "foo,bar{prop:value; }" ], // Keep track of things that aren't as minified as much as they - // could be (bug 35493) + // could be (T37493) [ 'foo { prop: value ;}', 'foo{prop:value }' ], [ 'foo { prop : value; }', 'foo{prop :value}' ], [ 'foo { prop: value ; }', 'foo{prop:value }' ], @@ -105,7 +105,7 @@ class CSSMinTest extends MediaWikiTestCase { 'foo { prop: url(http://example.org/bar.png); }', ], [ - 'With trailing slash on remote (bug 27052)', + 'With trailing slash on remote (T29052)', [ 'foo { prop: url(../bar.png); }', false, 'http://example.org/quux/', false ], 'foo { prop: url(http://example.org/bar.png); }', ], @@ -370,7 +370,7 @@ class CSSMinTest extends MediaWikiTestCase { '{ background: /*asd*/ url(http://localhost/w/something.png); background: /*jkl*/ url(http://localhost/w/something.png); }', ], [ - 'Sanity check for offending line from jquery.ui.theme.css (bug 60077)', + 'Sanity check for offending line from jquery.ui.theme.css (T62077)', '.ui-state-default, .ui-widget-content .ui-state-default, .ui-widget-header .ui-state-default { border: 1px solid #d3d3d3/*{borderColorDefault}*/; background: #e6e6e6/*{bgColorDefault}*/ url(images/ui-bg_glass_75_e6e6e6_1x400.png)/*{bgImgUrlDefault}*/ 50%/*{bgDefaultXPos}*/ 50%/*{bgDefaultYPos}*/ repeat-x/*{bgDefaultRepeat}*/; font-weight: normal/*{fwDefault}*/; color: #555555/*{fcDefault}*/; }', '.ui-state-default, .ui-widget-content .ui-state-default, .ui-widget-header .ui-state-default { border: 1px solid #d3d3d3/*{borderColorDefault}*/; background: #e6e6e6/*{bgColorDefault}*/ url(http://localhost/w/images/ui-bg_glass_75_e6e6e6_1x400.png)/*{bgImgUrlDefault}*/ 50%/*{bgDefaultXPos}*/ 50%/*{bgDefaultYPos}*/ repeat-x/*{bgDefaultRepeat}*/; font-weight: normal/*{fwDefault}*/; color: #555555/*{fcDefault}*/; }', ], @@ -418,7 +418,7 @@ class CSSMinTest extends MediaWikiTestCase { } /** - * Seperated because they are currently broken (bug 35492) + * Seperated because they are currently broken (T37492) * * @group Broken * @dataProvider provideStringCases diff --git a/tests/phpunit/includes/libs/JavaScriptMinifierTest.php b/tests/phpunit/includes/libs/JavaScriptMinifierTest.php index 3787962aff..ca12f6c777 100644 --- a/tests/phpunit/includes/libs/JavaScriptMinifierTest.php +++ b/tests/phpunit/includes/libs/JavaScriptMinifierTest.php @@ -10,7 +10,7 @@ class JavaScriptMinifierTest extends PHPUnit_Framework_TestCase { [ "/* Foo *\n*bar\n*/", "" ], /** - * Slashes used inside block comments (bug 26931). + * Slashes used inside block comments (T28931). * At some point there was a bug that caused this comment to be ended at '* /', * causing /M... to be left as the beginning of a regex. */ @@ -60,7 +60,7 @@ class JavaScriptMinifierTest extends PHPUnit_Framework_TestCase { [ "5.3.\nx;", "5.3.x;" ], // Semicolon insertion between an expression having an inline - // comment after it, and a statement on the next line (bug 27046). + // comment after it, and a statement on the next line (T29046). [ "var a = this //foo bar \n for ( b = 0; c < d; b++ ) {}", "var a=this\nfor(b=0;cassertNotNull( $dateIndex, 'Date entry exists in metadata' ); $this->assertEquals( '0000:01:00 00:02:27', $meta['visible'][$dateIndex]['value'], - 'File with invalid date metadata (bug 29471)' ); + 'File with invalid date metadata (T31471)' ); } /** diff --git a/tests/phpunit/includes/media/SVGMetadataExtractorTest.php b/tests/phpunit/includes/media/SVGMetadataExtractorTest.php index 3049e2f6ef..9bfd5f6196 100644 --- a/tests/phpunit/includes/media/SVGMetadataExtractorTest.php +++ b/tests/phpunit/includes/media/SVGMetadataExtractorTest.php @@ -75,7 +75,7 @@ class SVGMetadataExtractorTest extends MediaWikiTestCase { ], [ "$base/Toll_Texas_1.svg", - // This file triggered bug 31719, needs entity expansion in the xmlns checks + // This file triggered T33719, needs entity expansion in the xmlns checks [ 'width' => 385, 'height' => 385, diff --git a/tests/phpunit/includes/page/ArticleTablesTest.php b/tests/phpunit/includes/page/ArticleTablesTest.php index 3a3b514651..34b25251c1 100644 --- a/tests/phpunit/includes/page/ArticleTablesTest.php +++ b/tests/phpunit/includes/page/ArticleTablesTest.php @@ -5,7 +5,7 @@ */ class ArticleTablesTest extends MediaWikiLangTestCase { /** - * Make sure that bug 14404 doesn't strike again. We don't want + * Make sure that T16404 doesn't strike again. We don't want * templatelinks based on the user language when {{int:}} is used, only the * content language. * @@ -13,7 +13,7 @@ class ArticleTablesTest extends MediaWikiLangTestCase { * @covers Title::getLinksFrom */ public function testTemplatelinksUsesContentLanguage() { - $title = Title::newFromText( 'Bug 14404' ); + $title = Title::newFromText( 'T16404' ); $page = WikiPage::factory( $title ); $user = new User(); $user->mRights = [ 'createpage', 'edit', 'purge' ]; @@ -22,7 +22,7 @@ class ArticleTablesTest extends MediaWikiLangTestCase { $page->doEditContent( new WikitextContent( '{{:{{int:history}}}}' ), - 'Test code for bug 14404', + 'Test code for T16404', 0, false, $user @@ -35,7 +35,7 @@ class ArticleTablesTest extends MediaWikiLangTestCase { // We need an edit, a purge is not enough to regenerate the tables $page->doEditContent( new WikitextContent( '{{:{{int:history}}}}' ), - 'Test code for bug 14404', + 'Test code for T16404', EDIT_UPDATE, false, $user diff --git a/tests/phpunit/includes/parser/ParserPreloadTest.php b/tests/phpunit/includes/parser/ParserPreloadTest.php index d12fee368f..414ab8f9d6 100644 --- a/tests/phpunit/includes/parser/ParserPreloadTest.php +++ b/tests/phpunit/includes/parser/ParserPreloadTest.php @@ -51,7 +51,7 @@ class ParserPreloadTest extends MediaWikiTestCase { $this->assertPreloaded( '
monospaced
', '
monospaced
', - '
 in preloaded text must be unstripped (bug 27467)'
+			'
 in preloaded text must be unstripped (T29467)'
 		);
 	}
 
@@ -62,7 +62,7 @@ class ParserPreloadTest extends MediaWikiTestCase {
 		$this->assertPreloaded(
 			'[[Dummy title]]',
 			'[[Dummy title]]',
-			' in preloaded text must be unstripped (bug 27467)'
+			' in preloaded text must be unstripped (T29467)'
 		);
 	}
 
diff --git a/tests/phpunit/includes/parser/PreprocessorTest.php b/tests/phpunit/includes/parser/PreprocessorTest.php
index c491e6b829..11a21976db 100644
--- a/tests/phpunit/includes/parser/PreprocessorTest.php
+++ b/tests/phpunit/includes/parser/PreprocessorTest.php
@@ -214,7 +214,7 @@ class PreprocessorTest extends MediaWikiTestCase {
 			[ "Factorial" ], # https://en.wikipedia.org/w/index.php?title=Template:Factorial&oldid=98548758 GFDL + CC BY-SA by Polonium
 			[ "All_system_messages" ], # https://tl.wiktionary.org/w/index.php?title=Suleras:All_system_messages&oldid=2765 GPL text generated by MediaWiki
 			[ "Fundraising" ], # https://tl.wiktionary.org/w/index.php?title=MediaWiki:Sitenotice&oldid=5716 GFDL + CC BY-SA, copied there by Sky Harbor.
-			[ "NestedTemplates" ], # bug 27936
+			[ "NestedTemplates" ], # T29936
 		] );
 		// @codingStandardsIgnoreEnd
 	}
diff --git a/tests/phpunit/includes/search/SearchEnginePrefixTest.php b/tests/phpunit/includes/search/SearchEnginePrefixTest.php
index a88264bb78..68338938c3 100644
--- a/tests/phpunit/includes/search/SearchEnginePrefixTest.php
+++ b/tests/phpunit/includes/search/SearchEnginePrefixTest.php
@@ -230,7 +230,7 @@ class SearchEnginePrefixTest extends MediaWikiLangTestCase {
 				],
 			] ],
 			[ [
-				'Exact match not on top (bug 70958)',
+				'Exact match not on top (T72958)',
 				'provision' => [
 					'Barcelona',
 					'Bar',
@@ -244,7 +244,7 @@ class SearchEnginePrefixTest extends MediaWikiLangTestCase {
 				],
 			] ],
 			[ [
-				'Exact match missing (bug 70958)',
+				'Exact match missing (T72958)',
 				'provision' => [
 					'Barcelona',
 					'Barbara',
diff --git a/tests/phpunit/includes/specials/SpecialBooksourcesTest.php b/tests/phpunit/includes/specials/SpecialBooksourcesTest.php
index 074045d79f..9c71261e43 100644
--- a/tests/phpunit/includes/specials/SpecialBooksourcesTest.php
+++ b/tests/phpunit/includes/specials/SpecialBooksourcesTest.php
@@ -14,7 +14,7 @@ class SpecialBooksourcesTest extends SpecialPageTestBase {
 			[ '9780136091817', false ],
 			[ '123456789X', true ],
 
-			// Bug 67021
+			// T69021
 			[ '1413304541', false ],
 			[ '141330454X', false ],
 			[ '1413304540', true ],
diff --git a/tests/phpunit/includes/specials/SpecialRecentchangesTest.php b/tests/phpunit/includes/specials/SpecialRecentchangesTest.php
index ab92aeeb3f..9b0fb6a672 100644
--- a/tests/phpunit/includes/specials/SpecialRecentchangesTest.php
+++ b/tests/phpunit/includes/specials/SpecialRecentchangesTest.php
@@ -101,7 +101,7 @@ class SpecialRecentchangesTest extends MediaWikiTestCase {
 	}
 
 	/**
-	 * @bug 2429
+	 * T4429
 	 * @dataProvider provideNamespacesAssociations
 	 */
 	public function testRcNsFilterAssociation( $ns1, $ns2 ) {
@@ -120,7 +120,7 @@ class SpecialRecentchangesTest extends MediaWikiTestCase {
 	}
 
 	/**
-	 * @bug 2429
+	 * T4429
 	 * @dataProvider provideNamespacesAssociations
 	 */
 	public function testRcNsFilterAssociationWithInversion( $ns1, $ns2 ) {
diff --git a/tests/phpunit/includes/specials/SpecialSearchTest.php b/tests/phpunit/includes/specials/SpecialSearchTest.php
index 13c21c01b7..9daae9741e 100644
--- a/tests/phpunit/includes/specials/SpecialSearchTest.php
+++ b/tests/phpunit/includes/specials/SpecialSearchTest.php
@@ -73,7 +73,7 @@ class SpecialSearchTest extends MediaWikiTestCase {
 			[
 				$EMPTY_REQUEST, $NO_USER_PREF,
 				'default', $defaultNS,
-				'Bug 33270: No request nor user preferences should give default profile'
+				'T35270: No request nor user preferences should give default profile'
 			],
 			[
 				[ 'ns5' => 1 ], $NO_USER_PREF,
@@ -88,7 +88,7 @@ class SpecialSearchTest extends MediaWikiTestCase {
 				return "searchNs$ns";
 			}, $defaultNS ), 0 ),
 				'advanced', [ 2, 14 ],
-				'Bug 33583: search with no option should honor User search preferences'
+				'T35583: search with no option should honor User search preferences'
 					. ' and have all other namespace disabled'
 			],
 		];
diff --git a/tests/phpunit/includes/upload/UploadBaseTest.php b/tests/phpunit/includes/upload/UploadBaseTest.php
index 6be272fb61..a42c86c3a6 100644
--- a/tests/phpunit/includes/upload/UploadBaseTest.php
+++ b/tests/phpunit/includes/upload/UploadBaseTest.php
@@ -297,31 +297,31 @@ class UploadBaseTest extends MediaWikiTestCase {
 				'   ',
 				true,
 				true,
-				'SVG with non-local image href (bug 65839)'
+				'SVG with non-local image href (T67839)'
 			],
 			[
 				'   50 100 ',
 				true,
 				true,
-				'SVG with remote stylesheet (bug 57550)'
+				'SVG with remote stylesheet (T59550)'
 			],
 			[
 				'  B  ',
 				true,
 				true,
-				'SVG with rembeded iframe (bug 60771)'
+				'SVG with rembeded iframe (T62771)'
 			],
 			[
 				'   WebPlatform.org  ',
 				true,
 				true,
-				'SVG with @import in style element (bug 69008)'
+				'SVG with @import in style element (T71008)'
 			],
 			[
 				'   WebPlatform.org  ',
 				true,
 				true,
-				'SVG with @import in style element and child element (bug 69008#c11)'
+				'SVG with @import in style element and child element (T71008#c11)'
 			],
 			[
 				'   WebPlatform.org  ',
@@ -333,19 +333,19 @@ class UploadBaseTest extends MediaWikiTestCase {
 				'  ',
 				true,
 				true,
-				'SVG with remote background image (bug 69008)'
+				'SVG with remote background image (T71008)'
 			],
 			[
 				'  ',
 				true,
 				true,
-				'SVG with remote background image, encoded (bug 69008)'
+				'SVG with remote background image, encoded (T71008)'
 			],
 			[
 				'   ',
 				true,
 				true,
-				'SVG with remote background image, in style element (bug 69008)'
+				'SVG with remote background image, in style element (T71008)'
 			],
 			[
 				// This currently doesn't seem to work in any browsers, but in case
@@ -353,7 +353,7 @@ class UploadBaseTest extends MediaWikiTestCase {
 				'  ',
 				true,
 				true,
-				'SVG with remote background image using image() (bug 69008)'
+				'SVG with remote background image using image() (T71008)'
 			],
 			[
 				// As reported by Cure53
diff --git a/tests/phpunit/includes/upload/UploadStashTest.php b/tests/phpunit/includes/upload/UploadStashTest.php
index 9b25505c68..7c40a2dfb6 100644
--- a/tests/phpunit/includes/upload/UploadStashTest.php
+++ b/tests/phpunit/includes/upload/UploadStashTest.php
@@ -19,7 +19,7 @@ class UploadStashTest extends MediaWikiTestCase {
 	protected function setUp() {
 		parent::setUp();
 
-		// Setup a file for bug 29408
+		// Setup a file for T31408
 		$this->bug29408File = wfTempDir() . '/bug29408';
 		file_put_contents( $this->bug29408File, "\x00" );
 
@@ -62,7 +62,7 @@ class UploadStashTest extends MediaWikiTestCase {
 
 		// Throws exception caught by PHPUnit on failure
 		$file = $stash->stashFile( $this->bug29408File );
-		// We'll never reach this point if we hit bug 29408
+		// We'll never reach this point if we hit T31408
 		$this->assertTrue( true, 'Unrecognized file without extension' );
 
 		$stash->removeFile( $file->getFileKey() );
diff --git a/tests/phpunit/includes/user/UserTest.php b/tests/phpunit/includes/user/UserTest.php
index deb970820e..941e940d42 100644
--- a/tests/phpunit/includes/user/UserTest.php
+++ b/tests/phpunit/includes/user/UserTest.php
@@ -362,7 +362,7 @@ class UserTest extends MediaWikiTestCase {
 	}
 
 	/**
-	 * Bug 37963
+	 * T39963
 	 * Make sure defaults are loaded when setOption is called.
 	 * @covers User::loadOptions
 	 */
diff --git a/tests/phpunit/languages/LanguageTest.php b/tests/phpunit/languages/LanguageTest.php
index 88bab90799..22fd7b8510 100644
--- a/tests/phpunit/languages/LanguageTest.php
+++ b/tests/phpunit/languages/LanguageTest.php
@@ -1337,7 +1337,7 @@ class LanguageTest extends LanguageClassesTestCase {
 					"Acteur%7CAlbert%20Robbins%7CAnglais%7CAnn%20Donahue%7CAnthony%20E.%20Zuiker%7CCarol%20Mendelsohn"
 				)
 			],
-			// The following two data sets come from bug 36839. They fail if checkTitleEncoding uses a regexp to test for
+			// The following two data sets come from T38839. They fail if checkTitleEncoding uses a regexp to test for
 			// valid UTF-8 encoding and the pcre.recursion_limit is low (like, say, 1024). They succeed if checkTitleEncoding
 			// uses mb_check_encoding for its test.
 			[
diff --git a/tests/phpunit/languages/classes/LanguageBe_taraskTest.php b/tests/phpunit/languages/classes/LanguageBe_taraskTest.php
index 37c8fe0dd4..26db1062a1 100644
--- a/tests/phpunit/languages/classes/LanguageBe_taraskTest.php
+++ b/tests/phpunit/languages/classes/LanguageBe_taraskTest.php
@@ -15,19 +15,19 @@ class LanguageBe_taraskTest extends LanguageClassesTestCase {
 	}
 
 	/**
-	 * @see bug 23156 & r64981
+	 * @see T25156 & r64981
 	 * @covers Language::commafy
 	 */
 	public function testSearchRightSingleQuotationMarkAsApostroph() {
 		$this->assertEquals(
 			"'",
 			$this->getLang()->normalizeForSearch( '’' ),
-			'bug 23156: U+2019 conversion to U+0027'
+			'T25156: U+2019 conversion to U+0027'
 		);
 	}
 
 	/**
-	 * @see bug 23156 & r64981
+	 * @see T25156 & r64981
 	 * @covers Language::commafy
 	 */
 	public function testCommafy() {
@@ -36,7 +36,7 @@ class LanguageBe_taraskTest extends LanguageClassesTestCase {
 	}
 
 	/**
-	 * @see bug 23156 & r64981
+	 * @see T25156 & r64981
 	 * @covers Language::commafy
 	 */
 	public function testDoesNotCommafyFourDigitsNumber() {
diff --git a/tests/phpunit/languages/classes/LanguageMlTest.php b/tests/phpunit/languages/classes/LanguageMlTest.php
index 5eacd88b02..6bac031d02 100644
--- a/tests/phpunit/languages/classes/LanguageMlTest.php
+++ b/tests/phpunit/languages/classes/LanguageMlTest.php
@@ -10,7 +10,7 @@ class LanguageMlTest extends LanguageClassesTestCase {
 
 	/**
 	 * @dataProvider providerFormatNum
-	 * @see bug 29495
+	 * T31495
 	 * @covers Language::formatNum
 	 */
 	public function testFormatNum( $result, $value ) {
diff --git a/tests/phpunit/languages/classes/LanguageTrTest.php b/tests/phpunit/languages/classes/LanguageTrTest.php
index 9a1823b1cd..28d71df76f 100644
--- a/tests/phpunit/languages/classes/LanguageTrTest.php
+++ b/tests/phpunit/languages/classes/LanguageTrTest.php
@@ -9,7 +9,7 @@
 class LanguageTrTest extends LanguageClassesTestCase {
 
 	/**
-	 * See @bug 28040
+	 * See T30040
 	 * Credits to irc://irc.freenode.net/wikipedia-tr users:
 	 *  - berm
 	 *  - []LuCkY[]
@@ -48,7 +48,7 @@ class LanguageTrTest extends LanguageClassesTestCase {
 			[ 'lcfirst', 'ı', 'lower', 'ı' ],
 			[ 'lcfirst', 'i', 'lower', 'i' ],
 
-			# A real example taken from bug 28040 using
+			# A real example taken from T30040 using
 			# https://tr.wikipedia.org/wiki/%C4%B0Phone
 			[ 'lcfirst', 'iPhone', 'lower', 'iPhone' ],
 
diff --git a/tests/phpunit/maintenance/backupPrefetchTest.php b/tests/phpunit/maintenance/backupPrefetchTest.php
index b3f8e3e61d..2c5931d270 100644
--- a/tests/phpunit/maintenance/backupPrefetchTest.php
+++ b/tests/phpunit/maintenance/backupPrefetchTest.php
@@ -22,7 +22,7 @@ class BaseDumpTest extends MediaWikiTestCase {
 			$this->dump->close();
 		}
 
-		// Bug 37458, parent teardown need to be done after closing the
+		// T39458, parent teardown need to be done after closing the
 		// dump or it might cause some permissions errors.
 		parent::tearDown();
 	}
diff --git a/tests/phpunit/mocks/filebackend/MockFSFile.php b/tests/phpunit/mocks/filebackend/MockFSFile.php
index 6797f59554..8ee45a87cf 100644
--- a/tests/phpunit/mocks/filebackend/MockFSFile.php
+++ b/tests/phpunit/mocks/filebackend/MockFSFile.php
@@ -40,7 +40,7 @@ class MockFSFile extends FSFile {
 
 	/**
 	 * August 22 – The theft of the Mona Lisa is discovered in the Louvre."
-	 * @bug 20281
+	 * T22281
 	 */
 	public function getSize() {
 		return 1911;
diff --git a/tests/phpunit/skins/SideBarTest.php b/tests/phpunit/skins/SideBarTest.php
index aaa1febbd1..af03fe6325 100644
--- a/tests/phpunit/skins/SideBarTest.php
+++ b/tests/phpunit/skins/SideBarTest.php
@@ -123,7 +123,7 @@ class SideBarTest extends MediaWikiLangTestCase {
 	}
 
 	/**
-	 * bug 33321 - Make sure there's a | after transforming.
+	 * T35321 - Make sure there's a | after transforming.
 	 * @group Database
 	 * @covers SkinTemplate::addToSidebarPlain
 	 */
diff --git a/tests/qunit/suites/resources/jquery/jquery.byteLimit.test.js b/tests/qunit/suites/resources/jquery/jquery.byteLimit.test.js
index 81c116c48d..5c2a06c306 100644
--- a/tests/qunit/suites/resources/jquery/jquery.byteLimit.test.js
+++ b/tests/qunit/suites/resources/jquery/jquery.byteLimit.test.js
@@ -72,7 +72,7 @@
 	} );
 
 	byteLimitTest( {
-		description: 'Plain text input. Calling byteLimit with no parameters and no maxlength attribute (bug 36310)',
+		description: 'Plain text input. Calling byteLimit with no parameters and no maxlength attribute (T38310)',
 		$input: $( '' ).attr( 'type', 'text' )
 			.byteLimit(),
 		sample: simpleSample,
@@ -172,7 +172,7 @@
 		expected: '1234'
 	} );
 
-	// Regression tests for bug 41450
+	// Regression tests for T43450
 	byteLimitTest( {
 		description: 'Input filter of which the base exceeds the limit',
 		$input: $( '' ).attr( 'type', 'text' )
@@ -232,7 +232,7 @@
 		var $el;
 
 		// Use a new  because the bug only occurs on the first time
-		// the limit it reached (bug 40850)
+		// the limit it reached (T42850)
 		$el = $( '' ).attr( 'type', 'text' )
 			.appendTo( '#qunit-fixture' )
 			.byteLimit( 3 )
diff --git a/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js b/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js
index 5203def852..01ff45fca9 100644
--- a/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js
+++ b/tests/qunit/suites/resources/jquery/jquery.tablesorter.parsers.test.js
@@ -255,6 +255,6 @@
 		} );
 	} );
 
-	// TODO add numbers sorting tests for bug 8115 with a different language
+	// TODO add numbers sorting tests for T10115 with a different language
 
 }( jQuery, mediaWiki ) );
diff --git a/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js b/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js
index 483a37d1be..dffd67a56a 100644
--- a/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js
+++ b/tests/qunit/suites/resources/jquery/jquery.tablesorter.test.js
@@ -622,7 +622,7 @@
 
 	// Regression tests!
 	tableTest(
-		'Bug 28775: German-style (dmy) short numeric dates',
+		'T30775: German-style (dmy) short numeric dates',
 		[ 'Date' ],
 		[
 			// German-style dates are day-month-year
@@ -650,7 +650,7 @@
 	);
 
 	tableTest(
-		'Bug 28775: American-style (mdy) short numeric dates',
+		'T30775: American-style (mdy) short numeric dates',
 		[ 'Date' ],
 		[
 			// American-style dates are month-day-year
@@ -677,7 +677,7 @@
 	);
 
 	tableTest(
-		'Bug 17141: IPv4 address sorting',
+		'T19141: IPv4 address sorting',
 		[ 'IP' ],
 		ipv4,
 		ipv4Sorted,
@@ -687,7 +687,7 @@
 		}
 	);
 	tableTest(
-		'Bug 17141: IPv4 address sorting (reverse)',
+		'T19141: IPv4 address sorting (reverse)',
 		[ 'IP' ],
 		ipv4,
 		reversed( ipv4Sorted ),
@@ -874,7 +874,7 @@
 	} );
 
 	/** FIXME: the diff output is not very readeable. */
-	QUnit.test( 'bug 32047 - caption must be before thead', 1, function ( assert ) {
+	QUnit.test( 'T34047 - caption must be before thead', 1, function ( assert ) {
 		var $table;
 		$table = $(
 			'' +
@@ -890,7 +890,7 @@
 		assert.equal(
 			$table.children().get( 0 ).nodeName,
 			'CAPTION',
-			'First element after  must be  must be 
(bug 32047)' + 'First element after (T34047)' ); } ); @@ -991,7 +991,7 @@ ], 'Order matches expected order (based on data-sort-value attribute values)' ); // Example 3: Test that live changes are used from data-sort-value, - // even if they change after the tablesorter is constructed (bug 38152). + // even if they change after the tablesorter is constructed (T40152). $table = $( '' + '' + @@ -1007,7 +1007,7 @@ .tablesorter() .find( '.headerSort:eq(0)' ).click(); - // Change the sortValue data properties (bug 38152) + // Change the sortValue data properties (T40152) // - change data $table.find( 'td:contains(A)' ).data( 'sortValue', 3 ); // - add data @@ -1054,7 +1054,7 @@ } ); - tableTest( 'bug 8115: sort numbers with commas (ascending)', + tableTest( 'T10115: sort numbers with commas (ascending)', [ 'Numbers' ], numbers, numbersAsc, function ( $table ) { $table.tablesorter(); @@ -1062,16 +1062,16 @@ } ); - tableTest( 'bug 8115: sort numbers with commas (descending)', + tableTest( 'T10115: sort numbers with commas (descending)', [ 'Numbers' ], numbers, reversed( numbersAsc ), function ( $table ) { $table.tablesorter(); $table.find( '.headerSort:eq(0)' ).click().click(); } ); - // TODO add numbers sorting tests for bug 8115 with a different language + // TODO add numbers sorting tests for T10115 with a different language - QUnit.test( 'bug 32888 - Tables inside a tableheader cell', 2, function ( assert ) { + QUnit.test( 'T34888 - Tables inside a tableheader cell', 2, function ( assert ) { var $table; $table = $( '
Data
' + @@ -1087,12 +1087,12 @@ assert.equal( $table.find( '> thead:eq(0) > tr > th.headerSort' ).length, 1, - 'Child tables inside a headercell should not interfere with sortable headers (bug 32888)' + 'Child tables inside a headercell should not interfere with sortable headers (T34888)' ); assert.equal( $( '#mw-bug-32888-2' ).find( 'th.headerSort' ).length, 0, - 'The headers of child tables inside a headercell should not be sortable themselves (bug 32888)' + 'The headers of child tables inside a headercell should not be sortable themselves (T34888)' ); } ); @@ -1192,7 +1192,7 @@ ); } ); - QUnit.test( 'bug 38911 - The row with the largest amount of columns should receive the sort indicators', 3, function ( assert ) { + QUnit.test( 'T40911 - The row with the largest amount of columns should receive the sort indicators', 3, function ( assert ) { var $table = $( '
' + '' + @@ -1269,7 +1269,7 @@ ); } ); - // bug 53527 + // T55527 QUnit.test( 'td cells in thead should not be taken into account for longest row calculation', 2, function ( assert ) { var $table = $( '
' + @@ -1290,7 +1290,7 @@ ); } ); - // bug 41889 - exploding rowspans in more complex cases + // T43889 - exploding rowspans in more complex cases tableTestHTML( 'Rowspan exploding with row headers', '
' + @@ -1305,7 +1305,7 @@ ] ); - // bug 53211 - exploding rowspans in more complex cases + // T55211 - exploding rowspans in more complex cases QUnit.test( 'Rowspan exploding with row headers and colspans', 1, function ( assert ) { var $table = $( '
' + @@ -1424,7 +1424,7 @@ ] ); - QUnit.test( 'bug 105731 - incomplete rows in table body', 3, function ( assert ) { + QUnit.test( 'T105731 - incomplete rows in table body', 3, function ( assert ) { var $table, parsers; $table = $( '
' + diff --git a/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js b/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js index 100167928f..c28d3ceac2 100644 --- a/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js +++ b/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js @@ -244,9 +244,9 @@ caretSample = 'Some big text that we like to work with. Nothing fancy... you know what I mean?'; - /* @broken: Disabled per bug 34820 + /* @broken: Disabled per T36820 caretTest({ - description: 'getCaretPosition with original/empty selection - bug 31847 with IE 6/7/8', + description: 'getCaretPosition with original/empty selection - T33847 with IE 6/7/8', text: caretSample, start: [0, caretSample.length], // Opera and Firefox (prior to FF 6.0) default caret to the end of the box (caretSample.length) end: [0, caretSample.length], // Other browsers default it to the beginning (0), so check both. diff --git a/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.test.js b/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.test.js index 3f5e8c32a0..44d220970d 100644 --- a/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.test.js +++ b/tests/qunit/suites/resources/mediawiki.api/mediawiki.api.test.js @@ -201,7 +201,7 @@ ] ) ); - // Don't cache error (bug 65268) + // Don't cache error (T67268) return api.getToken( 'testerror' ) .then( null, function ( err ) { assert.equal( err, 'bite-me', 'Expected error' ); diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.Title.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.Title.test.js index 124c49f5b8..31a1a2861c 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.Title.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.Title.test.js @@ -338,7 +338,7 @@ extTest( 'MediaWiki:Vector.js', 'js' ); extTest( 'User:Example/common.css', 'css' ); - extTest( 'File:Example.longextension', 'longextension', 'Extension parsing not limited (bug 36151)' ); + extTest( 'File:Example.longextension', 'longextension', 'Extension parsing not limited (T38151)' ); extTest( 'Example/information.json', 'json', 'Extension parsing not restricted from any namespace' ); extTest( 'Foo.', null, 'Trailing dot is not an extension' ); extTest( 'Foo..', null, 'Trailing dots are not an extension' ); diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.Uri.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.Uri.test.js index 242096eab1..ef0d5e572a 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.Uri.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.Uri.test.js @@ -287,7 +287,7 @@ assert.deepEqual( original.query, { one: '1', two: '2' }, - 'Properties is deep cloned (bug 37708)' + 'Properties is deep cloned (T39708)' ); } ); @@ -488,7 +488,7 @@ assert.equal( uri.toString(), 'http://en.wiki.local/foo.com', 'handle absolute paths by supplying host from document in strict mode' ); } ); - QUnit.test( 'bug 35658', 2, function ( assert ) { + QUnit.test( 'T37658', 2, function ( assert ) { var testProtocol, testServer, testPort, testPath, UriClass, uri, href; testProtocol = 'https://'; diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.jscompat.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.jscompat.test.js index 3b5915ac28..34fe9bf8d2 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.jscompat.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.jscompat.test.js @@ -39,7 +39,7 @@ }); */ - QUnit.test( 'Stripping of single initial newline from textarea\'s literal contents (bug 12130)', function ( assert ) { + QUnit.test( 'Stripping of single initial newline from textarea\'s literal contents (T14130)', function ( assert ) { var maxn, n, expected, $textarea; diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js index 92ee7dd360..fcf80bde22 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.loader.test.js @@ -256,7 +256,7 @@ url: { print: [ urlStyleTest( '.mw-test-implement-b1', 'text-align', 'center' ) ], screen: [ - // bug 40834: Make sure it actually works with more than 1 stylesheet reference + // T42834: Make sure it actually works with more than 1 stylesheet reference urlStyleTest( '.mw-test-implement-b2', 'float', 'left' ), urlStyleTest( '.mw-test-implement-b3', 'float', 'right' ) ] @@ -315,7 +315,7 @@ 'test.implement.d', function () { assertStyleAsync( assert, $element, 'float', 'right', function () { - assert.notEqual( $element2.css( 'text-align' ), 'center', 'print style is not applied (bug 40500)' ); + assert.notEqual( $element2.css( 'text-align' ), 'center', 'print style is not applied (T42500)' ); done(); } ); }, @@ -328,7 +328,7 @@ mw.loader.load( 'test.implement.d' ); } ); - // @import (bug 31676) + // @import (T33676) QUnit.test( '.implement( styles has @import )', 7, function ( assert ) { var isJsExecuted, $element, done = assert.async(); @@ -429,14 +429,12 @@ } ); QUnit.test( '.implement( only messages )', 2, function ( assert ) { - assert.assertFalse( mw.messages.exists( 'bug_29107' ), 'Verify that the test message doesn\'t exist yet' ); + assert.assertFalse( mw.messages.exists( 'T31107' ), 'Verify that the test message doesn\'t exist yet' ); - // jscs: disable requireCamelCaseOrUpperCaseIdentifiers - mw.loader.implement( 'test.implement.msgs', [], {}, { bug_29107: 'loaded' } ); - // jscs: enable requireCamelCaseOrUpperCaseIdentifiers + mw.loader.implement( 'test.implement.msgs', [], {}, { T31107: 'loaded' } ); return mw.loader.using( 'test.implement.msgs', function () { - assert.ok( mw.messages.exists( 'bug_29107' ), 'Bug 29107: messages-only module should implement ok' ); + assert.ok( mw.messages.exists( 'T31107' ), 'T31107: messages-only module should implement ok' ); }, function () { assert.ok( false, 'Error callback fired while implementing "test.implement.msgs" module' ); } ); diff --git a/tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js b/tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js index 6dd17f11e1..88052de08a 100644 --- a/tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js +++ b/tests/qunit/suites/resources/mediawiki/mediawiki.util.test.js @@ -232,13 +232,13 @@ assert.strictEqual( mw.util.getParamValue( 'bar', url ), null, 'Return null when not found' ); url = 'http://example.org/#&foo=bad'; - assert.strictEqual( mw.util.getParamValue( 'foo', url ), null, 'Ignore hash if param is not in querystring but in hash (bug 27427)' ); + assert.strictEqual( mw.util.getParamValue( 'foo', url ), null, 'Ignore hash if param is not in querystring but in hash (T29427)' ); url = 'example.org?' + $.param( { TEST: 'a b+c' } ); - assert.strictEqual( mw.util.getParamValue( 'TEST', url ), 'a b+c', 'Bug 30441: getParamValue must understand "+" encoding of space' ); + assert.strictEqual( mw.util.getParamValue( 'TEST', url ), 'a b+c', 'T32441: getParamValue must understand "+" encoding of space' ); url = 'example.org?' + $.param( { TEST: 'a b+c d' } ); // check for sloppy code from r95332 :) - assert.strictEqual( mw.util.getParamValue( 'TEST', url ), 'a b+c d', 'Bug 30441: getParamValue must understand "+" encoding of space (multiple spaces)' ); + assert.strictEqual( mw.util.getParamValue( 'TEST', url ), 'a b+c d', 'T32441: getParamValue must understand "+" encoding of space (multiple spaces)' ); } ); QUnit.test( '$content', 2, function ( assert ) { @@ -320,7 +320,7 @@ assert.equal( $( '#p-test-custom #c-barmenu ul li' ).length, 1, - 'addPortletLink did not add the item to all
    elements in the portlet (bug 35082)' + 'addPortletLink did not add the item to all
      elements in the portlet (T37082)' ); tbRLDM = mw.util.addPortletLink( 'p-test-tb', '//mediawiki.org/wiki/RL/DM', -- 2.20.1