Changeset 3265 for trunk/modules

Timestamp:

08/05/2008 02:29:29 PM (4 months ago)

Author:

Geert

Message:

Now using a class constant (instead of a property) to disable a unit test case. A constant fits more in the unchangeable nature of the setting. Also cleaner on the Reflection part.

Location:

trunk/modules/unit_test

Files:

• libraries/Unit_Test.php (modified) (1 diff)
• tests/Example_Test.php (modified) (1 diff)

trunk/modules/unit_test/libraries/Unit_Test.php

@@ -88 +88 @@
 
                                 // Skip disabled Tests
-                                if ($reflector->hasProperty('DISABLED'))
-                                {
-                                        $property = new ReflectionProperty($class, 'DISABLED');
-                                        if ($property->getValue(new $class) === TRUE)
-                                                continue;
-                                }
+                                if ($reflector->getConstant('DISABLED') === TRUE)
+                                        continue;
 
                                 // Initialize setup and teardown method triggers

trunk/modules/unit_test/tests/Example_Test.php

@@ -12 +12 @@
 class Example_Test extends Unit_Test_Case {
 
-        // public $DISABLED = TRUE; // Disables this Test class
+        // Disable this Test class?
+        const DISABLED = FALSE;
 
         public $setup_has_run = FALSE;