-
Notifications
You must be signed in to change notification settings - Fork 3k
Add Watchdog Manager tests #9586
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
Add Watchdog Manager tests #9586
Conversation
@fkjagodzinski, thank you for your changes. |
37c0c86
to
dc1979a
Compare
ffadd46
to
259e22d
Compare
@kegilbert Where's the error in this |
@fkjagodzinski Please fix drivers/Watchdog.h, which has a misspelling of |
Spelling fixed. ✔️ |
db15f92
to
acddd2a
Compare
7e5833e
to
348e578
Compare
The Watchdog driver has been updated making these tests incompatible with the new API. The Watchdog driver functionality is tested with the unit tests located in UNITTESTS/drivers/Watchdog/ .
acddd2a
to
8087c3d
Compare
Minor update:
|
The Watchdog driver has been updated making this test incompatible with the new API. Use HAL API instead.
Test run: FAILEDSummary: 3 of 3 test jobs failed Failed test jobs:
|
rajkan01#1 has been merged and Watchdog Manager tests are now included in #10066. |
Description
After the Watchdog driver update from
#9243#10066 the greentea tests also need an update:tests-mbed_drivers-watchdog*
,tests-mbed_platform-watchdog_mgr_reset
.Please note, this PR needs #10066 merged first to work.
Pull request type
Reviewers
@c1728p9 @rajkan01