Skip to content

Commit 1f99703

Browse files
author
Seppo Takalo
committed
Fix TDBStore sizes for tests
Tests must require at least 2 erase sectors per TDBStore, so that the garbage collection may work.
1 parent 412a931 commit 1f99703

File tree

2 files changed

+8
-4
lines changed
  • features/storage/TESTS/kvstore

2 files changed

+8
-4
lines changed

features/storage/TESTS/kvstore/general_tests_phase_1/main.cpp

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,11 @@ static void kvstore_init()
120120
program_size = sec_bd->get_program_size();
121121
erase_size = sec_bd->get_erase_size();
122122
// We must be able to hold at least 10 small keys (20 program sectors) and master record + internal data
123-
ul_bd_size = align_up(program_size * 40, erase_size);
124-
rbp_bd_size = align_up(program_size * 40, erase_size);
123+
// but minimum of 2 erase sectors, so that the garbage collection way work
124+
ul_bd_size = align_up(program_size * 40, erase_size * 2);
125+
rbp_bd_size = align_up(program_size * 40, erase_size * 2);
126+
127+
TEST_ASSERT((ul_bd_size + rbp_bd_size) < sec_bd->size());
125128

126129
res = sec_bd->deinit();
127130
TEST_ASSERT_EQUAL_ERROR_CODE(MBED_SUCCESS, res);

features/storage/TESTS/kvstore/general_tests_phase_2/main.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,9 @@ static void kvstore_init()
120120
program_size = sec_bd->get_program_size();
121121
erase_size = sec_bd->get_erase_size();
122122
// We must be able to hold at least 10 small keys (20 program sectors) and master record + internal data
123-
ul_bd_size = align_up(program_size * 40, erase_size);
124-
rbp_bd_size = align_up(program_size * 40, erase_size);
123+
// but minimum of 2 erase sectors, so that the garbage collection way work
124+
ul_bd_size = align_up(program_size * 40, erase_size * 2);
125+
rbp_bd_size = align_up(program_size * 40, erase_size * 2);
125126

126127
res = sec_bd->deinit();
127128
TEST_ASSERT_EQUAL_ERROR_CODE(MBED_SUCCESS, res);

0 commit comments

Comments
 (0)