commit 351efff214898300b0f8d320d2e7085716faed38
author nharmata <nharmata@google.com> Fri Mar 30 15:04:38 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Fri Mar 30 15:06:26 2018 -0700
tree b54cdcb212621630f0f01aaf63729bca028a9dec
parent b5e893a51cff0f555ed56d05193a32630b09fb89

Don't test for reference equality of the underlying IOException. It's overkill for these unit tests.

RELNOTES: None
PiperOrigin-RevId: 191122418

src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
index 07c7062..db71eda 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
@@ -696,7 +696,7 @@
     String errorMessage = errorInfo.getException().getMessage();
     assertThat(errorMessage).contains("nope");
     assertThat(errorInfo.getException()).isInstanceOf(NoSuchPackageException.class);
-    assertThat(errorInfo.getException()).hasCauseThat().isSameAs(exn);
+    assertThat(errorInfo.getException()).hasCauseThat().isInstanceOf(IOException.class);
   }
 
   @Test
@@ -714,7 +714,7 @@
     String errorMessage = errorInfo.getException().getMessage();
     assertThat(errorMessage).contains("nope");
     assertThat(errorInfo.getException()).isInstanceOf(NoSuchPackageException.class);
-    assertThat(errorInfo.getException()).hasCauseThat().isSameAs(exn);
+    assertThat(errorInfo.getException()).hasCauseThat().isInstanceOf(IOException.class);
   }
 
   private static class CustomInMemoryFs extends InMemoryFileSystem {