Skip to content

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Mar 2, 2021

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) and tests/congure_test (integration tests), both tests should still pass:

$ cd dist/pythonlibs/riotctrl_shell && tox
$ cd ../../../tests/congure_test/ && BOARD="<your choice>" make -j flash test

Issues/PRs references

Requires #16111 as it rewrote some of the shell commands for congure_test (merged).

PR dependency graph

@miri64 miri64 added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: tests Area: tests and testing framework Area: tools Area: Supplementary tools CI: run tests If set, CI server will run tests on hardware for the labeled PR CI: disable test cache If set, CI will always run all tests regardless of whether they have been run successfully before labels Mar 2, 2021
@miri64 miri64 added this to the Release 2021.04 milestone Mar 2, 2021
@miri64 miri64 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 3, 2021
@miri64 miri64 force-pushed the riotctrl_shell/feat/congure_test branch from 05f08fc to b34aa79 Compare March 3, 2021 11:40
@miri64
Copy link
Member Author

miri64 commented Mar 3, 2021

Rebased to current master and dependencies to resolve merge conflict.

@miri64 miri64 force-pushed the riotctrl_shell/feat/congure_test branch from b34aa79 to 0d310bd Compare March 3, 2021 13:13
@miri64
Copy link
Member Author

miri64 commented Mar 3, 2021

Rebased to current master since #16111 was merged.

@kaspar030
Copy link
Contributor

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.

@miri64
Copy link
Member Author

miri64 commented Mar 9, 2021

As other tests are probably using the changed riotctrl_shell, I suggest giving this a full test run.

There is currently only congure_test that uses it.

@miri64 miri64 force-pushed the riotctrl_shell/feat/congure_test branch from 0d310bd to 5ceffde Compare March 9, 2021 10:24
@miri64
Copy link
Member Author

miri64 commented Mar 9, 2021

Squashed the provided doc improvement suggestion and removed the REMOVE ME! commit.

Copy link
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK.

@miri64 miri64 merged commit d27ea02 into RIOT-OS:master Mar 9, 2021
@miri64 miri64 deleted the riotctrl_shell/feat/congure_test branch March 9, 2021 11:28
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 Area: tools Area: Supplementary tools CI: disable test cache If set, CI will always run all tests regardless of whether they have been run successfully before CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants