Skip to content

Commit ed782d1

Browse files
committed
use async filter example to avoid confusion
1 parent 6fd4aa7 commit ed782d1

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

zuul-core/src/test/java/com/netflix/zuul/filters/BaseFilterTest.java

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
import org.junit.runner.RunWith;
2929
import org.mockito.Mock;
3030
import org.mockito.junit.MockitoJUnitRunner;
31+
import rx.Observable;
3132

3233
/**
3334
* Tests for {@link BaseFilter}. Currently named BaseFilter2Test as there is an existing class named BaseFilterTest.
@@ -79,12 +80,12 @@ public ZuulMessage apply(ZuulMessage req) {
7980
@Test
8081
void validateDefaultConcurrencyLimit() {
8182
final int[] limit = {0};
82-
class ConcInboundFilter extends BaseSyncFilter {
83+
class ConcInboundFilter extends BaseFilter {
8384

8485
@Override
85-
public ZuulMessage apply(ZuulMessage input) {
86+
public Observable applyAsync(ZuulMessage input) {
8687
limit[0] = Math.max(filterConcurrencyCustom.get(), filterConcurrencyDefault.get());
87-
return null;
88+
return Observable.just("Done");
8889
}
8990

9091
@Override
@@ -97,7 +98,7 @@ public boolean shouldFilter(ZuulMessage msg) {
9798
return true;
9899
}
99100
}
100-
new ConcInboundFilter().apply(new ZuulMessageImpl(new SessionContext(), new Headers()));
101+
new ConcInboundFilter().applyAsync(new ZuulMessageImpl(new SessionContext(), new Headers()));
101102
Truth.assertThat(limit[0]).isEqualTo(4000);
102103
}
103104

@@ -108,12 +109,12 @@ void validateFilterConcurrencyLimitOverride() {
108109
configuration.setProperty("zuul.ConcInboundFilter.in.concurrency.limit", 4000);
109110
final int[] limit = {0};
110111

111-
class ConcInboundFilter extends BaseSyncFilter {
112+
class ConcInboundFilter extends BaseFilter {
112113

113114
@Override
114-
public ZuulMessage apply(ZuulMessage input) {
115+
public Observable applyAsync(ZuulMessage input) {
115116
limit[0] = Math.max(filterConcurrencyCustom.get(), filterConcurrencyDefault.get());
116-
return null;
117+
return Observable.just("Done");
117118
}
118119

119120
@Override
@@ -126,7 +127,7 @@ public boolean shouldFilter(ZuulMessage msg) {
126127
return true;
127128
}
128129
}
129-
new ConcInboundFilter().apply(new ZuulMessageImpl(new SessionContext(), new Headers()));
130+
new ConcInboundFilter().applyAsync(new ZuulMessageImpl(new SessionContext(), new Headers()));
130131
Truth.assertThat(limit[0]).isEqualTo(7000);
131132
}
132133
}

0 commit comments

Comments
 (0)