Changeset 905 in tests

Timestamp:

07/18/2012 07:37:03 PM (14 years ago)

Author:

nacin

Message:

Update closures tests. Tests_Actions_Closures, don't use rand_str() for tag names, and remove when done so PHPUnit backing up globals doesn't try to serialize a closure.

Location:

trunk

Files:

• phpunit.xml (modified) (1 diff)
• tests/actions/closures.php (modified) (4 diffs)

trunk/phpunit.xml

@@ -9 +9 @@
         <testsuite>
             <directory suffix=".php">tests</directory>
-            <file phpVersion="5.3.0">tests/test_actions_closures.php</file>
+            <file phpVersion="5.3.0">tests/actions/closures.php</file>
         </testsuite>
     </testsuites>

trunk/tests/actions/closures.php

@@ -6 +6 @@
  * @group hooks
  */
-class WP_Test_Actions_Closures extends WP_UnitTestCase {
+class Tests_Actions_Closures extends WP_UnitTestCase {
 
         /**
@@ -12 +12 @@
          */
         function test_action_closure() {
-                $tag = rand_str();
+                $tag = 'test_action_closure';
                 $closure = function($a, $b) { $GLOBALS[$a] = $b;};
                 add_action($tag, $closure, 10, 2);
@@ -23 +23 @@
                 $this->assertSame($GLOBALS[$context[0]], $context[1]);
 
-                $tag2 = rand_str();
+                $tag2 = 'test_action_closure_2';
                 $closure2 = function() { $GLOBALS['closure_no_args'] = true;};
                 add_action($tag2, $closure2);
@@ -32 +32 @@
 
                 $this->assertTrue($GLOBALS['closure_no_args']);
+
+                remove_action( $tag, $closure );
+                remove_action( $tag2, $closure2 );
         }
 }