[riot-notifications] [RIOT-OS/RIOT] [BUG] testrunner matching local echo (#10952)

Gaƫtan Harter notifications at github.com
Wed Feb 20 12:08:01 CET 2019

I just tested with `picocom` and the test fails as `picocom` does a local echo itself:

RIOT_TERMINAL=picocom BOARD=samr21-xpro make --no-print-directory -C examples/hello-world test
picocom --nolock --imap lfcrlf --echo --baud "115200" "/dev/ttyACM0"
picocom v2.2

port is        : /dev/ttyACM0
flowcontrol    : none
baudrate is    : 115200
parity is      : none
databits are   : 8
stopbits are   : 1
escape is      : C-a
local echo is  : yes
noinit is      : no
noreset is     : no
nolock is      : yes
send_cmd is    : sz -vv
receive_cmd is : rz -vv -E
imap is        : lfcrlf,
omap is        : 
emap is        : crcrlf,delbs,

Type [C-a] [C-h] to see available commands

Terminal ready
main(): This is RIOT! (Version: 2019.04-devel-120-g2558e-pr/bmain(): This is RIOT! (Version: 2019.04-devel-120-g2558e-pr/bug/testrunner/matching_local_echo)
Hello World!
the test should not match this
You are running RIOT on a(n) samr21-xpro board.
This board features a(n) samd21 MCU.
the test should not match this

Traceback (most recent call last):
  File "/home/harter/work/git/RIOT/examples/hello-world/tests/01-run.py", line 20, in <module>
  File "/home/harter/work/git/RIOT/dist/pythonlibs/testrunner/__init__.py", line 57, in run
  File "/home/harter/work/git/RIOT/examples/hello-world/tests/01-run.py", line 16, in testfunc
    assert res == 0, "There should have been a timeout and not match stdin"
AssertionError: There should have been a timeout and not match stdin
/home/harter/work/git/RIOT/examples/hello-world/../../Makefile.include:568: recipe for target 'test' failed
make: *** [test] Error 1

So having a dedicated test could be good.

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...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190220/94598632/attachment-0001.html>

More information about the notifications mailing list