-
Notifications
You must be signed in to change notification settings - Fork 3k
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: skip timing accuracy test on FastModels #10030
Conversation
@@ -23,6 +23,11 @@ | |||
#include "hal/us_ticker_api.h" | |||
#include "hal/lp_ticker_api.h" | |||
|
|||
//FastModels not support timing test | |||
#if defined(__ARM_FM) |
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.
I thought that specifying a specific target in tests was not recommended ?
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.
Correct is not. This is however an architecture exclusion
@ARMmbed/mbed-os-hal Please review
@jamesbeyond, thank you for your changes. |
Similar reasoning to #10124 (comment) |
CI started |
Test run: FAILEDSummary: 1 of 6 test jobs failed Failed test jobs:
|
Will restart The failures were all related to K66F timeouts. |
CI job restarted: K66F failures shouldn't be related to this PR. |
Description
This is the test case for timing accuracy, Software models can't guarantee it's time accuracy as it's nature.
So skip this test for FastModels.
Pull request type