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

Move heap_and_stack tests to TESTS/mbed_bootstrap #5278

Merged
merged 1 commit into from Oct 20, 2017

Conversation

Projects
None yet
5 participants
@maciejbocianski
Member

maciejbocianski commented Oct 9, 2017

Description

Moved to new location TESTS/mbed_bootstrap

Status

READY

@maciejbocianski

This comment has been minimized.

Member

maciejbocianski commented Oct 9, 2017

@bulislaw could you confirm this move

@bulislaw

This comment has been minimized.

Member

bulislaw commented Oct 10, 2017

Now I'm thinking the tests require RTX for setup, can you confirm that? If so then we should put it in rtos test dir.

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 10, 2017

Build : SUCCESS

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

Triggering tests

/test mbed-os

@mbed-ci

This comment has been minimized.

@maciejbocianski

This comment has been minimized.

Member

maciejbocianski commented Oct 11, 2017

I can see that it uses mostly setup from mbed_boot.c

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 12, 2017

I can see that it uses mostly setup from mbed_boot.c

thus this shold be left as it was or?

@bulislaw

This comment has been minimized.

Member

bulislaw commented Oct 12, 2017

No, lets move it to test folder, but to rtos tests.

@maciejbocianski maciejbocianski force-pushed the maciejbocianski:heap_and_stack_tests branch to 9ab2a1d Oct 13, 2017

@maciejbocianski

This comment has been minimized.

Member

maciejbocianski commented Oct 13, 2017

@bulislaw done

@0xc0170 0xc0170 added needs: CI and removed needs: review labels Oct 13, 2017

@mbed-ci

This comment has been minimized.

mbed-ci commented Oct 13, 2017

Build : SUCCESS

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

Skipping test trigger, missing label 'NEED CI'

@mbed-ci

This comment has been minimized.

@mbed-ci

This comment has been minimized.

@maciejbocianski

This comment has been minimized.

Member

maciejbocianski commented Oct 17, 2017

ping

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Oct 17, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 19, 2017

/morph uvisor-test

@0xc0170 0xc0170 added needs: CI and removed ready for merge labels Oct 19, 2017

@adbridge adbridge merged commit 9c1fd48 into ARMmbed:master Oct 20, 2017

7 checks passed

AWS-CI uVisor Build & Test Success
Details
AWS-CI uvisor Test DIDNT RUN UVISOR TESTS
Details
Cam-CI uvisor Build & Test DIDNT RUN UVISOR TESTS
Details
ci-morph-build build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment