Changeset 905 in tests

Timestamp:

07/18/2012 07:37:03 PM (12 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 );
     }
 }