Changeset 29290 for trunk/tests/phpunit/tests/formatting/SanitizeFileName.php

Timestamp:

07/24/2014 10:08:09 PM (12 years ago)

Author:

wonderboymusic

Message:

In sanitize_file_name(), replace %20 and + with dashes. Remove unnecessary code from _wp_handle_upload().

Adds unit tests.

Props ericmann.
Fixes #16330.

File:

• trunk/tests/phpunit/tests/formatting/SanitizeFileName.php (modified) (1 diff)

trunk/tests/phpunit/tests/formatting/SanitizeFileName.php

@@ -20 +20 @@
         }
 
+        /**
+         * Test that spaces are correctly replaced with dashes.
+         *
+         * @ticket 16330
+         */
+        function test_replace_spaces() {
+                $urls = array(
+                        'unencoded space.png'   => 'unencoded-space.png',
+                        'encoded%20space.jpg'   => 'encoded-space.jpg',
+                        'plus+space.jpg'        => 'plus-space.jpg',
+                        'multi %20 +space.png'   => 'multi-space.png',
+                );
+
+                foreach( $urls as $test => $expected ) {
+                        $this->assertEquals( $expected, sanitize_file_name( $test ) );
+                }
+        }
+
         function test_replaces_any_number_of_hyphens_with_one_hyphen() {
                 $this->assertEquals("a-t-t", sanitize_file_name("a----t----t"));