aboutsummaryrefslogtreecommitdiffstats
path: root/tests/phpunit/includes/exception/PermissionsErrorTest.php
diff options
context:
space:
mode:
authorTim Starling <tstarling@wikimedia.org>2023-03-23 22:36:19 +1100
committerKrinkle <krinkle@fastmail.com>2023-03-24 02:53:57 +0000
commit5e30a927bc2d4027a660a3f7f34315a93bb06215 (patch)
tree5927c6b0a51ed6a5f44062c4b0a183c149170061 /tests/phpunit/includes/exception/PermissionsErrorTest.php
parent5ee6384bdf1f62e49ad6c2df600755be6bf1e815 (diff)
downloadmediawikicore-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.php4
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() ];