Chris@16
|
1 <?php
|
Chris@16
|
2
|
Chris@16
|
3 /*
|
Chris@16
|
4 * This file is part of Psy Shell.
|
Chris@16
|
5 *
|
Chris@16
|
6 * (c) 2012-2018 Justin Hileman
|
Chris@16
|
7 *
|
Chris@16
|
8 * For the full copyright and license information, please view the LICENSE
|
Chris@16
|
9 * file that was distributed with this source code.
|
Chris@16
|
10 */
|
Chris@16
|
11
|
Chris@16
|
12 namespace Psy\Test\CodeCleaner;
|
Chris@16
|
13
|
Chris@16
|
14 use Psy\CodeCleaner\ListPass;
|
Chris@16
|
15
|
Chris@16
|
16 class ListPassTest extends CodeCleanerTestCase
|
Chris@16
|
17 {
|
Chris@16
|
18 public function setUp()
|
Chris@16
|
19 {
|
Chris@16
|
20 $this->setPass(new ListPass());
|
Chris@16
|
21 }
|
Chris@16
|
22
|
Chris@16
|
23 /**
|
Chris@16
|
24 * @dataProvider invalidStatements
|
Chris@16
|
25 * @expectedException \Psy\Exception\ParseErrorException
|
Chris@16
|
26 */
|
Chris@16
|
27 public function testProcessInvalidStatement($code, $expectedMessage)
|
Chris@16
|
28 {
|
Chris@16
|
29 if (method_exists($this, 'setExpectedException')) {
|
Chris@16
|
30 $this->setExpectedException('Psy\Exception\ParseErrorException', $expectedMessage);
|
Chris@16
|
31 } else {
|
Chris@16
|
32 $this->expectExceptionMessage($expectedMessage);
|
Chris@16
|
33 }
|
Chris@16
|
34
|
Chris@16
|
35 $stmts = $this->parse($code);
|
Chris@16
|
36 $this->traverser->traverse($stmts);
|
Chris@16
|
37 }
|
Chris@16
|
38
|
Chris@16
|
39 public function invalidStatements()
|
Chris@16
|
40 {
|
Chris@16
|
41 // Not typo. It is ambiguous whether "Syntax" or "syntax".
|
Chris@16
|
42 $errorShortListAssign = "yntax error, unexpected '='";
|
Chris@16
|
43 $errorEmptyList = 'Cannot use empty list';
|
Chris@16
|
44 $errorAssocListAssign = 'Syntax error, unexpected T_CONSTANT_ENCAPSED_STRING, expecting \',\' or \')\'';
|
Chris@16
|
45 $errorNonVariableAssign = 'Assignments can only happen to writable values';
|
Chris@16
|
46 $errorPhpParserSyntax = 'PHP Parse error: Syntax error, unexpected';
|
Chris@16
|
47
|
Chris@16
|
48 $invalidExpr = [
|
Chris@16
|
49 ['list() = array()', $errorEmptyList],
|
Chris@16
|
50 ['list("a") = array(1)', $errorPhpParserSyntax],
|
Chris@16
|
51 ];
|
Chris@16
|
52
|
Chris@16
|
53 if (version_compare(PHP_VERSION, '7.1', '<')) {
|
Chris@16
|
54 return array_merge($invalidExpr, [
|
Chris@16
|
55 ['list("a" => _) = array("a" => 1)', $errorPhpParserSyntax],
|
Chris@16
|
56 ['[] = []', $errorShortListAssign],
|
Chris@16
|
57 ['[$a] = [1]', $errorShortListAssign],
|
Chris@16
|
58 ['list("a" => $a) = array("a" => 1)', $errorAssocListAssign],
|
Chris@16
|
59 ]);
|
Chris@16
|
60 }
|
Chris@16
|
61
|
Chris@16
|
62 return array_merge($invalidExpr, [
|
Chris@16
|
63 ['list("a" => _) = array("a" => 1)', $errorPhpParserSyntax],
|
Chris@16
|
64 ['["a"] = [1]', $errorNonVariableAssign],
|
Chris@16
|
65 ['[] = []', $errorEmptyList],
|
Chris@16
|
66 ['[,] = [1,2]', $errorEmptyList],
|
Chris@16
|
67 ]);
|
Chris@16
|
68 }
|
Chris@16
|
69
|
Chris@16
|
70 /**
|
Chris@16
|
71 * @dataProvider validStatements
|
Chris@16
|
72 */
|
Chris@16
|
73 public function testProcessValidStatement($code)
|
Chris@16
|
74 {
|
Chris@16
|
75 $stmts = $this->parse($code);
|
Chris@16
|
76 $this->traverser->traverse($stmts);
|
Chris@16
|
77 $this->assertTrue(true);
|
Chris@16
|
78 }
|
Chris@16
|
79
|
Chris@16
|
80 public function validStatements()
|
Chris@16
|
81 {
|
Chris@16
|
82 $validExpr = [
|
Chris@16
|
83 ['list($a) = array(1)'],
|
Chris@16
|
84 ['list($x, $y) = array(1, 2)'],
|
Chris@16
|
85 ];
|
Chris@16
|
86
|
Chris@16
|
87 if (version_compare(PHP_VERSION, '7.1', '>=')) {
|
Chris@16
|
88 return array_merge($validExpr, [
|
Chris@16
|
89 ['[$a] = array(1)'],
|
Chris@16
|
90 ['list($b) = [2]'],
|
Chris@16
|
91 ['[$x, $y] = array(1, 2)'],
|
Chris@16
|
92 ['[$a] = [1]'],
|
Chris@16
|
93 ['[$x, $y] = [1, 2]'],
|
Chris@16
|
94 ['["_" => $v] = ["_" => 1]'],
|
Chris@16
|
95 ['[$a,] = [1,2,3]'],
|
Chris@16
|
96 ['[,$b] = [1,2,3]'],
|
Chris@16
|
97 ['[$a,,$c] = [1,2,3]'],
|
Chris@16
|
98 ['[$a,,,] = [1,2,3]'],
|
Chris@16
|
99 ]);
|
Chris@16
|
100 }
|
Chris@16
|
101
|
Chris@16
|
102 return $validExpr;
|
Chris@16
|
103 }
|
Chris@16
|
104 }
|