[riot-notifications] [RIOT-OS/RIOT] tests/conjugre-turo: reset before opening terminal (#16564)

Francisco 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
the output.

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

fixes  #16415
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/16564

-- 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 --

https://github.com/RIOT-OS/RIOT/pull/16564.patch
https://github.com/RIOT-OS/RIOT/pull/16564.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16564
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210617/2ba2004a/attachment.htm>


More information about the notifications mailing list