Skip to content

Commit d2dced9

Browse files
committed
File/Get*Tests: minor tweak
No need for the `isset()` checks.
1 parent 0e1eb6f commit d2dced9

File tree

3 files changed

+3
-9
lines changed

3 files changed

+3
-9
lines changed

tests/Core/File/GetMemberPropertiesTest.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ public function testGetMemberProperties($identifier, $expected)
3636
];
3737

3838
foreach ($exclude as $b) {
39-
if (isset($found[$b]) === true) {
40-
unset($found[$b]);
41-
}
39+
unset($found[$b]);
4240
}
4341

4442
$this->assertEquals($expected, $found);

tests/Core/File/GetMethodParametersTest.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1259,9 +1259,7 @@ private function getMethodParametersTestHelper($commentString, $expected)
12591259

12601260
foreach ($found as $i => $data) {
12611261
foreach ($exclude as $b) {
1262-
if (isset($found[$i][$b]) === true) {
1263-
unset($found[$i][$b]);
1264-
}
1262+
unset($found[$i][$b]);
12651263
}
12661264
}
12671265

tests/Core/File/GetMethodPropertiesTest.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -954,9 +954,7 @@ private function getMethodPropertiesTestHelper($commentString, $expected)
954954
];
955955

956956
foreach ($exclude as $b) {
957-
if (isset($found[$b]) === true) {
958-
unset($found[$b]);
959-
}
957+
unset($found[$b]);
960958
}
961959

962960
$this->assertEquals($expected, $found);

0 commit comments

Comments
 (0)