Changeset 58022 for trunk/tests/phpunit/tests/admin/wpUpgrader.php

Timestamp:

04/18/2024 03:15:50 AM (14 months ago)

Author:

peterwilsoncc

Message:

Upgrade/Install: Validate source & destination values in WP_Ugrader.

Adds a missing string and some additional validation of paths in the upgrader class.

Follow up to [56992].

Props costdev, jipmoors, karlijnbok, swissspidy, afragen, mukesh27.
Fixes #59712.

File:

• trunk/tests/phpunit/tests/admin/wpUpgrader.php (modified) (3 diffs)

trunk/tests/phpunit/tests/admin/wpUpgrader.php

@@ -138 +138 @@
                 'fs_no_themes_dir',
                 'fs_no_folder',
+                'no_package',
                 'download_failed',
                 'installing_package',
@@ -778 +779 @@
     public function data_install_package_invalid_paths() {
         return array(
-            'empty string' => array( 'path' => '' ),
+            'empty string'                   => array( 'path' => '' ),
 
             // Type checks.
-            'empty array'  => array( 'path' => array() ),
-            '(int) 0'      => array( 'path' => 0 ),
-            '(int) -0'     => array( 'path' => -0 ),
-            '(float) 0.0'  => array( 'path' => 0.0 ),
-            '(float) -0.0' => array( 'path' => -0.0 ),
-            '(bool) false' => array( 'path' => false ),
-            'null'         => array( 'path' => null ),
+            'empty array'                    => array( 'path' => array() ),
+            'populated array'                => array( 'path' => array( '/' ) ),
+            '(int) 0'                        => array( 'path' => 0 ),
+            '(int) -0'                       => array( 'path' => -0 ),
+            '(int) -1'                       => array( 'path' => -1 ),
+            '(int) 1'                        => array( 'path' => 1 ),
+            '(float) 0.0'                    => array( 'path' => 0.0 ),
+            '(float) -0.0'                   => array( 'path' => -0.0 ),
+            '(float) 1.0'                    => array( 'path' => 1.0 ),
+            '(float) -1.0'                   => array( 'path' => -1.0 ),
+            '(bool) false'                   => array( 'path' => false ),
+            '(bool) true'                    => array( 'path' => true ),
+            'null'                           => array( 'path' => null ),
+            'empty object'                   => array( 'path' => new stdClass() ),
+            'populated object'               => array( 'path' => (object) array( '/' ) ),
+
+            // Ensures that `trim()` is run triggering an empty array.
+            'a string with spaces'           => array( 'path' => '   ' ),
+            'a string with tabs'             => array( 'path' => "\t\t" ),
+            'a string with new lines'        => array( 'path' => "\n\n" ),
+            'a string with carriage returns' => array( 'path' => "\r\r" ),
+
+            // Ensure that strings with leading/trailing whitespace are invalid.
+            'a path with a leading space'    => array( 'path' => ' /path' ),
+            'a path with a trailing space'   => array( 'path' => '/path ' ),
+            'a path with a leading tab'      => array( 'path' => "\t/path" ),
+            'a path with a trailing tab'     => array( 'path' => "/path\t" ),
         );
     }
@@ -1558 +1579 @@
 
     /**
+     * Tests that `WP_Upgrader::download_package()` returns a WP_Error object
+     * for an empty package.
+     *
+     * @ticket 59712
+     *
+     * @covers WP_Upgrader::download_package
+     */
+    public function test_download_package_should_return_a_wp_error_object_for_an_empty_package() {
+        self::$instance->init();
+
+        $result = self::$instance->download_package( '' );
+
+        $this->assertWPError(
+            $result,
+            'WP_Upgrader::download_package() did not return a WP_Error object'
+        );
+
+        $this->assertSame(
+            'no_package',
+            $result->get_error_code(),
+            'Unexpected WP_Error code'
+        );
+    }
+
+    /**
      * Tests that `WP_Upgrader::download_package()` returns a file with the
      * package name in it.