Page MenuHomePhabricator

Cover CommunityStructuredMentorProvider and CommunityStructuredMentorWriter by appropriate tests
Closed, ResolvedPublic2 Estimated Story Points

Description

While preparing for T367574: [Epic] Remove code for Special:EditGrowthConfig, I noticed CommunityStructuredMentorProvider and CommunityStructuredMentorWriter are both uncovered by any tests. The abstract parents (which contain most of the logic) are tested by tests using legacy Community Configuration, but once the legacy CC is gone, those tests wouldn't be useful for anything. Before being able to remove those, we need to write new tests to cover the newer classes.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Urbanecm_WMF added a project: Technical-Debt.
Urbanecm_WMF moved this task from Inbox to Estimated tasks backlog on the Growth-Team board.

Let's aim on fixing the few latest blockers for removing legacy Community Configuration soon.

Urbanecm_WMF set the point value for this task to 2.Jan 21 2025, 4:00 PM

Maintenance tasks for dropping CC1.0 were prioritized during the lack Backlog Refinement meeting, moving back to Up Next (to consider pulling into the next sprint).

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

[mediawiki/extensions/GrowthExperiments@master] Add tests for CommunityStructuredMentorProvider/Writer

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

Change #1120966 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Add tests for CommunityStructuredMentorProvider/Writer

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