aboutsummaryrefslogtreecommitdiffstats
path: root/tests/tests.c
diff options
context:
space:
mode:
authorDaniel Campello <campello@chromium.org>2022-03-29 20:47:46 -0600
committerAnastasia Klimchuk <aklm@chromium.org>2022-04-06 07:43:55 +0000
commit885fb2e82b6e3bf8189f34b5c3c3a67e76262802 (patch)
treee37f8788c731fb3dafd444692a9d3d24400ddc79 /tests/tests.c
parent9454336970806b9b869ae1ee2515280051e60780 (diff)
downloadflashrom-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.c31
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, ...)