Conversation
Search for GMock (and its bundled GTest) before searching for standalone GTest. This resolves #699. Signed-off-by: Kyle Fazzari <kyle.fazzari@navy.mil>
|
@dirk-thomas Thanks for the changes, lgtm. |
|
@mikepurvis Would it be possible for you to use this branch for your testing for a little while? That would give us more confidence that it doesn't introduce any regressions... |
|
I've switched our distribution to this branch. FYI @tspicer01 |
|
We're not actively using the new gmock support, but I believe none of our unittests have gone up in flames as a result of this change. |
|
@v-lopez Thank you for providing this. I will go ahead and merge this so it will be part of the next release of this package. |
|
What I didn't realize when review this is that the patch duplicates large blocks of the logic and copy-n-pastes it into the new function |
Builds on top of #896 and addresses some issues.
@v-lopez FYI