Skip to content

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jan 8, 2016

Conversation

deanberris
Copy link
Member

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.

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.
@glynos
Copy link
Member

glynos commented Jan 7, 2016

What is the problem with using CMake 3.0?

@deanberris
Copy link
Member Author

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

On 8 Jan 2016, at 2:07 AM, Glyn Matthews notifications@github.com wrote:

What is the problem with us in CMake 3.0?


Reply to this email directly or view it on GitHub.

glynos added a commit that referenced this pull request Jan 8, 2016
Convert all tests to use googletest
@glynos glynos merged commit dc5cf50 into cpp-netlib:master Jan 8, 2016
@glynos
Copy link
Member

glynos commented Jan 8, 2016

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants