Skip to content
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

WIP cFE Functional Test Structure Preview #1312

Closed
wants to merge 14 commits into from

Conversation

zanzaben
Copy link
Contributor

Describe the contribution
This is an overview of a few different task all merged together to get a look at the structure of the test files. To make sure everyone agrees on how they should be done and to discuss any changes they want that affect multiple tests.

Additional context
This should never be merged in. It is just to have an easy way to look at the big picture.

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

zanzaben and others added 12 commits April 12, 2021 10:17
Merge the "testrunner" app and put the runner logic into cfe_assert lib.
Converts the "testcase" library into an app that simply calls the logic
in cfe_assert to do the test.

This means that the test can be run on demand by issuing the ES
start app command, rather than putting it into the startup script.
…' into CFE_Functional_Tests

# Conflicts:
#	modules/cfe_testcase/CMakeLists.txt
#	modules/cfe_testcase/src/cfe_test.c
#	modules/cfe_testcase/src/cfe_test.h
…ctional_Tests

# Conflicts:
#	modules/cfe_testcase/CMakeLists.txt
#	modules/cfe_testcase/src/cfe_test.c
#	modules/cfe_testcase/src/cfe_test.h
@zanzaben zanzaben added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Apr 13, 2021
…tional_Tests

# Conflicts:
#	modules/cfe_testcase/src/cfe_test.c
#	modules/cfe_testcase/src/cfe_test.h
@skliper skliper added CCB:2021-04-14 and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Apr 14, 2021
@skliper
Copy link
Contributor

skliper commented Apr 14, 2021

CCB:2021-04-14 - concept reviewed and looks good

@zanzaben zanzaben closed this May 4, 2021
@skliper skliper added the invalid label May 4, 2021
@skliper
Copy link
Contributor

skliper commented May 4, 2021

Marked as "invalid" for merge, this was just for conceptual review.

@zanzaben zanzaben deleted the CFE_Functional_Tests branch May 21, 2021 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants