Page MenuHomePhabricator

Migrate GrowthExperiments API tests to CommunityConfiguration 2.0
Closed, ResolvedPublic1 Estimated Story Points

Description

In T369709: Switch `GEUseCommunityConfigurationExtension` to `true` in extension.json, we switched (most) PHPUnit tests to use the CommunityConfiguration extension (rather than the legacy CC support). Within this tasks, we need to migrate all API tests in GrowthExperiments to also make use of Community Configuration 2.0. This is important to ensure our tests run with configuration reasonably similar to production, in case an issue is caused by the way how we integrate with the CommunityConfiguration extension or similar.

Credits to @Michael for thinking of this.

Event Timeline

Urbanecm_WMF edited projects, added Growth-Team; removed Growth-Team (Maintenance).
Urbanecm_WMF moved this task from Inbox to Up Next (estimated tasks) on the Growth-Team board.

Based on my investigation (https://wikimedia.slack.com/archives/GVD7X37RB/p1739810937225419), https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GrowthExperiments/+/1120198), it seems that API tests don't actually need Community Configuration provisioned. This will likely involve removing the setting up infrastructure.

Urbanecm_WMF set the point value for this task to 1.Feb 17 2025, 5:13 PM

Change #1120198 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] tests: Use CC2.0 in api-testing

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

Change #1120208 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] testing(api): Remove importDump

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

Change #1120209 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] testing(api): Do not set unnecessary user properties

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

Change #1120198 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] testing(api): Use CommunityConfiguration 2.0

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

Change #1120208 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] testing(api): Remove importDump

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

Change #1120209 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] testing(api): Do not set unnecessary user properties

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