Skip to content

Commit 56ede3a

Browse files
michalpasztamobicaVeijo Pesonen
authored andcommitted
Fix unittests after ExhaustibleBlockDevice adjustments for wear leveling test
1 parent 0278e38 commit 56ede3a

File tree

1 file changed

+4
-21
lines changed

1 file changed

+4
-21
lines changed

UNITTESTS/features/storage/blockdevice/ExhaustibleBlockDevice/test_ExhaustibleBlockDevice.cpp

Lines changed: 4 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,6 @@ TEST_F(ExhaustibleBlockModuleTest, init)
7878
EXPECT_EQ(b.get_read_size(), 0);
7979
EXPECT_EQ(b.get_program_size(), 0);
8080
EXPECT_EQ(b.size(), 0);
81-
b.set_erase_cycles(0, 100); // This should not take effect.
82-
EXPECT_EQ(b.get_erase_cycles(0), 0);
8381
EXPECT_EQ(b.erase(0, BLOCK_SIZE), BD_ERROR_DEVICE_ERROR);
8482
EXPECT_EQ(b.program(magic, 0, BLOCK_SIZE), BD_ERROR_DEVICE_ERROR);
8583
EXPECT_EQ(b.read(buf, 0, BLOCK_SIZE), BD_ERROR_DEVICE_ERROR);
@@ -95,7 +93,6 @@ TEST_F(ExhaustibleBlockModuleTest, init)
9593
EXPECT_EQ(b.get_erase_size(), bd_mock.get_erase_size());
9694
EXPECT_EQ(b.get_erase_size(0), bd_mock.get_erase_size(0));
9795
EXPECT_EQ(b.get_erase_value(), bd_mock.get_erase_value());
98-
EXPECT_NE(b.get_erase_cycles(0), 100);
9996
EXPECT_EQ(b.get_program_size(), 512);
10097
EXPECT_EQ(b.get_read_size(), 512);
10198
EXPECT_EQ(b.size(), bd_mock.size());
@@ -110,43 +107,29 @@ TEST_F(ExhaustibleBlockModuleTest, program_unaligned)
110107

111108
TEST_F(ExhaustibleBlockModuleTest, erase_cycle_limit_reached)
112109
{
113-
EXPECT_NE(bd.get_erase_cycles(0), 0);
114-
115110
EXPECT_CALL(bd_mock, program(_, 0, BLOCK_SIZE))
116111
.Times(1)
117112
.WillRepeatedly(Return(BD_ERROR_OK));
118113

119114
EXPECT_EQ(bd.program(magic, 0, BLOCK_SIZE), 0);
120115

121116
EXPECT_CALL(bd_mock, erase(0, BLOCK_SIZE))
122-
.Times(ERASE_CYCLES)
117+
.Times(ERASE_CYCLES - 1) // Will fall silently after erase cycles are worn out.
123118
.WillRepeatedly(Return(BD_ERROR_OK));
124119

125120
for (int i = 0; i < ERASE_CYCLES; i++) {
126121
EXPECT_EQ(bd.erase(0, BLOCK_SIZE), 0);
127122
}
128-
EXPECT_EQ(bd.get_erase_cycles(0), 0);
129123

130-
// This calls are expect not to happen.
131-
EXPECT_EQ(bd.erase(0, BLOCK_SIZE), BD_ERROR_DEVICE_ERROR);
132-
EXPECT_EQ(bd.program(magic2, 0, BLOCK_SIZE), BD_ERROR_DEVICE_ERROR);
124+
// Erase silently fails, no error report.
125+
EXPECT_EQ(bd.erase(0, BLOCK_SIZE), BD_ERROR_OK);
126+
EXPECT_EQ(bd.program(magic2, 0, BLOCK_SIZE), BD_ERROR_OK);
133127
}
134128

135129
TEST_F(ExhaustibleBlockModuleTest, erase_invalid)
136130
{
137131
ASSERT_GT(ERASE_CYCLES, 1);
138132

139-
EXPECT_EQ(bd.get_erase_cycles(0), ERASE_CYCLES);
140-
141133
// Unaligned erase should fail
142134
EXPECT_EQ(bd.erase(0, BLOCK_SIZE-1), BD_ERROR_DEVICE_ERROR);
143-
// Number of erase cycles should not change
144-
EXPECT_EQ(bd.get_erase_cycles(0), ERASE_CYCLES);
145-
}
146-
147-
TEST_F(ExhaustibleBlockModuleTest, set_erase_cycles)
148-
{
149-
EXPECT_EQ(bd.get_erase_cycles(0), ERASE_CYCLES);
150-
bd.set_erase_cycles(0, ERASE_CYCLES+1);
151-
EXPECT_EQ(bd.get_erase_cycles(0), ERASE_CYCLES+1);
152135
}

0 commit comments

Comments
 (0)