diff options
author | Anastasia Klimchuk <aklm@chromium.org> | 2021-11-12 16:22:29 +1100 |
---|---|---|
committer | Edward O'Callaghan <quasisec@chromium.org> | 2022-02-14 03:15:57 +0000 |
commit | 3f8d1b4f2b13eca9603c6f73b07631a7a98cc160 (patch) | |
tree | 086b1621706270bfefc0c0c95d02a29d9c4bcc19 /tests/chip.c | |
parent | 94f91ef864b56ba71445b3e4d84f4b12f066aed2 (diff) | |
download | flashrom-3f8d1b4f2b13eca9603c6f73b07631a7a98cc160.tar.gz flashrom-3f8d1b4f2b13eca9603c6f73b07631a7a98cc160.tar.bz2 flashrom-3f8d1b4f2b13eca9603c6f73b07631a7a98cc160.zip |
tests: Set up mock chip memory in consistent and predictable way
This patch adds a macro MOCK_CHIP_CONTENT which represents a memory
state of a mock chip. The macro is used to initialise mock chip
memory at the beginning of a test (in setup_chip function).
Previously mock chip memory was not reset between tests. For
existing tests that did not matter, however new test for verify
operation (added later in this chain) needs mock chip memory to
be setup in a predictable way.
BUG=b:181803212
TEST=ninja test
Change-Id: I0d7623a601c207bfc62d54ab89d94cda56d85871
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/59237
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
Diffstat (limited to 'tests/chip.c')
-rw-r--r-- | tests/chip.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tests/chip.c b/tests/chip.c index 5d646ebe..fd7094cc 100644 --- a/tests/chip.c +++ b/tests/chip.c @@ -23,6 +23,7 @@ #include "programmer.h" #define MOCK_CHIP_SIZE (8*MiB) +#define MOCK_CHIP_CONTENT 0xff static struct { unsigned int unlock_calls; /* how many times unlock function was called */ @@ -93,6 +94,7 @@ static void setup_chip(struct flashrom_flashctx *flashctx, struct flashrom_layou flashctx->chip = chip; g_chip_state.unlock_calls = 0; + memset(g_chip_state.buf, MOCK_CHIP_CONTENT, sizeof(g_chip_state.buf)); printf("Creating layout with one included region... "); assert_int_equal(0, flashrom_layout_new(layout)); |