diff options
author | Tim Starling <tstarling@wikimedia.org> | 2023-03-23 22:36:19 +1100 |
---|---|---|
committer | Krinkle <krinkle@fastmail.com> | 2023-03-24 02:53:57 +0000 |
commit | 5e30a927bc2d4027a660a3f7f34315a93bb06215 (patch) | |
tree | 5927c6b0a51ed6a5f44062c4b0a183c149170061 /tests/phpunit/includes/exception/PermissionsErrorTest.php | |
parent | 5ee6384bdf1f62e49ad6c2df600755be6bf1e815 (diff) | |
download | mediawikicore-5e30a927bc2d4027a660a3f7f34315a93bb06215.tar.gz mediawikicore-5e30a927bc2d4027a660a3f7f34315a93bb06215.zip |
tests: Make some PHPUnit data providers static
Just methods where adding "static" to the declaration was enough, I
didn't do anything with providers that used $this.
Initially by search and replace. There were many mistakes which I
found mostly by running the PHPStorm inspection which searches for
$this usage in a static method. Later I used the PHPStorm "make static"
action which avoids the more obvious mistakes.
Bug: T332865
Change-Id: I47ed6692945607dfa5c139d42edbd934fa4f3a36
Diffstat (limited to 'tests/phpunit/includes/exception/PermissionsErrorTest.php')
-rw-r--r-- | tests/phpunit/includes/exception/PermissionsErrorTest.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/phpunit/includes/exception/PermissionsErrorTest.php b/tests/phpunit/includes/exception/PermissionsErrorTest.php index fdb29282bfdf..b5b40da2bdfb 100644 --- a/tests/phpunit/includes/exception/PermissionsErrorTest.php +++ b/tests/phpunit/includes/exception/PermissionsErrorTest.php @@ -12,7 +12,7 @@ class PermissionsErrorTest extends MediaWikiIntegrationTestCase { $this->setGroupPermissions( '*', 'testpermission', true ); } - public function provideConstruction() { + public static function provideConstruction() { $status = new PermissionStatus(); $status->error( 'cat', 1, 2 ); $status->warning( 'dog', 3, 4 ); @@ -32,7 +32,7 @@ class PermissionsErrorTest extends MediaWikiIntegrationTestCase { $this->assertArrayEquals( $expected, $e->errors ); } - public function provideInvalidConstruction() { + public static function provideInvalidConstruction() { yield [ null, null ]; yield [ null, [] ]; yield [ null, new PermissionStatus() ]; |