Page MenuHomePhabricator

Make PHPUnit dataProvider on AbstractChangesListSpecialPageTestCase static
Closed, ResolvedPublic

Description

The @dataProvider annotation should be a static function, make

  • AbstractChangesListSpecialPageTestCase::provideParseParameters
  • AbstractChangesListSpecialPageTestCase::validateOptionsProvider

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

Event Timeline

Thank you for tagging this task with good first task for Wikimedia newcomers!

Newcomers often may not be aware of things that may seem obvious to seasoned contributors, so please take a moment to reflect on how this task might look to somebody who has never contributed to Wikimedia projects.

A good first task is a self-contained, non-controversial task with a clear approach. It should be well-described with pointers to help a completely new contributor, for example it should clearly pointed to the codebase URL and provide clear steps to help a contributor get setup for success. We've included some guidelines at https://phabricator.wikimedia.org/tag/good_first_task/ !

Thank you for helping us drive new contributions to our projects <3

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

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

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

Change #1143164 merged by jenkins-bot:

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

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