Put back error messages on missing getter static_asserts

wravery · wravery · commit 006e46f3b205 · 2021-12-01T13:18:32.000-08:00
diff --git a/samples/learn/DroidObject.h b/samples/learn/DroidObject.h
@@ -127,7 +127,7 @@ class Droid
 			}
 			else
 			{
-				static_assert(methods::DroidMethod::NoParamsId<T>);
+				static_assert(methods::DroidMethod::NoParamsId<T>, R"msg(Droid::getId is not implemented)msg");
 				return { _pimpl->getId() };
 			}
 		}
@@ -140,7 +140,7 @@ class Droid
 			}
 			else
 			{
-				static_assert(methods::DroidMethod::NoParamsName<T>);
+				static_assert(methods::DroidMethod::NoParamsName<T>, R"msg(Droid::getName is not implemented)msg");
 				return { _pimpl->getName() };
 			}
 		}
@@ -153,7 +153,7 @@ class Droid
 			}
 			else
 			{
-				static_assert(methods::DroidMethod::NoParamsFriends<T>);
+				static_assert(methods::DroidMethod::NoParamsFriends<T>, R"msg(Droid::getFriends is not implemented)msg");
 				return { _pimpl->getFriends() };
 			}
 		}
@@ -166,7 +166,7 @@ class Droid
 			}
 			else
 			{
-				static_assert(methods::DroidMethod::NoParamsAppearsIn<T>);
+				static_assert(methods::DroidMethod::NoParamsAppearsIn<T>, R"msg(Droid::getAppearsIn is not implemented)msg");
 				return { _pimpl->getAppearsIn() };
 			}
 		}
@@ -179,7 +179,7 @@ class Droid
 			}
 			else
 			{
-				static_assert(methods::DroidMethod::NoParamsPrimaryFunction<T>);
+				static_assert(methods::DroidMethod::NoParamsPrimaryFunction<T>, R"msg(Droid::getPrimaryFunction is not implemented)msg");
 				return { _pimpl->getPrimaryFunction() };
 			}
 		}
diff --git a/samples/learn/HumanObject.h b/samples/learn/HumanObject.h
@@ -127,7 +127,7 @@ class Human
 			}
 			else
 			{
-				static_assert(methods::HumanMethod::NoParamsId<T>);
+				static_assert(methods::HumanMethod::NoParamsId<T>, R"msg(Human::getId is not implemented)msg");
 				return { _pimpl->getId() };
 			}
 		}
@@ -140,7 +140,7 @@ class Human
 			}
 			else
 			{
-				static_assert(methods::HumanMethod::NoParamsName<T>);
+				static_assert(methods::HumanMethod::NoParamsName<T>, R"msg(Human::getName is not implemented)msg");
 				return { _pimpl->getName() };
 			}
 		}
@@ -153,7 +153,7 @@ class Human
 			}
 			else
 			{
-				static_assert(methods::HumanMethod::NoParamsFriends<T>);
+				static_assert(methods::HumanMethod::NoParamsFriends<T>, R"msg(Human::getFriends is not implemented)msg");
 				return { _pimpl->getFriends() };
 			}
 		}
@@ -166,7 +166,7 @@ class Human
 			}
 			else
 			{
-				static_assert(methods::HumanMethod::NoParamsAppearsIn<T>);
+				static_assert(methods::HumanMethod::NoParamsAppearsIn<T>, R"msg(Human::getAppearsIn is not implemented)msg");
 				return { _pimpl->getAppearsIn() };
 			}
 		}
@@ -179,7 +179,7 @@ class Human
 			}
 			else
 			{
-				static_assert(methods::HumanMethod::NoParamsHomePlanet<T>);
+				static_assert(methods::HumanMethod::NoParamsHomePlanet<T>, R"msg(Human::getHomePlanet is not implemented)msg");
 				return { _pimpl->getHomePlanet() };
 			}
 		}
diff --git a/samples/learn/MutationObject.h b/samples/learn/MutationObject.h
@@ -74,7 +74,7 @@ class Mutation
 			}
 			else
 			{
-				static_assert(methods::MutationMethod::NoParamsCreateReview<T>);
+				static_assert(methods::MutationMethod::NoParamsCreateReview<T>, R"msg(Mutation::applyCreateReview is not implemented)msg");
 				return { _pimpl->applyCreateReview(std::move(epArg), std::move(reviewArg)) };
 			}
 		}
diff --git a/samples/learn/QueryObject.h b/samples/learn/QueryObject.h
@@ -106,7 +106,7 @@ class Query
 			}
 			else
 			{
-				static_assert(methods::QueryMethod::NoParamsHero<T>);
+				static_assert(methods::QueryMethod::NoParamsHero<T>, R"msg(Query::getHero is not implemented)msg");
 				return { _pimpl->getHero(std::move(episodeArg)) };
 			}
 		}
@@ -119,7 +119,7 @@ class Query
 			}
 			else
 			{
-				static_assert(methods::QueryMethod::NoParamsHuman<T>);
+				static_assert(methods::QueryMethod::NoParamsHuman<T>, R"msg(Query::getHuman is not implemented)msg");
 				return { _pimpl->getHuman(std::move(idArg)) };
 			}
 		}
@@ -132,7 +132,7 @@ class Query
 			}
 			else
 			{
-				static_assert(methods::QueryMethod::NoParamsDroid<T>);
+				static_assert(methods::QueryMethod::NoParamsDroid<T>, R"msg(Query::getDroid is not implemented)msg");
 				return { _pimpl->getDroid(std::move(idArg)) };
 			}
 		}
diff --git a/samples/learn/ReviewObject.h b/samples/learn/ReviewObject.h
@@ -88,7 +88,7 @@ class Review
 			}
 			else
 			{
-				static_assert(methods::ReviewMethod::NoParamsStars<T>);
+				static_assert(methods::ReviewMethod::NoParamsStars<T>, R"msg(Review::getStars is not implemented)msg");
 				return { _pimpl->getStars() };
 			}
 		}
@@ -101,7 +101,7 @@ class Review
 			}
 			else
 			{
-				static_assert(methods::ReviewMethod::NoParamsCommentary<T>);
+				static_assert(methods::ReviewMethod::NoParamsCommentary<T>, R"msg(Review::getCommentary is not implemented)msg");
 				return { _pimpl->getCommentary() };
 			}
 		}
diff --git a/src/SchemaGenerator.cpp b/src/SchemaGenerator.cpp
@@ -806,7 +806,9 @@ void Generator::outputObjectDeclaration(
 			{
 				static_assert(methods::)cpp"
 						   << objectType.cppType << R"cpp(Method::NoParams)cpp" << fieldName
-						   << R"cpp(<T>);)cpp";
+						   << R"cpp(<T>, R"msg()cpp" << objectType.cppType << R"cpp(::)cpp"
+						   << outputField.accessor << fieldName
+						   << R"cpp( is not implemented)msg");)cpp";
 			}
 			else
 			{

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ class Droid`
`127`	`127`	`}`
`128`	`128`	`else`
`129`	`129`	`{`
`130`		`- static_assert(methods::DroidMethod::NoParamsId<T>);`
	`130`	`+ static_assert(methods::DroidMethod::NoParamsId<T>, R"msg(Droid::getId is not implemented)msg");`
`131`	`131`	`return { _pimpl->getId() };`
`132`	`132`	`}`
`133`	`133`	`}`
`@@ -140,7 +140,7 @@ class Droid`
`140`	`140`	`}`
`141`	`141`	`else`
`142`	`142`	`{`
`143`		`- static_assert(methods::DroidMethod::NoParamsName<T>);`
	`143`	`+ static_assert(methods::DroidMethod::NoParamsName<T>, R"msg(Droid::getName is not implemented)msg");`
`144`	`144`	`return { _pimpl->getName() };`
`145`	`145`	`}`
`146`	`146`	`}`
`@@ -153,7 +153,7 @@ class Droid`
`153`	`153`	`}`
`154`	`154`	`else`
`155`	`155`	`{`
`156`		`- static_assert(methods::DroidMethod::NoParamsFriends<T>);`
	`156`	`+ static_assert(methods::DroidMethod::NoParamsFriends<T>, R"msg(Droid::getFriends is not implemented)msg");`
`157`	`157`	`return { _pimpl->getFriends() };`
`158`	`158`	`}`
`159`	`159`	`}`
`@@ -166,7 +166,7 @@ class Droid`
`166`	`166`	`}`
`167`	`167`	`else`
`168`	`168`	`{`
`169`		`- static_assert(methods::DroidMethod::NoParamsAppearsIn<T>);`
	`169`	`+ static_assert(methods::DroidMethod::NoParamsAppearsIn<T>, R"msg(Droid::getAppearsIn is not implemented)msg");`
`170`	`170`	`return { _pimpl->getAppearsIn() };`
`171`	`171`	`}`
`172`	`172`	`}`
`@@ -179,7 +179,7 @@ class Droid`
`179`	`179`	`}`
`180`	`180`	`else`
`181`	`181`	`{`
`182`		`- static_assert(methods::DroidMethod::NoParamsPrimaryFunction<T>);`
	`182`	`+ static_assert(methods::DroidMethod::NoParamsPrimaryFunction<T>, R"msg(Droid::getPrimaryFunction is not implemented)msg");`
`183`	`183`	`return { _pimpl->getPrimaryFunction() };`
`184`	`184`	`}`
`185`	`185`	`}`
Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ class Human`
`127`	`127`	`}`
`128`	`128`	`else`
`129`	`129`	`{`
`130`		`- static_assert(methods::HumanMethod::NoParamsId<T>);`
	`130`	`+ static_assert(methods::HumanMethod::NoParamsId<T>, R"msg(Human::getId is not implemented)msg");`
`131`	`131`	`return { _pimpl->getId() };`
`132`	`132`	`}`
`133`	`133`	`}`
`@@ -140,7 +140,7 @@ class Human`
`140`	`140`	`}`
`141`	`141`	`else`
`142`	`142`	`{`
`143`		`- static_assert(methods::HumanMethod::NoParamsName<T>);`
	`143`	`+ static_assert(methods::HumanMethod::NoParamsName<T>, R"msg(Human::getName is not implemented)msg");`
`144`	`144`	`return { _pimpl->getName() };`
`145`	`145`	`}`
`146`	`146`	`}`
`@@ -153,7 +153,7 @@ class Human`
`153`	`153`	`}`
`154`	`154`	`else`
`155`	`155`	`{`
`156`		`- static_assert(methods::HumanMethod::NoParamsFriends<T>);`
	`156`	`+ static_assert(methods::HumanMethod::NoParamsFriends<T>, R"msg(Human::getFriends is not implemented)msg");`
`157`	`157`	`return { _pimpl->getFriends() };`
`158`	`158`	`}`
`159`	`159`	`}`
`@@ -166,7 +166,7 @@ class Human`
`166`	`166`	`}`
`167`	`167`	`else`
`168`	`168`	`{`
`169`		`- static_assert(methods::HumanMethod::NoParamsAppearsIn<T>);`
	`169`	`+ static_assert(methods::HumanMethod::NoParamsAppearsIn<T>, R"msg(Human::getAppearsIn is not implemented)msg");`
`170`	`170`	`return { _pimpl->getAppearsIn() };`
`171`	`171`	`}`
`172`	`172`	`}`
`@@ -179,7 +179,7 @@ class Human`
`179`	`179`	`}`
`180`	`180`	`else`
`181`	`181`	`{`
`182`		`- static_assert(methods::HumanMethod::NoParamsHomePlanet<T>);`
	`182`	`+ static_assert(methods::HumanMethod::NoParamsHomePlanet<T>, R"msg(Human::getHomePlanet is not implemented)msg");`
`183`	`183`	`return { _pimpl->getHomePlanet() };`
`184`	`184`	`}`
`185`	`185`	`}`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ class Mutation`
`74`	`74`	`}`
`75`	`75`	`else`
`76`	`76`	`{`
`77`		`- static_assert(methods::MutationMethod::NoParamsCreateReview<T>);`
	`77`	`+ static_assert(methods::MutationMethod::NoParamsCreateReview<T>, R"msg(Mutation::applyCreateReview is not implemented)msg");`
`78`	`78`	`return { _pimpl->applyCreateReview(std::move(epArg), std::move(reviewArg)) };`
`79`	`79`	`}`
`80`	`80`	`}`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ class Query`
`106`	`106`	`}`
`107`	`107`	`else`
`108`	`108`	`{`
`109`		`- static_assert(methods::QueryMethod::NoParamsHero<T>);`
	`109`	`+ static_assert(methods::QueryMethod::NoParamsHero<T>, R"msg(Query::getHero is not implemented)msg");`
`110`	`110`	`return { _pimpl->getHero(std::move(episodeArg)) };`
`111`	`111`	`}`
`112`	`112`	`}`
`@@ -119,7 +119,7 @@ class Query`
`119`	`119`	`}`
`120`	`120`	`else`
`121`	`121`	`{`
`122`		`- static_assert(methods::QueryMethod::NoParamsHuman<T>);`
	`122`	`+ static_assert(methods::QueryMethod::NoParamsHuman<T>, R"msg(Query::getHuman is not implemented)msg");`
`123`	`123`	`return { _pimpl->getHuman(std::move(idArg)) };`
`124`	`124`	`}`
`125`	`125`	`}`
`@@ -132,7 +132,7 @@ class Query`
`132`	`132`	`}`
`133`	`133`	`else`
`134`	`134`	`{`
`135`		`- static_assert(methods::QueryMethod::NoParamsDroid<T>);`
	`135`	`+ static_assert(methods::QueryMethod::NoParamsDroid<T>, R"msg(Query::getDroid is not implemented)msg");`
`136`	`136`	`return { _pimpl->getDroid(std::move(idArg)) };`
`137`	`137`	`}`
`138`	`138`	`}`
Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ class Review`
`88`	`88`	`}`
`89`	`89`	`else`
`90`	`90`	`{`
`91`		`- static_assert(methods::ReviewMethod::NoParamsStars<T>);`
	`91`	`+ static_assert(methods::ReviewMethod::NoParamsStars<T>, R"msg(Review::getStars is not implemented)msg");`
`92`	`92`	`return { _pimpl->getStars() };`
`93`	`93`	`}`
`94`	`94`	`}`
`@@ -101,7 +101,7 @@ class Review`
`101`	`101`	`}`
`102`	`102`	`else`
`103`	`103`	`{`
`104`		`- static_assert(methods::ReviewMethod::NoParamsCommentary<T>);`
	`104`	`+ static_assert(methods::ReviewMethod::NoParamsCommentary<T>, R"msg(Review::getCommentary is not implemented)msg");`
`105`	`105`	`return { _pimpl->getCommentary() };`
`106`	`106`	`}`
`107`	`107`	`}`
Original file line number	Diff line number	Diff line change
`@@ -806,7 +806,9 @@ void Generator::outputObjectDeclaration(`
`806`	`806`	`{`
`807`	`807`	`static_assert(methods::)cpp"`
`808`	`808`	`<< objectType.cppType << R"cpp(Method::NoParams)cpp" << fieldName`
`809`		`- << R"cpp(<T>);)cpp";`
	`809`	`+ << R"cpp(<T>, R"msg()cpp" << objectType.cppType << R"cpp(::)cpp"`
	`810`	`+ << outputField.accessor << fieldName`
	`811`	`+ << R"cpp( is not implemented)msg");)cpp";`
`810`	`812`	`}`
`811`	`813`	`else`
`812`	`814`	`{`