Changeset 43506 for trunk/tests/phpunit/tests/link/getThePrivacyPolicyLink.php

Timestamp:

07/18/2018 11:49:46 AM (8 years ago)

Author:

SergeyBiryukov

Message:

Privacy: Use the actual Privacy Policy page title in get_the_privacy_policy_link().

Props desrosj, birgire, ianbelanger, Ov3rfly.
Fixes #44192.

File:

• trunk/tests/phpunit/tests/link/getThePrivacyPolicyLink.php (modified) (3 diffs)

trunk/tests/phpunit/tests/link/getThePrivacyPolicyLink.php

@@ -72 +72 @@
         /**
          * The function should return a valid link if a privacy policy page has been
-         * created and set as the `wp_page_for_privacy_policy`.
+         * created and set as the `wp_page_for_privacy_policy`. The post title should
+         * be used as the link text.
          */
         public function test_get_the_privacy_policy_link_should_return_valid_link_when_privacy_page_set() {
@@ -81 +82 @@
                 $this->assertStringStartsWith( '<a', $actual_link );
                 $this->assertContains( self::$privacy_policy_url, $actual_link );
-                $this->assertStringEndsWith( '</a>', $actual_link );
+                $this->assertStringEndsWith( '>' . WP_TESTS_DOMAIN . ' Privacy Policy</a>', $actual_link );
         }
 
@@ -106 +107 @@
 
                 $this->assertSame( '', $actual_link );
+        }
+
+        /**
+         * The function should return an empty string when there is an empty page title
+         * for the privacy policy.
+         *
+         * @ticket 44192
+         */
+        public function test_function_should_return_empty_string_when_privacy_page_title_empty() {
+                $nameless_page_id = $this->factory->post->create(
+                        array(
+                                'post_type'  => 'page',
+                                'post_title' => '',
+                        )
+                );
+
+                update_option( 'wp_page_for_privacy_policy', $nameless_page_id );
+
+                $this->assertSame( '', get_the_privacy_policy_link( self::$before, self::$after ) );
         }