Merge remote-tracking branch 'origin/fix-invalid-api-credentials-test' into remove-rocket-loader

Author: n7studios

.github/workflows/tests.yml

@@ -44,8 +44,9 @@ jobs:
 
     # Defines the WordPress and PHP Versions matrix to run tests on.
     strategy:
+      fail-fast: false
       matrix:
-        wp-versions: [ '6.6-RC3' ] #[ 'latest', '6.1.1' ]
+        wp-versions: [ 'latest' ] #[ 'latest', '6.1.1' ]
         php-versions: [ '7.4', '8.0', '8.1', '8.2', '8.3' ] #[ '7.4', '8.0', '8.1', '8.2' ]
 
     # Steps to install, configure and run tests

tests/wpunit/APITest.php

@@ -491,14 +491,28 @@ public function testRefreshTokenWithInvalidToken()
 		$this->assertEquals($result->get_error_code(), 'convertkit_api_error');
 	}
 
+	/**
+	 * Test that supplying no API credentials to the API class returns a WP_Error.
+	 *
+	 * @since   2.0.2
+	 */
+	public function testNoAPICredentials()
+	{
+		$api    = new ConvertKit_API_V4( $_ENV['CONVERTKIT_OAUTH_CLIENT_ID'], $_ENV['CONVERTKIT_OAUTH_REDIRECT_URI'] );
+		$result = $api->get_account();
+		$this->assertInstanceOf(WP_Error::class, $result);
+		$this->assertEquals($result->get_error_code(), $this->errorCode);
+		$this->assertEquals($result->get_error_message(), 'Authentication Failed');
+	}
+
 	/**
 	 * Test that supplying invalid API credentials to the API class returns a WP_Error.
 	 *
 	 * @since   1.0.0
 	 */
 	public function testInvalidAPICredentials()
 	{
-		$api    = new ConvertKit_API_V4( $_ENV['CONVERTKIT_OAUTH_CLIENT_ID'], $_ENV['CONVERTKIT_OAUTH_REDIRECT_URI'] );
+		$api    = new ConvertKit_API_V4( $_ENV['CONVERTKIT_OAUTH_CLIENT_ID'], $_ENV['CONVERTKIT_OAUTH_REDIRECT_URI'], 'fakeAccessToken', 'fakeRefreshToken' );
 		$result = $api->get_account();
 		$this->assertInstanceOf(WP_Error::class, $result);
 		$this->assertEquals($result->get_error_code(), $this->errorCode);