From fc23633035eebb24b7199cb15683e25100184e2b Mon Sep 17 00:00:00 2001
From: Kunal Mehta <legoktm@member.fsf.org>
Date: Thu, 28 Dec 2017 00:24:40 -0800
Subject: [PATCH] Add @covers tags to languages tests

I removed comments that merely repeated the location of the class being
tested. There are other tests in this directory that don't have a
corresponding class and need further investigation.

Change-Id: Ic16f0887b5030ac53fab4382cfaedfb5426cdb08
---
 tests/phpunit/languages/LanguageConverterTest.php         | 2 ++
 tests/phpunit/languages/LanguageTest.php                  | 8 ++++++--
 tests/phpunit/languages/classes/LanguageArTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageBe_taraskTest.php | 3 +++
 tests/phpunit/languages/classes/LanguageBsTest.php        | 6 +++++-
 tests/phpunit/languages/classes/LanguageCrhTest.php       | 4 ++++
 tests/phpunit/languages/classes/LanguageCuTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageDsbTest.php       | 4 +++-
 tests/phpunit/languages/classes/LanguageGanTest.php       | 4 ++++
 tests/phpunit/languages/classes/LanguageHsbTest.php       | 4 +++-
 tests/phpunit/languages/classes/LanguageHuTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageHyTest.php        | 6 +++++-
 tests/phpunit/languages/classes/LanguageIuTest.php        | 4 ++++
 tests/phpunit/languages/classes/LanguageKkTest.php        | 5 +++++
 tests/phpunit/languages/classes/LanguageKshTest.php       | 4 +++-
 tests/phpunit/languages/classes/LanguageKuTest.php        | 4 ++++
 tests/phpunit/languages/classes/LanguageMlTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageRuTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageShiTest.php       | 4 ++++
 tests/phpunit/languages/classes/LanguageSlTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageSrTest.php        | 5 ++++-
 tests/phpunit/languages/classes/LanguageTgTest.php        | 4 ++++
 tests/phpunit/languages/classes/LanguageTrTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageUkTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageUzTest.php        | 5 ++++-
 tests/phpunit/languages/classes/LanguageWaTest.php        | 4 +++-
 tests/phpunit/languages/classes/LanguageZhTest.php        | 5 +++++
 27 files changed, 99 insertions(+), 18 deletions(-)

diff --git a/tests/phpunit/languages/LanguageConverterTest.php b/tests/phpunit/languages/LanguageConverterTest.php
index fc2ed33b65..82ab7def22 100644
--- a/tests/phpunit/languages/LanguageConverterTest.php
+++ b/tests/phpunit/languages/LanguageConverterTest.php
@@ -160,6 +160,8 @@ class LanguageConverterTest extends MediaWikiLangTestCase {
 
 	/**
 	 * Test exhausting pcre.backtrack_limit
+	 *
+	 * @covers LanguageConverter::autoConvert
 	 */
 	public function testAutoConvertT124404() {
 		$testString = '';
diff --git a/tests/phpunit/languages/LanguageTest.php b/tests/phpunit/languages/LanguageTest.php
index e39f57ea99..0c341303f7 100644
--- a/tests/phpunit/languages/LanguageTest.php
+++ b/tests/phpunit/languages/LanguageTest.php
@@ -1630,7 +1630,7 @@ class LanguageTest extends LanguageClassesTestCase {
 	}
 
 	/**
-	 * @dataProvider testFormatNumProvider
+	 * @dataProvider provideFormatNum
 	 * @covers Language::formatNum
 	 */
 	public function testFormatNum(
@@ -1643,7 +1643,7 @@ class LanguageTest extends LanguageClassesTestCase {
 		$this->assertEquals( $expected, $formattedNum );
 	}
 
-	public function testFormatNumProvider() {
+	public function provideFormatNum() {
 		return [
 			[ true, 'en', 100, false, '100' ],
 			[ true, 'en', 101, true, '101' ],
@@ -1657,6 +1657,7 @@ class LanguageTest extends LanguageClassesTestCase {
 	}
 
 	/**
+	 * @covers Language::parseFormattedNumber
 	 * @dataProvider parseFormattedNumberProvider
 	 */
 	public function testParseFormattedNumber( $langCode, $number ) {
@@ -1795,6 +1796,9 @@ class LanguageTest extends LanguageClassesTestCase {
 		];
 	}
 
+	/**
+	 * @covers Language::equals
+	 */
 	public function testEquals() {
 		$en1 = new Language();
 		$en1->setCode( 'en' );
diff --git a/tests/phpunit/languages/classes/LanguageArTest.php b/tests/phpunit/languages/classes/LanguageArTest.php
index 5a66759873..f3f5a3f1f1 100644
--- a/tests/phpunit/languages/classes/LanguageArTest.php
+++ b/tests/phpunit/languages/classes/LanguageArTest.php
@@ -4,7 +4,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/LanguageAr.php */
+/**
+ * @covers LanguageAr
+ */
 class LanguageArTest extends LanguageClassesTestCase {
 	/**
 	 * @covers Language::formatNum
diff --git a/tests/phpunit/languages/classes/LanguageBe_taraskTest.php b/tests/phpunit/languages/classes/LanguageBe_taraskTest.php
index 26db1062a1..de684359fa 100644
--- a/tests/phpunit/languages/classes/LanguageBe_taraskTest.php
+++ b/tests/phpunit/languages/classes/LanguageBe_taraskTest.php
@@ -1,6 +1,9 @@
 <?php
 
 // @codingStandardsIgnoreStart Ignore Squiz.Classes.ValidClassName.NotCamelCaps
+/**
+ * @covers LanguageBe_tarask
+ */
 class LanguageBe_taraskTest extends LanguageClassesTestCase {
 	// @codingStandardsIgnoreEnd
 	/**
diff --git a/tests/phpunit/languages/classes/LanguageBsTest.php b/tests/phpunit/languages/classes/LanguageBsTest.php
index 207f50547c..29b2ccf306 100644
--- a/tests/phpunit/languages/classes/LanguageBsTest.php
+++ b/tests/phpunit/languages/classes/LanguageBsTest.php
@@ -5,7 +5,11 @@
  * @file
  */
 
-/** Tests for Croatian (hrvatski) */
+/**
+ * Tests for Croatian (hrvatski)
+ *
+ * @covers LanguageBs
+ */
 class LanguageBsTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageCrhTest.php b/tests/phpunit/languages/classes/LanguageCrhTest.php
index f34288c9b8..d99fc26729 100644
--- a/tests/phpunit/languages/classes/LanguageCrhTest.php
+++ b/tests/phpunit/languages/classes/LanguageCrhTest.php
@@ -1,5 +1,9 @@
 <?php
 
+/**
+ * @covers LanguageCrh
+ * @covers CrhConverter
+ */
 class LanguageCrhTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
diff --git a/tests/phpunit/languages/classes/LanguageCuTest.php b/tests/phpunit/languages/classes/LanguageCuTest.php
index de65d162c8..565a8856ed 100644
--- a/tests/phpunit/languages/classes/LanguageCuTest.php
+++ b/tests/phpunit/languages/classes/LanguageCuTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/LanguageCu.php */
+/**
+ * @covers LanguageCu
+ */
 class LanguageCuTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageDsbTest.php b/tests/phpunit/languages/classes/LanguageDsbTest.php
index 949d5dbe40..877a70cdf9 100644
--- a/tests/phpunit/languages/classes/LanguageDsbTest.php
+++ b/tests/phpunit/languages/classes/LanguageDsbTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/classes/LanguageDsb.php */
+/**
+ * @covers LanguageDsb
+ */
 class LanguageDsbTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageGanTest.php b/tests/phpunit/languages/classes/LanguageGanTest.php
index 43eb93e35b..c5d9e5e6b0 100644
--- a/tests/phpunit/languages/classes/LanguageGanTest.php
+++ b/tests/phpunit/languages/classes/LanguageGanTest.php
@@ -1,5 +1,9 @@
 <?php
 
+/**
+ * @covers LanguageGan
+ * @covers GanConverter
+ */
 class LanguageGanTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
diff --git a/tests/phpunit/languages/classes/LanguageHsbTest.php b/tests/phpunit/languages/classes/LanguageHsbTest.php
index 133cc1c3ff..0841f6f9a9 100644
--- a/tests/phpunit/languages/classes/LanguageHsbTest.php
+++ b/tests/phpunit/languages/classes/LanguageHsbTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/classes/LanguageHsb.php */
+/**
+ * @covers LanguageHsb
+ */
 class LanguageHsbTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageHuTest.php b/tests/phpunit/languages/classes/LanguageHuTest.php
index 7ea63e10d9..a1925bdf87 100644
--- a/tests/phpunit/languages/classes/LanguageHuTest.php
+++ b/tests/phpunit/languages/classes/LanguageHuTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/LanguageHu.php */
+/**
+ * @covers LanguageHu
+ */
 class LanguageHuTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageHyTest.php b/tests/phpunit/languages/classes/LanguageHyTest.php
index 64530912de..b493615453 100644
--- a/tests/phpunit/languages/classes/LanguageHyTest.php
+++ b/tests/phpunit/languages/classes/LanguageHyTest.php
@@ -5,7 +5,11 @@
  * @file
  */
 
-/** Tests for Armenian (Հայերեն) */
+/**
+ * Tests for Armenian (Հայերեն)
+ *
+ * @covers LanguageHy
+ */
 class LanguageHyTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageIuTest.php b/tests/phpunit/languages/classes/LanguageIuTest.php
index ff9c4d0d7f..01d97fc0c4 100644
--- a/tests/phpunit/languages/classes/LanguageIuTest.php
+++ b/tests/phpunit/languages/classes/LanguageIuTest.php
@@ -1,5 +1,9 @@
 <?php
 
+/**
+ * @covers LanguageIu
+ * @covers IuConverter
+ */
 class LanguageIuTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
diff --git a/tests/phpunit/languages/classes/LanguageKkTest.php b/tests/phpunit/languages/classes/LanguageKkTest.php
index a03eac2226..f21950e0bd 100644
--- a/tests/phpunit/languages/classes/LanguageKkTest.php
+++ b/tests/phpunit/languages/classes/LanguageKkTest.php
@@ -1,5 +1,10 @@
 <?php
 
+/**
+ * @covers LanguageKk
+ * @covers LanguageKk_cyrl
+ * @covers KkConverter
+ */
 class LanguageKkTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
diff --git a/tests/phpunit/languages/classes/LanguageKshTest.php b/tests/phpunit/languages/classes/LanguageKshTest.php
index f77c5b623e..6419e28178 100644
--- a/tests/phpunit/languages/classes/LanguageKshTest.php
+++ b/tests/phpunit/languages/classes/LanguageKshTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/classes/LanguageKsh.php */
+/**
+ * @covers LanguageKsh
+ */
 class LanguageKshTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageKuTest.php b/tests/phpunit/languages/classes/LanguageKuTest.php
index 797ab4a428..db693088f9 100644
--- a/tests/phpunit/languages/classes/LanguageKuTest.php
+++ b/tests/phpunit/languages/classes/LanguageKuTest.php
@@ -1,5 +1,9 @@
 <?php
 
+/**
+ * @covers LanguageKu
+ * @covers KuConverter
+ */
 class LanguageKuTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
diff --git a/tests/phpunit/languages/classes/LanguageMlTest.php b/tests/phpunit/languages/classes/LanguageMlTest.php
index 6bac031d02..673b5c775f 100644
--- a/tests/phpunit/languages/classes/LanguageMlTest.php
+++ b/tests/phpunit/languages/classes/LanguageMlTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/LanguageMl.php */
+/**
+ * @covers LanguageMl
+ */
 class LanguageMlTest extends LanguageClassesTestCase {
 
 	/**
diff --git a/tests/phpunit/languages/classes/LanguageRuTest.php b/tests/phpunit/languages/classes/LanguageRuTest.php
index a76293c190..9124040f61 100644
--- a/tests/phpunit/languages/classes/LanguageRuTest.php
+++ b/tests/phpunit/languages/classes/LanguageRuTest.php
@@ -6,7 +6,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/classes/LanguageRu.php */
+/**
+ * @covers LanguageRu
+ */
 class LanguageRuTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageShiTest.php b/tests/phpunit/languages/classes/LanguageShiTest.php
index 207a1b0bff..1d0f863578 100644
--- a/tests/phpunit/languages/classes/LanguageShiTest.php
+++ b/tests/phpunit/languages/classes/LanguageShiTest.php
@@ -1,5 +1,9 @@
 <?php
 
+/**
+ * @covers LanguageShi
+ * @covers ShiConverter
+ */
 class LanguageShiTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
diff --git a/tests/phpunit/languages/classes/LanguageSlTest.php b/tests/phpunit/languages/classes/LanguageSlTest.php
index ed138c500e..50100ce712 100644
--- a/tests/phpunit/languages/classes/LanguageSlTest.php
+++ b/tests/phpunit/languages/classes/LanguageSlTest.php
@@ -6,7 +6,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/classes/LanguageSl.php */
+/**
+ * @covers LanguageSl
+ */
 class LanguageSlTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providerPlural
diff --git a/tests/phpunit/languages/classes/LanguageSrTest.php b/tests/phpunit/languages/classes/LanguageSrTest.php
index b64fd679b5..e81d537060 100644
--- a/tests/phpunit/languages/classes/LanguageSrTest.php
+++ b/tests/phpunit/languages/classes/LanguageSrTest.php
@@ -16,7 +16,10 @@
  *  - Tests for LanguageConverter and Language should probably be separate..
  */
 
-/** Tests for MediaWiki languages/LanguageSr.php */
+/**
+ * @covers LanguageSr
+ * @covers SrConverter
+ */
 class LanguageSrTest extends LanguageClassesTestCase {
 	/**
 	 * @covers LanguageConverter::convertTo
diff --git a/tests/phpunit/languages/classes/LanguageTgTest.php b/tests/phpunit/languages/classes/LanguageTgTest.php
index 0ed24ff1c0..89697675ab 100644
--- a/tests/phpunit/languages/classes/LanguageTgTest.php
+++ b/tests/phpunit/languages/classes/LanguageTgTest.php
@@ -1,5 +1,9 @@
 <?php
 
+/**
+ * @covers LanguageTg
+ * @covers TgConverter
+ */
 class LanguageTgTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
diff --git a/tests/phpunit/languages/classes/LanguageTrTest.php b/tests/phpunit/languages/classes/LanguageTrTest.php
index 28d71df76f..3ddf2d03e2 100644
--- a/tests/phpunit/languages/classes/LanguageTrTest.php
+++ b/tests/phpunit/languages/classes/LanguageTrTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/LanguageTr.php */
+/**
+ * @covers LanguageTr
+ */
 class LanguageTrTest extends LanguageClassesTestCase {
 
 	/**
diff --git a/tests/phpunit/languages/classes/LanguageUkTest.php b/tests/phpunit/languages/classes/LanguageUkTest.php
index 6b259823be..379162230d 100644
--- a/tests/phpunit/languages/classes/LanguageUkTest.php
+++ b/tests/phpunit/languages/classes/LanguageUkTest.php
@@ -6,7 +6,9 @@
  * @file
  */
 
-/** Tests for Ukrainian */
+/**
+ * @covers LanguageUk
+ */
 class LanguageUkTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageUzTest.php b/tests/phpunit/languages/classes/LanguageUzTest.php
index 7ef87bf460..367226d773 100644
--- a/tests/phpunit/languages/classes/LanguageUzTest.php
+++ b/tests/phpunit/languages/classes/LanguageUzTest.php
@@ -16,7 +16,10 @@
  *  - Tests for LanguageConverter and Language should probably be separate..
  */
 
-/** Tests for MediaWiki languages/LanguageUz.php */
+/**
+ * @covers LanguageUz
+ * @covers UzConverter
+ */
 class LanguageUzTest extends LanguageClassesTestCase {
 
 	/**
diff --git a/tests/phpunit/languages/classes/LanguageWaTest.php b/tests/phpunit/languages/classes/LanguageWaTest.php
index 27e57f6405..80c9860334 100644
--- a/tests/phpunit/languages/classes/LanguageWaTest.php
+++ b/tests/phpunit/languages/classes/LanguageWaTest.php
@@ -5,7 +5,9 @@
  * @file
  */
 
-/** Tests for MediaWiki languages/classes/LanguageWa.php */
+/**
+ * @covers LanguageWa
+ */
 class LanguageWaTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider providePlural
diff --git a/tests/phpunit/languages/classes/LanguageZhTest.php b/tests/phpunit/languages/classes/LanguageZhTest.php
index 26edc90aa5..2e73ac51cc 100644
--- a/tests/phpunit/languages/classes/LanguageZhTest.php
+++ b/tests/phpunit/languages/classes/LanguageZhTest.php
@@ -1,5 +1,10 @@
 <?php
 
+/**
+ * @covers LanguageZh
+ * @covers LanguageZh_hans
+ * @covers ZhConverter
+ */
 class LanguageZhTest extends LanguageClassesTestCase {
 	/**
 	 * @dataProvider provideAutoConvertToAllVariants
-- 
2.20.1