Merge branch 'test/HttpResponse'

Author: eafalkens

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

@@ -1,25 +1,41 @@
 package org.juv25d.http;
 
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.api.Assertions.assertThatCode;
+import static org.junit.jupiter.api.Assertions.*;
 
-class HttpResponseTest {
+public class HttpResponseTest {
+
+    private HttpResponse response;
+
+    @BeforeEach
+    void setUp() {
+        response = new HttpResponse();
+    }
+
+    @Test
+    void shouldReturnDefaultStatusCode() {
+        assertEquals(200, response.statusCode());
+    }
 
     @Test
-    void defaultConstructor_hasSafeDefaults_andSetHeaderDoesNotThrow() {
-        HttpResponse response = new HttpResponse();
+    void shouldReturnDefaultText() {
+        assertEquals("OK", response.statusText());
+    }
 
-        assertThat(response.statusCode()).isEqualTo(200);
-        assertThat(response.statusText()).isEqualTo("OK");
-        assertThat(response.headers()).isNotNull();
-        assertThat(response.body()).isNotNull();
-        assertThat(response.body()).isEmpty();
+    @Test
+    void shouldThrowExceptionWhenStatusTextIsNull() {
+        assertThrows(NullPointerException.class, () -> response.setStatusText(null));
+    }
 
-        assertThatCode(() -> response.setHeader("Content-Type", "text/plain"))
-            .doesNotThrowAnyException();
+    @Test
+    void shouldHaveEmptyHeaderByDefault() {
+        assertTrue(response.headers().isEmpty());
+    }
 
-        assertThat(response.headers()).containsEntry("Content-Type", "text/plain");
+    @Test
+    void shouldHaveEmptyBodyByDefault() {
+        assertArrayEquals(new byte[0], response.body());
     }
 }

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

@@ -55,5 +55,4 @@ void writes404NotFoundResponse() throws Exception {
 
         assertThat(result).startsWith("HTTP/1.1 404 Not Found");
     }
-
 }