pin_up 1.4.2 → 1.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b9d8c52115e2b37980524773fe44a0ccbffd91ecb3b5a033ac60f49fd1cc17c4
-  data.tar.gz: d1b6259a2b5f15478731d1068cd2983b4f5e306c8fab2bf7a6bac2d62357bb1c
+  metadata.gz: e164965651e50f81fdc85cb4b21a4f4717428244c72f868cbdeda2b971841312
+  data.tar.gz: 4c5b20e30b65d3eafe442a71a53705ec5163c13571cc872564047da2abe956f9
 SHA512:
-  metadata.gz: d83a6f44dc31bab5c5140a0d3047e4fb8ea4827d0f94ae4a17573406f712f3a40ac514e634e2e2143afef6f9e89f1cfbefd32e0682d8fc0b44ffe7d27c1fa508
-  data.tar.gz: a2eb58981ca0ef056654bed2c86abadcf60c57de40be630b65bbc99e7a72a94a322e0815f5c794a8394dd83206df86c6408d027d194209deb402eeaf018b2db4
+  metadata.gz: 5db2600657d67aa262c3aef1ee3715ad3d6c3c75029ff7ac75ec0058384728a6e046233cbbef7b37cc8dfd08db7d53dabfd9a4c90a1735600fdf2589cc4ed08b
+  data.tar.gz: fb81889c9bfd3de39550f85a9fe1954e73ef5cc17d80bcccdf9ba2fbcfafe80dfd4b73861e0b47bbc1be1a28ad525bf857b9ff5b21794879b600abbe1e34fa48

data/.travis.yml

@@ -2,7 +2,7 @@
 script: bundle exec rspec spec
 language: ruby
 sudo: false
-before_install: gem update bundler
+before_install: gem install bundler:1.17.2
 rvm:
   - 2.3.0
   - 2.4.0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pin_up (1.4.2)
+    pin_up (1.4.3)
       httparty (= 0.17.0)
 
 GEM

data/VERSION

@@ -1 +1 @@
-1.4.2
+1.4.3

data/lib/pin_up/pin_errors.rb

@@ -36,6 +36,8 @@ module Pin
       when 422
         message = handle_bad_response(response)
         raise Pin::InvalidResource.new(response, message)
+      else
+        raise Pin::Unknown.new(response, "Unknown error with status code #{http_status}")
       end
     end
 
@@ -72,6 +74,9 @@ module Pin
     end
   end
 
+  class Unknown < SimpleError
+  end
+
   class Unauthorized < SimpleError
   end
 

data/pin_up.gemspec

@@ -6,7 +6,7 @@
 
 Gem::Specification.new do |s|
   s.name = "pin_up".freeze
-  s.version = "1.4.2"
+  s.version = "1.4.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]

data/spec/bank_accounts_spec.rb

@@ -6,7 +6,7 @@ describe Pin::BankAccounts, :vcr do
   end
 
   it 'creates a bank account token and returns its details' do
-    options = { name: 'Roland Robot', bsb: '123456', number: '987654321' }
+    options = { name: 'Roland Robot', bsb: '012984', number: '987654321' }
     expect(Pin::BankAccounts.create(options)['token']).to match(/^[a-z]{2}[_]/)
   end
 end

data/spec/errors_spec.rb

@@ -339,7 +339,7 @@ describe 'Errors', :vcr, class: Pin::PinError do
     deleted_subscription_token = Pin::Subscription.delete(subscription_1_token)['token']
     expect { Pin::Subscription.delete(deleted_subscription_token) }.to raise_error do |error|
       expect(error).to be_a Pin::InvalidResource
-      expect(error.message).to eq 'Cannot cancel subscription when state is trial_cancelling'
+      expect(error.message).to eq 'Cannot cancel subscription when state is cancelling'
       expect(error.response).to be_a Hash
     end
   end

data/spec/plan_spec.rb

@@ -114,8 +114,12 @@ describe 'Plan', :vcr, class: Pin::Plan do
                                  "created_at"=>match(/\d/),
                                  "customer_permissions"=>["cancel"],
                                  "token"=>match(/(plan)[_]([\w-]{22})/),
-                                 "active_subscriptions"=>0,
-                                 "trial_subscriptions"=>0)
+                                 "subscription_counts" => {
+                                   "trial"=>0,
+                                    "active"=>0,
+                                    "cancelling"=>0,
+                                    "cancelled"=>0
+                                 })
   end
 
   it 'should return a paginated list of all plans' do

data/spec/recipients_spec.rb

@@ -6,7 +6,7 @@ describe Pin::Recipient, :vcr do
       name: 'Roland Robot',
       bank_account: {
         name: 'Roland Robot',
-        bsb: '123456',
+        bsb: '012984',
         number: 987654321
       } }
   }

data/spec/subscription_spec.rb

@@ -173,27 +173,30 @@ describe 'Subscription', :vcr, class: Pin::Subscription do
     expect(Pin::Subscription.reactivate(deactivated['token'])['state']).to eq('active')
   end
 
-  it 'should fetch history for a subscription given a token' do
-    expect(Pin::Subscription.history(subscription_2_token)).to_not eq []
-  end
-
-  it 'should go to a specific page when page parameter is passed' do
-    request = Pin::Subscription.history(subscription_2_token, 20, true)
-    expect(request[:pagination]['current']).to eq 20
-  end
-
-  it 'should list subscriptions on a page given a page' do
-    request = Pin::Subscription.history(subscription_2_token, 1, true)
-    expect(request[:response]).to_not eq []
-  end
-
-  it 'should return pagination if true is passed for pagination' do
-    request = Pin::Subscription.history(subscription_2_token, 1, true)
-    request[:pagination].key?(%W('current previous next per_page pages count))
-  end
-
-  it 'should not list subscriptions for a given page if there are no subscriptions' do
-    request = Pin::Subscription.history(subscription_2_token, 25, true)
-    expect(request[:response]).to eq []
-  end
+  ## History no longer exists as an endpoint
+  ## Leaving this in for reference
+
+  # it 'should fetch history for a subscription given a token' do
+  #   expect(Pin::Subscription.history(subscription_2_token)).to_not eq []
+  # end
+
+  # it 'should go to a specific page when page parameter is passed' do
+  #   request = Pin::Subscription.history(subscription_2_token, 20, true)
+  #   expect(request[:pagination]['current']).to eq 20
+  # end
+
+  # it 'should list subscriptions on a page given a page' do
+  #   request = Pin::Subscription.history(subscription_2_token, 1, true)
+  #   expect(request[:response]).to_not eq []
+  # end
+
+  # it 'should return pagination if true is passed for pagination' do
+  #   request = Pin::Subscription.history(subscription_2_token, 1, true)
+  #   request[:pagination].key?(%W('current previous next per_page pages count))
+  # end
+
+  # it 'should not list subscriptions for a given page if there are no subscriptions' do
+  #   request = Pin::Subscription.history(subscription_2_token, 25, true)
+  #   expect(request[:response]).to eq []
+  # end
 end

data/spec/transfers_spec.rb

@@ -5,7 +5,7 @@ describe Pin::Transfer, :vcr do
     { email: 'test@example.com',
       name: 'Roland Robot',
       bank_account: { name: 'Roland Robot',
-                      bsb: '123456',
+                      bsb: '012984',
                       number: 987654321 } }
   }
 
@@ -55,5 +55,5 @@ describe Pin::Transfer, :vcr do
 
   it 'should list transfers for search on a page given a page' do
     expect(Pin::Transfer.search(1, query: 'Pay day')).to_not eq []
-  end  
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pin_up
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 1.4.3
 platform: ruby
 authors:
 - Daniel Nitsikopoulos