diff --git a/tests/ConvertKitAPIKeyTest.php b/tests/ConvertKitAPIKeyTest.php index 5c8872d..b7c4536 100644 --- a/tests/ConvertKitAPIKeyTest.php +++ b/tests/ConvertKitAPIKeyTest.php @@ -357,9 +357,8 @@ public function testAddSubscribersToForms() } /** - * Test that add_subscribers_to_forms() returns a ClientException - * when a referrer URL is specified, as this is only supported - * using OAuth. + * Skip this test from ConvertKitAPITest, as testAddSubscribersToForms() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -367,38 +366,12 @@ public function testAddSubscribersToForms() */ public function testAddSubscribersToFormsWithReferrer() { - // Create subscriber. - $emailAddress = $this->generateEmailAddress(); - $subscriber = $this->api->create_subscriber( - email_address: $emailAddress - ); - - // Set subscriber_id to ensure subscriber is unsubscribed after test. - $this->subscriber_ids[] = $subscriber->subscriber->id; - - $this->expectException(ClientException::class); - - // Add subscribers to forms. - $result = $this->api->add_subscribers_to_forms( - forms_subscribers_ids: [ - [ - 'form_id' => (int) $_ENV['CONVERTKIT_API_FORM_ID'], - 'subscriber_id' => $subscriber->subscriber->id, - 'referrer' => 'https://mywebsite.com/bfpromo/', - ], - [ - 'form_id' => (int) $_ENV['CONVERTKIT_API_FORM_ID_2'], - 'subscriber_id' => $subscriber->subscriber->id, - 'referrer' => 'https://mywebsite.com/bfpromo/', - ], - ] - ); + $this->markTestSkipped('testAddSubscribersToForms() above confirms a ClientException is thrown.'); } /** - * Test that add_subscribers_to_forms() returns a ClientException - * when a referrer URL with UTM parameters is specified, as this is only - * supported using OAuth. + * Skip this test from ConvertKitAPITest, as testAddSubscribersToForms() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -406,48 +379,12 @@ public function testAddSubscribersToFormsWithReferrer() */ public function testAddSubscribersToFormsWithReferrerUTMParams() { - // Define referrer. - $referrerUTMParams = [ - 'utm_source' => 'facebook', - 'utm_medium' => 'cpc', - 'utm_campaign' => 'black_friday', - 'utm_term' => 'car_owners', - 'utm_content' => 'get_10_off', - ]; - $referrer = 'https://mywebsite.com/bfpromo/?' . http_build_query($referrerUTMParams); - - // Create subscriber. - $emailAddress = $this->generateEmailAddress(); - $subscriber = $this->api->create_subscriber( - email_address: $emailAddress - ); - - // Set subscriber_id to ensure subscriber is unsubscribed after test. - $this->subscriber_ids[] = $subscriber->subscriber->id; - - $this->expectException(ClientException::class); - - // Add subscribers to forms. - $result = $this->api->add_subscribers_to_forms( - forms_subscribers_ids: [ - [ - 'form_id' => (int) $_ENV['CONVERTKIT_API_FORM_ID'], - 'subscriber_id' => $subscriber->subscriber->id, - 'referrer' => $referrer, - ], - [ - 'form_id' => (int) $_ENV['CONVERTKIT_API_FORM_ID_2'], - 'subscriber_id' => $subscriber->subscriber->id, - 'referrer' => $referrer, - ], - ] - ); + $this->markTestSkipped('testAddSubscribersToForms() above confirms a ClientException is thrown.'); } /** - * Test that add_subscribers_to_forms() returns a ClientException - * when invalid Form IDs are specified, as this is only supported - * using OAuth. + * Skip this test from ConvertKitAPITest, as testAddSubscribersToForms() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -455,35 +392,12 @@ public function testAddSubscribersToFormsWithReferrerUTMParams() */ public function testAddSubscribersToFormsWithInvalidFormIDs() { - // Create subscriber. - $emailAddress = $this->generateEmailAddress(); - $subscriber = $this->api->create_subscriber( - email_address: $emailAddress - ); - - // Set subscriber_id to ensure subscriber is unsubscribed after test. - $this->subscriber_ids[] = $subscriber->subscriber->id; - - $this->expectException(ClientException::class); - - // Add subscribers to forms. - $result = $this->api->add_subscribers_to_forms( - forms_subscribers_ids: [ - [ - 'form_id' => 9999999, - 'subscriber_id' => $subscriber->subscriber->id, - ], - [ - 'form_id' => 9999999, - 'subscriber_id' => $subscriber->subscriber->id, - ], - ] - ); + $this->markTestSkipped('testAddSubscribersToForms() above confirms a ClientException is thrown.'); } /** - * Test that add_subscribers_to_forms() returns a ClientException - * when invalid Subscriber IDs are specified, as this is only supported + * Skip this test from ConvertKitAPITest, as testAddSubscribersToForms() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -491,30 +405,7 @@ public function testAddSubscribersToFormsWithInvalidFormIDs() */ public function testAddSubscribersToFormsWithInvalidSubscriberIDs() { - // Create subscriber. - $emailAddress = $this->generateEmailAddress(); - $subscriber = $this->api->create_subscriber( - email_address: $emailAddress - ); - - // Set subscriber_id to ensure subscriber is unsubscribed after test. - $this->subscriber_ids[] = $subscriber->subscriber->id; - - $this->expectException(ClientException::class); - - // Add subscribers to forms. - $result = $this->api->add_subscribers_to_forms( - forms_subscribers_ids: [ - [ - 'form_id' => (int) $_ENV['CONVERTKIT_API_FORM_ID'], - 'subscriber_id' => 999999, - ], - [ - 'form_id' => (int) $_ENV['CONVERTKIT_API_FORM_ID_2'], - 'subscriber_id' => 999999, - ], - ] - ); + $this->markTestSkipped('testAddSubscribersToForms() above confirms a ClientException is thrown.'); } /** @@ -541,7 +432,8 @@ public function testCreateSubscribers() } /** - * Test that create_subscribers() throws a ClientException when no data is specified. + * Skip this test from ConvertKitAPITest, as testCreateSubscribersWithBlankData() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -549,15 +441,12 @@ public function testCreateSubscribers() */ public function testCreateSubscribersWithBlankData() { - $this->expectException(ClientException::class); - $result = $this->api->create_subscribers([ - [], - ]); + $this->markTestSkipped('testCreateSubscribers() above confirms a ClientException is thrown.'); } /** - * Test that create_subscribers() throws a ClientException when invalid email addresses - * are specified, as this is only supported using OAuth. + * Skip this test from ConvertKitAPITest, as testCreateSubscribersWithBlankData() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -565,16 +454,7 @@ public function testCreateSubscribersWithBlankData() */ public function testCreateSubscribersWithInvalidEmailAddresses() { - $this->expectException(ClientException::class); - $subscribers = [ - [ - 'email_address' => 'not-an-email-address', - ], - [ - 'email_address' => 'not-an-email-address-again', - ], - ]; - $result = $this->api->create_subscribers($subscribers); + $this->markTestSkipped('testCreateSubscribers() above confirms a ClientException is thrown.'); } /** @@ -610,9 +490,8 @@ public function testGetPurchases() } /** - * Test that get_purchases() throws a ClientException - * when the total count is included, as this is only - * supported using OAuth. + * Skip this test from ConvertKitAPITest, as testGetPurchases() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -620,16 +499,12 @@ public function testGetPurchases() */ public function testGetPurchasesWithTotalCount() { - $this->expectException(ClientException::class); - $result = $this->api->get_purchases( - include_total_count: true - ); + $this->markTestSkipped('testGetPurchases() above confirms a ClientException is thrown.'); } /** - * Test that get_purchases() throws a ClientException - * when pagination parameters and per_page limits are specified, - * as this is only supported using OAuth. + * Skip this test from ConvertKitAPITest, as testGetPurchases() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -637,10 +512,7 @@ public function testGetPurchasesWithTotalCount() */ public function testGetPurchasesPagination() { - $this->expectException(ClientException::class); - $result = $this->api->get_purchases( - per_page: 1 - ); + $this->markTestSkipped('testGetPurchases() above confirms a ClientException is thrown.'); } /** @@ -659,9 +531,8 @@ public function testGetPurchase() } /** - * Test that get_purchases() throws a ClientException when an invalid - * purchase ID is specified, as this is only supported - * using OAuth. + * Skip this test from ConvertKitAPITest, as testGetPurchase() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -669,8 +540,7 @@ public function testGetPurchase() */ public function testGetPurchaseWithInvalidID() { - $this->expectException(ClientException::class); - $this->api->get_purchase(12345); + $this->markTestSkipped('testGetPurchase() above confirms a ClientException is thrown.'); } /** @@ -720,8 +590,8 @@ public function testCreatePurchase() } /** - * Test that create_purchase() throws a ClientException when an invalid - * email address is specified, as this is only supported using OAuth. + * Skip this test from ConvertKitAPITest, as testCreatePurchase() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -729,27 +599,12 @@ public function testCreatePurchase() */ public function testCreatePurchaseWithInvalidEmailAddress() { - $this->expectException(ClientException::class); - $this->api->create_purchase( - email_address: 'not-an-email-address', - transaction_id: str_shuffle('wfervdrtgsdewrafvwefds'), - currency: 'usd', - products: [ - [ - 'name' => 'Floppy Disk (512k)', - 'sku' => '7890-ijkl', - 'pid' => 9999, - 'lid' => 7777, - 'quantity' => 2, - 'unit_price' => 5.00, - ], - ], - ); + $this->markTestSkipped('testCreatePurchase() above confirms a ClientException is thrown.'); } /** - * Test that create_purchase() throws a ClientException when a blank - * transaction ID is specified, as this is only supported using OAuth. + * Skip this test from ConvertKitAPITest, as testCreatePurchase() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -757,27 +612,12 @@ public function testCreatePurchaseWithInvalidEmailAddress() */ public function testCreatePurchaseWithBlankTransactionID() { - $this->expectException(ClientException::class); - $this->api->create_purchase( - email_address: $this->generateEmailAddress(), - transaction_id: '', - currency: 'usd', - products: [ - [ - 'name' => 'Floppy Disk (512k)', - 'sku' => '7890-ijkl', - 'pid' => 9999, - 'lid' => 7777, - 'quantity' => 2, - 'unit_price' => 5.00, - ], - ], - ); + $this->markTestSkipped('testCreatePurchase() above confirms a ClientException is thrown.'); } /** - * Test that create_purchase() throws a ClientException when no products - * are specified, as this is only supported using OAuth. + * Skip this test from ConvertKitAPITest, as testCreatePurchase() above + * confirms a ClientException is thrown. * * @since 2.2.0 * @@ -785,12 +625,6 @@ public function testCreatePurchaseWithBlankTransactionID() */ public function testCreatePurchaseWithNoProducts() { - $this->expectException(ClientException::class); - $this->api->create_purchase( - email_address: $this->generateEmailAddress(), - transaction_id: str_shuffle('wfervdrtgsdewrafvwefds'), - currency: 'usd', - products: [], - ); + $this->markTestSkipped('testCreatePurchase() above confirms a ClientException is thrown.'); } }