diff options
| author | Tim Starling <tstarling@wikimedia.org> | 2023-03-24 14:21:20 +1100 |
|---|---|---|
| committer | Krinkle <krinkle@fastmail.com> | 2023-03-25 00:39:06 +0000 |
| commit | be3018b268590a3c4523eb19d26891f323ab6115 (patch) | |
| tree | 6e25dbf00787dcc789d376e67bfcad66f817b5c2 /tests/phpunit/includes/api/query/ApiQuerySiteinfoTest.php | |
| parent | 317b460500725f81797d27c33bd68756055fb44c (diff) | |
| download | mediawikicore-be3018b268590a3c4523eb19d26891f323ab6115.tar.gz mediawikicore-be3018b268590a3c4523eb19d26891f323ab6115.zip | |
Just another 80 or so PHPStorm inspection fixes (#4)
* Unnecessary regex modifier. I agree with this inspection which flags
/s modifiers on regexes that don't use a dot.
* Property declared dynamically.
* Unused local variable. But it's acceptable for an unused local
variable to take the return value of a method under test, when it is
being tested for its side-effects. And it's acceptable for an unused
local variable to document unused list expansion elements, or the
nature of array keys in a foreach.
Change-Id: I067b5b45dd1138c00e7269b66d3d1385f202fe7f
Diffstat (limited to 'tests/phpunit/includes/api/query/ApiQuerySiteinfoTest.php')
| -rw-r--r-- | tests/phpunit/includes/api/query/ApiQuerySiteinfoTest.php | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/phpunit/includes/api/query/ApiQuerySiteinfoTest.php b/tests/phpunit/includes/api/query/ApiQuerySiteinfoTest.php index 4f7b7c0ed14e..a818b5bfccba 100644 --- a/tests/phpunit/includes/api/query/ApiQuerySiteinfoTest.php +++ b/tests/phpunit/includes/api/query/ApiQuerySiteinfoTest.php @@ -353,7 +353,6 @@ class ApiQuerySiteinfoTest extends ApiTestCase { } if ( $val['name'] === 'viscount' ) { - $viscountFound = true; $this->assertSame( [ 'perambulate' ], $val['rights'] ); $this->assertSame( $userAllGroups, $val['add'] ); } elseif ( $val['name'] === 'bot' ) { |