Bug/test errors

[eafalkens]

Summary by CodeRabbit

• Chores

  • Updated project coordinates and build configuration.
  • Lowered Java version requirement from 25 to 23.
  • Updated build tool versions and removed unused build plugins for a simplified build process.
  • Updated testing dependencies to newer versions.

• Tests

  • Refactored test infrastructure and improved test setup.

[coderabbitai]

Warning

Rate limit exceeded

@eafalkens has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 21 minutes and 42 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 2cdf3a1 and 93f94c6.

📒 Files selected for processing (3)

• pom.xml
• src/test/java/org/juv25d/http/HttpResponseTest.java
• src/test/java/org/juv25d/http/HttpResponseWriterTest.java

📝 Walkthrough

Walkthrough

This PR refactors the project's build configuration and test structure. The pom.xml is substantially simplified: project coordinates shift from org.juv25d/JavaHttpServer to org.example/JavaTemplate, Java version requirement is lowered from 25 to 23, multiple dependencies are removed, and build plugins are pruned. HttpResponseTest is converted to a public class with JUnit Jupiter assertions and a @BeforeEach setup method.

Changes

Cohort / File(s)	Summary
Build Configuration pom.xml	Project coordinates changed (org.juv25d→org.example, JavaHttpServer→JavaTemplate); Java version downgraded (25→23); dependencies removed (archunit, snakeyaml, bucket4j, testcontainers, jspecify); plugins removed (spotless, pitest, buildnumber, maven-shade) and simplified; maven-jar-plugin added; maven-surefire-plugin and maven-failsafe-plugin updated to 3.5.5 with expanded javaagent configuration.
Test Refactoring src/test/java/org/juv25d/http/HttpResponseTest.java, src/test/java/org/juv25d/http/HttpResponseWriterTest.java	HttpResponseTest made public; @BeforeEach setup method added to instantiate HttpResponse; assertions converted from AssertJ to JUnit Jupiter; new test cases for default status, text, headers, and body. HttpResponseWriterTest: trailing whitespace removed.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feature/basic-http-responses #24: Updates HttpResponse test suite and test runner configuration that directly exercise the HttpResponse implementations.
• chore: Update POM to Java 25 and rename artifactId/groupId #11: Modifies the same pom.xml project coordinates and maven.compiler.release version in opposite directions.
• Update to fix yaml error in pom and README.md content #65: Directly conflicts with pom.xml plugin configuration, removing maven-shade-plugin while adding maven-jar-plugin.

Suggested reviewers

• fmazmz
• bamsemats
• jesperlarsson1910

Poem

🐰 With pom.xml trimmed and tidy,
Tests now shine in Jupiter's light,
Coordinates reset, the build runs brightly,
Simpler paths make the project right! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Bug/test errors' is vague and generic, failing to describe the specific changes made in the pull request.	Use a more descriptive title that captures the main changes, such as 'Refactor pom.xml configuration and update test assertions' or 'Simplify build configuration and migrate to JUnit Jupiter assertions'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch bug/test-errors

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@pom.xml`:
- Around line 7-8: The pom.xml change replaced the project's Maven coordinates
with template defaults (groupId and artifactId set to org.example and
JavaTemplate); revert these to the project's original coordinates by restoring
the correct groupId and artifactId values used by the project (replace the
current org.example and JavaTemplate entries with the repository's canonical
groupId and artifactId), ensuring consistency with release automation and
downstream consumers who rely on the original coordinates.

---

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e609337 and 2cdf3a1.

📒 Files selected for processing (3)

• pom.xml
• src/test/java/org/juv25d/http/HttpResponseTest.java
• src/test/java/org/juv25d/http/HttpResponseWriterTest.java

💤 Files with no reviewable changes (1)

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