-
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
tests/pkg_u8g2: run the test in CI #11687
tests/pkg_u8g2: run the test in CI #11687
Conversation
6ef93fe
to
d6d1f83
Compare
I removed the |
tests/pkg_u8g2/Makefile
Outdated
@@ -59,4 +59,7 @@ CFLAGS += -DTEST_PIN_RESET=$(TEST_PIN_RESET) | |||
|
|||
CFLAGS += -DTEST_DISPLAY=$(TEST_DISPLAY) | |||
|
|||
# HACK Blacklist native as `murdock` fails on utf-8 characters for native tests | |||
TEST_ON_CI_WHITELIST += nrf52dk samr21-xpro |
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.
Does it make sense to whitelist. What if other boards are added in (I think there is a pr for the esp)... Why not just use a blacklist?
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.
Because this is currently not supported as TEST_ON_CI_BLACKLIST
does not exist.
If you want I can split the variable introduction out of
- makefiles/murdock.inc.mk: change policy to run tests by default makefiles/murdock.inc.mk: change policy to run tests by default #11680
HACK, the test currently fails in CI for `native` due to `utf-8` handling.
2f93ff8
to
50449b3
Compare
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.
LGTM, murdock is green so ACK.
Thank you for the review. |
Contribution description
Enable running the test as the test is available.
Testing procedure
The test must be successfully executed by CI (check the listed tests).
Issues/PRs references
Found it was not enabled while working on #11680
I blacklisted
native
due to #11691