diff options
| author | Anastasia Klimchuk <aklm@chromium.org> | 2021-04-21 07:58:30 +1000 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2021-04-27 12:46:45 +0000 |
| commit | cffaac1e0f802e3fe27c4997e931586ee738f94a (patch) | |
| tree | a454b3319888785615fdf98bd6e2a1d64cdc11f9 | |
| parent | ab9d1460bad85d598e73adf08ec6209fc70d1014 (diff) | |
| download | flashrom-cffaac1e0f802e3fe27c4997e931586ee738f94a.tar.gz flashrom-cffaac1e0f802e3fe27c4997e931586ee738f94a.tar.bz2 flashrom-cffaac1e0f802e3fe27c4997e931586ee738f94a.zip | |
tests: Add unit test to run init/shutdown for dummyflasher.c
Introduce test to exercise that init and shutdown of drivers
correctly manage the drivers life-time state. We constrain
ourselves to dummyflasher in particular here as it does not need
any mocking.
BUG=b:181803212
TEST=builds and ninja test
Change-Id: I3c0ef73397f00c1db7aabb5f9f00cb43525af29c
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/52497
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
| -rw-r--r-- | tests/init_shutdown.c | 37 | ||||
| -rw-r--r-- | tests/meson.build | 1 | ||||
| -rw-r--r-- | tests/tests.c | 5 | ||||
| -rw-r--r-- | tests/tests.h | 3 |
4 files changed, 46 insertions, 0 deletions
diff --git a/tests/init_shutdown.c b/tests/init_shutdown.c new file mode 100644 index 000000000..522686017 --- /dev/null +++ b/tests/init_shutdown.c @@ -0,0 +1,37 @@ +/* + * This file is part of the flashrom project. + * + * Copyright 2021 Google LLC + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <include/test.h> +#include <string.h> + +#include "programmer.h" + +static void run_lifecycle(void **state, enum programmer prog, const char *param) +{ + (void) state; /* unused */ + + printf("Testing programmer_init for programmer=%u ...\n", prog); + assert_int_equal(0, programmer_init(prog, strdup(param))); + printf("... programmer_init for programmer=%u successful\n", prog); + + printf("Testing programmer_shutdown for programmer=%u ...\n", prog); + assert_int_equal(0, programmer_shutdown()); + printf("... programmer_shutdown for programmer=%u successful\n", prog); +} + +void dummy_init_and_shutdown_test_success(void **state) +{ + run_lifecycle(state, PROGRAMMER_DUMMY, "bus=parallel+lpc+fwh+spi"); +} diff --git a/tests/meson.build b/tests/meson.build index f0cb76dd8..815ea7660 100644 --- a/tests/meson.build +++ b/tests/meson.build @@ -18,6 +18,7 @@ srcs = [ 'helpers.c', 'flashrom.c', 'spi25.c', + 'init_shutdown.c', ] mocks = [ diff --git a/tests/tests.c b/tests/tests.c index 5be42161b..1dc819edb 100644 --- a/tests/tests.c +++ b/tests/tests.c @@ -68,5 +68,10 @@ int main(void) }; ret |= cmocka_run_group_tests_name("spi25.c tests", spi25_tests, NULL, NULL); + const struct CMUnitTest init_shutdown_tests[] = { + cmocka_unit_test(dummy_init_and_shutdown_test_success), + }; + ret |= cmocka_run_group_tests_name("init_shutdown.c tests", init_shutdown_tests, NULL, NULL); + return ret; } diff --git a/tests/tests.h b/tests/tests.h index cb905fd6f..32fed0a43 100644 --- a/tests/tests.h +++ b/tests/tests.h @@ -40,4 +40,7 @@ void probe_spi_res3_test_success(void **state); void probe_spi_at25f_test_success(void **state); void probe_spi_st95_test_success(void **state); /* spi95.c */ +/* init_shutdown.c */ +void dummy_init_and_shutdown_test_success(void **state); + #endif /* TESTS_H */ |