Merge pull request #327 from recurly/test-cleanup-and-release

Release 2.8.2

Author: csmb

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Recurly PHP Client Library CHANGELOG
 
+## Version 2.8.2 (July 21th, 2017)
+
+* Fixes a bug creating subscriptions for existing accounts (thanks to @g30rg) [#326](https://github.com/recurly/recurly-client-php/pull/326)
+
 ## Version 2.8.1 (July 14th, 2017)
 
 This bumps us to API v2.7 but does not contain any breaking changes.

Tests/Recurly/Subscription_Test.php

@@ -114,17 +114,8 @@ public function testUpdateShippingAddressXml() {
     );
   }
 
-  public function testCreateSubscripionOnPreExistingAccount() {
-    $account_code = 'abcdef1234567890';
-    $fixture = 'accounts/create-201.xml';
-    $this->client->addResponse('POST', '/accounts', $fixture);
-    $this->client->addResponse('GET', "/accounts/$account_code", $fixture);
-
-    $account = new Recurly_Account(null, $this->client);
-    $account->account_code = 'abcdef1234567890';
-
-    $account->create();
-
+  public function testCreateSubscripionWithExistingAccountXml() {
+    $this->client->addResponse('GET', "/accounts/abcdef1234567890", 'accounts/show-200.xml');
     $account = Recurly_Account::get('abcdef1234567890', $this->client);
 
     $subscription = new Recurly_Subscription();

lib/recurly/client.php

@@ -44,7 +44,7 @@ class Recurly_Client
    */
   private $_acceptLanguage = 'en-US';
 
-  const API_CLIENT_VERSION = '2.8.1';
+  const API_CLIENT_VERSION = '2.8.2';
   const DEFAULT_ENCODING = 'UTF-8';
 
   const GET = 'GET';