Changeset 38100 for trunk/tests/phpunit/tests/general/resourceHints.php

Timestamp:

07/19/2016 02:34:42 AM (8 years ago)

Author:

peterwilsoncc

Message:

Script Loader: Limit resource hinting to enqueued assets.

Externally hosted script and style dependencies trigger dns-prefetch hinting only when enqueued. This removed a bug in which hinting was added on registration.

Renames the function wp_resource_hints_scripts_styles to wp_dependencies_unique_hosts as the function provides the hosts, not the hinting.

Props swissspidy.
Fixes #37385.

File:

• trunk/tests/phpunit/tests/general/resourceHints.php (modified) (1 diff)

trunk/tests/phpunit/tests/general/resourceHints.php

@@ -151 +151 @@
     }
 
+    function test_dns_prefetch_scripts_does_not_included_registered_only() {
+        $expected = "<link rel='preconnect' href='http://s.w.org'>\n";
+        $unexpected = "<link rel='dns-prefetch' href='//wordpress.org'>\n";
+
+        wp_register_script( 'jquery-elsewhere', 'https://wordpress.org/wp-includes/js/jquery/jquery.js' );
+
+        $actual = get_echo( 'wp_resource_hints' );
+
+        wp_deregister_script( 'jquery-elsewhere' );
+
+        $this->assertEquals( $expected, $actual );
+        $this->assertNotContains( $unexpected, $actual );
+    }
 }