Changeset 25072 for trunk/tests/tests/dependencies/jquery.php

Timestamp:

08/21/2013 03:40:35 PM (11 years ago)

Author:

nacin

Message:

Remove sourceMappingURL from jquery.min.js.

props Jayjdk.
fixes #24994.

File:

• trunk/tests/tests/dependencies/jquery.php (modified) (2 diffs)

trunk/tests/tests/dependencies/jquery.php

@@ -8 +8 @@
 
     function test_location_of_jquery() {
-        if ( SCRIPT_DEBUG ) {
-            $jquery_scripts = array(
-                'jquery-core'    => '/wp-includes/js/jquery/jquery.js',
-                'jquery-migrate' => '/wp-includes/js/jquery/jquery-migrate.min.js'
-            );
-        } else {
-            $jquery_scripts = array(
-                'jquery-core'    => '/wp-includes/js/jquery/jquery.js',
-                'jquery-migrate' => '/wp-includes/js/jquery/jquery-migrate.js'
-            );
-        }
         $scripts = new WP_Scripts;
         wp_default_scripts( $scripts );
+
+        $jquery_scripts = array(
+            'jquery-core' => '/wp-includes/js/jquery/jquery.js',
+        );
+        if ( SCRIPT_DEBUG )
+            $jquery_scripts['jquery-migrate'] = '/wp-includes/js/jquery/jquery-migrate.js';
+        else
+            $jquery_scripts['jquery-migrate'] = '/wp-includes/js/jquery/jquery-migrate.min.js';
+
         $object = $scripts->query( 'jquery', 'registered' );
         $this->assertInstanceOf( '_WP_Dependency', $object );
@@ -60 +58 @@
         set_current_screen( 'front' );
     }
+
+    /**
+     * @ticket 24994
+     */
+    function test_exclusion_of_sourcemaps() {
+        $contents = trim( file_get_contents( ABSPATH . WPINC . '/js/jquery/jquery.js' ) );
+        $this->assertFalse( strpos( $contents, 'sourceMappingURL' ), 'Presence of sourceMappingURL' );
+    }
 }