Skip to content

Commit 5c3bdf3

Browse files
committed
Assert the full execution of extend() tests
1 parent c4b0c26 commit 5c3bdf3

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

src/test/java/io/reactivex/flowable/FlowableTests.java

+4-2
Original file line numberDiff line numberDiff line change
@@ -1114,7 +1114,7 @@ public void testForEachWithNull() {
11141114
public void testExtend() {
11151115
final TestSubscriber<Object> subscriber = new TestSubscriber<Object>();
11161116
final Object value = new Object();
1117-
Flowable.just(value).to(new Function<Flowable<Object>, Object>() {
1117+
Object returned = Flowable.just(value).to(new Function<Flowable<Object>, Object>() {
11181118
@Override
11191119
public Object apply(Flowable<Object> onSubscribe) {
11201120
onSubscribe.subscribe(subscriber);
@@ -1124,13 +1124,14 @@ public Object apply(Flowable<Object> onSubscribe) {
11241124
return subscriber.values().get(0);
11251125
}
11261126
});
1127+
assertSame(returned, value);
11271128
}
11281129

11291130
@Test
11301131
public void testAsExtend() {
11311132
final TestSubscriber<Object> subscriber = new TestSubscriber<Object>();
11321133
final Object value = new Object();
1133-
Flowable.just(value).as(new FlowableConverter<Object, Object>() {
1134+
Object returned = Flowable.just(value).as(new FlowableConverter<Object, Object>() {
11341135
@Override
11351136
public Object apply(Flowable<Object> onSubscribe) {
11361137
onSubscribe.subscribe(subscriber);
@@ -1140,6 +1141,7 @@ public Object apply(Flowable<Object> onSubscribe) {
11401141
return subscriber.values().get(0);
11411142
}
11421143
});
1144+
assertSame(returned, value);
11431145
}
11441146

11451147
@Test

src/test/java/io/reactivex/observable/ObservableTest.java

+4-2
Original file line numberDiff line numberDiff line change
@@ -1153,7 +1153,7 @@ public void testForEachWithNull() {
11531153
public void testExtend() {
11541154
final TestObserver<Object> subscriber = new TestObserver<Object>();
11551155
final Object value = new Object();
1156-
Observable.just(value).to(new Function<Observable<Object>, Object>() {
1156+
Object returned = Observable.just(value).to(new Function<Observable<Object>, Object>() {
11571157
@Override
11581158
public Object apply(Observable<Object> onSubscribe) {
11591159
onSubscribe.subscribe(subscriber);
@@ -1163,13 +1163,14 @@ public Object apply(Observable<Object> onSubscribe) {
11631163
return subscriber.values().get(0);
11641164
}
11651165
});
1166+
assertSame(returned, value);
11661167
}
11671168

11681169
@Test
11691170
public void testAsExtend() {
11701171
final TestObserver<Object> subscriber = new TestObserver<Object>();
11711172
final Object value = new Object();
1172-
Observable.just(value).as(new ObservableConverter<Object, Object>() {
1173+
Object returned = Observable.just(value).as(new ObservableConverter<Object, Object>() {
11731174
@Override
11741175
public Object apply(Observable<Object> onSubscribe) {
11751176
onSubscribe.subscribe(subscriber);
@@ -1179,6 +1180,7 @@ public Object apply(Observable<Object> onSubscribe) {
11791180
return subscriber.values().get(0);
11801181
}
11811182
});
1183+
assertSame(returned, value);
11821184
}
11831185

11841186
@Test

0 commit comments

Comments
 (0)