Skip to content

Commit ad54bc9

Browse files
Update RedirectFilterTest.java
1 parent 23df773 commit ad54bc9

1 file changed

Lines changed: 17 additions & 10 deletions

File tree

src/test/java/org/example/filter/redirect/RedirectFilterTest.java

Lines changed: 17 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -50,20 +50,27 @@ void returns_301_redirect_and_stops_pipeline() {
5050
}
5151

5252
@Test
53-
void returns_302_redirect() {
54-
RedirectFilter filter = new RedirectFilter(List.of(
55-
new RedirectRule(Pattern.compile("^/temp$"), "https://example.com/temporary", 302)
56-
));
53+
void returns_302_redirect() {
54+
RedirectFilter filter = new RedirectFilter(List.of(
55+
new RedirectRule(Pattern.compile("^/temp$"), "https://example.com/temporary", 302)
56+
));
5757

58-
FilterChain chain = (req, res) -> res.setStatusCode(200);
58+
AtomicBoolean chainCalled = new AtomicBoolean(false);
59+
FilterChain chain = (req, res) -> {
60+
chainCalled.set(true);
61+
res.setStatusCode(200);
62+
};
5963

60-
HttpResponseBuilder res = new HttpResponseBuilder();
64+
HttpResponseBuilder res = new HttpResponseBuilder();
6165

62-
filter.doFilter(request("/temp"), res, chain);
66+
filter.doFilter(request("/temp"), res, chain);
67+
68+
String raw = responseAsString(res);
69+
assertThat(raw).contains("HTTP/1.1 302 Found");
70+
assertThat(raw).contains("Location: https://example.com/temporary");
71+
assertThat(chainCalled.get()).isFalse();
72+
}
6373

64-
String raw = responseAsString(res);
65-
assertThat(raw).contains("HTTP/1.1 302 Found");
66-
assertThat(raw).contains("Location: https://example.com/temporary");
6774
}
6875

6976
@Test

0 commit comments

Comments
 (0)