Merge pull request #1719 from kluever/patch-2

Fix "ReturnValueIgnored" errors flagged by error-prone * Update AssertTest.java to use `assertNotNull()` on places where the return value of `e.getMessage()` is ignored
junit-team · Nov 21, 2022 · 5cc2cdc · 5cc2cdc
2 parents d5db99d + 86b979e
commit 5cc2cdc
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/test/java/junit/tests/framework/AssertTest.java b/src/test/java/junit/tests/framework/AssertTest.java
@@ -82,15 +82,15 @@ public void testAssertStringNotEqualsNull() {
             assertEquals("foo", null);
             fail();
         } catch (ComparisonFailure e) {
-            e.getMessage(); // why no assertion?
+            assertNotNull(e.getMessage());
         }
     }
 
     public void testAssertNullNotEqualsNull() {
         try {
             assertEquals(null, new Object());
         } catch (AssertionFailedError e) {
-            e.getMessage(); // why no assertion?
+            assertNotNull(e.getMessage());
             return;
         }
         fail();
@@ -168,4 +168,4 @@ public void testAssertNotSameFailsNull() {
         }
         fail();
     }
-}
+}