diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/PlaceTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/PlaceTest.java index 806c35e819..9fe8c11336 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/PlaceTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/PlaceTest.java @@ -33,10 +33,14 @@ * Test case for {@link Place}. * * @since 0.11 + * @todo #2297:60min Add message to all JUnit assertions and remove corresponding + * {@code @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage")} annotation. */ +@SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") final class PlaceTest { @Test + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void makesPath() { MatcherAssert.assertThat( new Place("hello.foo.bar") @@ -48,6 +52,7 @@ void makesPath() { } @Test + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void makesSimplePath() { MatcherAssert.assertThat( new Place("hey") diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/UnspileMojoTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/UnspileMojoTest.java index 195a317155..7c111cb652 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/UnspileMojoTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/UnspileMojoTest.java @@ -53,6 +53,10 @@ void cleans(@TempDir final Path temp) throws IOException { .with("classesDir", classes.toFile()) .execute(UnspileMojo.class); MatcherAssert.assertThat( + String.format( + "UnspileMojo unable to remove %s class", + foo + ), Files.exists(foo), Matchers.is(false) ); diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsDepgraphTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsDepgraphTest.java index f694161141..dbe72fb2bd 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsDepgraphTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsDepgraphTest.java @@ -39,6 +39,8 @@ * Test case for {@link DcsDepgraph.DcsJson}. * * @since 0.28.11 + * @todo #2297:60min Add message to all JUnit assertions and remove corresponding + * {@code @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage")} annotation. */ final class DcsDepgraphTest { @@ -47,6 +49,7 @@ final class DcsDepgraphTest { "eo-math-dependencies-transient-dependency.json, 3", "eo-math-dependencies-without-foreign.json, 7" }) + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void readsAllDependenciesFromJsonFile( final String name, final long number, diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsEachWithoutTransitiveTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsEachWithoutTransitiveTest.java index 22339cc5ee..a2298ff0ef 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsEachWithoutTransitiveTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsEachWithoutTransitiveTest.java @@ -37,10 +37,13 @@ * Test case for {@link DcsEachWithoutTransitive}. * * @since 0.30 + * @todo #2297:60min Add message to all JUnit assertions and remove corresponding + * {@code @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage")} annotation. */ final class DcsEachWithoutTransitiveTest { @Test + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void failsIfHasTransitiveDependencies() { Assertions.assertThrows( IllegalStateException.class, @@ -52,6 +55,7 @@ void failsIfHasTransitiveDependencies() { } @Test + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void keepsDependenciesThatHaveTeStDependenciesAsTransitive() { final DcsFake original = new DcsFake(); MatcherAssert.assertThat( @@ -64,6 +68,7 @@ void keepsDependenciesThatHaveTeStDependenciesAsTransitive() { } @Test + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void keepsDependencyThatHasTheSameDependencyAsTransitive() { final DcsFake original = new DcsFake(); MatcherAssert.assertThat( @@ -76,6 +81,7 @@ void keepsDependencyThatHasTheSameDependencyAsTransitive() { } @Test + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void keepsDependencyThatHasRuntimeDependencyAsTransitive() { final DcsFake original = new DcsFake(); MatcherAssert.assertThat( diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsWithRuntimeTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsWithRuntimeTest.java index 44f7dd0adb..259203e5ab 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsWithRuntimeTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsWithRuntimeTest.java @@ -33,11 +33,14 @@ * Test case for {@link DcsWithRuntime}. * * @since 0.28.11 + * @todo #2297:60min Add message to all JUnit assertions and remove corresponding + * {@code @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage")} annotation. */ final class DcsWithRuntimeTest { @Test @ExtendWith(WeAreOnline.class) + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void addsHardcodedVersionOfRuntimeDependency() { MatcherAssert.assertThat( new DcsWithRuntime( @@ -50,6 +53,7 @@ void addsHardcodedVersionOfRuntimeDependency() { @Test @ExtendWith(WeAreOnline.class) + @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage") void addsRemoteVersionOfRuntimeDependency() { MatcherAssert.assertThat( new DcsWithRuntime( diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/hash/ChNarrowTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/hash/ChNarrowTest.java index 268fdd9b22..1c9eca7ba3 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/hash/ChNarrowTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/hash/ChNarrowTest.java @@ -46,6 +46,10 @@ final class ChNarrowTest { }) void cutsHashCorrectly(final String input, final String output) { MatcherAssert.assertThat( + String.format( + "The hash of %s was calculated incorrectly", + input + ), new ChNarrow( new CommitHash.ChConstant(input) ).value(), diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/latex/LatexTemplateTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/latex/LatexTemplateTest.java index 0dbe9ada86..873363b2f5 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/latex/LatexTemplateTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/latex/LatexTemplateTest.java @@ -40,6 +40,7 @@ final class LatexTemplateTest { @Test void generatesFullTemplate() { MatcherAssert.assertThat( + "LatexTemplate.asString() gives incorrect template", new LatexTemplate( "+package f\n[args] > main\n stdout \"Hello!\"" ).asString(), diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/rust/CommentedTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/rust/CommentedTest.java index e605c483b4..5ce91d88c1 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/rust/CommentedTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/rust/CommentedTest.java @@ -54,6 +54,7 @@ String content() { "//" ).save(new FtDefault(temp)); MatcherAssert.assertThat( + "The auto-generated file has wrong comment", new TextOf( temp.resolve("name.ext") ).asString(), diff --git a/eo-maven-plugin/src/test/java/org/eolang/maven/util/WalkTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/util/WalkTest.java index 079201d943..32c1a57fde 100644 --- a/eo-maven-plugin/src/test/java/org/eolang/maven/util/WalkTest.java +++ b/eo-maven-plugin/src/test/java/org/eolang/maven/util/WalkTest.java @@ -43,6 +43,7 @@ void findsFiles(@TempDir final Path temp) throws Exception { new HmBase(temp).save("", Paths.get("foo/hello/0.1/EObar/x.bin")); new HmBase(temp).save("", Paths.get("EOxxx/bar")); MatcherAssert.assertThat( + "Walk is not iterable with more than 1 item, but it must be", new Walk(temp).includes(new ListOf<>("EO**/*")), Matchers.iterableWithSize(1) ); diff --git a/eo-parser/src/test/java/org/eolang/parser/ParsingTrainTest.java b/eo-parser/src/test/java/org/eolang/parser/ParsingTrainTest.java index 65e3cf545b..2f83771121 100644 --- a/eo-parser/src/test/java/org/eolang/parser/ParsingTrainTest.java +++ b/eo-parser/src/test/java/org/eolang/parser/ParsingTrainTest.java @@ -46,6 +46,7 @@ final class ParsingTrainTest { @Test void buildsList() { MatcherAssert.assertThat( + "ParsingTrain is not iterable with more than 1 item, but it must be", new ParsingTrain(), Matchers.iterableWithSize(Matchers.greaterThan(1)) ); @@ -69,6 +70,7 @@ void stopsPipeline() { ) ); MatcherAssert.assertThat( + "XSL transformation don't work properly.", new Xsline( new ParsingTrain() ).pass(xml), @@ -97,6 +99,7 @@ void stopsPipeline() { }) void runsXslFunction(final String bytes, final String num) { MatcherAssert.assertThat( + "Failed to convert byte to int using the apply-func.xsl transformation", new Xsline( new ParsingTrain("/org/eolang/parser/apply-func.xsl") ).pass(new XMLDocument(String.format("%s", bytes))), @@ -116,6 +119,10 @@ void parsesPacks(final String pack) throws Exception { ); } MatcherAssert.assertThat( + String.format( + "The %s check pack is failed.", + pack + ), check.failures(), Matchers.empty() ); @@ -125,6 +132,10 @@ void parsesPacks(final String pack) throws Exception { @ClasspathSource(value = "org/eolang/parser/xax/", glob = "**.yml") void createsXaxStoryWithXslStylesheets(final String yaml) { MatcherAssert.assertThat( + String.format( + "The %s xax check is failed", + yaml + ), new XaxStory(yaml), Matchers.is(true) ); diff --git a/pom.xml b/pom.xml index d8bbebb1c8..0e9f9478b2 100644 --- a/pom.xml +++ b/pom.xml @@ -369,7 +369,7 @@ SOFTWARE. com.qulice qulice-maven-plugin - 0.22.0 + 0.22.2 checkstyle:/src/site/resources/.*