From d26235d5a1970425c9ecec9400c86cd5123cdea5 Mon Sep 17 00:00:00 2001 From: Gael Robin Date: Fri, 28 Jun 2024 04:02:56 +0200 Subject: [PATCH] Fix tests + rename test class --- inc/Engine/License/API/UserClient.php | 5 +++++ .../inc/Engine/License/API/UserClient/getUserData.php | 2 ++ tests/Unit/inc/Engine/License/API/UserClient/getUserData.php | 2 +- 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/inc/Engine/License/API/UserClient.php b/inc/Engine/License/API/UserClient.php index 62ab12f0f4..f8e31b7b4d 100644 --- a/inc/Engine/License/API/UserClient.php +++ b/inc/Engine/License/API/UserClient.php @@ -88,6 +88,11 @@ private function get_raw_user_data() { ? $this->options->get( 'consumer_email', '' ) : rocket_get_constant( 'WP_ROCKET_EMAIL', '' ); + // Bail out if customer_key & email are empty. + if ( empty( $customer_key ) || empty( $customer_email ) ) { + return false; + } + $response = $this->send_post_request( [ 'body' => 'user_id=' . rawurlencode( $customer_email ) . '&consumer_key=' . sanitize_key( $customer_key ), diff --git a/tests/Integration/inc/Engine/License/API/UserClient/getUserData.php b/tests/Integration/inc/Engine/License/API/UserClient/getUserData.php index 8fc3eea35c..a666576631 100644 --- a/tests/Integration/inc/Engine/License/API/UserClient/getUserData.php +++ b/tests/Integration/inc/Engine/License/API/UserClient/getUserData.php @@ -37,6 +37,8 @@ public function set_up() { public function tear_down() { delete_transient( 'wp_rocket_customer_data' ); + delete_transient( 'wpr_user_information_timeout_active' ); + delete_transient( 'wpr_user_information_timeout' ); remove_filter( 'pre_get_rocket_option_consumer_email', [ $this, 'set_consumer_email' ] ); remove_filter( 'pre_get_rocket_option_consumer_key', [ $this, 'set_consumer_key' ] ); remove_filter( 'pre_http_request', [ $this, 'set_response' ] ); diff --git a/tests/Unit/inc/Engine/License/API/UserClient/getUserData.php b/tests/Unit/inc/Engine/License/API/UserClient/getUserData.php index ab9b1cfe02..ce85508c20 100644 --- a/tests/Unit/inc/Engine/License/API/UserClient/getUserData.php +++ b/tests/Unit/inc/Engine/License/API/UserClient/getUserData.php @@ -14,7 +14,7 @@ * * @group License */ -class GetPricingData extends TestCase { +class GetUserData extends TestCase { use ApiTrait; protected static $api_credentials_config_file = 'license.php';