Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix #591, add test teardown failure to test summary #694

Merged

Conversation

zanzaben
Copy link
Contributor

@zanzaben zanzaben commented Dec 17, 2020

Describe the contribution
Fixes #591
added test teardown failures to the test summary
also changed the print out to use the same style as startup failures.

Testing performed
Used a test that always fails teardown and verified that it was reported in the test summary.

Expected behavior changes
N/A

System(s) tested on
Ubuntu 20.04

Contributor Info - All information REQUIRED for consideration of pull request
Alex Campbell GSFC

@zanzaben zanzaben requested a review from skliper December 17, 2020 16:28
@zanzaben zanzaben added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Dec 17, 2020
@astrogeco
Copy link
Contributor

CCB 2021-01-06 APPROVED

@zanzaben zanzaben changed the title Fix #591 add test teardown failure to test summary Fix #591, add test teardown failure to test summary Jan 12, 2021
@zanzaben zanzaben force-pushed the fix591_TTF_not_reported branch from 3a18fe8 to 55aa1a7 Compare January 13, 2021 14:55
@astrogeco astrogeco added CCB:2021-01-13 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Jan 13, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate January 25, 2021 01:33
@astrogeco astrogeco merged commit 4165e67 into nasa:integration-candidate Jan 25, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Jan 25, 2021
@zanzaben zanzaben deleted the fix591_TTF_not_reported branch February 23, 2021 17:15
@skliper skliper added this to the 6.0.0 milestone Sep 24, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TTF (test teardown failure) not reported in individual or full test summary
3 participants