Skip to content

Message refactor #631

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.

Sign up for GitHub

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 11 commits into from
Apr 11, 2016
Merged

Message refactor #631

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
858eea7
Added image to the license in README.rst.
glynos Apr 4, 2016
0187ec1
Removed some redundant headers.
glynos Apr 2, 2016
047d89c
Added a first test in order to start refactoring the HTTP request/res…
glynos Apr 3, 2016
40bab67
Since I removed the include directory in a previous commit, I have no…
glynos Apr 3, 2016
175cdd3
Updated the examples that used the include directory that I removed.
glynos Apr 3, 2016
5614b43
Removed boost/network.hpp
glynos Apr 9, 2016
2f61e45
Fixed string constants.
glynos Apr 10, 2016
4c9fbda
Added a unit tests for the HTTP client request.
glynos Apr 10, 2016
5f6ab56
Added a unit test for the HTTP client response.
glynos Apr 10, 2016
99c777c
Removed some headers that looked redundant.
glynos Apr 10, 2016
1e14f3c
Refactored HTTP client request and response.
glynos Apr 10, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Added a unit test for the HTTP client response.
  • Loading branch information
@glynos
glynos committed Apr 10, 2016
commit 5f6ab563f15417453fb2ba23c8d6f9c03efaa104
1 change: 1 addition & 0 deletions libs/network/test/http/client/CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
set(
TESTS
request_test
response_test
)

foreach(test ${TESTS})
Expand Down
2 changes: 1 addition & 1 deletion libs/network/test/http/client/request_test.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,6 @@ TEST(http_client_request, construct_from_uri) {
ASSERT_NO_THROW(client_request("http://cpp-netlib.org/"));
}

TEST(http_client_request, construct_from_invalid_uri) {
TEST(http_client_request, DISABLED_construct_from_invalid_uri) {
ASSERT_THROW(client_request("I am not a valid URI"), std::exception);
}
17 changes: 17 additions & 0 deletions libs/network/test/http/client/response_test.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
// Copyright 2016 Glyn Matthews.
// Distributed under the Boost Software License, Version 1.0.
// (See accompanying file LICENSE_1_0.txt or copy at
// http://www.boost.org/LICENSE_1_0.txt)


#include <gtest/gtest.h>
// #include <boost/network/protocol/http/client/response.hpp>
#include <boost/network/protocol/http/client.hpp>


using client_response = boost::network::http::client::response;


TEST(http_client_response, construct) {
ASSERT_NO_THROW(client_response());
}