- Notifications
You must be signed in to change notification settings - Fork425
Convert all tests to use googletest#580
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This change systematically removes the dependency to Boost.Testthroughout the tests in libs/network/test/*. Most of the changes aremechanical, and functionally equivalent.We can actually start simplifying the test structure after this changegets pulled into the main repository.
glynos commentedJan 7, 2016
What is the problem with using CMake 3.0? |
deanberris commentedJan 7, 2016
The Travis containerised environment only has CMake 2.8.x, so until we figure out how to install CMake 3.0 during the integration testing, I've bumped down the version instead. Sent from my iPhone
|
Convert all tests to use googletest
glynos commentedJan 8, 2016
LGTM |
This change systematically removes the dependency to Boost.Test
throughout the tests in libs/network/test/*. Most of the changes are
mechanical, and functionally equivalent.
We can actually start simplifying the test structure after this change
gets pulled into the main repository.