Skip to content

Commit f40e141

Browse files
Add redirect filter rules and tests
1 parent 9b041a8 commit f40e141

File tree

1 file changed

+13
-24
lines changed

1 file changed

+13
-24
lines changed

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

Lines changed: 13 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,7 @@
1717
class RedirectFilterTest {
1818

1919
private static HttpRequest request(String path) {
20-
return new HttpRequest(
21-
"GET",
22-
path,
23-
"HTTP/1.1",
24-
Map.of(),
25-
null
26-
);
20+
return new HttpRequest("GET", path, "HTTP/1.1", Map.of(), null);
2721
}
2822

2923
private static String responseAsString(HttpResponseBuilder response) {
@@ -50,27 +44,22 @@ void returns_301_redirect_and_stops_pipeline() {
5044
}
5145

5246
@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-
));
57-
58-
AtomicBoolean chainCalled = new AtomicBoolean(false);
59-
FilterChain chain = (req, res) -> {
60-
chainCalled.set(true);
61-
res.setStatusCode(200);
62-
};
47+
void returns_302_redirect_and_stops_pipeline() {
48+
RedirectFilter filter = new RedirectFilter(List.of(
49+
new RedirectRule(Pattern.compile("^/temp$"), "https://example.com/temporary", 302)
50+
));
6351

64-
HttpResponseBuilder res = new HttpResponseBuilder();
52+
AtomicBoolean chainCalled = new AtomicBoolean(false);
53+
FilterChain chain = (req, res) -> chainCalled.set(true);
6554

66-
filter.doFilter(request("/temp"), res, chain);
55+
HttpResponseBuilder res = new HttpResponseBuilder();
6756

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-
}
57+
filter.doFilter(request("/temp"), res, chain);
7358

59+
String raw = responseAsString(res);
60+
assertThat(raw).contains("HTTP/1.1 302 Found");
61+
assertThat(raw).contains("Location: https://example.com/temporary");
62+
assertThat(chainCalled.get()).isFalse();
7463
}
7564

7665
@Test

0 commit comments

Comments
 (0)