Skip to content

Commit

Permalink
Squiz/ScopeKeywordSpacing: add some tests which shouldn't throw errors
Browse files Browse the repository at this point in the history
Includes:
* Making the spacing differences in the new tests a little more varied.
* Changing an invalid modifier keyword combination to a valid one.
  • Loading branch information
jrfnl committed Jan 8, 2025
1 parent 6bed8f0 commit 6e7157f
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -152,17 +152,25 @@ class FinalTest {
final public static function create(ContainerInterface $container) {}
}

final class FinalTest2 {
final class FinalTest2 {
}

final abstract class FinalTest3 {
}
final
readonly class FinalTest3 {}

class FinalTest4 {
final const X = "foo";
final public const Y = "bar";
final public const Y = "bar";
}

abstract class AbstractTest {
abstract public function foo();
}

final class FinalSpacingCorrect {
public final const SPACING_CORRECT = true;
}

abstract class AbstractSpacingCorrect {
public abstract function spacingCorrect() {}
}
Original file line number Diff line number Diff line change
Expand Up @@ -149,8 +149,7 @@ class FinalTest {
final class FinalTest2 {
}

final abstract class FinalTest3 {
}
final readonly class FinalTest3 {}

class FinalTest4 {
final const X = "foo";
Expand All @@ -160,3 +159,11 @@ class FinalTest4 {
abstract class AbstractTest {
abstract public function foo();
}

final class FinalSpacingCorrect {
public final const SPACING_CORRECT = true;
}

abstract class AbstractSpacingCorrect {
public abstract function spacingCorrect() {}
}

0 comments on commit 6e7157f

Please sign in to comment.