Fix tests + rename test class

wp-media · Jun 28, 2024 · d26235d · d26235d
1 parent ee9131d
commit d26235d
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git 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
@@ -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
@@ -14,7 +14,7 @@
  *
  * @group License
  */
-class GetPricingData extends TestCase {
+class GetUserData extends TestCase {
 	use ApiTrait;
 
 	protected static $api_credentials_config_file = 'license.php';