Do not insert page titles into querycache.qc_value

[lhc/web/wiklou.git] / tests / phpunit / includes / MagicWordFactoryTest.php

diff --git a/tests/phpunit/includes/MagicWordFactoryTest.php b/tests/phpunit/includes/MagicWordFactoryTest.php
index ec7f2ab..065024b 100644 (file)
--- a/tests/phpunit/includes/MagicWordFactoryTest.php
+++ b/tests/phpunit/includes/MagicWordFactoryTest.php
@@ -6,29 +6,22 @@
  * @author Derick N. Alangi
  */
 class MagicWordFactoryTest extends MediaWikiTestCase {
-       /**
-        * Make magic word factory
-        */
-       private function makeMagicWordFactory( $contLang ) {
-               if ( $contLang === null ) {
-                       $mwf = new MagicWordFactory( Language::factory( 'en' ) );
-                       return $mwf;
-               }
-               return new MagicWordFactory( $contLang );
+       private function makeMagicWordFactory( Language $contLang = null ) {
+               return new MagicWordFactory( $contLang ?: Language::factory( 'en' ) );
        }
 
        public function testGetContentLanguage() {
                $contLang = Language::factory( 'en' );
 
                $magicWordFactory = $this->makeMagicWordFactory( $contLang );
-               $mwfActual = $magicWordFactory->getContentLanguage();
+               $magicWordContLang = $magicWordFactory->getContentLanguage();
 
-               $this->assertSame( $contLang, $mwfActual );
+               $this->assertSame( $contLang, $magicWordContLang );
        }
 
        public function testGetMagicWord() {
                $magicWordIdValid = 'pageid';
-               $magicWordFactory = $this->makeMagicWordFactory( null );
+               $magicWordFactory = $this->makeMagicWordFactory();
                $mwActual = $magicWordFactory->get( $magicWordIdValid );
                $contLang = $magicWordFactory->getContentLanguage();
                $expected = new MagicWord( $magicWordIdValid, [ 'PAGEID' ], false, $contLang );
@@ -37,7 +30,7 @@ class MagicWordFactoryTest extends MediaWikiTestCase {
        }
 
        public function testGetInvalidMagicWord() {
-               $magicWordFactory = $this->makeMagicWordFactory( null );
+               $magicWordFactory = $this->makeMagicWordFactory();
 
                $this->setExpectedException( MWException::class );
                \Wikimedia\suppressWarnings();
@@ -49,28 +42,28 @@ class MagicWordFactoryTest extends MediaWikiTestCase {
        }
 
        public function testGetVariableIDs() {
-               $magicWordFactory = $this->makeMagicWordFactory( null );
+               $magicWordFactory = $this->makeMagicWordFactory();
                $varIds = $magicWordFactory->getVariableIDs();
 
-               $this->assertContainsOnly( 'string', $varIds );
-               $this->assertNotEmpty( $varIds );
                $this->assertInternalType( 'array', $varIds );
+               $this->assertNotEmpty( $varIds );
+               $this->assertContainsOnly( 'string', $varIds );
        }
 
        public function testGetSubstIDs() {
-               $magicWordFactory = $this->makeMagicWordFactory( null );
+               $magicWordFactory = $this->makeMagicWordFactory();
                $substIds = $magicWordFactory->getSubstIDs();
 
-               $this->assertContainsOnly( 'string', $substIds );
-               $this->assertNotEmpty( $substIds );
                $this->assertInternalType( 'array', $substIds );
+               $this->assertNotEmpty( $substIds );
+               $this->assertContainsOnly( 'string', $substIds );
        }
 
        /**
         * Test both valid and invalid caching hints paths
         */
        public function testGetCacheTTL() {
-               $magicWordFactory = $this->makeMagicWordFactory( null );
+               $magicWordFactory = $this->makeMagicWordFactory();
                $actual = $magicWordFactory->getCacheTTL( 'localday' );
 
                $this->assertSame( 3600, $actual );
@@ -83,7 +76,7 @@ class MagicWordFactoryTest extends MediaWikiTestCase {
        }
 
        public function testGetDoubleUnderscoreArray() {
-               $magicWordFactory = $this->makeMagicWordFactory( null );
+               $magicWordFactory = $this->makeMagicWordFactory();
                $actual = $magicWordFactory->getDoubleUnderscoreArray();
 
                $this->assertInstanceOf( MagicWordArray::class, $actual );