-
Notifications
You must be signed in to change notification settings - Fork 58
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
Fix #168, PSP coverage unit test #174
Fix #168, PSP coverage unit test #174
Conversation
Only commit 892880a is relevant. Will rebase after the next baseline. The following mcp750-vxworks PSP files are currently covered as an example: |
foreach(TGT ${INSTALL_TARGET_LIST}) | ||
install(TARGETS coverage-${CFE_PSP_TARGETNAME}-testrunner DESTINATION ${TGTNAME}/${UT_INSTALL_SUBDIR}) | ||
endforeach() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TGT
not used in install line, why foreach?
Also related to #14 |
What's the status on this, can it be reviewed today? |
Implement a basic example of PSP coverage test.
892880a
to
597921e
Compare
Rebased to "main" -- yes this can be reviewed today. |
CCB 2020-07-29 APPROVED |
@skliper do you want to add this to Bootes or push to Caelum? |
If you can merge it today, Bootes. My logic is there is a customer waiting on it and there is no impact to flight code. If anyone ever needs to certify Bootes it's a very valuable framework to have. |
We're on the same page. Done. |
Describe the contribution
Add a framework and example for coverage test of the PSP code.
Fix #168
Testing performed
Build and run unit test
Build and sanity check CFE (no changes to FSW)
Expected behavior changes
New PSP coverage test executable is built, and several files within PSP are targeted.
System(s) tested on
Ubuntu 20.04
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.