Skip to content

Commit 5a54a50

Browse files
committed
Ignore unused params, to make both PHPCS and Psalm happy
1 parent 4c1601e commit 5a54a50

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

test/RoaveTest/SecurityAdvisories/AdvisorySources/GetAdvisoriesFromGithubApiTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@
4040
use Roave\SecurityAdvisories\Advisory;
4141
use Roave\SecurityAdvisories\AdvisorySources\GetAdvisoriesFromGithubApi;
4242
use UnexpectedValueException;
43+
4344
use function array_values;
4445

4546
#[CoversClass(GetAdvisoriesFromGithubApi::class)]

test/RoaveTest/SecurityAdvisories/VersionConstraintTest.php

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,13 +47,16 @@ public function testFromRangeWithComplexRanges(string $stringConstraint, string
4747
self::assertSame($expectedNormalization, $constraint->getConstraintString());
4848
}
4949

50-
/** @dataProvider mergeableRangesProvider */
50+
/**
51+
* @dataProvider mergeableRangesProvider
52+
* @psalm-suppress UnusedParam
53+
*/
5154
#[DataProvider('mergeableRangesProvider')]
5255
public function testMergeWithMergeableRanges(
5356
string $constraintString1,
5457
string $constraintString2,
55-
bool $_,
56-
bool $__,
58+
bool $ignored1,
59+
bool $ignored2,
5760
): void {
5861
$constraint1 = VersionConstraint::fromString($constraintString1);
5962
$constraint2 = VersionConstraint::fromString($constraintString2);

0 commit comments

Comments
 (0)