Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests/lwip target board for python test is hardcoded to native #6533

Open
BytesGalore opened this issue Feb 1, 2017 · 5 comments
Open

tests/lwip target board for python test is hardcoded to native #6533

BytesGalore opened this issue Feb 1, 2017 · 5 comments
Labels
Area: tests Area: tests and testing framework Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Comments

@BytesGalore
Copy link
Member

The python driven test does not run for boards other then native
But since the default BOARD for is set to iotlab-m3 for this test, I guess they should be the primary target.

Setting the boards and ttys manually in the 01_test.py [1]

Board("iotlab-m3", None, "/dev/ttyUSBX")

forces the script to start with these parameters but it fails when trying to call reset [2] before going on with testing:

RIOT/tests/lwip$ make BOARD=iotlab-m3 test
./tests/01-run.py
Testing for (<Board 'iotlab-m3',port=None,serial='/dev/ttyUSB0'>, <Board 'iotlab-m3',port=None,serial='/dev/ttyUSB2'>): 
Traceback (most recent call last):
  File "./tests/01-run.py", line 270, in <module>
    [test_ipv6_send, test_udpv6_send, test_dual_send])
  File "./tests/01-run.py", line 172, in execute
    board_group.reset()
  File "./tests/01-run.py", line 153, in reset
    board.reset(application, env)
  File "./tests/01-run.py", line 124, in reset
    self.reset_strategy.execute(application, env)
  File "./tests/01-run.py", line 71, in execute
    super(ResetStrategy, self).__run_make(application, ("reset",), env)
AttributeError: 'super' object has no attribute '_ResetStrategy__run_make'

[1] https://github.com/RIOT-OS/RIOT/blob/master/tests/lwip/tests/01-run.py#L264
[2] https://github.com/RIOT-OS/RIOT/blob/master/tests/lwip/tests/01-run.py#L168

@BytesGalore BytesGalore added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: tests Area: tests and testing framework labels Feb 1, 2017
@kYc0o
Copy link
Contributor

kYc0o commented Jan 16, 2018

Actually I'm not sure we can use the test script for any board as is, since it configures two tap interfaces to execute the tests.

Therefore, another test for radio platforms should be created.

Can we close the issue @BytesGalore ?

@stale
Copy link

stale bot commented Aug 10, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions.

@stale stale bot added the State: stale State: The issue / PR has no activity for >185 days label Aug 10, 2019
@miri64 miri64 added the State: don't stale State: Tell state-bot to ignore this issue label Aug 10, 2019
@stale stale bot removed the State: stale State: The issue / PR has no activity for >185 days label Aug 10, 2019
@aabadie aabadie removed the State: don't stale State: Tell state-bot to ignore this issue label Oct 8, 2019
@miri64
Copy link
Member

miri64 commented Jul 1, 2020

Hopefully fixable once #11406 and follow-ups are in.

@MrKevinWeiss MrKevinWeiss added this to the Release 2021.07 milestone Jun 22, 2021
@MrKevinWeiss MrKevinWeiss removed this from the Release 2021.07 milestone Jul 15, 2021
@maribu
Copy link
Member

maribu commented Sep 16, 2022

@miri64: ping?

@miri64
Copy link
Member

miri64 commented Sep 19, 2022

See #14874

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: tests Area: tests and testing framework Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

No branches or pull requests

6 participants