Change in coreboot[master]: tests: Add lib/compute_ip_checksum-test test case

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/51089 )

Change subject: tests: Add lib/compute_ip_checksum-test test case
......................................................................

tests: Add lib/compute_ip_checksum-test test case

Signed-off-by: Jakub Czapiga 
Change-Id: I61c578ec93837cb2581a1ab9e2f3db2a0dd69f3d
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51089
Tested-by: build bot (Jenkins) 
Reviewed-by: Paul Fagerburg 
---
M tests/lib/Makefile.inc
A tests/lib/compute_ip_checksum-test.c
2 files changed, 106 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Paul Fagerburg: Looks good to me, approved

diff --git a/tests/lib/Makefile.inc b/tests/lib/Makefile.inc
index 2d49ba3..921da1c 100644
--- a/tests/lib/Makefile.inc
+++ b/tests/lib/Makefile.inc
@@ -21,6 +21,7 @@
 tests-y += malloc-test
 tests-y += memmove-test
 tests-y += crc_byte-test
+tests-y += compute_ip_checksum-test

string-test-srcs += tests/lib/string-test.c
 string-test-srcs += src/lib/string.c
@@ -102,3 +103,6 @@
 crc_byte-test-srcs += tests/lib/crc_byte-test.c
 crc_byte-test-srcs += src/lib/crc_byte.c

+compute_ip_checksum-test-srcs += tests/lib/compute_ip_checksum-test.c
+compute_ip_checksum-test-srcs += src/lib/compute_ip_checksum.c
+
diff --git a/tests/lib/compute_ip_checksum-test.c b/tests/lib/compute_ip_checksum-test.c
new file mode 100644
index 0000000..fb67439
--- /dev/null
+++ b/tests/lib/compute_ip_checksum-test.c
@@ -0,0 +1,102 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#include <tests/test.h>
+#include <string.h>
+#include <stdlib.h>
+#include <types.h>
+#include <ip_checksum.h>
+
+static const uint8_t test_data_simple[] = {
+	0x64, 0x3b, 0x33, 0x17, 0x34, 0x74, 0x62, 0x30,
+	0x75, 0x73, 0xf3, 0x11, 0x30, 0x2c, 0x34, 0x35,
+	0x6d, 0x39, 0x69, 0x32, 0x23, 0x24, 0x76, 0x71,
+	0x77, 0x30, 0x39, 0x75, 0x76, 0x35, 0x71, 0x32,
+	0x40, 0x46, 0x34, 0x34, 0xBB, 0x03, 0x66, 0x52
+};
+static const size_t test_data_simple_sz = ARRAY_SIZE(test_data_simple);
+static const unsigned long test_data_simple_checksum = 0x4267;
+
+static uint8_t test_data_zeros[1024];
+static const size_t test_data_zeros_sz = ARRAY_SIZE(test_data_zeros);
+static const unsigned long test_data_zeros_checksum = 0xFFFF;
+
+static int setup_test_group(void **state)
+{
+	memset(test_data_zeros, 0, test_data_zeros_sz);
+
+	return 0;
+}
+
+static void test_compute_ip_checksum_zero_length(void **state)
+{
+	unsigned long res = compute_ip_checksum(test_data_simple, 0);
+
+	/* Expect checksum to be in initial state as there are were no data provided. */
+	assert_int_equal(0xFFFF, res);
+}
+
+static void test_compute_ip_checksum_zero_buffer(void **state)
+{
+	unsigned long res = compute_ip_checksum(test_data_zeros, test_data_zeros_sz);
+	assert_int_equal(test_data_zeros_checksum, res);
+}
+
+static void test_compute_ip_checksum_simple_data(void **state)
+{
+	unsigned long res;
+	unsigned long check_res;
+	const size_t helper_buffer_size = sizeof(uint8_t) * (test_data_simple_sz + 2);
+	char *helper_buffer = malloc(helper_buffer_size);
+
+	/* Self test */
+	assert_non_null(helper_buffer);
+
+	/* Expect function to generate the same checksum as stored in */
+	res = compute_ip_checksum(test_data_simple, test_data_simple_sz);
+	assert_int_equal(test_data_simple_checksum, res);
+
+	/* Copy test data and checksum to new buffer. Expect computed checksum to be zero,
+	   as it proves the data and the checksum are correct. */
+	memcpy(helper_buffer, test_data_simple, test_data_simple_sz);
+	helper_buffer[helper_buffer_size - 2] = res & 0xFF;
+	helper_buffer[helper_buffer_size - 1] = (res >> 8) & 0xFF;
+	check_res = compute_ip_checksum(helper_buffer, helper_buffer_size);
+	assert_int_equal(0, check_res);
+
+	free(helper_buffer);
+}
+
+static void test_add_ip_checksums_empty_values(void **state)
+{
+	unsigned long res;
+
+	res = add_ip_checksums(0, 0xFFFF, 0xFFFF);
+	assert_int_equal(0xFFFF, res);
+
+	res = add_ip_checksums(1, 0xFFFF, 0xFFFF);
+	assert_int_equal(0xFFFF, res);
+}
+
+static void test_add_ip_checksums(void **state)
+{
+	unsigned long res_1 = compute_ip_checksum(test_data_simple, test_data_simple_sz / 2);
+	unsigned long res_2 = compute_ip_checksum(test_data_simple + test_data_simple_sz / 2,
+							test_data_simple_sz / 2);
+	unsigned long res_sum = add_ip_checksums(test_data_simple_sz / 2, res_1, res_2);
+
+	assert_int_equal(test_data_simple_checksum, res_sum);
+}
+
+int main(void)
+{
+	const struct CMUnitTest tests[] = {
+		cmocka_unit_test(test_compute_ip_checksum_zero_length),
+		cmocka_unit_test(test_compute_ip_checksum_zero_buffer),
+		cmocka_unit_test(test_compute_ip_checksum_simple_data),
+
+		cmocka_unit_test(test_add_ip_checksums_empty_values),
+		cmocka_unit_test(test_add_ip_checksums),
+	};
+
+	return cmocka_run_group_tests(tests, setup_test_group, NULL);
+}