Krinkle has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/300089

Change subject: tests: Remove unused params to ResourceLoader::makeLoaderQuery
......................................................................

tests: Remove unused params to ResourceLoader::makeLoaderQuery

The last three parameters (printable, handheld, extraQuery) aren't relevant
here. Also, why was it passing printable=true?

Change-Id: I16dcd8c86b53c6854b0a1a70bf0c1705d40b3b37
---
M tests/ResourceLoaderGlobalModuleTestCase.php
1 file changed, 1 insertion(+), 4 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/GlobalCssJs 
refs/changes/89/300089/1

diff --git a/tests/ResourceLoaderGlobalModuleTestCase.php 
b/tests/ResourceLoaderGlobalModuleTestCase.php
index 23b26d3..07a5c03 100644
--- a/tests/ResourceLoaderGlobalModuleTestCase.php
+++ b/tests/ResourceLoaderGlobalModuleTestCase.php
@@ -44,10 +44,7 @@
                        $options['user'],
                        null, // version
                        false, // debug
-                       ResourceLoaderModule::TYPE_COMBINED,
-                       true, // printable
-                       false, // handheld
-                       [] // extra
+                       ResourceLoaderModule::TYPE_COMBINED
                );
                $rl = new ResourceLoader( 
ConfigFactory::getDefaultInstance()->makeConfig( 'main' ) );
                return new ResourceLoaderContext( $rl, new FauxRequest( $query 
) );

-- 
To view, visit https://gerrit.wikimedia.org/r/300089
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I16dcd8c86b53c6854b0a1a70bf0c1705d40b3b37
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GlobalCssJs
Gerrit-Branch: master
Gerrit-Owner: Krinkle <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to