Changeset 10400 for code/branches/core7/test/core/class/SubclassIdentifierTest.cc

Timestamp:

Apr 26, 2015, 3:22:27 PM (9 years ago)

Author:

landauf

Message:

fixed tests. however there are some open issues:

• the class-hierarchy must be built exactly 1 times in core_test. this is currently done in CommandTest.cc because that's the first test to run in core_test which actually needs the class hierarchy. the order of tests is not guaranteed though, so this should be solved more generic
• during creation of class hierarchy, config values are used. this fails in the tests, so it had to be disabled with a static flag in Identifier. this should be solved in a cleaner way.
• because the class hierarchy is now statically generated for all tests in core_test in CommandTest.cc, there is no way to test identifiers in an uninitialized state. because of this, three tests had to be disabled (*_NoFixture tests)

⇒ make the creation of the class hierarchy more modular and fix these issues

File:

• code/branches/core7/test/core/class/SubclassIdentifierTest.cc (modified) (3 diffs)

code/branches/core7/test/core/class/SubclassIdentifierTest.cc

@@ -21 +21 @@
         };
 
+        RegisterClass(TestClass);
+        RegisterClass(TestSubclass);
+
         // Fixture
         class SubclassIdentifierTest : public ::testing::Test
@@ -27 +30 @@
                 virtual void SetUp()
                 {
-                    registerClass("OrxonoxClass", new ClassFactoryNoArgs<OrxonoxClass>());
-                    registerClass("TestClass", new ClassFactoryWithContext<TestClass>());
-                    registerClass("TestSubclass", new ClassFactoryWithContext<TestSubclass>());
-
-                    IdentifierManager::getInstance().createClassHierarchy();
-
                     Context::setRootContext(new Context(NULL));
                 }
@@ -39 +36 @@
                 {
                     Context::setRootContext(NULL);
-
-                    IdentifierManager::getInstance().destroyAllIdentifiers();
                 }
         };