Skip to content

Commit 2a535e8

Browse files
committed
Add tests for unused before require
1 parent 01613e7 commit 2a535e8

File tree

2 files changed

+41
-0
lines changed

2 files changed

+41
-0
lines changed
Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
<?php
2+
namespace VariableAnalysis\Tests\VariableAnalysisSniff;
3+
4+
use VariableAnalysis\Tests\BaseTestCase;
5+
6+
class UnusedFollowedByRequire extends BaseTestCase {
7+
public function testUnusedFollowedByRequireDefault() {
8+
$fixtureFile = $this->getFixture('UnusedFollowedByRequireFixture.php');
9+
$phpcsFile = $this->prepareLocalFileForSniffs($fixtureFile);
10+
$phpcsFile->process();
11+
$lines = $this->getWarningLineNumbersFromFile($phpcsFile);
12+
$expectedWarnings = [
13+
2,
14+
3,
15+
4,
16+
];
17+
$this->assertEquals($expectedWarnings, $lines);
18+
}
19+
20+
public function testUnusedFollowedByRequireWhenSet() {
21+
$fixtureFile = $this->getFixture('UnusedFollowedByRequireFixture.php');
22+
$phpcsFile = $this->prepareLocalFileForSniffs($fixtureFile);
23+
$phpcsFile->ruleset->setSniffProperty(
24+
'VariableAnalysis\Sniffs\CodeAnalysis\VariableAnalysisSniff',
25+
'allowUnusedVariablesBeforeRequire',
26+
'true'
27+
);
28+
$phpcsFile->process();
29+
$lines = $this->getWarningLineNumbersFromFile($phpcsFile);
30+
$expectedWarnings = [
31+
4,
32+
];
33+
$this->assertEquals($expectedWarnings, $lines);
34+
}
35+
}
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
<?php
2+
function require_file_function($param) { // unused variable $param
3+
$var = 'something'; // unused variable $var
4+
activate_code($data); // undefined variable $data
5+
require_once __DIR__ . '/views/my-view.php';
6+
}

0 commit comments

Comments
 (0)