Support for building with code coverage (also used internally for CDash reporting)

Removed some compilation warnings (unused paramters)
Added Service test

Author: Rodrigo Fernandes

CMakeLists.txt

@@ -90,4 +90,15 @@ add_subdirectory(examples)
 
 enable_testing()
 include(CTest)
+
+option(CodeCoverage "Build support for code coverage (requires -DCMAKE_BUILD_TYPE=Debug)" OFF)
+
+if (CodeCoverage)
+	message (STATUS "Building with code coverage support")
+	set (CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} -g -O0 -Wall -W -Wshadow -Wunused-variable -Wunused-parameter -Wunused-function -Wunused -Wno-system-headers -Wno-deprecated -Woverloaded-virtual -Wwrite-strings -fprofile-arcs -ftest-coverage")
+	set (CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG} -g -O0 -Wall -W -fprofile-arcs -ftest-coverage")
+	set (CMAKE_SHARED_LINKER_FLAGS_DEBUG "${CMAKE_SHARED_LINKER_FLAGS_DEBUG} -fprofile-arcs -ftest-coverage")
+	set (CMAKE_EXE_LINKER_FLAGS_DEBUG "${CMAKE_EXE_LINKER_FLAGS_DEBUG} -fprofile-arcs -ftest-coverage")
+endif()
+
 add_subdirectory(tests)

examples/include/Instantiation.hh

@@ -0,0 +1,35 @@
+#ifndef _CALCULATOR_HH_
+#define _CALCULATOR_HH_
+#include <answer/Operation.hh>
+#include <string>
+#include <set>
+
+namespace WebServices{
+
+///Define request and response types
+struct Data{
+	double result;
+
+	template<class Archive>
+	void serialize(Archive &ar, const unsigned int /*version*/){
+		ar & BOOST_SERIALIZATION_NVP(result);
+	}
+};
+
+///Complex type service testing
+class Instantiation: public answer::instantiation::Singleton{
+	public:
+		Data calculator(const Data &request);
+};
+
+///Complex type service testing
+class Instantiation{
+	public:
+		Data calculator(const Data &request);
+};
+
+ANSWER_REGISTER_OPERATION(MyService::calculator);
+
+}
+
+#endif // _CALCULATOR_HH_

examples/src/Instantiation.cpp

@@ -0,0 +1,32 @@
+#include ".hh"
+
+using namespace std;
+using namespace answer;
+
+namespace WebServices{
+	
+CalculatorResponse MyService::calculator(const CalculatorRequest &request){
+	CalculatorResponse response;
+	switch(request.operation){
+		case 'x':
+		case '*':
+			response.result = request.operand1 * request.operand2;
+			break;
+		case '-':
+			response.result = request.operand1 - request.operand2;
+			break;
+		case 'd':
+		case '/':
+			response.result = request.operand1 / request.operand2;
+			break;
+		case '+':
+			response.result = request.operand1 + request.operand2;
+			break;
+		default:
+			throw WebMethodInvalidInput("Invalid operation requested");
+	}
+	
+	return response;
+}
+
+}

include/answer/Codec.hh

@@ -34,7 +34,7 @@ namespace answer{
 		}
 
 		template<typename T>
-		bool ResponseWrapper( std::ostream& out, const std::string& response, const std::string& mimeType, const answer::WebMethodException*)
+		bool ResponseWrapper( std::ostream& out, const std::string& response, const std::string& /*mimeType*/, const answer::WebMethodException*)
 		{
       out << response;
 			return true;

include/answer/Operation.hh

@@ -27,7 +27,7 @@ public:
 	Operation(const std::string& name):_name(name){}
     virtual ~Operation() {};
     //The invocation wrapper
-    virtual Response invoke(const std::string&, const std::string & ="")=0;
+    virtual Response invoke(const std::string& params, const std::string & prefix="")=0;
 };
 
 template <typename RequestT>

tests/CMakeLists.txt

@@ -0,0 +1,3 @@
+add_subdirectory(serialization)
+add_subdirectory(service)
+add_subdirectory(module)

tests/registration/RegistrationTest.cpp

@@ -24,8 +24,6 @@ struct testInput{
 	}
 };
 
-
-
 BOOST_AUTO_TEST_CASE( serialization )
 {
   testInput testIntSet1;

tests/serialization/SerializationTest.cpp

@@ -1,17 +1,17 @@
 find_package(Boost COMPONENTS unit_test_framework REQUIRED QUIET)
 
-add_executable(registrationTest
-	RegistrationTest.cpp
+add_executable(serviceTest
+	ServiceTest.cpp
 )
 
 include_directories(
 	${Boost_INCLUDE_DIRS}
 )
 
-target_link_libraries(registrationTest
+target_link_libraries(serviceTest
 	${Boost_LIBRARIES}
 	answer
 )
 
-add_test(registration registrationTest)
+add_test(service serviceTest)
 

tests/registration/CMakeLists.txt tests/service/CMakeLists.txttests/registration/CMakeLists.txt renamed to tests/service/CMakeLists.txt

@@ -0,0 +1,108 @@
+#define BOOST_TEST_MODULE RegistrationTest
+#include <boost/test/included/unit_test.hpp>
+
+//Normally this is defined in the build file, it's usually the project name
+#define ANSWER_SERVICE_NAME "RegistrationTest"
+
+#include "answer/Operation.hh"
+#include <boost/algorithm/string.hpp>
+
+using namespace std;
+
+class Operation{
+public:
+	int test(string X){
+		return X.size();
+	}
+};
+
+ANSWER_REGISTER_OPERATION(Operation::test);
+
+
+class TestContext: public
+		answer::Context,
+		answer::TransportInfo,
+		answer::OperationInfo{
+	
+	string _service;
+	string _operation;
+	string _accept;
+	list<string> _accepts;
+	
+public:
+	TestContext(const string &serviceName, const string &operationName, const string& acceptsType):
+		_service(serviceName), _operation(operationName), _accepts({acceptsType}){
+	}
+	
+	//Context
+	virtual answer::CookieJar& cookieJar(){
+		throw std::runtime_error("CookieJar not implemented");
+	}
+	virtual answer::Environment& environment(){
+		throw std::runtime_error("Environment not implemented");
+	}
+	virtual answer::OperationInfo& operationInfo(){
+		return *this;
+	}
+	virtual answer::ProviderStore& providerStore(){
+		throw std::runtime_error("ProviderStore not implemented");
+	}
+	virtual answer::TransportInfo& transportInfo(){
+		return *this;
+	}
+	
+	//TransportInfo
+	virtual const list<string>& accepts() const{
+		return _accepts;
+	}
+	virtual void addHeader ( const string& /*key*/, const string& /*value*/ = "", bool /*replace*/ = true ){
+		throw std::runtime_error("addHeader not implemented");
+	}
+	virtual const string& redirect() const{
+		throw std::runtime_error("redirect not implemented");
+	}
+	virtual const string& redirect ( const string& /*uri*/ ){
+		throw std::runtime_error("redirect (set) not implemented");
+	}
+	virtual bool redirectSet() const{
+		return false;
+	}
+
+	//OperationInfo
+	virtual const string& operation() const{
+		return _operation;
+	}
+	virtual const string& service() const{
+		return _service;
+	}
+	virtual const string& url() const{
+		throw std::runtime_error("url not implemented");
+	}
+		
+};
+
+
+BOOST_AUTO_TEST_CASE( registration )
+{
+  list<string> operations = answer::OperationStore::Instance().operationList();
+	BOOST_CHECK(operations.front() == string("RegistrationTest/test"));
+}
+
+BOOST_AUTO_TEST_CASE( request_response ){
+	TestContext context("RegistrationTest", "test", "application/xml");
+	answer::Operation &operation = answer::OperationStore::Instance().operation("RegistrationTest", "test");
+	answer::Response response = operation.invoke("<test>foobar</test>");
+	string body = response.body();
+	boost::trim(body);
+	BOOST_CHECK(body == "<test>6</test>");
+}
+
+BOOST_AUTO_TEST_CASE( request_response_codec ){
+	TestContext context("RegistrationTest", "test", "application/json");
+	answer::Operation &operation = answer::OperationStore::Instance().operation("RegistrationTest", "test");
+	answer::Response response = operation.invoke("<test>foobar</test>");
+	string body = response.body();
+	boost::trim(body);
+	BOOST_CHECK(body == "\"6\"");
+}
+