Add port tests for ticker and lp ticker #4628
Conversation
|
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputBuild failed! |
|
retest uvisor |
TESTS/mbed_hal/ticker_port/main.cpp
Outdated
| return greentea_case_setup_handler(source, index_of_case); | ||
| } | ||
|
|
||
| utest::v1::status_t lp_ticker_setup(const Case *const source, const size_t index_of_case) |
There was a problem hiding this comment.
This should be protected also, same as 2 lp ticker cases?
Add a test to make sure timer events scheduled for the past fire immediately, and add a test to ensure many events occurring at once do not cause a stack overflow.
|
Rebased, fix that compilation error, we can rerun the tests /morph test |
|
Thanks Russ, this is great for testing the events in the past. I was able to rebase this, and run on some devices that I knew had some problems. |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputTest failed! |
|
@studavekar Was this failure related to this PR? or unstable tests? |
|
/morph test |
|
/morph test-nightly |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputTest failed! |
|
@c1728p9 looks like _hal-ticker_port seems to be failing on a large number of devices http://mbed-ci-master-2.austin.arm.com:8081/job/test_matrix/600/#showFailuresLink |
|
@c1728p9 Bump. What's the status? |
|
This will be updated when #4644 is. |
|
@c1728p9 Bump. any news? |
|
Closing this PR since the tests aren't relevant after #4644. Will open a new PR with updated tests in the future. |
Add a test to make sure timer events scheduled for the past fire immediately, and add a test to ensure many events occurring at once do not cause a stack overflow.