commit 42edea6fb5aee44c8debde9cc4e2b0867768b85b
author jcater <jcater@google.com> Wed May 01 08:46:18 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Wed May 01 08:47:34 2019 -0700
tree 73a89b9c21d4feb6e6f9aa0ce0880b9e1c6d1447
parent 63db9a4d68ce6c4b562533d4ef8b966941919600

Automatic cleanup change.

PiperOrigin-RevId: 246137705

src/test/java/com/google/devtools/build/lib/analysis/AnalysisUtilsTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/analysis/AnalysisUtilsTest.java b/src/test/java/com/google/devtools/build/lib/analysis/AnalysisUtilsTest.java
index 4c6f2e4..ddac970 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/AnalysisUtilsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/AnalysisUtilsTest.java
@@ -16,7 +16,7 @@
 
 import static com.google.common.truth.Truth.assertThat;
 import static com.google.devtools.build.lib.analysis.AnalysisUtils.checkProvider;
-import static org.junit.Assert.fail;
+import static com.google.devtools.build.lib.testutil.MoreAsserts.assertThrows;
 
 import com.google.auto.value.AutoValue;
 import org.junit.Test;
@@ -33,12 +33,11 @@
 
   @Test
   public void checkProviderFailsOnClassGeneratedByAutoValue() {
-    try {
-      checkProvider(AutoValue_AnalysisUtilsTest_AutoValuedClass.class);
-      fail("Expected IllegalArgumentException, but nothing was thrown.");
-    } catch (IllegalArgumentException e) {
-      assertThat(e).hasMessageThat().contains("generated by @AutoValue");
-    }
+    IllegalArgumentException e =
+        assertThrows(
+            IllegalArgumentException.class,
+            () -> checkProvider(AutoValue_AnalysisUtilsTest_AutoValuedClass.class));
+    assertThat(e).hasMessageThat().contains("generated by @AutoValue");
   }
 
   // Note: this has to be defined outside of checkProviderFailsOnClassGeneratedByAutoValue() so it