Merge "FauxRequest: don’t override getValues()"

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

diff --git a/tests/phpunit/includes/specials/SpecialPageDataTest.php b/tests/phpunit/includes/specials/SpecialPageDataTest.php
index 2566875..b11973c 100644 (file)
--- a/tests/phpunit/includes/specials/SpecialPageDataTest.php
+++ b/tests/phpunit/includes/specials/SpecialPageDataTest.php
@@ -2,16 +2,19 @@
 
 /**
  * @covers SpecialPageData
- *
  * @group Database
- *
  * @group SpecialPage
  *
- * @license GPL-2.0+
  * @author Daniel Kinzler
  */
 class SpecialPageDataTest extends SpecialPageTestBase {
 
+       protected function setUp() {
+               parent::setUp();
+
+               $this->setContentLang( 'qqx' );
+       }
+
        protected function newSpecialPage() {
                $page = new SpecialPageData();
 
@@ -37,21 +40,6 @@ class SpecialPageDataTest extends SpecialPageTestBase {
                        [ 'Location' => '!.+!' ]
                ];
 
-               $subpageCases = [];
-               foreach ( $cases as $c ) {
-                       $case = $c;
-                       $case[0] = '';
-
-                       if ( isset( $case[1]['title'] ) ) {
-                               $case[0] .= $case[1]['title'];
-                               unset( $case[1]['title'] );
-                       }
-
-                       $subpageCases[] = $case;
-               }
-
-               $cases = array_merge( $cases, $subpageCases );
-
                $cases['Accept only HTML'] = [
                        '',
                        [ 'target' => 'Helsinki' ],
@@ -74,7 +62,16 @@ class SpecialPageDataTest extends SpecialPageTestBase {
                ];
 
                $cases['Nothing specified'] = [
-                       'Helsinki',
+                       'main/Helsinki',
+                       [],
+                       [],
+                       '!!',
+                       303,
+                       [ 'Location' => '!Helsinki&action=raw!' ]
+               ];
+
+               $cases['Nothing specified'] = [
+                       '/Helsinki',
                        [],
                        [],
                        '!!',
@@ -83,7 +80,7 @@ class SpecialPageDataTest extends SpecialPageTestBase {
                ];
 
                $cases['Invalid Accept header'] = [
-                       'Helsinki',
+                       'main/Helsinki',
                        [],
                        [ 'Accept' => 'text/foobar' ],
                        '!!',
@@ -139,17 +136,12 @@ class SpecialPageDataTest extends SpecialPageTestBase {
        }
 
        public function testSpecialPageWithoutParameters() {
-               $this->setContentLang( Language::factory( 'en' ) );
                $request = new FauxRequest();
                $request->response()->header( 'Status: 200 OK', true, 200 ); // init/reset
 
                list( $output, ) = $this->executeSpecialPage( '', $request );
 
-               $this->assertContains(
-                       "Content negotiation applies based on you client's Accept header.",
-                       $output,
-                       "output"
-               );
+               $this->assertContains( '(pagedata-text)', $output );
        }
 
 }