Don't require aggregate type for CountMembers<> and add a test for it

Signed-off-by: Christian Parpart <[email protected]>

Author: christianparpart

include/reflection-cpp/reflection.hpp

@@ -145,7 +145,6 @@ namespace detail
 } // namespace detail
 
 template <class T, class... Args>
-    requires(std::is_aggregate_v<std::remove_cvref_t<T>>)
 inline constexpr auto CountMembers = [] {
     using V = std::remove_cvref_t<T>;
     if constexpr (requires { V { Args {}..., detail::any_t {} }; })

test-reflection-cpp.cpp

@@ -65,6 +65,7 @@ name="John Doe" email="[email protected]" age=43
 TEST_CASE("nested", "[reflection]")
 {
     auto ts = TestStruct { 1, 2.0f, 3.0, "hello", { "John Doe", "[email protected]", 42 } };
+    CHECK(Reflection::CountMembers<std::remove_reference_t<decltype(ts)>> == 5);
     auto const result = Reflection::Inspect(ts);
     CHECK(result == R"(a=1 b=2 c=3 d="hello" e={name="John Doe" email="[email protected]" age=42})");
 }