bartoszek/zephyr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

tests: drivers: can: add a delay for the shell backend to initialize

Browse Source 
The CAN shell test is flaky on some qemu platform (qemu_riscv64_smp in
particular), where the first test command occasionally fails as it tries
to run with the backend not initialized.

The tests for subsys/shell/shell/src/main.c have a setup delay, copying
it to the CAN shell test seems to make it work reliably.

Tested with:
  ./scripts/twister -v -n -M -p qemu_riscv64_smp -T tests/drivers/can/shell

Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
This commit is contained in:
Fabio Baltieri 2022-09-05 18:04:31 +00:00 committed by Fabio Baltieri
parent 92f289597e
commit e9e7ad388d
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tests/drivers/can/shell/src/main.c
View File

@ -578,4 +578,11 @@ static void can_shell_before(void *fixture)
memset(&frame_capture, 0, sizeof(frame_capture));
}
ZTEST_SUITE(can_shell, NULL, NULL, can_shell_before, NULL, NULL);
static void *can_shell_setup(void)
{
/* Let the shell backend initialize. */
k_msleep(20);
return NULL;
}
ZTEST_SUITE(can_shell, NULL, can_shell_setup, can_shell_before, NULL, NULL);