objectionary#2937 remove SuppressWarnings

c71n93 · Mar 25, 2024 · a6e15d6 · a6e15d6
1 parent 0a6eb97
commit a6e15d6
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 18 deletions.
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
@@ -33,14 +33,10 @@
  * 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")
@@ -52,7 +48,6 @@ 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/dependencies/DcsDepgraphTest.java b/eo-maven-plugin/src/test/java/org/eolang/maven/dependencies/DcsDepgraphTest.java
@@ -39,8 +39,6 @@
  * 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 {
 
@@ -49,7 +47,6 @@ 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/...aven-plugin/src/test/java/org/eolang/maven/dependencies/DcsEachWithoutTransitiveTest.java b/...aven-plugin/src/test/java/org/eolang/maven/dependencies/DcsEachWithoutTransitiveTest.java
@@ -37,13 +37,10 @@
  * 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,
@@ -55,7 +52,6 @@ void failsIfHasTransitiveDependencies() {
     }
 
     @Test
-    @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage")
     void keepsDependenciesThatHaveTeStDependenciesAsTransitive() {
         final DcsFake original = new DcsFake();
         MatcherAssert.assertThat(
@@ -68,7 +64,6 @@ void keepsDependenciesThatHaveTeStDependenciesAsTransitive() {
     }
 
     @Test
-    @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage")
     void keepsDependencyThatHasTheSameDependencyAsTransitive() {
         final DcsFake original = new DcsFake();
         MatcherAssert.assertThat(
@@ -81,7 +76,6 @@ 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
@@ -33,14 +33,11 @@
  * 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(
@@ -53,7 +50,6 @@ void addsHardcodedVersionOfRuntimeDependency() {
 
     @Test
     @ExtendWith(WeAreOnline.class)
-    @SuppressWarnings("PMD.JUnitAssertionsShouldIncludeMessage")
     void addsRemoteVersionOfRuntimeDependency() {
         MatcherAssert.assertThat(
             new DcsWithRuntime(