[riot-notifications] [RIOT-OS/RIOT] tests/conjugre-turo: reset before opening terminal (#16564)
notifications at github.com
Thu Jun 17 09:01:15 CEST 2021
### Contribution description
Some BOARD terminal is slow to start so if the BOARD is reset after
opening the terminal this can lead to off-by-one error when parsing
So instead do as testrunner and reset before opening the terminal.
Note that no interactive_sync is needed since a similar mechanism
happens in ShellInteraction.
### Testing procedure
- Murdock should now be green
- Ran the tests on a list of [BOARD's](https://ci.inria.fr/ci-riot-tribe/job/build-pipeline.jk/271/artifact/), (atmega based boards failures are unrelated to this PR, and `ek-lm4f120xl` failed to flash, and `z1` fails to boot the firmware on master as well)
### Realted issues
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* tests/conjugre-turo: reset before opening terminal
-- File Changes --
M tests/congure_test/tests/01-run.py (2)
M tests/turo/tests/01-run.py (2)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications