Page MenuHomePhabricator

MediaWiki\Tests\Structure\BundleSizeTest::testBundleSize with data set "mediawiki.base" (array('N/A', 'mediawiki.base', '16.8 kB'))
Closed, ResolvedPublic

Description

Seen on https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikisource/+/1204069
https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php81/51397/console

19:37:33 1) MediaWiki\Tests\Structure\BundleSizeTest::testBundleSize with data set "mediawiki.base" (array('N/A', 'mediawiki.base', '16.8 kB'))
19:37:33 mediawiki.base should be less than 17203.2 (uncompressed) bytes
19:37:33 Failed asserting that 17327 is less than 17203.2.
19:37:33 
19:37:33 /workspace/src/tests/phpunit/structure/BundleSizeTestBase.php:147

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change #1205189 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/core@master] bundlesize: Allow mediawiki.base to be slightly larger

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

Change #1205189 merged by jenkins-bot:

[mediawiki/core@master] bundlesize: Allow mediawiki.base to be slightly larger

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

CI at https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikisource/+/1204069 is now passing.

Thanks for the patch @Jdforrester-WMF.

The mediawiki.base module is in charge of transporting mw.config data and mw.user.options defaults data. Both can be extended by hooks (GetPreferences hook, and onResourceLoaderGetConfigVars). I'm guessing the Wikisource extension, or one of the extension it co-installs in CI, adds enough to put it over the threshold. This makes it an awkward fit for the bundlesize metric, so perhaps that should be disabled/null, but for now an increase LGTM.

Krinkle claimed this task.
Krinkle moved this task from Inbox to Backlog on the MediaWiki-ResourceLoader board.