refactor: rename body.{hpp,cpp} to response_body to match the class rename

etr · claude · etr · commit ff7ba77b5b4d · 2026-07-19T01:26:37.000-07:00
Commit ea3e67c renamed the detail::body hierarchy to detail::response_body (response_body base + string_/file_/iovec_/pipe_/deferred_/empty_/ digest_challenge_response_body) but left the files carrying it named body.*. Align the filenames with the types they hold. - src/httpserver/detail/body.hpp -> detail/response_body.hpp (include guard SRC_HTTPSERVER_DETAIL_RESPONSE_BODY_HPP_ and internal #error message updated to match) - src/detail/body.cpp -> src/detail/response_body.cpp - test/unit/body_test.cpp -> test/unit/response_body_test.cpp (test target body -> response_body in test/Makefile.am) Updated every #include, both Makefile.am file lists, and the stale body.cpp / body_test.cpp mentions in comments, iovec_entry.hpp, and test/PORTABILITY.md. Deliberately unchanged: webserver_body_pipeline.cpp and the body_chunk / request_body names -- those are the request/upload-body path, not the response-body hierarchy. Verified: out-of-source reconfigure + build green, make check 113/113, all header/readme/release-notes/doxygen/lint gates pass. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015tAodxYJMEY4VxCX4dk62e
diff --git a/src/Makefile.am b/src/Makefile.am
@@ -25,11 +25,11 @@ lib_LTLIBRARIES = libhttpserver.la
 # builds. The WS-off branch in websocket_handler.cpp provides stub
 # definitions (every member throws feature_unavailable except is_valid()
 # which returns false).
-libhttpserver_la_SOURCES = string_utilities.cpp webserver.cpp http_utils.cpp file_info.cpp http_request.cpp http_request_auth.cpp http_response.cpp http_response_factories.cpp http_resource.cpp create_webserver.cpp create_test_request.cpp websocket_handler.cpp hook_handle.cpp peer_address.cpp resource_hook_table.cpp cookie.cpp detail/http_endpoint.cpp detail/body.cpp detail/ip_representation.cpp detail/ip_access_control.cpp detail/ws_registry.cpp detail/hook_bus.cpp detail/route_table.cpp detail/daemon_lifecycle.cpp detail/dispatch_util.cpp detail/error_pages.cpp detail/hook_dispatcher.cpp detail/http_request_impl.cpp detail/http_request_impl_args.cpp detail/http_request_impl_tls.cpp detail/request_dispatcher.cpp detail/request_pipeline.cpp detail/response_materializer.cpp detail/upload_pipeline.cpp detail/websocket_upgrader.cpp detail/webserver_impl.cpp detail/webserver_callbacks.cpp detail/webserver_body_pipeline.cpp
+libhttpserver_la_SOURCES = string_utilities.cpp webserver.cpp http_utils.cpp file_info.cpp http_request.cpp http_request_auth.cpp http_response.cpp http_response_factories.cpp http_resource.cpp create_webserver.cpp create_test_request.cpp websocket_handler.cpp hook_handle.cpp peer_address.cpp resource_hook_table.cpp cookie.cpp detail/http_endpoint.cpp detail/response_body.cpp detail/ip_representation.cpp detail/ip_access_control.cpp detail/ws_registry.cpp detail/hook_bus.cpp detail/route_table.cpp detail/daemon_lifecycle.cpp detail/dispatch_util.cpp detail/error_pages.cpp detail/hook_dispatcher.cpp detail/http_request_impl.cpp detail/http_request_impl_args.cpp detail/http_request_impl_tls.cpp detail/request_dispatcher.cpp detail/request_pipeline.cpp detail/response_materializer.cpp detail/upload_pipeline.cpp detail/websocket_upgrader.cpp detail/webserver_impl.cpp detail/webserver_callbacks.cpp detail/webserver_body_pipeline.cpp
 # noinst_HEADERS: shipped in the tarball but NEVER installed under $prefix/include.
 # Detail headers (httpserver/detail/*.hpp) live here so they cannot leak to
 # downstream consumers — the public surface comes in through <httpserver.hpp>.
-noinst_HEADERS = httpserver/string_utilities.hpp httpserver/detail/connection_context.hpp httpserver/detail/http_endpoint.hpp httpserver/detail/body.hpp httpserver/detail/webserver_impl.hpp httpserver/detail/webserver_impl_dispatch.hpp httpserver/detail/connection_state.hpp httpserver/detail/ip_access_control.hpp httpserver/detail/ws_registry.hpp httpserver/detail/hook_bus.hpp httpserver/detail/route_table.hpp httpserver/detail/daemon_lifecycle.hpp httpserver/detail/dispatch_util.hpp httpserver/detail/error_pages.hpp httpserver/detail/hook_dispatcher.hpp httpserver/detail/request_dispatcher.hpp httpserver/detail/request_pipeline.hpp httpserver/detail/response_materializer.hpp httpserver/detail/upload_pipeline.hpp httpserver/detail/websocket_upgrader.hpp httpserver/detail/secure_zero.hpp httpserver/detail/http_request_impl.hpp httpserver/detail/resource_hook_table.hpp httpserver/detail/route_entry.hpp httpserver/detail/lambda_resource.hpp httpserver/detail/segment_trie.hpp httpserver/detail/route_cache.hpp httpserver/detail/route_tier.hpp httpserver/detail/unescape_helpers.hpp gettext.h
+noinst_HEADERS = httpserver/string_utilities.hpp httpserver/detail/connection_context.hpp httpserver/detail/http_endpoint.hpp httpserver/detail/response_body.hpp httpserver/detail/webserver_impl.hpp httpserver/detail/webserver_impl_dispatch.hpp httpserver/detail/connection_state.hpp httpserver/detail/ip_access_control.hpp httpserver/detail/ws_registry.hpp httpserver/detail/hook_bus.hpp httpserver/detail/route_table.hpp httpserver/detail/daemon_lifecycle.hpp httpserver/detail/dispatch_util.hpp httpserver/detail/error_pages.hpp httpserver/detail/hook_dispatcher.hpp httpserver/detail/request_dispatcher.hpp httpserver/detail/request_pipeline.hpp httpserver/detail/response_materializer.hpp httpserver/detail/upload_pipeline.hpp httpserver/detail/websocket_upgrader.hpp httpserver/detail/secure_zero.hpp httpserver/detail/http_request_impl.hpp httpserver/detail/resource_hook_table.hpp httpserver/detail/route_entry.hpp httpserver/detail/lambda_resource.hpp httpserver/detail/segment_trie.hpp httpserver/detail/route_cache.hpp httpserver/detail/route_tier.hpp httpserver/detail/unescape_helpers.hpp gettext.h
 nobase_include_HEADERS = httpserver.hpp httpserver/body_kind.hpp httpserver/cookie.hpp httpserver/constants.hpp httpserver/create_webserver.hpp httpserver/create_webserver_setters.hpp httpserver/create_test_request.hpp httpserver/webserver.hpp httpserver/webserver_routes.hpp httpserver/webserver_runtime.hpp httpserver/webserver_websocket.hpp httpserver/webserver_hooks.hpp httpserver/websocket_handler.hpp httpserver/http_utils.hpp httpserver/http_utils_helpers.hpp httpserver/ip_representation.hpp httpserver/file_info.hpp httpserver/http_request.hpp httpserver/http_response.hpp httpserver/http_resource.hpp httpserver/feature_unavailable.hpp httpserver/iovec_entry.hpp httpserver/http_arg_value.hpp httpserver/http_method.hpp httpserver/hook_phase.hpp httpserver/hook_action.hpp httpserver/hook_handle.hpp httpserver/hook_context.hpp
 
 AM_CXXFLAGS += -fPIC -Wall
diff --git a/src/detail/hook_dispatcher.cpp b/src/detail/hook_dispatcher.cpp
@@ -43,7 +43,7 @@
 #include "httpserver/http_request.hpp"
 #include "httpserver/http_resource.hpp"
 #include "httpserver/http_response.hpp"
-#include "httpserver/detail/body.hpp"
+#include "httpserver/detail/response_body.hpp"
 #include "httpserver/detail/dispatch_util.hpp"
 #include "httpserver/detail/hook_bus.hpp"
 #include "httpserver/detail/connection_context.hpp"
diff --git a/src/detail/response_body.cpp b/src/detail/response_body.cpp
@@ -18,7 +18,7 @@
      USA
 */
 
-#include "httpserver/detail/body.hpp"
+#include "httpserver/detail/response_body.hpp"
 
 #include <fcntl.h>
 #include <microhttpd.h>
diff --git a/src/detail/response_materializer.cpp b/src/detail/response_materializer.cpp
@@ -37,7 +37,7 @@
 #include "httpserver/create_webserver.hpp"
 #include "httpserver/http_response.hpp"
 #include "httpserver/http_utils.hpp"
-#include "httpserver/detail/body.hpp"
+#include "httpserver/detail/response_body.hpp"
 #include "httpserver/detail/dispatch_util.hpp"
 #include "httpserver/detail/error_pages.hpp"
 #include "httpserver/detail/hook_dispatcher.hpp"
diff --git a/src/detail/webserver_callbacks.cpp b/src/detail/webserver_callbacks.cpp
@@ -76,7 +76,7 @@
 #include "httpserver/http_response.hpp"
 #include "httpserver/http_utils.hpp"
 #include "httpserver/string_utilities.hpp"
-#include "httpserver/detail/body.hpp"
+#include "httpserver/detail/response_body.hpp"
 #include "httpserver/detail/connection_state.hpp"
 #include "httpserver/detail/path_normalize.hpp"
 #include "httpserver/detail/resource_hook_table.hpp"
diff --git a/src/http_response.cpp b/src/http_response.cpp
@@ -38,7 +38,7 @@
 #include <utility>
 #include <vector>
 
-#include "httpserver/detail/body.hpp"   // complete type for body_->~response_body()
+#include "httpserver/detail/response_body.hpp"   // complete type for body_->~response_body()
 #include "httpserver/detail/http_field_validation.hpp"
 #include "httpserver/http_utils.hpp"
 #include "httpserver/iovec_entry.hpp"
@@ -73,7 +73,7 @@ static_assert(alignof(http_response) >= 16,
 // discriminator impossible to get out of sync. Both helpers are
 // noexcept: destroy_body relies on body subclass dtors being
 // noexcept, adopt_body_from relies on the noexcept move_into() virtual
-// (statically asserted per-subclass in detail/body.hpp).
+// (statically asserted per-subclass in detail/response_body.hpp).
 //
 // Members are private; they live as out-of-line member functions so
 // they have access without an extra friend declaration.
@@ -133,7 +133,7 @@ http_response::~http_response() {
 // noexcept because every member's move is noexcept (header_map is a
 // std::map, std::map move is noexcept; std::byte[64] is trivially
 // movable; per-subclass body move ctors are noexcept by static_assert in
-// detail/body.hpp).
+// detail/response_body.hpp).
 // -----------------------------------------------------------------------
 http_response::http_response(http_response&& other) noexcept
     : status_code_(other.status_code_),
diff --git a/src/http_response_factories.cpp b/src/http_response_factories.cpp
@@ -37,7 +37,7 @@
 #include <utility>
 #include <vector>
 
-#include "httpserver/detail/body.hpp"   // complete type for body_->~response_body()
+#include "httpserver/detail/response_body.hpp"   // complete type for body_->~response_body()
 #include "httpserver/detail/http_field_validation.hpp"
 #include "httpserver/feature_unavailable.hpp"
 #include "httpserver/http_utils.hpp"
@@ -63,7 +63,7 @@ namespace httpserver {
 // Defined out-of-line in this TU because every factory in this file
 // instantiates it (so no separate-TU instantiation is needed) and the
 // template body needs the complete type detail::response_body. Per-T size+align
-// guards duplicate the SBO budget asserts in detail/body.hpp so an
+// guards duplicate the SBO budget asserts in detail/response_body.hpp so an
 // over-sized future body subclass fails to compile at the factory site
 // rather than silently triggering the heap fallback.
 // -----------------------------------------------------------------------
diff --git a/src/httpserver/detail/response_body.hpp b/src/httpserver/detail/response_body.hpp
@@ -28,11 +28,11 @@
 //
 // Header-hygiene contract: only library .cpp files (and build-tree unit
 // tests compiled with -DHTTPSERVER_COMPILATION) may include this file.
-#ifndef SRC_HTTPSERVER_DETAIL_BODY_HPP_
-#define SRC_HTTPSERVER_DETAIL_BODY_HPP_
+#ifndef SRC_HTTPSERVER_DETAIL_RESPONSE_BODY_HPP_
+#define SRC_HTTPSERVER_DETAIL_RESPONSE_BODY_HPP_
 
 #ifndef HTTPSERVER_COMPILATION
-#error "detail/body.hpp is internal; build with -DHTTPSERVER_COMPILATION."
+#error "detail/response_body.hpp is internal; build with -DHTTPSERVER_COMPILATION."
 #endif
 
 #include <microhttpd.h>
@@ -490,4 +490,4 @@ static_assert(std::is_nothrow_move_constructible_v<digest_challenge_response_bod
 }  // namespace detail
 
 }  // namespace httpserver
-#endif  // SRC_HTTPSERVER_DETAIL_BODY_HPP_
+#endif  // SRC_HTTPSERVER_DETAIL_RESPONSE_BODY_HPP_
diff --git a/src/httpserver/iovec_entry.hpp b/src/httpserver/iovec_entry.hpp
@@ -36,13 +36,13 @@ namespace httpserver {
 // Layout is pinned to match POSIX `struct iovec` and libmicrohttpd's
 // `MHD_IoVec` so the dispatch path can `reinterpret_cast` a contiguous
 // array of iovec_entry into either C type at zero copy. The pinning
-// asserts live next to the cast site in `src/detail/body.cpp`
+// asserts live next to the cast site in `src/detail/response_body.cpp`
 // (iovec_response_body::materialize).
 //
 // `base` is `const void*` because libhttpserver never writes through
 // these buffers on the response path.
 //
-// Both fields are accessed from src/http_response.cpp and src/detail/body.cpp
+// Both fields are accessed from src/http_response.cpp and src/detail/response_body.cpp
 // (offsetof layout pinning + iovec construction); cppcheck analyses each TU
 // in isolation and cannot see the uses, so unusedStructMember is suppressed
 // per-member below.
diff --git a/src/webserver.cpp b/src/webserver.cpp
@@ -81,7 +81,7 @@
 #include "httpserver/http_response.hpp"
 #include "httpserver/http_utils.hpp"
 #include "httpserver/string_utilities.hpp"
-#include "httpserver/detail/body.hpp"
+#include "httpserver/detail/response_body.hpp"
 #include "httpserver/constants.hpp"
 #include "httpserver/hook_action.hpp"
 #include "httpserver/hook_context.hpp"
diff --git a/test/Makefile.am b/test/Makefile.am
@@ -26,7 +26,7 @@ LDADD += -lcurl
 
 AM_CPPFLAGS = -I$(top_srcdir)/src -I$(top_srcdir)/src/httpserver/ -DHTTPSERVER_COMPILATION
 METASOURCES = AUTO
-check_PROGRAMS = basic file_upload http_utils threaded nodelay string_utilities http_endpoint ip_access_control ws_start_stop authentication digest_challenge_format deferred http_resource http_response create_webserver create_webserver_explicit new_response_types daemon_info uri_log post_iterator_null_key feature_unavailable header_hygiene_iovec header_hygiene iovec_entry http_method constants body http_response_sbo http_response_factories http_response_digest_factory http_response_move_sanitizer webserver_pimpl http_request_pimpl create_test_request http_request_arena http_request_unescape_arena http_request_const_getters http_request_tls_accessors http_request_operator_stream webserver_register_smartptr webserver_register_path_prefix webserver_on_methods webserver_route route_table lookup_pipeline route_table_concurrency hook_bus ws_registry ws_registry_concurrency daemon_lifecycle routing_regression route_lookup_canonicalize auth_skip_normalize http_resource_allow_cache v2_dispatch_contract threadsafety_stress webserver_features webserver_ws_unavailable webserver_ws_available webserver_register_ws_smartptr webserver_dauth_unavailable webserver_dauth_available consumer_fixture header_hygiene_hooks hook_api_shape hooks_no_firing hooks_accept_ctx_shape hooks_connection_lifecycle hooks_accept_decision_denied hooks_accept_decision_throwing hooks_body_chunk_ctx_shape hooks_request_received_short_circuit hooks_body_chunk_observes_progress hooks_body_chunk_short_circuit_no_leak hooks_before_handler_ctx_shape hooks_route_resolved_miss_and_hit hooks_before_handler_short_circuit hooks_alias_count hooks_alias_functional hooks_handler_exception_chain hooks_handler_exception_user_handler_throws_continues_chain hooks_handler_exception_fallback_to_hardcoded_500 hooks_handler_exception_slot hooks_response_sent_ctx_shape hooks_request_completed_ctx_shape hooks_after_handler_replaces_response hooks_after_handler_mutates_response_in_place hooks_response_sent_carries_status_bytes_timing hooks_request_completed_fires_on_early_failure hooks_log_access_alias_slot hooks_per_route_invalid_phase_throws hooks_per_route_order hooks_per_route_early_413_per_endpoint hooks_per_route_resource_destroyed_first hooks_per_route_concurrent_registration hooks_not_found_alias auth_handler_optional_signature no_v1_compat_shim cookie_header_sentinel cookie_render cookie_deprecation_sentinel http_response_cookie_wire http_request_cookies_parsed peer_address_to_string secure_zero_dce connection_state_sentinel connection_state_body_residue debug_dump_request_body_unset debug_dump_request_body_set debug_dump_request_body_zero littletest_skip_semantics digest_client_self_test unescaper_func
+check_PROGRAMS = basic file_upload http_utils threaded nodelay string_utilities http_endpoint ip_access_control ws_start_stop authentication digest_challenge_format deferred http_resource http_response create_webserver create_webserver_explicit new_response_types daemon_info uri_log post_iterator_null_key feature_unavailable header_hygiene_iovec header_hygiene iovec_entry http_method constants response_body http_response_sbo http_response_factories http_response_digest_factory http_response_move_sanitizer webserver_pimpl http_request_pimpl create_test_request http_request_arena http_request_unescape_arena http_request_const_getters http_request_tls_accessors http_request_operator_stream webserver_register_smartptr webserver_register_path_prefix webserver_on_methods webserver_route route_table lookup_pipeline route_table_concurrency hook_bus ws_registry ws_registry_concurrency daemon_lifecycle routing_regression route_lookup_canonicalize auth_skip_normalize http_resource_allow_cache v2_dispatch_contract threadsafety_stress webserver_features webserver_ws_unavailable webserver_ws_available webserver_register_ws_smartptr webserver_dauth_unavailable webserver_dauth_available consumer_fixture header_hygiene_hooks hook_api_shape hooks_no_firing hooks_accept_ctx_shape hooks_connection_lifecycle hooks_accept_decision_denied hooks_accept_decision_throwing hooks_body_chunk_ctx_shape hooks_request_received_short_circuit hooks_body_chunk_observes_progress hooks_body_chunk_short_circuit_no_leak hooks_before_handler_ctx_shape hooks_route_resolved_miss_and_hit hooks_before_handler_short_circuit hooks_alias_count hooks_alias_functional hooks_handler_exception_chain hooks_handler_exception_user_handler_throws_continues_chain hooks_handler_exception_fallback_to_hardcoded_500 hooks_handler_exception_slot hooks_response_sent_ctx_shape hooks_request_completed_ctx_shape hooks_after_handler_replaces_response hooks_after_handler_mutates_response_in_place hooks_response_sent_carries_status_bytes_timing hooks_request_completed_fires_on_early_failure hooks_log_access_alias_slot hooks_per_route_invalid_phase_throws hooks_per_route_order hooks_per_route_early_413_per_endpoint hooks_per_route_resource_destroyed_first hooks_per_route_concurrent_registration hooks_not_found_alias auth_handler_optional_signature no_v1_compat_shim cookie_header_sentinel cookie_render cookie_deprecation_sentinel http_response_cookie_wire http_request_cookies_parsed peer_address_to_string secure_zero_dce connection_state_sentinel connection_state_body_residue debug_dump_request_body_unset debug_dump_request_body_set debug_dump_request_body_zero littletest_skip_semantics digest_client_self_test unescaper_func
 
 MOSTLYCLEANFILES = *.gcda *.gcno *.gcov
 
@@ -131,12 +131,12 @@ header_hygiene_LDADD =
 iovec_entry_SOURCES = unit/iovec_entry_test.cpp
 http_method_SOURCES = unit/http_method_test.cpp
 constants_SOURCES = unit/constants_test.cpp
-# body: TASK-008 unit test for the internal detail::response_body hierarchy. It
-# constructs/destroys MHD_Response objects directly via the libmicrohttpd
-# inspection APIs (no daemon), so it needs an explicit -lmicrohttpd link
-# the same way uri_log does.
-body_SOURCES = unit/body_test.cpp
-body_LDADD = $(LDADD) -lmicrohttpd
+# response_body: TASK-008 unit test for the internal detail::response_body
+# hierarchy. It constructs/destroys MHD_Response objects directly via the
+# libmicrohttpd inspection APIs (no daemon), so it needs an explicit
+# -lmicrohttpd link the same way uri_log does.
+response_body_SOURCES = unit/response_body_test.cpp
+response_body_LDADD = $(LDADD) -lmicrohttpd
 
 # http_response_sbo: TASK-009 unit test for the SBO value-type layout of
 # http_response. Pokes the private body_/body_storage_/body_inline_/kind_
@@ -151,7 +151,7 @@ http_response_sbo_LDADD = $(LDADD) -lmicrohttpd
 # functions on http_response (string/file/iovec/pipe/empty/deferred/
 # unauthorized). Each factory placement-news the corresponding
 # detail::response_body subclass into the SBO buffer, so this TU includes the
-# private detail/body.hpp via the build-tree -DHTTPSERVER_COMPILATION
+# private detail/response_body.hpp via the build-tree -DHTTPSERVER_COMPILATION
 # path. Needs -lmicrohttpd for the same transitive reasons as
 # http_response_sbo.
 http_response_factories_SOURCES = unit/http_response_factories_test.cpp
diff --git a/test/PORTABILITY.md b/test/PORTABILITY.md
@@ -136,7 +136,7 @@ The skip sites in `test/unit/` predate the `check-skip-rationales.sh` lint
 parity, but the lint does NOT enforce a `// reason:` comment on these
 blocks.
 
-### `body_test.cpp` — pipe_response_body POSIX-pipe tests (lines 263, 277)
+### `response_body_test.cpp` — pipe_response_body POSIX-pipe tests (lines 263, 277)
 
 - **Symptom**: the two pipe_response_body tests (`pipe_body_kind_and_materialize`,
   `pipe_body_destructor_closes_fd_when_not_materialized`) call POSIX
@@ -157,7 +157,7 @@ blocks.
   every platform, so the Windows gap is a defence-in-depth coverage
   loss rather than a contract gap.
 
-### `body_test.cpp` — file_response_body destructor test (line 198)
+### `response_body_test.cpp` — file_response_body destructor test (line 198)
 
 - **Symptom**: `file_body_destructor_closes_fd_when_not_materialized`
   uses POSIX `::open()` / `::close()` and inspects `errno == EBADF`.
@@ -175,15 +175,15 @@ blocks.
 
 - **Symptom**: `pipe_factory_kind` calls POSIX `::pipe()` to feed
   `http_response::pipe(int)`.
-- **Root cause**: same as body_test pipe_response_body — pipe creation is
+- **Root cause**: same as response_body_test pipe_response_body — pipe creation is
   platform-specific, so `pipe_factory_kind` stays gated behind
   `#ifndef _WIN32`. The factory's signature pin (the `static_assert`
   that `http_response::pipe` takes exactly one `int` argument, plus
   the `pipe_factory_signature_is_single_arg` test that hosts it) has
   no platform dependency and has been hoisted out of the `#ifndef
   _WIN32` block to file scope, so it now compiles and runs on every
   CI lane, including Windows.
-- **Restoration plan**: same follow-up as body_test pipe_response_body for the
+- **Restoration plan**: same follow-up as response_body_test pipe_response_body for the
   remaining runtime `pipe_factory_kind` test. Until then, the
   Linux/macOS CI lanes cover the runtime factory behaviour and the
   Windows lanes do not; the compile-time signature contract is now
diff --git a/test/unit/create_test_request_test.cpp b/test/unit/create_test_request_test.cpp
diff --git a/test/unit/http_response_digest_factory_test.cpp b/test/unit/http_response_digest_factory_test.cpp
diff --git a/test/unit/http_response_factories_test.cpp b/test/unit/http_response_factories_test.cpp
diff --git a/test/unit/http_response_move_sanitizer_test.cpp b/test/unit/http_response_move_sanitizer_test.cpp
diff --git a/test/unit/http_response_sbo_test.cpp b/test/unit/http_response_sbo_test.cpp
diff --git a/test/unit/response_body_test.cpp b/test/unit/response_body_test.cpp
