test: more tests around component_traits

TODO

@@ -36,4 +36,3 @@ TODO:
 * any cdynamic to support const ownership construction
 * allow passing arguments to meta setter/getter (we can fallback on meta invoke for everything probably)
 * delegate/sigh: forward connect/disconnect from & to *
-* entity based component_traits, add tests for different entities

test/entt/entity/component.cpp

@@ -3,6 +3,7 @@
 #include <entt/entity/component.hpp>
 #include "../../common/boxed_type.h"
 #include "../../common/empty.h"
+#include "../../common/entity.h"
 #include "../../common/non_movable.h"
 
 struct self_contained {
@@ -13,50 +14,93 @@ struct self_contained {
 struct traits_based {};
 
 template<>
-struct entt::component_traits<traits_based> {
-    using type = traits_based;
-    static constexpr auto in_place_delete = false;
+struct entt::component_traits<traits_based /*, entt::entity */> {
+    using entity_type = entt::entity;
+    using element_type = traits_based;
+
+    static constexpr auto in_place_delete = true;
     static constexpr auto page_size = 8u;
 };
 
-TEST(Component, VoidType) {
-    using traits_type = entt::component_traits<void>;
+template<>
+struct entt::component_traits<traits_based, test::entity> {
+    using entity_type = test::entity;
+    using element_type = traits_based;
+
+    static constexpr auto in_place_delete = false;
+    static constexpr auto page_size = 16u;
+};
+
+template<typename Entity>
+struct entt::component_traits<traits_based, Entity> {
+    using entity_type = Entity;
+    using element_type = traits_based;
+
+    static constexpr auto in_place_delete = true;
+    static constexpr auto page_size = 32u;
+};
+
+template<typename Type>
+struct Component: testing::Test {
+    using entity_type = Type;
+};
+
+using EntityTypes = ::testing::Types<entt::entity, test::entity, test::other_entity>;
+
+TYPED_TEST_SUITE(Component, EntityTypes, );
+
+TYPED_TEST(Component, VoidType) {
+    using entity_type = typename TestFixture::entity_type;
+    using traits_type = entt::component_traits<void, entity_type>;
 
     ASSERT_FALSE(traits_type::in_place_delete);
     ASSERT_EQ(traits_type::page_size, 0u);
 }
 
-TEST(Component, Empty) {
-    using traits_type = entt::component_traits<test::empty>;
+TYPED_TEST(Component, Empty) {
+    using entity_type = typename TestFixture::entity_type;
+    using traits_type = entt::component_traits<test::empty, entity_type>;
 
     ASSERT_FALSE(traits_type::in_place_delete);
     ASSERT_EQ(traits_type::page_size, 0u);
 }
 
-TEST(Component, NonEmpty) {
-    using traits_type = entt::component_traits<test::boxed_int>;
+TYPED_TEST(Component, NonEmpty) {
+    using entity_type = typename TestFixture::entity_type;
+    using traits_type = entt::component_traits<test::boxed_int, entity_type>;
 
     ASSERT_FALSE(traits_type::in_place_delete);
     ASSERT_EQ(traits_type::page_size, ENTT_PACKED_PAGE);
 }
 
-TEST(Component, NonMovable) {
-    using traits_type = entt::component_traits<test::non_movable>;
+TYPED_TEST(Component, NonMovable) {
+    using entity_type = typename TestFixture::entity_type;
+    using traits_type = entt::component_traits<test::non_movable, entity_type>;
 
     ASSERT_TRUE(traits_type::in_place_delete);
     ASSERT_EQ(traits_type::page_size, ENTT_PACKED_PAGE);
 }
 
-TEST(Component, SelfContained) {
-    using traits_type = entt::component_traits<self_contained>;
+TYPED_TEST(Component, SelfContained) {
+    using entity_type = typename TestFixture::entity_type;
+    using traits_type = entt::component_traits<self_contained, entity_type>;
 
     ASSERT_TRUE(traits_type::in_place_delete);
     ASSERT_EQ(traits_type::page_size, 4u);
 }
 
-TEST(Component, TraitsBased) {
-    using traits_type = entt::component_traits<traits_based>;
+TYPED_TEST(Component, TraitsBased) {
+    using entity_type = typename TestFixture::entity_type;
+    using traits_type = entt::component_traits<traits_based, entity_type>;
 
-    ASSERT_TRUE(!traits_type::in_place_delete);
-    ASSERT_EQ(traits_type::page_size, 8u);
+    if constexpr(std::is_same_v<entity_type, entt::entity>) {
+        ASSERT_TRUE(traits_type::in_place_delete);
+        ASSERT_EQ(traits_type::page_size, 8u);
+    } else if constexpr(std::is_same_v<entity_type, test::entity>) {
+        ASSERT_FALSE(traits_type::in_place_delete);
+        ASSERT_EQ(traits_type::page_size, 16u);
+    } else {
+        ASSERT_TRUE(traits_type::in_place_delete);
+        ASSERT_EQ(traits_type::page_size, 32u);
+    }
 }