diff options
| author | Dan Goodliffe <dan@randomdan.homeip.net> | 2016-06-26 17:26:58 +0100 |
|---|---|---|
| committer | Dan Goodliffe <dan@randomdan.homeip.net> | 2016-06-26 17:39:03 +0100 |
| commit | 2088232b792133cb227e4258233cc5d090a237ff (patch) | |
| tree | f281a58d3ea9782d414308b6b8340888fdb9cd96 | |
| parent | Tidy up (diff) | |
| download | icespider-2088232b792133cb227e4258233cc5d090a237ff.tar.bz2 icespider-2088232b792133cb227e4258233cc5d090a237ff.tar.xz icespider-2088232b792133cb227e4258233cc5d090a237ff.zip | |
Add missing status token to HTTP response
| -rw-r--r-- | icespider/core/ihttpRequest.cpp | 2 | ||||
| -rw-r--r-- | icespider/unittests/testApp.cpp | 14 |
2 files changed, 8 insertions, 8 deletions
diff --git a/icespider/core/ihttpRequest.cpp b/icespider/core/ihttpRequest.cpp index 0dd2aea..a255de0 100644 --- a/icespider/core/ihttpRequest.cpp +++ b/icespider/core/ihttpRequest.cpp @@ -50,7 +50,7 @@ namespace IceSpider { void IHttpRequest::response(short statusCode, const std::string & statusMsg) const { getOutputStream() - << statusCode << " " << statusMsg << "\r\n" + << "Status: " << statusCode << " " << statusMsg << "\r\n" << "\r\n"; } diff --git a/icespider/unittests/testApp.cpp b/icespider/unittests/testApp.cpp index 896b623..71ae7d4 100644 --- a/icespider/unittests/testApp.cpp +++ b/icespider/unittests/testApp.cpp @@ -166,29 +166,29 @@ BOOST_AUTO_TEST_CASE( testCallMethods ) TestRequest requestGetIndex(this, HttpMethod::GET, "/"); process(&requestGetIndex); - BOOST_REQUIRE_EQUAL(requestGetIndex.output.str(), "200 OK\r\n\r\n{\"value\":\"index\"}"); + BOOST_REQUIRE_EQUAL(requestGetIndex.output.str(), "Status: 200 OK\r\n\r\n{\"value\":\"index\"}"); TestRequest requestGetItem(this, HttpMethod::GET, "/view/something/1234"); process(&requestGetItem); - BOOST_REQUIRE_EQUAL(requestGetItem.output.str(), "200 OK\r\n\r\n{\"value\":\"withParams\"}"); + BOOST_REQUIRE_EQUAL(requestGetItem.output.str(), "Status: 200 OK\r\n\r\n{\"value\":\"withParams\"}"); TestRequest requestGetItemGiven(this, HttpMethod::GET, "/item/something/1234"); process(&requestGetItemGiven); - BOOST_REQUIRE_EQUAL(requestGetItemGiven.output.str(), "200 OK\r\n\r\n{\"value\":\"withParams\"}"); + BOOST_REQUIRE_EQUAL(requestGetItemGiven.output.str(), "Status: 200 OK\r\n\r\n{\"value\":\"withParams\"}"); TestRequest requestGetItemDefault(this, HttpMethod::GET, "/item/something"); process(&requestGetItemDefault); - BOOST_REQUIRE_EQUAL(requestGetItemDefault.output.str(), "200 OK\r\n\r\n{\"value\":\"withParams\"}"); + BOOST_REQUIRE_EQUAL(requestGetItemDefault.output.str(), "Status: 200 OK\r\n\r\n{\"value\":\"withParams\"}"); TestRequest requestDeleteItem(this, HttpMethod::DELETE, "/some value"); process(&requestDeleteItem); - BOOST_REQUIRE_EQUAL(requestDeleteItem.output.str(), "200 OK\r\n\r\n"); + BOOST_REQUIRE_EQUAL(requestDeleteItem.output.str(), "Status: 200 OK\r\n\r\n"); TestRequest requestUpdateItem(this, HttpMethod::POST, "/1234"); requestUpdateItem.hdr["Content-Type"] = "application/json"; requestUpdateItem.input << "{\"value\": \"some value\"}"; process(&requestUpdateItem); - BOOST_REQUIRE_EQUAL(requestDeleteItem.output.str(), "200 OK\r\n\r\n"); + BOOST_REQUIRE_EQUAL(requestDeleteItem.output.str(), "Status: 200 OK\r\n\r\n"); adp->deactivate(); } @@ -197,7 +197,7 @@ BOOST_AUTO_TEST_CASE( test404 ) { TestRequest requestGetIndex(this, HttpMethod::GET, "/404"); process(&requestGetIndex); - BOOST_REQUIRE_EQUAL(requestGetIndex.output.str(), "404 Not found\r\n\r\n"); + BOOST_REQUIRE_EQUAL(requestGetIndex.output.str(), "Status: 404 Not found\r\n\r\n"); } BOOST_AUTO_TEST_SUITE_END(); |