-
Notifications
You must be signed in to change notification settings - Fork 215
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 #559, Rename set force fail #646
Fix #559, Rename set force fail #646
Conversation
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.
Recommending a ticket to change UT_ClearForceFail to UT_ClearDefaultReturnValue.
I'd also recommend instead of using a simple
If nothing else this just makes it more like the patterns used elsewhere. The whole thing can go into a |
139a7e7
to
52ee871
Compare
I recommend avoiding merging main into a feature branch. Preference is to rebase the feature on main. You can always do a soft reset and recommit to fix this. |
e9e05c6
to
48ac38e
Compare
Fix nasa#639, Remove ES_APP_DEBUG functionality
Describe the contribution
Fixes #559 by changing UT_SetForceFail to UT_SetDefaultReturnValue
Testing performed
Build and run unit test
Expected behavior changes
No impact to behavior
System(s) tested on
Ubuntu 20.04
Additional context
Contributor Info - All information REQUIRED for consideration of pull request
Alex Campbell - NASA/GSFC