-
Notifications
You must be signed in to change notification settings - Fork 3k
Split integration tests for filesystem and network #11703
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
Split integration tests for filesystem and network #11703
Conversation
@MarceloSalazar, thank you for your changes. |
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.
Add also details in the commit message.
I would suggest moving tests one commit, removal of flashiap another one - two separate things.
Discussed with @jamesbeyond and agreed to re-enable flashiap. @0xc0170 this is ready for review - hopefully on time for code freeze ;-) |
CI started meanwhile |
Test run: FAILEDSummary: 2 of 4 test jobs failed Failed test jobs:
|
CI restarted |
Test run: SUCCESSSummary: 5 of 5 test jobs passed |
Travis failed to report , nothing at all. reopen in a second to get a new update |
I can't find it even on Travis (11703 PR not in the list there , I browsed back to 11700-2 PRs), something went wrong. @MarceloSalazar can you create new PR, that should go quickly in? based on all here. |
I'll try to close/reopen one more time otherwise we need a new PR :/ |
Description
This PR splits integration tests to let developers run individual tests for filesystem, network or stress (fs+net) depending on what the platform supports.
This is an example of platforms and tests that can be run:
The way to run the integration tests has not changed, but no valid test cases are now skipped.
mbed test -t <toolchain> -m <target> -DINTEGRATION_TESTS
Pull request type
Reviewers
@jamesbeyond
FYI @maclobdell @toyowata