Skip to content

Commit 69fafc6

Browse files
committed
Clean up HttpResponseTests; keep defaults and validation, remove
duplicates
1 parent 4bd580e commit 69fafc6

2 files changed

Lines changed: 0 additions & 30 deletions

File tree

src/test/java/org/juv25d/http/HttpResponseTest.java

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -11,26 +11,12 @@ void shouldReturnDefaultValue() {
1111
assertEquals(0, response.statusCode());
1212
}
1313

14-
@Test
15-
void shouldSetAndReturnStatusCode() {
16-
HttpResponse response = new HttpResponse();
17-
response.setStatusCode(404);
18-
assertEquals(404, response.statusCode());
19-
}
20-
2114
@Test
2215
void shouldReturnDefaultText() {
2316
HttpResponse response = new HttpResponse();
2417
assertNull(response.statusText());
2518
}
2619

27-
@Test
28-
void shouldSetAndReturnStatusText() {
29-
HttpResponse response = new HttpResponse();
30-
response.setStatusText("Not found");
31-
assertEquals("Not found", response.statusText());
32-
}
33-
3420
@Test
3521
void shouldThrowExceptionWhenStatusTextIsNull() {
3622
HttpResponse response = new HttpResponse();
@@ -43,24 +29,9 @@ void shouldHaveEmptyHeaderByDefault() {
4329
assertTrue(response.headers().isEmpty());
4430
}
4531

46-
@Test
47-
void shouldSetAndReturnHeader() {
48-
HttpResponse response = new HttpResponse();
49-
response.setHeader("Content-Type", "text/plain");
50-
assertEquals("text/plain", response.headers().get("Content-Type"));
51-
}
52-
5332
@Test
5433
void shouldHaveEmptyBodyByDefault() {
5534
HttpResponse response = new HttpResponse();
5635
assertEquals(0, response.body().length);
5736
}
58-
59-
@Test
60-
void shouldSetAndReturnBody() {
61-
HttpResponse response = new HttpResponse();
62-
byte[] body = "Hello World".getBytes();
63-
response.setBody(body);
64-
assertArrayEquals(body, response.body());
65-
}
6637
}

src/test/java/org/juv25d/http/HttpResponseWriterTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,5 +55,4 @@ void writes404NotFoundResponse() throws Exception {
5555

5656
assertThat(result).startsWith("HTTP/1.1 404 Not Found");
5757
}
58-
5958
}

0 commit comments

Comments
 (0)