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.

Sign up for GitHub

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

Jump to bottom

Make functional tests runnable via CFE_ES_START_APP_CC command #1281

Closed
jphickey opened this issue Apr 7, 2021 · 0 comments · Fixed by #1292 or #1406
Closed

Make functional tests runnable via CFE_ES_START_APP_CC command #1281

jphickey opened this issue Apr 7, 2021 · 0 comments · Fixed by #1292 or #1406
Assignees
@jphickey
Labels
enhancement unit-test
Milestone
Caelum

Comments

@jphickey
Copy link
Contributor

jphickey commented Apr 7, 2021

Is your feature request related to a problem? Please describe.
In the functional test proof-of-concept, the test cases are libraries that are loaded in the startup script, and the "testrunner" app actually runs them. But because libraries are not loadable/unloadable via command, it requires a custom startup script, and restarting CFE in order to run tests.

Describe the solution you'd like
The functional tests should be integrated with test runner such that they are each their own app, therefore runnable by simply issuing the ES command to start the app if they were not part of the initial startup script.

Additional context
This will simplify the process of running tests on targets, since it will alleviate the need to customize the startup script.

The cfe_assert library itself will likely need to remain as a library, because it is a common dependency between all test apps. But it shouldn't hurt to simply load this by default in the sample startup script, it won't interfere with FSW.

Requester Info
Joseph Hickey, Vantage Systems, Inc.
@jphickey jphickey self-assigned this Apr 7, 2021
@skliper skliper added this to the 7.0.0 milestone Apr 8, 2021
jphickey added a commit to jphickey/cFE that referenced this issue Apr 9, 2021
@jphickey
Fix nasa#1281, functional test as app instead of library
a3007ac 
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.
@jphickey jphickey mentioned this issue Apr 9, 2021
Merged
jphickey added a commit to jphickey/cFE that referenced this issue Apr 12, 2021
@jphickey
Fix nasa#1281, functional test as app instead of library
99be4cd 
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.
jphickey added a commit to jphickey/cFE that referenced this issue Apr 12, 2021
@jphickey
Fix nasa#1281, functional test as app instead of library
bf7f7f1 
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.
jphickey added a commit to jphickey/cFE that referenced this issue Apr 12, 2021
@jphickey
Fix nasa#1281, functional test as app instead of library
422dcac 
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.
jphickey added a commit to jphickey/cFE that referenced this issue Apr 14, 2021
@jphickey
Fix nasa#1281, adjust event types and documentation
ec2d2f6 
Adjust the event types so that Start/End are "info" events, not debug
events (in the default platform config, debug events are not shown).

Also update documentation to correct the startup script entries.
jphickey added a commit to jphickey/cFE that referenced this issue Apr 20, 2021
@jphickey
Fix nasa#1281, correct string matches in test
e4afd35 
Use a constant at the top of the file, and change the
names to match current environment.
jphickey added a commit to jphickey/cFE that referenced this issue Apr 20, 2021
@jphickey
Fix nasa#1281, correct string matches in test
23c835b 
Use a constant at the top of the file, and change the
names to match current environment.
astrogeco added a commit that referenced this issue Apr 22, 2021
@astrogeco
Merge pull request #1292 from jphickey/fix-1281-test-app
066ffff 
Fix #1281, remove testrunner and convert testcase to app
@astrogeco astrogeco mentioned this issue Apr 22, 2021
Merged
zanzaben pushed a commit to zanzaben/cFE that referenced this issue Apr 22, 2021
@jphickey @zanzaben
Fix nasa#1281, functional test as app instead of library
f510be9 
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.
zanzaben pushed a commit to zanzaben/cFE that referenced this issue Apr 22, 2021
@jphickey @zanzaben
Fix nasa#1281, adjust event types and documentation
aaf0be6 
Adjust the event types so that Start/End are "info" events, not debug
events (in the default platform config, debug events are not shown).

Also update documentation to correct the startup script entries.
zanzaben pushed a commit to zanzaben/cFE that referenced this issue Apr 22, 2021
@jphickey @zanzaben
Fix nasa#1281, correct string matches in test
65693a7 
Use a constant at the top of the file, and change the
names to match current environment.
zanzaben pushed a commit to zanzaben/cFE that referenced this issue Apr 22, 2021
@astrogeco @zanzaben
Merge pull request nasa#1292 from jphickey/fix-1281-test-app
63c356d 
Fix nasa#1281, remove testrunner and convert testcase to app
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
enhancement unit-test
Projects
None yet
Milestone
Caelum
2 participants
@jphickey @skliper