Skip to content

Commit bada414

Browse files
akarnokdakarnokd
akarnokd
authored and
akarnokd
committed
Fixed test bug and test expectations
1 parent 6eed05f commit bada414

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/test/java/rx/internal/operators/OperatorConcatTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -471,7 +471,7 @@ public void unsubscribe() {
471471

472472
@Override
473473
public boolean isUnsubscribed() {
474-
return subscribed;
474+
return !subscribed;
475475
}
476476

477477
};

src/test/java/rx/internal/operators/OperatorReplayTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -607,6 +607,7 @@ public void testIssue2191_SchedulerUnsubscribe() throws Exception {
607607
verifyObserverMock(mockObserverBeforeConnect, 2, 6);
608608
verifyObserverMock(mockObserverAfterConnect, 2, 6);
609609

610+
verify(spiedWorker, times(1)).isUnsubscribed();
610611
verify(spiedWorker, times(1)).unsubscribe();
611612
verify(sourceUnsubscribed, times(1)).call();
612613

@@ -666,6 +667,7 @@ public void testIssue2191_SchedulerUnsubscribeOnError() throws Exception {
666667
verifyObserver(mockObserverBeforeConnect, 2, 2, illegalArgumentException);
667668
verifyObserver(mockObserverAfterConnect, 2, 2, illegalArgumentException);
668669

670+
verify(spiedWorker, times(1)).isUnsubscribed();
669671
verify(spiedWorker, times(1)).unsubscribe();
670672
verify(sourceUnsubscribed, times(1)).call();
671673

0 commit comments

Comments
 (0)