resourceloader: Add coverage for internal Context::getReqBase()
authorTimo Tijhof <krinklemail@gmail.com>
Sun, 25 Aug 2019 18:01:48 +0000 (19:01 +0100)
committerTimo Tijhof <krinklemail@gmail.com>
Sun, 25 Aug 2019 18:03:04 +0000 (19:03 +0100)
Change-Id: I068000f55f2100a6791470896d7f1050bdcb19a4

includes/resourceloader/ResourceLoaderContext.php
tests/phpunit/includes/resourceloader/ResourceLoaderContextTest.php

index 94e8a3e..c3948cb 100644 (file)
@@ -76,8 +76,8 @@ class ResourceLoaderContext implements MessageLocalizer {
                // Various parameters
                $this->user = $request->getRawVal( 'user' );
                $this->debug = $request->getRawVal( 'debug' ) === 'true';
-               $this->only = $request->getRawVal( 'only', null );
-               $this->version = $request->getRawVal( 'version', null );
+               $this->only = $request->getRawVal( 'only' );
+               $this->version = $request->getRawVal( 'version' );
                $this->raw = $request->getFuzzyBool( 'raw' );
 
                // Image requests
index f6fd824..c748e2c 100644 (file)
@@ -35,6 +35,7 @@ class ResourceLoaderContextTest extends PHPUnit\Framework\TestCase {
                // Misc
                $this->assertEquals( 'ltr', $ctx->getDirection() );
                $this->assertEquals( 'qqx|fallback||||||||', $ctx->getHash() );
+               $this->assertSame( [], $ctx->getReqBase() );
                $this->assertInstanceOf( User::class, $ctx->getUserObj() );
        }
 
@@ -75,6 +76,7 @@ class ResourceLoaderContextTest extends PHPUnit\Framework\TestCase {
                // Misc
                $this->assertEquals( 'ltr', $ctx->getDirection() );
                $this->assertEquals( 'zh|fallback|||styles|||||', $ctx->getHash() );
+               $this->assertSame( [ 'lang' => 'zh' ], $ctx->getReqBase() );
        }
 
        public static function provideDirection() {