Merge pull request #2999 from zsxwing/assert-error

Fix a wrong assertion in assertError

Author: akarnokd

src/main/java/rx/observers/TestSubscriber.java

@@ -380,7 +380,7 @@ public void assertError(Throwable throwable) {
         if (err.size() > 1) {
             throw new AssertionError("Multiple errors: " + err.size(), new CompositeException(err));
         } else
-        if (throwable.equals(err.get(0))) {
+        if (!throwable.equals(err.get(0))) {
             throw new AssertionError("Exceptions differ; expected: " + throwable + ", actual: " + err.get(0), err.get(0));
         }
     }

src/test/java/rx/observers/TestSubscriberTest.java

@@ -121,4 +121,11 @@ public void testWrappingMockWhenUnsubscribeInvolved() {
         inOrder.verifyNoMoreInteractions();
     }
 
+    @Test
+    public void testAssertError() {
+        RuntimeException e = new RuntimeException("Oops");
+        TestSubscriber subscriber = new TestSubscriber();
+        Observable.error(e).subscribe(subscriber);
+        subscriber.assertError(e);
+    }
 }