Skip to content

Commit 12a0446

Browse files
mprseadbridge
authored andcommitted
tests-mbed_hal-sleep: add message with failure details
1 parent 4a35dcb commit 12a0446

File tree

1 file changed

+21
-7
lines changed

1 file changed

+21
-7
lines changed

TESTS/mbed_hal/sleep/main.cpp

Lines changed: 21 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,8 @@
4747

4848
using namespace utest::v1;
4949

50+
static char info[512] = {0};
51+
5052
/* The following ticker frequencies are possible:
5153
* high frequency ticker: 250 KHz (1 tick per 4 us) - 8 Mhz (1 tick per 1/8 us)
5254
* low power ticker: 8 KHz (1 tick per 125 us) - 64 KHz (1 tick per ~15.6 us)
@@ -152,7 +154,8 @@ void sleep_usticker_test()
152154
/* Testing wake-up time 10 us. */
153155
for (timestamp_t i = 100; i < 1000; i += 100) {
154156
/* note: us_ticker_read() operates on ticks. */
155-
const timestamp_t next_match_timestamp = overflow_protect(us_ticker_read() + us_to_ticks(i, ticker_freq),
157+
const timestamp_t start_timestamp = us_ticker_read();
158+
const timestamp_t next_match_timestamp = overflow_protect(start_timestamp + us_to_ticks(i, ticker_freq),
156159
ticker_width);
157160

158161
us_ticker_set_interrupt(next_match_timestamp);
@@ -161,9 +164,11 @@ void sleep_usticker_test()
161164

162165
const unsigned int wakeup_timestamp = us_ticker_read();
163166

164-
TEST_ASSERT(
165-
compare_timestamps(us_to_ticks(sleep_mode_delta_us, ticker_freq), ticker_width, next_match_timestamp,
166-
wakeup_timestamp));
167+
sprintf(info, "Delta ticks: %u, Ticker width: %u, Expected wake up tick: %d, Actual wake up tick: %d, delay ticks: %d, wake up after ticks: %d",
168+
us_to_ticks(sleep_mode_delta_us, ticker_freq), ticker_width, next_match_timestamp, wakeup_timestamp, us_to_ticks(i, ticker_freq) , wakeup_timestamp - start_timestamp);
169+
170+
TEST_ASSERT_MESSAGE(compare_timestamps(us_to_ticks(sleep_mode_delta_us, ticker_freq),
171+
ticker_width, next_match_timestamp, wakeup_timestamp), info);
167172
}
168173

169174
set_us_ticker_irq_handler(us_ticker_irq_handler_org);
@@ -194,15 +199,20 @@ void deepsleep_lpticker_test()
194199
/* Testing wake-up time 10 ms. */
195200
for (timestamp_t i = 20000; i < 200000; i += 20000) {
196201
/* note: lp_ticker_read() operates on ticks. */
197-
const timestamp_t next_match_timestamp = overflow_protect(lp_ticker_read() + us_to_ticks(i, ticker_freq), ticker_width);
202+
const timestamp_t start_timestamp = lp_ticker_read();
203+
const timestamp_t next_match_timestamp = overflow_protect(start_timestamp + us_to_ticks(i, ticker_freq), ticker_width);
198204

199205
lp_ticker_set_interrupt(next_match_timestamp);
200206

201207
sleep();
202208

203209
const timestamp_t wakeup_timestamp = lp_ticker_read();
204210

205-
TEST_ASSERT(compare_timestamps(us_to_ticks(deepsleep_mode_delta_us, ticker_freq), ticker_width, next_match_timestamp, wakeup_timestamp));
211+
sprintf(info, "Delta ticks: %u, Ticker width: %u, Expected wake up tick: %d, Actual wake up tick: %d, delay ticks: %d, wake up after ticks: %d",
212+
us_to_ticks(deepsleep_mode_delta_us, ticker_freq), ticker_width, next_match_timestamp, wakeup_timestamp, us_to_ticks(i, ticker_freq) , wakeup_timestamp - start_timestamp);
213+
214+
TEST_ASSERT_MESSAGE(compare_timestamps(us_to_ticks(deepsleep_mode_delta_us, ticker_freq), ticker_width,
215+
next_match_timestamp, wakeup_timestamp), info);
206216
}
207217

208218
set_lp_ticker_irq_handler(lp_ticker_irq_handler_org);
@@ -245,8 +255,12 @@ void deepsleep_high_speed_clocks_turned_off_test()
245255

246256
TEST_ASSERT_UINT32_WITHIN(1000, 0, ticks_to_us(us_ticks_diff, us_ticker_freq));
247257

258+
sprintf(info, "Delta ticks: %u, Ticker width: %u, Expected wake up tick: %d, Actual wake up tick: %d",
259+
us_to_ticks(deepsleep_mode_delta_us, lp_ticker_freq), lp_ticker_width, wakeup_time, lp_ticks_after_sleep);
260+
248261
/* Check if we have woken-up after expected time. */
249-
TEST_ASSERT(compare_timestamps(us_to_ticks(deepsleep_mode_delta_us, lp_ticker_freq), lp_ticker_width, wakeup_time, lp_ticks_after_sleep));
262+
TEST_ASSERT_MESSAGE(compare_timestamps(us_to_ticks(deepsleep_mode_delta_us, lp_ticker_freq), lp_ticker_width,
263+
wakeup_time, lp_ticks_after_sleep), info);
250264
}
251265

252266
#endif

0 commit comments

Comments
 (0)