Add explicit Value(const char*) constructor

Author: wravery

CMakeLists.txt

@@ -138,6 +138,9 @@ if(BUILD_TESTS OR UPDATE_SAMPLES)
     add_test(NAME PegtlCase
       COMMAND tests --gtest_filter=PegtlCase.*
       WORKING_DIRECTORY $<TARGET_FILE_DIR:tests>)
+    add_test(NAME ResponseCase
+      COMMAND tests --gtest_filter=ResponseCase.*
+      WORKING_DIRECTORY $<TARGET_FILE_DIR:tests>)
   endif()
 
   if(UPDATE_SAMPLES)

GraphQLResponse.cpp

@@ -44,6 +44,12 @@ Value::~Value()
 	// omitted, declare it explicitly and define it in graphqlservice.
 }
 
+Value::Value(const char* value)
+	: _type(Type::String)
+	, _string(new StringType(value))
+{
+}
+
 Value::Value(StringType&& value)
 	: _type(Type::String)
 	, _string(new StringType(std::move(value)))

include/graphqlservice/GraphQLResponse.h

@@ -44,6 +44,7 @@ struct Value
 	Value(Type type = Type::Null);
 	~Value();
 
+	explicit Value(const char* value);
 	explicit Value(StringType&& value);
 	explicit Value(BooleanType value);
 	explicit Value(IntType value);

tests.cpp

@@ -1624,3 +1624,12 @@ TEST(PegtlCase, AnalyzeGrammar)
 {
 	ASSERT_EQ(0, analyze<document>(true)) << "there shuldn't be any infinite loops in the PEG version of the grammar";
 }
+
+TEST(ResponseCase, ValueConstructorFromStringLiteral)
+{
+	auto expected = "Test String";
+	auto actual = response::Value(expected);
+
+	ASSERT_TRUE(response::Type::String == actual.type());
+	ASSERT_EQ(expected, actual.release<response::StringType>());
+}