Skip to content

Commit 6e7157f

Browse files
committed
Squiz/ScopeKeywordSpacing: add some tests which shouldn't throw errors
Includes: * Making the spacing differences in the new tests a little more varied. * Changing an invalid modifier keyword combination to a valid one.
1 parent 6bed8f0 commit 6e7157f

File tree

2 files changed

+21
-6
lines changed

2 files changed

+21
-6
lines changed

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

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,17 +152,25 @@ class FinalTest {
152152
final public static function create(ContainerInterface $container) {}
153153
}
154154

155-
final class FinalTest2 {
155+
final class FinalTest2 {
156156
}
157157

158-
final abstract class FinalTest3 {
159-
}
158+
final
159+
readonly class FinalTest3 {}
160160

161161
class FinalTest4 {
162162
final const X = "foo";
163-
final public const Y = "bar";
163+
final public const Y = "bar";
164164
}
165165

166166
abstract class AbstractTest {
167167
abstract public function foo();
168168
}
169+
170+
final class FinalSpacingCorrect {
171+
public final const SPACING_CORRECT = true;
172+
}
173+
174+
abstract class AbstractSpacingCorrect {
175+
public abstract function spacingCorrect() {}
176+
}

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

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,8 +149,7 @@ class FinalTest {
149149
final class FinalTest2 {
150150
}
151151

152-
final abstract class FinalTest3 {
153-
}
152+
final readonly class FinalTest3 {}
154153

155154
class FinalTest4 {
156155
final const X = "foo";
@@ -160,3 +159,11 @@ class FinalTest4 {
160159
abstract class AbstractTest {
161160
abstract public function foo();
162161
}
162+
163+
final class FinalSpacingCorrect {
164+
public final const SPACING_CORRECT = true;
165+
}
166+
167+
abstract class AbstractSpacingCorrect {
168+
public abstract function spacingCorrect() {}
169+
}

0 commit comments

Comments
 (0)