Page MenuHomePhabricator

Fix all Wikimedia production code and MW extensions PHPUnit tests data providers real providers (and use static methods)
Closed, ResolvedPublic

Description

CodeSearch survey: https://codesearch.wmcloud.org/deployed/?q=public+function+provide*&files=tests%2F&excludeFiles=&repos=

In the spirit of T332865: PHPUnit data providers should be simple static functions that return plain data, we've mostly migrated our code to use pure data providers, but there are still some extensions (per code search) above that still don't follow this pattern.

It would be nice to migrate all of them to the new pattern. Not sure if it would be helpful to tag all extensions on the codesearch list.


Code search for code used at Wikimedia:

Details

Related Changes in Gerrit:
SubjectAuthorRepoBranchLines +/-
Umherirrendermediawiki/extensions/EventBusmaster+22 -22
Umherirrendermediawiki/extensions/ConfirmEditmaster+1 -1
Umherirrendermediawiki/coremaster+1 -1
Umherirrendermediawiki/extensions/CirrusSearchmaster+2 -2
Umherirrendermediawiki/extensions/OATHAuthmaster+1 -1
Umherirrendermediawiki/coremaster+33 -27
Umherirrendermediawiki/extensions/WikimediaCustomizationsmaster+1 -1
Umherirrendermediawiki/extensions/WikiLambdamaster+24 -19
Umherirrendermediawiki/extensions/WikimediaCustomizationsmaster+37 -65
Umherirrendermediawiki/extensions/Wikibasemaster+1 -1
Umherirrendermediawiki/extensions/CirrusSearchmaster+3 -3
Umherirrendermediawiki/extensions/Wikibasemaster+13 -13
Umherirrendermediawiki/extensions/Wikibasemaster+95 -103
Umherirrendermediawiki/extensions/IPReputationmaster+1 -1
Umherirrendermediawiki/extensions/GrowthExperimentsmaster+3 -3
Umherirrendermediawiki/extensions/Mathmaster+2 -2
Umherirrendermediawiki/extensions/OATHAuthmaster+3 -3
Umherirrendermediawiki/extensions/CommunityRequestsmaster+46 -33
Umherirrendermediawiki/extensions/ConfirmEditmaster+1 -1
Umherirrendermediawiki/extensions/CirrusSearchmaster+6 -6
Umherirrendermediawiki/extensions/EmailAuthmaster+2 -2
Umherirrendermediawiki/extensions/CampaignEventsmaster+2 -2
Umherirrendermediawiki/extensions/Kartographermaster+2 -2
Umherirrendermediawiki/extensions/Chartmaster+1 -1
Umherirrendermediawiki/extensions/UploadWizardmaster+1 -1
Umherirrendermediawiki/extensions/WikimediaCustomizationsmaster+3 -3
Umherirrendermediawiki/extensions/Flowmaster+1 -1
Umherirrendermediawiki/extensions/ParserMigrationmaster+1 -5
Umherirrendermediawiki/extensions/CheckUsermaster+7 -7
Umherirrendermediawiki/extensions/Citemaster+1 -1
Umherirrendermediawiki/extensions/Scoremaster+1 -1
Umherirrendermediawiki/extensions/TestKitchenmaster+1 -1
Umherirrendermediawiki/extensions/CodeMirrormaster+2 -2
Umherirrendermediawiki/extensions/Thanksmaster+1 -1
Umherirrendermediawiki/extensions/SecurePollmaster+3 -3
Jforrestermediawiki/extensions/WikiLambdamaster+1 -1
D3r1ck01operations/mediawiki-configmaster+10 -12
Samwilsonmediawiki/extensions/CommunityRequestsmaster+10 -10
D3r1ck01mediawiki/extensions/OATHAuthmaster+4 -4
D3r1ck01mediawiki/extensions/OAuthmaster+2 -7
Show related patches Customize query in gerrit

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
Jdforrester-WMF renamed this task from Make production extensions PHPUnit tests data providers real providers (and use static methods) to Fix all Wikimedia production extensions PHPUnit tests data providers real providers (and use static methods).Mar 6 2026, 2:14 PM
Jdforrester-WMF updated the task description. (Show Details)

Change #1248542 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] tests: Convert last provider over to static, as required by PHPUnit 10

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

Jdforrester-WMF renamed this task from Fix all Wikimedia production extensions PHPUnit tests data providers real providers (and use static methods) to Fix all Wikimedia production code and MW extensions PHPUnit tests data providers real providers (and use static methods).Mar 6 2026, 2:26 PM
Jdforrester-WMF updated the task description. (Show Details)

Change #1257977 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/EmailAuth@master] tests: Make data provider static in SecondaryAuthenticationProviderTest

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

Change #1257978 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Flow@master] tests: Make PHPUnit data provider static in FlowPostMetaActionsTest

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

Change #1257979 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/IPReputation@master] tests: Make PHPUnit data provider static in IPoidDataLookupTest

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

Change #1257984 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/OATHAuth@master] tests: Make PHPUnit data provider static

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

Change #1257985 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/ParserMigration@master] tests: Make PHPUnit data provider static in OracleTest

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

Change #1257986 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Score@master] tests: Make PHPUnit data provider static in ScoreFormatterTest

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

Change #1257987 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/SecurePoll@master] tests: Make PHPUnit data provider static

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

Change #1257988 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/TestKitchen@master] tests: Make PHPUnit data provider static

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

Change #1257989 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Thanks@master] tests: Make PHPUnit data provider static in HooksTest

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

Change #1257990 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/WikimediaCustomizations@master] tests: Make PHPUnit data provider static for BadEmailDomainChecker

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

Change #1257991 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/UploadWizard@master] tests: Make PHPUnit data provider static in ConfigTest

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

Change #1257994 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Kartographer@master] tests: Make PHPUnit data provider static

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

Change #1257995 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/GrowthExperiments@master] tests: Make PHPUnit data provider static

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

Change #1257996 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/ConfirmEdit@master] tests: Make PHPUnit data provider static in SimpleCaptchaTest

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

Change #1257997 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CodeMirror@master] tests: Make PHPUnit data provider static

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

Change #1257998 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Cite@master] tests: Make PHPUnit data provider static in ValidatorTest

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

Change #1257999 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Chart@master] tests: Make PHPUnit data provider static

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

Change #1258000 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CampaignEvents@master] tests: Make PHPUnit data provider static

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

Change #1258001 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CirrusSearch@master] tests: Make PHPUnit data provider static

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

Change #1257987 merged by jenkins-bot:

[mediawiki/extensions/SecurePoll@master] tests: Make PHPUnit data provider static

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

Change #1258004 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Math@master] tests: Make PHPUnit data provider static

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

Change #1258005 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CheckUser@master] tests: Make PHPUnit data provider static

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

Change #1258007 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Wikibase@master] tests: Make PHPUnit data provider static

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

Change #1258040 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CommunityRequests@master] tests: Make PHPUnit data provider static

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

Change #1257989 merged by jenkins-bot:

[mediawiki/extensions/Thanks@master] tests: Make PHPUnit data provider static in HooksTest

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

Change #1257997 merged by jenkins-bot:

[mediawiki/extensions/CodeMirror@master] tests: Make PHPUnit data provider static

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

Change #1258128 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Wikibase@master] tests: Make complex PHPUnit data provider static

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

Change #1257988 merged by jenkins-bot:

[mediawiki/extensions/TestKitchen@master] tests: Make PHPUnit data provider static

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

Change #1257986 merged by jenkins-bot:

[mediawiki/extensions/Score@master] tests: Make PHPUnit data provider static in ScoreFormatterTest

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

Change #1257998 merged by jenkins-bot:

[mediawiki/extensions/Cite@master] tests: Make PHPUnit data provider static in ValidatorTest

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

Change #1258005 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] tests: Make PHPUnit data provider static

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

Change #1257978 merged by jenkins-bot:

[mediawiki/extensions/Flow@master] tests: Make PHPUnit data provider static in FlowPostMetaActionsTest

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

Change #1257985 merged by jenkins-bot:

[mediawiki/extensions/ParserMigration@master] tests: Make PHPUnit data provider static in OracleTest

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

Change #1257990 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@master] tests: Make PHPUnit data provider static for BadEmailDomainChecker

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

Change #1257991 merged by jenkins-bot:

[mediawiki/extensions/UploadWizard@master] tests: Make PHPUnit data provider static in ConfigTest

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

Change #1257999 merged by jenkins-bot:

[mediawiki/extensions/Chart@master] tests: Make PHPUnit data provider static

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

Change #1257994 merged by jenkins-bot:

[mediawiki/extensions/Kartographer@master] tests: Make PHPUnit data provider static

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

Change #1258000 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] tests: Make PHPUnit data provider static

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

Change #1257977 merged by jenkins-bot:

[mediawiki/extensions/EmailAuth@master] tests: Make data provider static in SecondaryAuthenticationProviderTest

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

Change #1257996 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@master] tests: Make PHPUnit data provider static in SimpleCaptchaTest

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

Change #1258001 merged by jenkins-bot:

[mediawiki/extensions/CirrusSearch@master] tests: Make PHPUnit data provider static

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

Change #1258040 merged by jenkins-bot:

[mediawiki/extensions/CommunityRequests@master] tests: Make PHPUnit data provider static

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

Change #1257984 merged by jenkins-bot:

[mediawiki/extensions/OATHAuth@master] tests: Make PHPUnit data provider static

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

Change #1257995 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] tests: Make PHPUnit data provider static

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

Change #1258004 merged by jenkins-bot:

[mediawiki/extensions/Math@master] tests: Make PHPUnit data provider static

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

Change #1257979 merged by jenkins-bot:

[mediawiki/extensions/IPReputation@master] tests: Make PHPUnit data provider static in IPoidDataLookupTest

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

Change #1258128 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] tests: Make complex PHPUnit data provider static

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

Change #1258007 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] tests: Make simple PHPUnit data provider static

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

Change #1277199 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CirrusSearch@master] tests: Make PHPUnit data provider static

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

Change #1277204 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/Wikibase@master] tests: Use static data provider in ItemByExternalIdQueryTest

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

Change #1277206 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/WikiLambda@master] tests: Make PHPUnit data provider static in ZObjectAuthorizationTest

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

Change #1277199 merged by jenkins-bot:

[mediawiki/extensions/CirrusSearch@master] tests: Make PHPUnit data provider static

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

Change #1277204 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] tests: Use static data provider in ItemByExternalIdQueryTest

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

Change #1277213 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/WikimediaCustomizations@master] tests: Make PHPUnit data provider static in AttributionRestHandlerTest

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

Change #1277213 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@master] tests: Make PHPUnit data provider static in AttributionRestHandlerTest

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

Change #1277206 merged by jenkins-bot:

[mediawiki/extensions/WikiLambda@master] tests: Make PHPUnit data provider static in ZObjectAuthorizationTest

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

Change #1292901 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/WikimediaCustomizations@master] tests: Make PHPUnit data provider static in LicenceHelperTest

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

Change #1292903 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] tests: Make PHPUnit data provider static

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

Change #1292903 merged by jenkins-bot:

[mediawiki/core@master] tests: Make PHPUnit data provider static

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

Change #1292901 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@master] tests: Make PHPUnit data provider static in LicenceHelperTest

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

Change #1295071 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/OATHAuth@master] tests: Make PHPUnit data provider static in WebAuthnKeyTest

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

Change #1295071 merged by jenkins-bot:

[mediawiki/extensions/OATHAuth@master] tests: Make PHPUnit data provider static in WebAuthnKeyTest

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

Change #1297203 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/CirrusSearch@master] tests: Make PHPUnit data provider static

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

Change #1297206 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/core@master] tests: Make PHPUnit data provider static

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

Change #1297203 merged by jenkins-bot:

[mediawiki/extensions/CirrusSearch@master] tests: Make PHPUnit data provider static

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

Change #1297206 merged by jenkins-bot:

[mediawiki/core@master] tests: Make PHPUnit data provider static

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

Change #1312250 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/EventBus@master] tests: Make PHPUnit data provider static

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

Change #1312251 had a related patch set uploaded (by Umherirrender; author: Umherirrender):

[mediawiki/extensions/ConfirmEdit@master] tests: Make PHPUnit data provider static

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

Change #1312251 merged by jenkins-bot:

[mediawiki/extensions/ConfirmEdit@master] tests: Make PHPUnit data provider static

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

Change #1312250 merged by jenkins-bot:

[mediawiki/extensions/EventBus@master] tests: Make PHPUnit data provider static

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