Skip to content

Commit a9c5b72

Browse files
thugheskartben
authored andcommitted
tests: net: lib: http_server: Check response
Building with clang, it warns: tests/net/lib/http_server/core/src/main.c:1400:19: error: variable 'http1_header_capture_common_response' is not needed and will not be emitted [-Werror,-Wunneeded-internal-declaration] static const char http1_header_capture_common_response[] ^ = "HTTP/1.1 200\r\n" Add a check to make sure the response actually matches the expected response. Signed-off-by: Tom Hughes <tomhughes@chromium.org>
1 parent 1b6feab commit a9c5b72

File tree

1 file changed

+2
-0
lines changed
  • tests/net/lib/http_server/core/src

1 file changed

+2
-0
lines changed

tests/net/lib/http_server/core/src/main.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1431,6 +1431,8 @@ static void test_http1_header_capture_common(const char *request)
14311431
zassert_not_equal(ret, -1, "send() failed (%d)", errno);
14321432

14331433
test_read_data(&offset, sizeof(http1_header_capture_common_response) - 1);
1434+
zassert_mem_equal(buf, http1_header_capture_common_response,
1435+
sizeof(http1_header_capture_common_response) - 1);
14341436
}
14351437

14361438
ZTEST(server_function_tests, test_http1_header_capture)

0 commit comments

Comments
 (0)