Skip to content

Commit 730a77c

Browse files
committedMay 11, 2024·
Merge branch '__rultor'
2 parents ac58393 + 74aab50 commit 730a77c

File tree

1 file changed

+28
-7
lines changed

1 file changed

+28
-7
lines changed
 

‎qulice-checkstyle/src/test/java/com/qulice/checkstyle/CheckstyleValidatorTest.java

+28-7
Original file line numberDiff line numberDiff line change
@@ -459,23 +459,44 @@ void allowsOnlyProperlyOrderedAtClauses() throws Exception {
459459
* @throws Exception If something wrong happens inside
460460
*/
461461
@Test
462-
@Disabled
463462
void passesWindowsEndsOfLineWithoutException() throws Exception {
464-
this.validate("WindowsEol.java", false, "LICENSE found:");
463+
final String file = "WindowsEol.java";
464+
final Collection<Violation> results = this.runValidation(file, false);
465+
MatcherAssert.assertThat(
466+
"violation should be reported correctly",
467+
results,
468+
Matchers.contains(
469+
new ViolationMatcher(
470+
"Line does not match expected header line of ' */'.",
471+
file,
472+
"3",
473+
"HeaderCheck"
474+
)
475+
)
476+
);
465477
}
466478

467479
/**
468480
* Fail validation with Windows-style formatting of the license and
469481
* Linux-style formatting of the sources.
470482
* @throws Exception If something wrong happens inside
471-
* @todo #61:30min This test and passesWindowsEndsOfLineWithoutException
472-
* should be refactored to gather log4j logs and validate that they work
473-
* correctly. (see changes done in #61)
474483
*/
475484
@Test
476-
@Disabled
477485
void testWindowsEndsOfLineWithLinuxSources() throws Exception {
478-
this.runValidation("WindowsEolLinux.java", false);
486+
final String file = "WindowsEolLinux.java";
487+
final Collection<Violation> results = this.runValidation(file, false);
488+
MatcherAssert.assertThat(
489+
"violation should be reported correctly",
490+
results,
491+
Matchers.contains(
492+
new ViolationMatcher(
493+
"Line does not match expected header line of ' * Hello.'.",
494+
file,
495+
"2",
496+
"HeaderCheck"
497+
)
498+
)
499+
);
479500
}
480501

481502
/**

0 commit comments

Comments
 (0)
Please sign in to comment.