Skip to content

Commit

Permalink
Fix incorrect argument type
Browse files Browse the repository at this point in the history
Apply suggestions from code review

Signed-off-by: Ihor Sviziev <[email protected]>
  • Loading branch information
ihor-sviziev committed Sep 21, 2021
1 parent 913050e commit 0ea3012
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions test/Reflection/ParameterReflectionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -59,11 +59,11 @@ public function testTypeReturn($param, $type)
}

/**
* This test covers
* This test covers type detection when not all params declared in phpDoc block
*
* @dataProvider paramType2
* @dataProvider paramTypeWithNotAllParamsDeclared
*/
public function testTypeReturn2(string $param, string $type): void
public function testTypeReturnWithNotAllParamsDeclared(string $param, string $type): void
{
$parameter = new Reflection\ParameterReflection(
[TestAsset\TestSampleClass5::class, 'methodWithNotAllParamsDeclared'],
Expand Down Expand Up @@ -100,7 +100,7 @@ public function paramType(): array
* @return string[][]
* @psalm-return non-empty-list<array{non-empty-string, non-empty-string}>
*/
public function paramType2(): array
public function paramTypeWithNotAllParamsDeclared(): array
{
return [
['one', 'string'],
Expand Down

0 comments on commit 0ea3012

Please sign in to comment.