Changeset 57674

Timestamp:

02/21/2024 01:05:53 AM (10 months ago)

Author:

costdev

Message:

Tests: Mock API response in Plugin Dependencies tests.

Previously, the tests for WP_Plugin_Dependencies::get_dependency_names() performed an API request to WordPress.org. If an HTTP failure occurred when connecting to WordPress.org, this could trigger test failures.

This mocks the API response to prevent HTTP failures from triggering test failures.

Follow-up to [57545].

Props swissspidy, peterwilsoncc, costdev.
See #59647.

File:

• trunk/tests/phpunit/tests/admin/plugin-dependencies/getDependencyNames.php (modified) (5 diffs)

trunk/tests/phpunit/tests/admin/plugin-dependencies/getDependencyNames.php

@@ -20 +20 @@
 
     /**
+     * Mocks an API response.
+     *
+     * @param string $type The type of response. Accepts 'success' or 'failure'.
+     */
+    private function mock_api_response( $type ) {
+        add_filter(
+            'plugins_api',
+            function ( $bypass, $action, $args ) use ( $type ) {
+                if ( 'plugin_information' === $action && isset( $args->slug ) && str_starts_with( $args->slug, 'dependency' ) ) {
+                    if ( 'success' === $type ) {
+                        return (object) array(
+                            'slug' => $args->slug,
+                            'name' => 'Dependency ' . str_replace( 'dependency', '', $args->slug ),
+                        );
+                    } elseif ( 'failure' === $type ) {
+                        return new WP_Error( 'plugin_not_found', 'Plugin not found.' );
+                    }
+                }
+
+                return $bypass;
+            },
+            10,
+            3
+        );
+    }
+
+    /**
      * Tests that dependency names are retrieved.
      *
@@ -41 +68 @@
         );
 
+        $this->mock_api_response( 'success' );
         self::$instance::initialize();
 
@@ -109 +137 @@
         );
 
+        $this->mock_api_response( 'failure' );
         self::$instance::initialize();
 
@@ -158 +187 @@
         );
 
+        $this->mock_api_response( 'failure' );
         self::$instance::initialize();
 
@@ -212 +242 @@
         set_site_transient( 'wp_plugin_dependencies_plugin_data', array( 'dependency' => false ) );
         set_site_transient( 'wp_plugin_dependencies_plugin_timeout_dependency', true, 12 * HOUR_IN_SECONDS );
+
+        $this->mock_api_response( 'success' );
         self::$instance::get_dependency_names( 'dependent' );