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

Fix #1014, Remove Test_SB_Cmds_SubRptUnexpCmdCode #1113

Merged

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jan 21, 2021

Describe the contribution
Fix #1014, Remove Test_SB_Cmds_SubRptUnexpCmdCode

Test was just a copy/paste of Test_SB_Cmds_CmdUnexpCmdCode
and did not actually implement a new test. Subscription
reporting is a no parameter command.

Testing performed
Build and run unit tests, passed.

Expected behavior changes
None

System(s) tested on

  • Hardware: cFS Dev Server
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this commit

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added this to the 7.0.0 milestone Jan 21, 2021
Test was just a copy/paste of Test_SB_Cmds_CmdUnexpCmdCode
and did not actually implement a new test.  Subscription
reporting is a no parameter command.
@skliper skliper force-pushed the fix1014-subrptunexpcmdcode branch from bc24083 to 9740572 Compare January 27, 2021 15:10
@astrogeco astrogeco changed the base branch from main to integration-candidate January 27, 2021 22:28
@astrogeco astrogeco force-pushed the integration-candidate branch from 54dffe8 to f6c6737 Compare February 3, 2021 02:48
@astrogeco astrogeco merged commit bb8e351 into nasa:integration-candidate Feb 3, 2021
@skliper skliper deleted the fix1014-subrptunexpcmdcode branch April 1, 2021 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Test_SB_Cmds_SubRptUnexpCmdCode (no longer applicable)
2 participants