From 5725a202377cd135e3195834f7b2f5b42b31ec2c Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 11 Feb 2026 00:16:17 +0000 Subject: [PATCH 01/18] Bump org.apache.maven.plugins:maven-dependency-plugin (#6) Bumps the maven-deps group with 1 update: [org.apache.maven.plugins:maven-dependency-plugin](https://github.com/apache/maven-dependency-plugin). Updates `org.apache.maven.plugins:maven-dependency-plugin` from 3.9.0 to 3.10.0 - [Release notes](https://github.com/apache/maven-dependency-plugin/releases) - [Commits](https://github.com/apache/maven-dependency-plugin/compare/maven-dependency-plugin-3.9.0...maven-dependency-plugin-3.10.0) --- updated-dependencies: - dependency-name: org.apache.maven.plugins:maven-dependency-plugin dependency-version: 3.10.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: maven-deps ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 6b7ade11..2a87ca57 100644 --- a/pom.xml +++ b/pom.xml @@ -65,7 +65,7 @@ org.apache.maven.plugins maven-dependency-plugin - 3.9.0 + 3.10.0 From 3b7839910c42e5b6135bc862b4c63d896d95bbac Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 18 Feb 2026 00:17:44 +0000 Subject: [PATCH 02/18] Bump org.junit.jupiter:junit-jupiter in the maven-deps group (#13) Bumps the maven-deps group with 1 update: [org.junit.jupiter:junit-jupiter](https://github.com/junit-team/junit-framework). Updates `org.junit.jupiter:junit-jupiter` from 6.0.2 to 6.0.3 - [Release notes](https://github.com/junit-team/junit-framework/releases) - [Commits](https://github.com/junit-team/junit-framework/compare/r6.0.2...r6.0.3) --- updated-dependencies: - dependency-name: org.junit.jupiter:junit-jupiter dependency-version: 6.0.3 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: maven-deps ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index 2a87ca57..ba8eeb10 100644 --- a/pom.xml +++ b/pom.xml @@ -11,7 +11,7 @@ 23 UTF-8 - 6.0.2 + 6.0.3 3.27.7 5.21.0 From 427ca1c4b65319c230210e8999226f5d70170b56 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 12:26:47 +0100 Subject: [PATCH 03/18] Add test for default statusCode in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 13 +++++++++++++ 1 file changed, 13 insertions(+) create mode 100644 src/test/java/org/juv25d/http/HttpResponseTest.java diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java new file mode 100644 index 00000000..fd4392d8 --- /dev/null +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -0,0 +1,13 @@ +package org.juv25d.http; + +import org.junit.jupiter.api.Test; +import static org.junit.jupiter.api.Assertions.assertEquals; + +public class HttpResponseTest { + + @Test + void shouldReturnDefaultValue() { + HttpResponse response = new HttpResponse(); + assertEquals(0, response.statusCode()); + } +} From fb5b5da6557c2fcf661088d48518a64c04201525 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 12:31:17 +0100 Subject: [PATCH 04/18] Add test for setting statusCode in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index fd4392d8..7e6c2914 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -10,4 +10,12 @@ void shouldReturnDefaultValue() { HttpResponse response = new HttpResponse(); assertEquals(0, response.statusCode()); } + + @Test + void shouldSetAndReturnStatusCode() { + HttpResponse response = new HttpResponse(); + response.setStatusCode(404); + assertEquals(404, response.statusCode()); + } + } From 89bfc446736c87fdd245ba1f8d1d41e9d15803af Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 12:36:09 +0100 Subject: [PATCH 05/18] Add test for default statusText in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index 7e6c2914..54ff3f49 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -2,6 +2,7 @@ import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertNull; public class HttpResponseTest { @@ -18,4 +19,9 @@ void shouldSetAndReturnStatusCode() { assertEquals(404, response.statusCode()); } + @Test + void shouldReturnDefaultText() { + HttpResponse response = new HttpResponse(); + assertNull(response.statusText()); + } } From e0761cd9150ec80eefaf13e6aba03ce6e6e6cb88 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 12:39:53 +0100 Subject: [PATCH 06/18] Add test for setting statusText in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index 54ff3f49..788ac071 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -24,4 +24,11 @@ void shouldReturnDefaultText() { HttpResponse response = new HttpResponse(); assertNull(response.statusText()); } + + @Test + void shouldSetAndReturnStatusText() { + HttpResponse response = new HttpResponse(); + response.setStatusText("Not found"); + assertEquals("Not found", response.statusText()); + } } From b1618b2fffd83fd4dfa2bbbfde7bf989a45a262c Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 12:53:33 +0100 Subject: [PATCH 07/18] Add test for deafult empty header in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index 788ac071..44bce9f1 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -1,8 +1,8 @@ package org.juv25d.http; import org.junit.jupiter.api.Test; -import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertNull; + +import static org.junit.jupiter.api.Assertions.*; public class HttpResponseTest { @@ -31,4 +31,11 @@ void shouldSetAndReturnStatusText() { response.setStatusText("Not found"); assertEquals("Not found", response.statusText()); } + + @Test + void shouldHaveEmptyHeaderByDefault() { + HttpResponse response = new HttpResponse(); + assertTrue(response.headers().isEmpty()); + } + } From 743a6e5792cf4d92a4264a248bba08fa0fdd037e Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 12:54:35 +0100 Subject: [PATCH 08/18] Add test for setting header in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index 44bce9f1..752b3c39 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -38,4 +38,10 @@ void shouldHaveEmptyHeaderByDefault() { assertTrue(response.headers().isEmpty()); } + @Test + void shouldSetAndReturnHeader() { + HttpResponse response = new HttpResponse(); + response.setHeader("Content-Type", "text/plain"); + assertEquals("text/plain", response.headers().get("Content-Type")); + } } From f28f3a091359e99c26fa34f2d5fa278c4c8a3461 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 13:06:59 +0100 Subject: [PATCH 09/18] Add test for default empty body in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index 752b3c39..17f80530 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -44,4 +44,12 @@ void shouldSetAndReturnHeader() { response.setHeader("Content-Type", "text/plain"); assertEquals("text/plain", response.headers().get("Content-Type")); } + + @Test + void shouldHaveEmptyBodyByDefault() { + HttpResponse response = new HttpResponse(); + assertEquals(0, response.body().length); + } + + } From e71b1d700c9c4f6cb3c1cde595a4a3ca4b8edfa4 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 13:07:58 +0100 Subject: [PATCH 10/18] Add test for setting body in HttpResponse --- src/test/java/org/juv25d/http/HttpResponseTest.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index 17f80530..f6601861 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -51,5 +51,11 @@ void shouldHaveEmptyBodyByDefault() { assertEquals(0, response.body().length); } - + @Test + void shouldSetAndReturnBody() { + HttpResponse response = new HttpResponse(); + byte[] body = "Hello World".getBytes(); + response.setBody(body); + assertArrayEquals(body, response.body()); + } } From 4bd580ed5c33da5661435ccbdb021d714603edb0 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Thu, 19 Feb 2026 13:16:56 +0100 Subject: [PATCH 11/18] Add test for NullPointerException when setting null statusText --- src/test/java/org/juv25d/http/HttpResponseTest.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index f6601861..c2327157 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -1,7 +1,6 @@ package org.juv25d.http; import org.junit.jupiter.api.Test; - import static org.junit.jupiter.api.Assertions.*; public class HttpResponseTest { @@ -32,6 +31,12 @@ void shouldSetAndReturnStatusText() { assertEquals("Not found", response.statusText()); } + @Test + void shouldThrowExceptionWhenStatusTextIsNull() { + HttpResponse response = new HttpResponse(); + assertThrows(NullPointerException.class, () -> response.setStatusText(null)); + } + @Test void shouldHaveEmptyHeaderByDefault() { HttpResponse response = new HttpResponse(); From 69fafc604c61cd86fd67449adeb40c6c71f92647 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Mon, 23 Feb 2026 13:21:12 +0100 Subject: [PATCH 12/18] Clean up HttpResponseTests; keep defaults and validation, remove duplicates --- .../org/juv25d/http/HttpResponseTest.java | 29 ------------------- .../juv25d/http/HttpResponseWriterTest.java | 1 - 2 files changed, 30 deletions(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index c2327157..c4b2f73a 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -11,26 +11,12 @@ void shouldReturnDefaultValue() { assertEquals(0, response.statusCode()); } - @Test - void shouldSetAndReturnStatusCode() { - HttpResponse response = new HttpResponse(); - response.setStatusCode(404); - assertEquals(404, response.statusCode()); - } - @Test void shouldReturnDefaultText() { HttpResponse response = new HttpResponse(); assertNull(response.statusText()); } - @Test - void shouldSetAndReturnStatusText() { - HttpResponse response = new HttpResponse(); - response.setStatusText("Not found"); - assertEquals("Not found", response.statusText()); - } - @Test void shouldThrowExceptionWhenStatusTextIsNull() { HttpResponse response = new HttpResponse(); @@ -43,24 +29,9 @@ void shouldHaveEmptyHeaderByDefault() { assertTrue(response.headers().isEmpty()); } - @Test - void shouldSetAndReturnHeader() { - HttpResponse response = new HttpResponse(); - response.setHeader("Content-Type", "text/plain"); - assertEquals("text/plain", response.headers().get("Content-Type")); - } - @Test void shouldHaveEmptyBodyByDefault() { HttpResponse response = new HttpResponse(); assertEquals(0, response.body().length); } - - @Test - void shouldSetAndReturnBody() { - HttpResponse response = new HttpResponse(); - byte[] body = "Hello World".getBytes(); - response.setBody(body); - assertArrayEquals(body, response.body()); - } } diff --git a/src/test/java/org/juv25d/http/HttpResponseWriterTest.java b/src/test/java/org/juv25d/http/HttpResponseWriterTest.java index aa5504a1..35ebe3a7 100644 --- a/src/test/java/org/juv25d/http/HttpResponseWriterTest.java +++ b/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"); } - } From 5a0fead6e43d16fdf22eaf6dfa02c0143b05d258 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Mon, 23 Feb 2026 13:58:59 +0100 Subject: [PATCH 13/18] Refactor HttpResponseTest to CodeRabbit changes --- .../org/juv25d/http/HttpResponseTest.java | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index c4b2f73a..223e3842 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -1,37 +1,40 @@ package org.juv25d.http; +import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.*; public class HttpResponseTest { + private HttpResponse response; + + @BeforeEach + void setUp() { + response = new HttpResponse(); + } + @Test - void shouldReturnDefaultValue() { - HttpResponse response = new HttpResponse(); + void shouldReturnDefaultStatusCode() { assertEquals(0, response.statusCode()); } @Test void shouldReturnDefaultText() { - HttpResponse response = new HttpResponse(); - assertNull(response.statusText()); + assertEquals("", response.statusText()); } @Test void shouldThrowExceptionWhenStatusTextIsNull() { - HttpResponse response = new HttpResponse(); assertThrows(NullPointerException.class, () -> response.setStatusText(null)); } @Test void shouldHaveEmptyHeaderByDefault() { - HttpResponse response = new HttpResponse(); assertTrue(response.headers().isEmpty()); } @Test void shouldHaveEmptyBodyByDefault() { - HttpResponse response = new HttpResponse(); - assertEquals(0, response.body().length); + assertArrayEquals(new byte[0], response.body()); } } From 04addf4bacfe98db12cb2a1cb8228da3f350f544 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Mon, 23 Feb 2026 14:43:29 +0100 Subject: [PATCH 14/18] Fix HttpResponseTest --- .../org/juv25d/http/HttpResponseTest.java | 22 +------------------ 1 file changed, 1 insertion(+), 21 deletions(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index 5f9591d4..d32a8391 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -2,6 +2,7 @@ import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; + import static org.junit.jupiter.api.Assertions.*; public class HttpResponseTest { @@ -36,26 +37,5 @@ void shouldHaveEmptyHeaderByDefault() { @Test void shouldHaveEmptyBodyByDefault() { assertArrayEquals(new byte[0], response.body()); -import org.junit.jupiter.api.Test; - -import static org.assertj.core.api.Assertions.assertThat; -import static org.assertj.core.api.Assertions.assertThatCode; - -class HttpResponseTest { - - @Test - void defaultConstructor_hasSafeDefaults_andSetHeaderDoesNotThrow() { - HttpResponse response = new HttpResponse(); - - assertThat(response.statusCode()).isEqualTo(200); - assertThat(response.statusText()).isEqualTo("OK"); - assertThat(response.headers()).isNotNull(); - assertThat(response.body()).isNotNull(); - assertThat(response.body()).isEmpty(); - - assertThatCode(() -> response.setHeader("Content-Type", "text/plain")) - .doesNotThrowAnyException(); - - assertThat(response.headers()).containsEntry("Content-Type", "text/plain"); } } From d924bfe660de5a5ea5fe3d8134a1a7be49706b1e Mon Sep 17 00:00:00 2001 From: eafalkens Date: Mon, 23 Feb 2026 14:50:19 +0100 Subject: [PATCH 15/18] Trigger CI From 3d7222390e4c8a3297bf6e74e505c5ed769a3298 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Mon, 23 Feb 2026 14:54:12 +0100 Subject: [PATCH 16/18] Fix HttpResponseTest --- src/test/java/org/juv25d/http/HttpResponseTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index d32a8391..dbc781d4 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -16,12 +16,12 @@ void setUp() { @Test void shouldReturnDefaultStatusCode() { - assertEquals(0, response.statusCode()); + assertEquals(200, response.statusCode()); } @Test void shouldReturnDefaultText() { - assertEquals("", response.statusText()); + assertEquals("OK", response.statusText()); } @Test From 5f18f918876b7d42570e93cc228d7d811caa0591 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 25 Feb 2026 00:18:15 +0000 Subject: [PATCH 17/18] Bump the maven-deps group with 2 updates (#21) Bumps the maven-deps group with 2 updates: [org.apache.maven.plugins:maven-surefire-plugin](https://github.com/apache/maven-surefire) and [org.apache.maven.plugins:maven-failsafe-plugin](https://github.com/apache/maven-surefire). Updates `org.apache.maven.plugins:maven-surefire-plugin` from 3.5.4 to 3.5.5 - [Release notes](https://github.com/apache/maven-surefire/releases) - [Commits](https://github.com/apache/maven-surefire/compare/surefire-3.5.4...surefire-3.5.5) Updates `org.apache.maven.plugins:maven-failsafe-plugin` from 3.5.4 to 3.5.5 - [Release notes](https://github.com/apache/maven-surefire/releases) - [Commits](https://github.com/apache/maven-surefire/compare/surefire-3.5.4...surefire-3.5.5) --- updated-dependencies: - dependency-name: org.apache.maven.plugins:maven-surefire-plugin dependency-version: 3.5.5 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: maven-deps - dependency-name: org.apache.maven.plugins:maven-failsafe-plugin dependency-version: 3.5.5 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: maven-deps ... Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --- pom.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pom.xml b/pom.xml index ba8eeb10..66086e31 100644 --- a/pom.xml +++ b/pom.xml @@ -77,7 +77,7 @@ org.apache.maven.plugins maven-surefire-plugin - 3.5.4 + 3.5.5 @{argLine} -javaagent:${org.mockito:mockito-core:jar} -Xshare:off @@ -85,7 +85,7 @@ org.apache.maven.plugins maven-failsafe-plugin - 3.5.4 + 3.5.5 From 6d84d6bf4b5d346be79a445855f80bfb3f508596 Mon Sep 17 00:00:00 2001 From: eafalkens Date: Sun, 1 Mar 2026 17:02:43 +0100 Subject: [PATCH 18/18] Add test for empty statusText in HttpResponse and remove null test --- src/test/java/org/juv25d/http/HttpResponseTest.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/juv25d/http/HttpResponseTest.java b/src/test/java/org/juv25d/http/HttpResponseTest.java index dbc781d4..d61c89d8 100644 --- a/src/test/java/org/juv25d/http/HttpResponseTest.java +++ b/src/test/java/org/juv25d/http/HttpResponseTest.java @@ -25,8 +25,9 @@ void shouldReturnDefaultText() { } @Test - void shouldThrowExceptionWhenStatusTextIsNull() { - assertThrows(NullPointerException.class, () -> response.setStatusText(null)); + void shouldAllowEmptyStatusText() { + response.setStatusText(""); + assertEquals("", response.statusText()); } @Test