[flashrom-gerrit] Change in flashrom[master]: tests/: Add flashrom.c unit tests

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/41646 )

Change subject: tests/: Add flashrom.c unit tests
......................................................................

tests/: Add flashrom.c unit tests

Change-Id: I2d9213f98c6c9639f2417466ba4895117e8d600a
Signed-off-by: Edward O'Callaghan 
---
A tests/flashrom.c
M tests/meson.build
M tests/tests.c
M tests/tests.h
4 files changed, 45 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/46/41646/1

diff --git a/tests/flashrom.c b/tests/flashrom.c
new file mode 100644
index 0000000..ad46dd8
--- /dev/null
+++ b/tests/flashrom.c
@@ -0,0 +1,36 @@
+#include <include/test.h>
+
+#include "programmer.h"
+
+void flashbuses_to_text_test_success(void **state)
+{
+	(void) state; /* unused */
+
+	enum chipbustype bustype;
+
+	bustype = BUS_NONSPI;
+	assert_string_equal(flashbuses_to_text(bustype), "Non-SPI");
+
+	bustype |= BUS_PARALLEL;
+	assert_string_not_equal(flashbuses_to_text(bustype), "Non-SPI, Parallel");
+
+	bustype = BUS_PARALLEL;
+	bustype |= BUS_LPC;
+	assert_string_equal(flashbuses_to_text(bustype), "Parallel, LPC");
+
+	bustype |= BUS_FWH;
+	//BUS_NONSPI = BUS_PARALLEL | BUS_LPC | BUS_FWH,
+	assert_string_equal(flashbuses_to_text(bustype), "Non-SPI");
+
+	bustype |= BUS_SPI;
+	assert_string_equal(flashbuses_to_text(bustype), "Parallel, LPC, FWH, SPI");
+
+	bustype |= BUS_PROG;
+	assert_string_equal(
+			flashbuses_to_text(bustype),
+			"Parallel, LPC, FWH, SPI, Programmer-specific"
+	);
+
+	bustype = BUS_NONE;
+	assert_string_equal(flashbuses_to_text(bustype), "None");
+}
diff --git a/tests/meson.build b/tests/meson.build
index 3ed1c3b..aad1372 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -3,6 +3,7 @@
 srcs = [
   'tests.c',
   'spi.c',
+  'flashrom.c',
   'spi25.c',
 ]

diff --git a/tests/tests.c b/tests/tests.c
index 16f4863..a3ad791 100644
--- a/tests/tests.c
+++ b/tests/tests.c
@@ -24,6 +24,11 @@
    };
    ret |= cmocka_run_group_tests_name("spi.c tests", spi_tests, NULL, NULL);

+	const struct CMUnitTest flashrom_tests[] = {
+		cmocka_unit_test(flashbuses_to_text_test_success),
+	};
+	ret |= cmocka_run_group_tests_name("flashrom.c tests", flashrom_tests, NULL, NULL);
+
    const struct CMUnitTest spi25_tests[] = {
    	cmocka_unit_test(spi_write_enable_test_success),
    	cmocka_unit_test(spi_write_disable_test_success),
diff --git a/tests/tests.h b/tests/tests.h
index 9377817..688c280 100644
--- a/tests/tests.h
+++ b/tests/tests.h
@@ -4,6 +4,9 @@
 void registered_masters_test_success(void **state);
 void programmer_init_test_success(void **state);

+/* flashrom.c */
+void flashbuses_to_text_test_success(void **state);
+
 /* spi25.c */
 void spi_write_enable_test_success(void **state);
 void spi_write_disable_test_success(void **state);