Skip to content

Conversation

@astrogeco
Copy link
Contributor

@astrogeco astrogeco commented Aug 18, 2020

Describe the contribution

Fix #587
Fix #608
Fix #759
Fix #778
Fix #787

Testing performed

Bundle Test - https://github.com/nasa/cFS/pull/127/checks

Expected behavior changes

PR #784 - Deprecates CFE_SB_GetLastSenderId() API by introducing new CFE_OMIT_DEPRECATED_6_8 tag

PR #789 - Documentation update remove deleted requiremements

PR #804 - Add a new cfe_assert module for functional testing by making it possible to load the UT assert object code as a CFE library. These are compiled as separate, independent modules and only loaded on demand. Also includes a draft example for cFE testing, which calls some basic ES AppId functions.

System(s) tested on
Bundle CI - https://github.com/nasa/cFS/pull/127/checks

Additional context
Part of nasa/cFS#127

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Christopher D. Knight, NASA-ARC
Jacob Hageman, NASA-GSFC
Joseph Hickey, Vantage Systems, Inc.

CDKnightNASA and others added 12 commits July 31, 2020 12:26
 #787
 - cES1005.1: Remove application load address from start app command
 - cTBL6003.1: Clarify table validation pior to activate
 - cEVS3103.4, cEVS3103.5, cEVS3001: Remove SB qualifier
 #587
 - cSB4344: Added get packet type API requirement
 - cSB4345: Added message ID validation API requirement
Provides ability to load UT assert as a CFE app, and an example
of using this to test some basic CFE ES functions.
Fix #778, add cfe assert and example lib
…recated

fix #759 - deprecates GetLastSenderId()
Provides ability to load UT assert as a CFE app, and an example
of using this to test some basic CFE ES functions.

This introduces a separate library for the basic UT assert functions
(cfe_assert) and an app that executes the test (cfe_testrunner) rather
than combining these into a single module.
Revert "Fix #778, add cfe assert and example lib"
Fix #778, Add CFE_Assert library and test module
@astrogeco astrogeco marked this pull request as draft August 18, 2020 18:19
@astrogeco astrogeco changed the title Integration candidate Integration Candidate 2020-08-12 Aug 18, 2020
@astrogeco astrogeco marked this pull request as ready for review August 19, 2020 14:55
@astrogeco astrogeco merged commit a148b97 into main Aug 19, 2020
astrogeco added a commit to nasa/cFS that referenced this pull request Aug 19, 2020
@skliper skliper added this to the 7.0.0 milestone Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment