diff options
author | Daniel Campello <campello@chromium.org> | 2022-03-29 20:47:46 -0600 |
---|---|---|
committer | Anastasia Klimchuk <aklm@chromium.org> | 2022-04-06 07:43:55 +0000 |
commit | 885fb2e82b6e3bf8189f34b5c3c3a67e76262802 (patch) | |
tree | e37f8788c731fb3dafd444692a9d3d24400ddc79 /tests/tests.c | |
parent | 9454336970806b9b869ae1ee2515280051e60780 (diff) | |
download | flashrom-885fb2e82b6e3bf8189f34b5c3c3a67e76262802.tar.gz flashrom-885fb2e82b6e3bf8189f34b5c3c3a67e76262802.tar.bz2 flashrom-885fb2e82b6e3bf8189f34b5c3c3a67e76262802.zip |
tests: assert pathname and flags when calling open()
With this change the wrappers for mock and friends are able to take an
optional io_mock_fallback_open_state struct to assert expected pathnames
and flags whenever an open operation is called.
Based partially on https://review.coreboot.org/c/flashrom/+/62319/5
BUG=b:227404721,b:217629892,b:215255210
TEST=./test_build.sh; FEATURES=test emerge-amd64-generic flashrom
BRANCH=none
Signed-off-by: Daniel Campello <campello@chromium.org>
Co-Author: Edward O'Callaghan <quasisec@google.com>
Change-Id: Ib46ca5b854c8453ec02ae09f3151cd4d25f988eb
Reviewed-on: https://review.coreboot.org/c/flashrom/+/63227
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Diffstat (limited to 'tests/tests.c')
-rw-r--r-- | tests/tests.c | 31 |
1 files changed, 23 insertions, 8 deletions
diff --git a/tests/tests.c b/tests/tests.c index f32c14da..3eef47e4 100644 --- a/tests/tests.c +++ b/tests/tests.c @@ -74,28 +74,43 @@ uint8_t __wrap_sio_read(uint16_t port, uint8_t reg) return (uint8_t)mock(); } -int __wrap_open(const char *pathname, int flags) +static int mock_open(const char *pathname, int flags) { - LOG_ME; if (get_io() && get_io()->open) return get_io()->open(get_io()->state, pathname, flags); + + if (get_io() && get_io()->fallback_open_state) { + struct io_mock_fallback_open_state *io_state; + unsigned int open_state_flags; + + io_state = get_io()->fallback_open_state; + assert_true(io_state->noc < MAX_MOCK_OPEN); + assert_non_null(io_state->paths[io_state->noc]); + assert_string_equal(pathname, io_state->paths[io_state->noc]); + open_state_flags = io_state->flags[io_state->noc]; + assert_int_equal(flags & open_state_flags, open_state_flags); + io_state->noc++; // proceed to the next path upon next call. + } + return MOCK_FD; } +int __wrap_open(const char *pathname, int flags) +{ + LOG_ME; + return mock_open(pathname, flags); +} + int __wrap_open64(const char *pathname, int flags) { LOG_ME; - if (get_io() && get_io()->open) - return get_io()->open(get_io()->state, pathname, flags); - return MOCK_FD; + return mock_open(pathname, flags); } int __wrap___open64_2(const char *pathname, int flags) { LOG_ME; - if (get_io() && get_io()->open) - return get_io()->open(get_io()->state, pathname, flags); - return MOCK_FD; + return mock_open(pathname, flags); } int __wrap_ioctl(int fd, unsigned long int request, ...) |