Skip to content

Commit 9e8ed90

Browse files
committed
Update unit tests: inverted selection
after correcting update_trigger double write
1 parent d14045a commit 9e8ed90

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

tools/unit-tests/unit-enc-nvm.c

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
#define MOCK_ADDRESS_SWAP 0xCE000000
2828
const char ENCRYPT_KEY[] = "0123456789abcdef0123456789abcdef0123456789abcdef";
2929
#include <stdio.h>
30+
#include "image.h"
3031
#include "encrypt.h"
3132
#include "libwolfboot.c"
3233
#include <fcntl.h>
@@ -274,10 +275,10 @@ START_TEST (test_nvm_update_with_encryption)
274275
/* Triggering update to set flags */
275276
wolfBoot_update_trigger();
276277

277-
/* Current selected should now be 0 */
278+
/* Current selected should now be 1 */
278279
ret = nvm_select_fresh_sector(PART_UPDATE);
279-
ck_assert_msg(ret == 0, "Failed to select updating fresh sector\n");
280-
ck_assert_msg(erased_nvm_bank1 != 0, "Did not erase the non-selected bank");
280+
ck_assert_msg(ret == 1, "Failed to select updating fresh sector\n");
281+
ck_assert_msg(erased_nvm_bank0 != 0, "Did not erase the non-selected bank");
281282

282283
magic = get_partition_magic(PART_UPDATE);
283284
ck_assert_msg(*magic == *boot_word,

tools/unit-tests/unit-nvm.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -269,10 +269,10 @@ START_TEST (test_nvm_select_fresh_sector)
269269
/* Triggering update to set flags */
270270
wolfBoot_update_trigger();
271271

272-
/* Current selected should now be 0 */
272+
/* Current selected should now be 1 */
273273
ret = nvm_select_fresh_sector(PART_UPDATE);
274-
ck_assert_msg(ret == 0, "Failed to select updating fresh sector\n");
275-
ck_assert_msg(erased_nvm_bank1 > 0, "Did not erase the non-selected bank");
274+
ck_assert_msg(ret == 1, "Failed to select updating fresh sector\n");
275+
ck_assert_msg(erased_nvm_bank0 > 0, "Did not erase the non-selected bank");
276276

277277
magic = get_partition_magic(PART_UPDATE);
278278
ck_assert_msg(*magic == *boot_word,

0 commit comments

Comments
 (0)