Changeset 55562 for trunk/tests/phpunit/tests/formatting/linksAddTarget.php

Timestamp:

03/19/2023 12:03:30 PM (21 months ago)

Author:

SergeyBiryukov

Message:

Tests: Use the data_ prefix for various data provider methods.

This aims to bring more consistency to the test suite, as the vast majority of data providers already use that prefix.

Includes moving some data providers next to the tests they are used in.

Follow-up to [55464].

See #57841.

File:

• trunk/tests/phpunit/tests/formatting/linksAddTarget.php (modified) (3 diffs)

trunk/tests/phpunit/tests/formatting/linksAddTarget.php

@@ -6 +6 @@
  */
 class Tests_Formatting_LinksAddTarget extends WP_UnitTestCase {
+
+    /**
+     * Validate the normalize_whitespace function
+     *
+     * @dataProvider data_links_add_target
+     */
+    public function test_links_add_target( $content, $target, $tags, $exp_str ) {
+        if ( true === is_null( $target ) ) {
+            $this->assertSame( $exp_str, links_add_target( $content ) );
+        } elseif ( true === is_null( $tags ) ) {
+            $this->assertSame( $exp_str, links_add_target( $content, $target ) );
+        } else {
+            $this->assertSame( $exp_str, links_add_target( $content, $target, $tags ) );
+        }
+    }
+
     /**
      * Test Content DataProvider
@@ -11 +27 @@
      * array ( input_txt, converted_output_txt)
      */
-    public function get_input_output() {
+    public function data_links_add_target() {
         return array(
             array(
@@ -93 +109 @@
         );
     }
-
-    /**
-     * Validate the normalize_whitespace function
-     *
-     * @dataProvider get_input_output
-     */
-    public function test_normalize_whitespace( $content, $target, $tags, $exp_str ) {
-        if ( true === is_null( $target ) ) {
-            $this->assertSame( $exp_str, links_add_target( $content ) );
-        } elseif ( true === is_null( $tags ) ) {
-            $this->assertSame( $exp_str, links_add_target( $content, $target ) );
-        } else {
-            $this->assertSame( $exp_str, links_add_target( $content, $target, $tags ) );
-        }
-    }
 }