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

Add at least one conditional for unit testing #8

Closed
jphickey opened this issue Nov 20, 2019 · 0 comments · Fixed by #14
Closed

Add at least one conditional for unit testing #8

jphickey opened this issue Nov 20, 2019 · 0 comments · Fixed by #14
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@jphickey
Copy link
Contributor

Is your feature request related to a problem? Please describe.
In the process of providing useful examples for unit testing, the sample lib should contain
at least one "if" statement to demonstrate how the alternate paths can be exercised in UT.

Describe the solution you'd like
Add a call to a simple C library function during SAMPLE_LibInit, and have an alternate return code (not CFE_SUCCESS) as based off the result. Unit test can then force the alternate path to execute.

Describe alternatives you've considered
N/A

Additional context
This will add considerable value to the UT examples

Requester Info
Joseph Hickey, Vantage Systems, Inc.

@jphickey jphickey self-assigned this Nov 20, 2019
@skliper skliper added the enhancement New feature or request label Dec 18, 2019
@skliper skliper added this to the 1.2.0 milestone Dec 18, 2019
skliper added a commit that referenced this issue Dec 30, 2019
Fix #8 
Reviewed and approved at 2019-12-18 CCB
@skliper skliper closed this as completed in 2264f97 Jan 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants