Skip to content

Fix Ice Tea test execution order to be sorted by names. #8588

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 1, 2018

Conversation

mprse
Copy link
Contributor

@mprse mprse commented Oct 30, 2018

Description

This patch provides proposition to sort list of Ice Tea tests to be executed (and shown in results table) by name .
While working on SPI communication test PR #8443 I found that test cases listed in final results are scattered and the readability is poor.

The tests verifies spi modes, buffers, freq, symbol sizes, etc. On the picture below we can see that all cases are mixed and it is hard to look for results for the specific group of cases.

image

Below are the results with the fix (sorted by names). Now it is easy to check the results for the specific group (e.g. BUFFERS tests).
image

Pull request type

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

@0xc0170 0xc0170 requested review from OPpuolitaival and a team October 30, 2018 14:45
Copy link
Contributor

@OPpuolitaival OPpuolitaival left a comment

Choose a reason for hiding this comment

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

Looks good

@cmonr
Copy link
Contributor

cmonr commented Oct 31, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

Build : SUCCESS

Build number : 3521
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8588/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Oct 31, 2018

@mbed-ci
Copy link

mbed-ci commented Nov 1, 2018

@cmonr
Copy link
Contributor

cmonr commented Nov 1, 2018

/morph test

@mbed-ci
Copy link

mbed-ci commented Nov 1, 2018

@cmonr cmonr merged commit 274f8b9 into ARMmbed:master Nov 1, 2018
@cmonr cmonr removed the needs: CI label Nov 1, 2018
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.

5 participants