[go: up one dir, main page]

Page MenuHomePhabricator

Make PHPUnit dataProvider static in WikibaseQualityConstraints tests
Closed, ResolvedPublic

Description

The @dataProvider annotation should be a static function, check and make data provider in the extension, at least:

  • ContemporaryCheckerTest with provideStandardStatements
  • HtmlTableBuilderTest with toHtmlDataProvider
  • WikibaseQualityConstraintsHooksTest with provideChanges

static and adjust the usages (More infos at T332865).

Initial work was done in 0d89717c1f999529bdcc5f5cf313f7b1fbdb7284

Event Timeline

Apparently half this work was already done, but unfortunately I only found that out after I made very similar changes myself, because it wasn’t linked to this task.

Change 923539 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/WikibaseQualityConstraints@master] tests: Make most data providers static

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

Change 923540 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/WikibaseQualityConstraints@master] tests: Make DefaultConfig::getDefaultConfig() static

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

Change 923541 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/WikibaseQualityConstraints@master] TimeValueComparerTest: Make data provider static

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

Change 923542 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/WikibaseQualityConstraints@master] RangeCheckerHelperTest: Make data provider static

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

Change 923543 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/WikibaseQualityConstraints@master] ContemporaryCheckerTest: Make data provider static

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

Change 923541 abandoned by Lucas Werkmeister (WMDE):

[mediawiki/extensions/WikibaseQualityConstraints@master] TimeValueComparerTest: Make data provider static

Reason:

The important part of this change was already done in I8302e26edc; the static types will just have to wait for another time.

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

Change 923542 abandoned by Lucas Werkmeister (WMDE):

[mediawiki/extensions/WikibaseQualityConstraints@master] RangeCheckerHelperTest: Make data provider static

Reason:

The important part of this change was already done in I8302e26edc; the static types will just have to wait for another time.

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

Change 923540 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] tests: Make DefaultConfig::getDefaultConfig() static

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

Change 923539 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] WikibaseQualityConstraintsHooksTest: Make data provider static

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

Change 923543 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] ContemporaryCheckerTest: Make data provider static

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