Skip to content

Commit 19b5775

Browse files
jrfnlkukulich
authored andcommitted
Squiz/InlineComment: fix enum test
The enum test was placed _below_ a test which was marked as testing a specific situation at the end of a file, which invalidated the "end of file" test. Fixed by moving the new test up.
1 parent 37bc4e0 commit 19b5775

File tree

2 files changed

+16
-16
lines changed

2 files changed

+16
-16
lines changed

src/Standards/Squiz/Tests/Commenting/InlineCommentUnitTest.inc

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,14 @@ final class MyClass
165165
final public function test() {}
166166
}
167167

168+
/**
169+
* Comment should be ignored.
170+
*
171+
*/
172+
enum MyEnum {
173+
174+
}
175+
168176
/*
169177
* N.B.: The below test line must be the last test in the file.
170178
* Testing that a new line after an inline comment when it's the last non-whitespace
@@ -173,11 +181,3 @@ final class MyClass
173181
*/
174182

175183
// For this test line having an empty line below it, is fine.
176-
177-
/**
178-
* Comment should be ignored.
179-
*
180-
*/
181-
enum MyEnum {
182-
183-
}

src/Standards/Squiz/Tests/Commenting/InlineCommentUnitTest.inc.fixed

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,14 @@ final class MyClass
158158
final public function test() {}
159159
}
160160

161+
/**
162+
* Comment should be ignored.
163+
*
164+
*/
165+
enum MyEnum {
166+
167+
}
168+
161169
/*
162170
* N.B.: The below test line must be the last test in the file.
163171
* Testing that a new line after an inline comment when it's the last non-whitespace
@@ -166,11 +174,3 @@ final class MyClass
166174
*/
167175

168176
// For this test line having an empty line below it, is fine.
169-
170-
/**
171-
* Comment should be ignored.
172-
*
173-
*/
174-
enum MyEnum {
175-
176-
}

0 commit comments

Comments
 (0)