Skip to content

Commit 8a4dbff

Browse files
committed
validate base_time key received
This is required as greentea can send multiple __sync packets and cause the to be in unexpected state.
1 parent f535cb5 commit 8a4dbff

File tree

3 files changed

+21
-4
lines changed

3 files changed

+21
-4
lines changed

TESTS/mbed_drivers/ticker/main.cpp

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,12 +93,16 @@ void test_case_1x_ticker() {
9393
char _key[11] = { };
9494
char _value[128] = { };
9595
uint8_t results_size = 0;
96+
int expected_key = 1;
9697

9798
greentea_send_kv("timing_drift_check_start", 0);
9899
ticker1->attach_us(&ticker_callback_0, ONE_MILLI_SEC);
99100

100101
// wait for 1st signal from host
101-
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
102+
do {
103+
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
104+
expected_key = strcmp(_key, "base_time");
105+
} while (expected_key);
102106
greentea_send_kv(_key, callback_trigger_count * ONE_MILLI_SEC);
103107

104108
// wait for 2nd signal from host
@@ -116,6 +120,7 @@ void test_case_2x_callbacks() {
116120
char _key[11] = { };
117121
char _value[128] = { };
118122
uint8_t results_size = 0;
123+
int expected_key = 1;
119124

120125
led1 = 0;
121126
led2 = 0;
@@ -125,7 +130,10 @@ void test_case_2x_callbacks() {
125130
ticker1->attach_us(ticker_callback_1_switch_to_2, ONE_MILLI_SEC);
126131

127132
// wait for 1st signal from host
128-
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
133+
do {
134+
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
135+
expected_key = strcmp(_key, "base_time");
136+
} while (expected_key);
129137
greentea_send_kv(_key, callback_trigger_count * ONE_MILLI_SEC);
130138

131139
// wait for 2nd signal from host

TESTS/mbed_drivers/timeout/main.cpp

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,18 @@ void test_case_timeout() {
5252

5353
char _key[11] = { };
5454
char _value[128] = { };
55+
int expected_key = 1;
5556
uint8_t results_size = 0;
5657

5758
greentea_send_kv("timing_drift_check_start", 0);
5859
timeout.attach_us(set_incremeant_count, ONE_MILLI_SEC);
5960

6061
// wait for 1st signal from host
61-
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
62+
do {
63+
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
64+
expected_key = strcmp(_key, "base_time");
65+
} while (expected_key);
66+
6267
greentea_send_kv(_key, callback_trigger_count * ONE_MILLI_SEC);
6368

6469
// wait for 2nd signal from host

TESTS/mbedmicro-rtos-mbed/basic/main.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,11 +56,15 @@ void update_tick_thread() {
5656
void gt_comm_wait_thread() {
5757
char _key[11] = { };
5858
char _value[128] = { };
59+
int expected_key = 1;
5960

6061
greentea_send_kv("timing_drift_check_start", 0);
6162

6263
// wait for 1st signal from host
63-
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
64+
do {
65+
greentea_parse_kv(_key, _value, sizeof(_key), sizeof(_value));
66+
expected_key = strcmp(_key, "base_time");
67+
} while (expected_key);
6468
greentea_send_kv(_key, callback_trigger_count * ONE_MILLI_SEC);
6569

6670
// wait for 2nd signal from host

0 commit comments

Comments
 (0)