-
Notifications
You must be signed in to change notification settings - Fork 217
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 #1052, implement missing parameter/retcode test permutations #1066
Fix #1052, implement missing parameter/retcode test permutations #1066
Conversation
After Binsem fix:
|
After CountSem fix:
|
After Mutex fix:
|
0058ceb
to
205cdc3
Compare
After Queue Fix:
|
After Task Fix:
|
8354d06
to
ecd298a
Compare
After Module fix:
|
After Network Fix:
|
After Select fix:
|
After Shell fix:
|
After Socket fix:
|
After Timebase fix:
|
After timer fix:
|
After directory fix:
|
After FileSys fix:
|
After File fix:
|
This appears to be the last one! That should cover all APIs. Note that the timer tests also assume/require that #1065 gets merged first. It moves some asserts back into timer callbacks, instead of saving the RC and asserting later (simpler and more reliable that way). |
1e232ed
to
b1139bd
Compare
This is complete now but it still needs a rebase to (depends on some items from the 2021-06-08 review set). |
CCB:2021-06-09 APPROVED
|
b1139bd
to
d35de1b
Compare
Rebased to latest baseline, should be mergable now. |
d35de1b
to
46eaa70
Compare
nasa/osal#1066, implement missing parameter/retcode test permutations
CCB:2021-06-23 APPROVED |
*Combines* - nasa/elf2cfetbl#81 - nasa/tblCRCTool#52 - nasa/ci_lab#88 - nasa/sch_lab#83 - nasa/sample_app#150 - nasa/sample_lib#64 - nasa/to_lab#100 *Includes* - nasa/cFE#1630, correct path to users guide warning log - nasa/cFE#1621, add additional test cases for Child Tasks - nasa/cFE#1608, Add cfe functional tests to CI - nasa/cFE#1627, rename/clean CFE coverage assert macros - nasa/cFE#1623, Added UT tests for cFE ES Api - nasa/cFE#1634, Expand CDS Functional Tests. - nasa/cFE#1633, add test log file - nasa/cFE#1594, Event ID updates - nasa/cFE#1624, scrub all UT_Report calls - nasa/osal#1066, implement missing parameter/retcode test permutations - nasa/cFS-GroundSystem#182, Add test start command script for cmdUtil - nasa/tblCRCTool#51, add printf conversion casts ** Implement Coding Standard in CodeQL ** - nasa/cFS-GroundSystem#180 - nasa/elf2cfetbl#80 - nasa/tblCRCTool#49 - nasa/ci_lab#87 - nasa/sch_lab#79 - nasa/sample_app#149 - nasa/sample_lib#63 - nasa/to_lab#99
*Combines* - nasa/cFE#1632, v6.8.0-rc1+dev726 - nasa/osal#1079, v5.1.0-rc1+dev548 - nasa/ci_lab#88, v2.4.0-rc1+dev42 - nasa/sch_lab#83, v2.4.0-rc1+dev40 - nasa/sample_app#150, v1.2.0-rc1+dev66 - nasa/sample_lib#64, v1.2.0-rc1+dev38 - nasa/to_lab#100, v2.4.0-rc1+dev49 - nasa/elf2cfetbl#81, v3.2.0-rc1+dev30 - nasa/tblCRCTool#52, v1.2.0-rc1+dev33 - nasa/cFS-GroundSystem#183, v2.2.0-rc1+dev52 *Includes* - nasa/cFE#1630, correct path to users guide warning log - nasa/cFE#1621, add additional test cases for Child Tasks - nasa/cFE#1608, Add cfe functional tests to CI - nasa/cFE#1627, rename/clean CFE coverage assert macros - nasa/cFE#1623, Added UT tests for cFE ES Api - nasa/cFE#1634, Expand CDS Functional Tests. - nasa/cFE#1633, add test log file - nasa/cFE#1594, Event ID updates - nasa/cFE#1624, scrub all UT_Report calls - nasa/osal#1066, implement missing parameter/retcode test permutations - nasa/cFS-GroundSystem#182, Add test start command script for cmdUtil - nasa/tblCRCTool#51, add printf conversion casts ** Implement Coding Standard in CodeQL ** - nasa/cFS-GroundSystem#180 - nasa/elf2cfetbl#80 - nasa/tblCRCTool#49 - nasa/ci_lab#87 - nasa/sch_lab#79 - nasa/sample_app#149 - nasa/sample_lib#63 - nasa/to_lab#99 Co-authored-by: Jacob Hageman <skliper@users.noreply.github.com> Co-authored-by: Joseph Hickey <jphickey@users.noreply.github.com> Co-authored-by: Ariel Adams <ArielSAdamsNASA@users.noreply.github.com> Co-authored-by: Alex Campbell <zanzaben@users.noreply.github.com> Co-authored-by: Jose F Martinez Pedraza <pepepr08@users.noreply.github.com>
cFE Integration Candidate: 2021-01-05
Describe the contribution
Another pass through the API to ensure that:
OS_INVALID_POINTER
retcode testOS_ERR_INVALID_SIZE
retcode testosal_id_t
type has twoOS_ERR_INVALID_ID
retcode tests, and that it is tested withOS_OBJECT_ID_UNDEFINED
andUT_OBJID_INCORRECT
(which is not zero, but not valid).Testing performed
Run all unit tests, and run validation script on tests logs to confirm parameters have been exercised and confirmed for each required retcode.
Expected behavior changes
Missing tests are implemented
System(s) tested on
Ubuntu
Additional context
This PR will have each subsystem done as separate commits
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.