Skip to content

Split integration tests in fs/net #11943

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

Merged
merged 1 commit into from
Nov 26, 2019

Conversation

MarceloSalazar
Copy link

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:

fs-single net-single stress (fs+net)
K64F (sd+eth) ✔️ ✔️ ✔️
NUCLEO_F429ZI (eth) ✔️
DISCO_L496AG (qspi) ✔️

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

This is another attempt to merge after these ones: #11703 #11858

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[X] Test update
[ ] Breaking change

Reviewers

@0xc0170

@ciarmcom ciarmcom requested review from 0xc0170 and a team November 26, 2019 10:00
@ciarmcom
Copy link
Member

@MarceloSalazar, thank you for your changes.
@0xc0170 @ARMmbed/mbed-os-maintainers please review.

Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR #11703 was approved

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 26, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Nov 26, 2019

Test run: SUCCESS

Summary: 5 of 5 test jobs passed
Build number : 1
Build artifacts

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 26, 2019

I have no idea why Travis do not pick this up again 😞 I am checking it

@0xc0170 0xc0170 closed this Nov 26, 2019
@0xc0170 0xc0170 reopened this Nov 26, 2019
@0xc0170
Copy link
Contributor

0xc0170 commented Nov 26, 2019

continuous-integration/travis-ci/pr Pending

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants