Changeset 53805 for trunk/tests/phpunit/tests/actions.php

Timestamp:

07/31/2022 04:25:27 PM (4 years ago)

Author:

SergeyBiryukov

Message:

Tests: Update the terminology used for action or filter names in MockAction class.

This replaces the "tag" wording with "hook name" where appropriate, to match the core function signatures.

Includes:

• Introducing a ::get_hook_names() method instead of ::get_tags(), keeping the latter as an alias.
• Adding a hook_name key to the ::$events property, keeping tag for backward compatibility for now.
• Adding missing @since tags for class methods.

Follow-up to [24/tests], [62/tests], [70/tests], [50807], [53804].

See #55652.

File:

• trunk/tests/phpunit/tests/actions.php (modified) (7 diffs)

trunk/tests/phpunit/tests/actions.php

@@ -21 +21 @@
         $this->assertSame( 1, $a->get_call_count() );
         // Only our hook was called.
-        $this->assertSame( array( $hook_name ), $a->get_tags() );
+        $this->assertSame( array( $hook_name ), $a->get_hook_names() );
 
         $argsvar = $a->get_args();
@@ -40 +40 @@
         // Make sure our hook was called correctly.
         $this->assertSame( 1, $a->get_call_count() );
-        $this->assertSame( array( $hook_name ), $a->get_tags() );
+        $this->assertSame( array( $hook_name ), $a->get_hook_names() );
 
         // Now remove the action, do it again, and make sure it's not called this time.
@@ -46 +46 @@
         do_action( $hook_name );
         $this->assertSame( 1, $a->get_call_count() );
-        $this->assertSame( array( $hook_name ), $a->get_tags() );
+        $this->assertSame( array( $hook_name ), $a->get_hook_names() );
 
     }
@@ -216 +216 @@
             // 'action2' is called first because it has priority 9.
             array(
-                'action' => 'action2',
-                'tag'    => $hook_name,
-                'args'   => array( '' ),
+                'action'    => 'action2',
+                'hook_name' => $hook_name,
+                'tag'       => $hook_name, // Back compat.
+                'args'      => array( '' ),
             ),
             // 'action' is called second.
             array(
-                'action' => 'action',
-                'tag'    => $hook_name,
-                'args'   => array( '' ),
+                'action'    => 'action',
+                'hook_name' => $hook_name,
+                'tag'       => $hook_name, // Back compat.
+                'args'      => array( '' ),
             ),
         );
@@ -275 +277 @@
         $this->assertSame( 4, $a->get_call_count() );
         // Only our hook was called.
-        $this->assertSame( array( $hook_name1, $hook_name2, $hook_name1, $hook_name1 ), $a->get_tags() );
+        $this->assertSame( array( $hook_name1, $hook_name2, $hook_name1, $hook_name1 ), $a->get_hook_names() );
 
         remove_action( 'all', array( &$a, 'action' ) );
@@ -295 +297 @@
         // Make sure our hook was called correctly.
         $this->assertSame( 1, $a->get_call_count() );
-        $this->assertSame( array( $hook_name ), $a->get_tags() );
+        $this->assertSame( array( $hook_name ), $a->get_hook_names() );
 
         // Now remove the action, do it again, and make sure it's not called this time.
@@ -302 +304 @@
         do_action( $hook_name );
         $this->assertSame( 1, $a->get_call_count() );
-        $this->assertSame( array( $hook_name ), $a->get_tags() );
+        $this->assertSame( array( $hook_name ), $a->get_hook_names() );
     }