-
Notifications
You must be signed in to change notification settings - Fork 2k
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
riotctrl_shell: add ShellInteraction for congure_test
#16119
riotctrl_shell: add ShellInteraction for congure_test
#16119
Conversation
05f08fc
to
b34aa79
Compare
Rebased to current master and dependencies to resolve merge conflict. |
b34aa79
to
0d310bd
Compare
Rebased to current master since #16111 was merged. |
LGTM, please squash! feel free to add the "BOARD" thingy. I tested this on nrf52840dk, CI tested wroom32 and samr21-xpro. As other tests are probably using the changed riotctrl_shell, I suggest giving this a full test run. |
There is currently only |
0d310bd
to
5ceffde
Compare
Squashed the provided doc improvement suggestion and removed the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK.
Contribution description
This introduces a shell interaction for the shell commands of
congure_test
.Testing procedure
I added tests both in
dist/pythonlibs/riotctrl_shell/tests
(unittests) andtests/congure_test
(integration tests), both tests should still pass:Issues/PRs references
Requires
#16111 as it rewrote some of the shell commands for(merged).congure_test