Migrate off the overload of fail(...) that accepts a message.

We're removing it from open-source Truth, since we're about to permanently commit to the remaining open-source Truth APIs, and we're not sure if assert_().fail(...) offers positive value when assertWithMessage(...).fail() is more in line with normal Truth practices.

MOE_MIGRATED_REVID=254043172

1 files changed, 8 insertions, 6 deletions

diff --git a/common/src/test/java/com/google/auto/common/OverridesTest.java b/common/src/test/java/com/google/auto/common/OverridesTest.java
index 4f7c4130..0619f254 100644
--- a/common/src/test/java/com/google/auto/common/OverridesTest.java
+++ b/common/src/test/java/com/google/auto/common/OverridesTest.java
@@ -330,12 +330,14 @@ public class OverridesTest {
           boolean javacSays = javacOverrides.overrides(javacOverrider, javacOverridden, javacIn);
           boolean weSay = explicitOverrides.overrides(overrider, overridden, in);
           if (javacSays != weSay) {
-            expect.fail(
-                "%s.%s overrides %s.%s in %s: javac says %s, we say %s",
-                overrider.getEnclosingElement(), overrider,
-                overridden.getEnclosingElement(), overridden,
-                in,
-                javacSays, weSay);
+            expect
+                .withMessage(
+                    "%s.%s overrides %s.%s in %s: javac says %s, we say %s",
+                    overrider.getEnclosingElement(), overrider,
+                    overridden.getEnclosingElement(), overridden,
+                    in,
+                    javacSays, weSay)
+                .fail();
           }
         }
       }