sys/test_helper: Add test_helper module#11224
sys/test_helper: Add test_helper module#11224MrKevinWeiss wants to merge 3 commits intoRIOT-OS:masterfrom
Conversation
This commit adds a test_helper module that provides functions to help with writing shell based tests. This should make it easier to stick to a standard way of writing and reduce code duplication.
This commit replaces the echo command and uses the test_helper module in the shell test.
MrKevinWeiss
added
Type: enhancement
MrKevinWeiss
force-pushed
the
pr/addtest_helper
branch
from
f698ea1 to
fdeb243
Compare
|
@smlng should I also add an implementation with it (maybe the spi so it doesn't break the HiL)? |
|
I should maybe add a different pseudomodule that brings in the shell everything something like |
|
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
bot
added
the
State: stale
MrKevinWeiss
added
the
State: don't stale
stale
bot
removed
the
State: stale
|
Implemented in RFtests but I don't want to close until some conclusion is reached. |
|
@MrKevinWeiss would be nice to have this in a way where it could optionally be toogled on and off. |
|
I will try to get this up to speed when I come back to work next week. |
|
closed with #15950 |
2 participants
Contribution description
This PR adds a test_helper module adds a collection of helpers used for shell based tests. Specifically it adds basic, lightweight json formatting, print_echo and collects module dependencies. This should make it easier to stick to a standard way of writing and reduce code duplication.
Testing procedure
make all -C tests/shell/ test