Skip to content

Commit 9deeb9b

Browse files
committed
UnusedVariableSniff: More tests
1 parent fd3d8ff commit 9deeb9b

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

tests/Sniffs/Variables/UnusedVariableSniffTest.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ public function testErrors(): void
1717
{
1818
$report = self::checkFile(__DIR__ . '/data/unusedVariableErrors.php');
1919

20-
self::assertSame(19, $report->getErrorCount());
20+
self::assertSame(21, $report->getErrorCount());
2121

2222
self::assertSniffError($report, 7, UnusedVariableSniff::CODE_UNUSED_VARIABLE, 'Unused variable $firstMatch.');
2323
self::assertSniffError($report, 12, UnusedVariableSniff::CODE_UNUSED_VARIABLE, 'Unused variable $unused.');
@@ -38,6 +38,8 @@ public function testErrors(): void
3838
self::assertSniffError($report, 78, UnusedVariableSniff::CODE_UNUSED_VARIABLE, 'Unused variable $unused.');
3939
self::assertSniffError($report, 83, UnusedVariableSniff::CODE_UNUSED_VARIABLE, 'Unused variable $a.');
4040
self::assertSniffError($report, 88, UnusedVariableSniff::CODE_UNUSED_VARIABLE, 'Unused variable $a.');
41+
self::assertSniffError($report, 94, UnusedVariableSniff::CODE_UNUSED_VARIABLE, 'Unused variable $sameNameAsParameter.');
42+
self::assertSniffError($report, 101, UnusedVariableSniff::CODE_UNUSED_VARIABLE, 'Unused variable $a.');
4143
}
4244

4345
}

tests/Sniffs/Variables/data/unusedVariableErrors.php

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,3 +91,14 @@ function () {
9191
TEXT;
9292
};
9393

94+
$sameNameAsParameter = null;
95+
96+
function ($sameNameAsParameter) {
97+
98+
};
99+
100+
function () {
101+
$a = null;
102+
self::$a = null;
103+
};
104+

0 commit comments

Comments
 (0)