Skip to content

Commit 5aceb18

Browse files
committed
Squiz/ScopeKeywordSpacing: move parse error test to separate file
Done in a way to maintain the error line numbers for the rest of the file.
1 parent 5e5e6dc commit 5aceb18

File tree

3 files changed

+15
-4
lines changed

3 files changed

+15
-4
lines changed

src/Standards/Squiz/Tests/WhiteSpace/ScopeKeywordSpacingUnitTest.1.inc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,10 +77,10 @@ class MyOtherClass
7777
$varQ = 'string',
7878
$varR = 123;
7979

80-
// Intentionally missing a semicolon for testing.
8180
public
8281
$varS,
83-
$varT
82+
$varT,
83+
$varU;
8484
}
8585

8686
// Issue #3188 - static as return type.

src/Standards/Squiz/Tests/WhiteSpace/ScopeKeywordSpacingUnitTest.1.inc.fixed

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,10 @@ class MyOtherClass
7272
$varQ = 'string',
7373
$varR = 123;
7474

75-
// Intentionally missing a semicolon for testing.
7675
public
7776
$varS,
78-
$varT
77+
$varT,
78+
$varU;
7979
}
8080

8181
// Issue #3188 - static as return type.
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
<?php
2+
3+
// Intentional parse error.
4+
// Testing handling of multi-property statements during live coding (missing semicolon after statement).
5+
// This must be the only test in this file.
6+
class MyOtherClass
7+
{
8+
public
9+
$varS,
10+
$varT
11+
}

0 commit comments

Comments
 (0)