Page MenuHomePhabricator

Port WikiPageConfigLoaderTest from GrowthExperiments to CommunityConfiguration extension
Closed, ResolvedPublic3 Estimated Story Points

Description

In https://github.com/wikimedia/mediawiki-extensions-CommunityConfiguration/commit/336a5da19da807fe0675f215ddf9f55265e28954, we added the Loader class, mostly taken from WikiPageConfigLoader as used in GrowthExperiments. That commit, however, did not transfer WikiPageConfigLoaderTest. To ensure a reasonable test coverage of the newly created CommunityConfiguration extension, we should add a LoaderTest as well.

Acceptance Criteria
  • The Loader class of CommunityConfiguration has reasonable unit test coverage

Event Timeline

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

Change 1007298 had a related patch set uploaded (by Cyndywikime; author: Cyndywikime):

[mediawiki/extensions/CommunityConfiguration@master] Add unit tests for Loader.php

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

Change 1007298 merged by jenkins-bot:

[mediawiki/extensions/CommunityConfiguration@master] Add unit tests for Loader.php

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