braque 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a98f34cb71d248a703cb08b321109252638cd0c5
-  data.tar.gz: c08a5b5527ffae9cb00e0722919397a78a937036
+  metadata.gz: 7e576d0ca74ea35d912a52c67e9971a7740a0dfa
+  data.tar.gz: 5d52452de3a6ed8611a2b127ed002d5ebee8c3e9
 SHA512:
-  metadata.gz: 4764cb4013926ed115f96ec4b076c3a2bcf461e59106e26fbfe1e5f5cfd3c0c21627ab12805ef2b32fe3dad8d43be286491219085919d7c09839eb349ddaa6fd
-  data.tar.gz: 2a96a53a0da8fe227c8d35b20273ac7b9f9f6580c24d91e8aa7627444c307453075f8812f03f3aa25b018f44dfba6668400194a81eb16c0a1410f459bed27858
+  metadata.gz: 60fbbb04472b235c26ceb02490013f7b68b046478e461889a5ccaac88728aa5b51671c022dc4fcb3f74827f8f7cb85f3b37c5f0af4239f5ce53ad8bf5c14abcd
+  data.tar.gz: 0076fe9afecb8bded09adf25806d9afc920be19de40b2de2bcad5d33e3e8e13765676f793b01f96c681751e6ee1dacbb5fe96f0310d9cd0b2fabf7afbddb14f3

data/lib/braque/collection.rb

@@ -13,7 +13,10 @@ module Braque
         response.each do |item|
           retrieved_items << klass.new(item)
         end
-        @retrieved_items, @next_link, @previous_link, @total_count = retrieved_items, next_link, previous_link, total_count
+        @retrieved_items = retrieved_items
+        @next_link = next_link
+        @previous_link = previous_link
+        @total_count = total_count
         super retrieved_items
       end
     end

data/lib/braque/version.rb

@@ -1,3 +1,3 @@
 module Braque
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

data/spec/braque/model_spec.rb

@@ -70,6 +70,21 @@ RSpec.describe Braque::Model, type: :model do
           expect(breeze.title).to eq collection_response['_embedded']['breezes'].first['title']
         end
       end
+      context 'with an errored response' do
+        before(:each) do
+          root_request
+          @collection_request = WebMock.stub_request(:get, "#{Breeze.api_root}/breezes")
+                                .with(headers: { 'Http-Authorization' => Breeze.api_token })
+                                .to_return(status: 500)
+        end
+        it 'returns a Faraday::ClientError error' do
+          expect do
+            Breeze.list
+          end.to raise_error(Faraday::ClientError, 'the server responded with status 500')
+          expect(root_request).to have_been_requested
+          expect(@collection_request).to have_been_requested
+        end
+      end
       context 'with array params' do
         before(:each) do
           root_request
@@ -108,6 +123,21 @@ RSpec.describe Braque::Model, type: :model do
           expect(@breeze.title).to eq resource_response['title']
         end
       end
+      context 'with an errored response' do
+        before(:each) do
+          root_request
+          @resource_request = WebMock.stub_request(:get, "#{Breeze.api_root}/breezes/1")
+                              .with(headers: { 'Http-Authorization' => Breeze.api_token })
+                              .to_return(status: 500)
+        end
+        it 'returns a Faraday::ClientError error' do
+          expect do
+            @breeze = Breeze.find id: 1
+          end.to raise_error(Faraday::ClientError, 'the server responded with status 500')
+          expect(root_request).to have_been_requested
+          expect(@resource_request).to have_been_requested
+        end
+      end
       context 'when overriding resource_find_options' do
         before(:each) do
           class Breeze
@@ -140,27 +170,44 @@ RSpec.describe Braque::Model, type: :model do
     end
 
     context '.create' do
+      context 'with a successful response' do
+        before(:each) do
+          root_request
+          @create_request = WebMock.stub_request(:post, "#{Breeze.api_root}/breezes")
+                            .with(headers: { 'Http-Authorization' => Breeze.api_token })
+                            .to_return(status: 201, body: resource_response)
+          @params = { title: 'What a nice breeze.' }
+          @breeze = Breeze.create(@params)
+        end
+        it 'performs the API root request' do
+          expect(root_request).to have_been_requested
+        end
+        it 'performs the create request' do
+          expect(@create_request).to have_been_requested
+        end
+        it 'returns an item with the correct class' do
+          expect(@breeze).to be_a_kind_of Breeze
+        end
+        it 'returns an item with the correct attributes' do
+          expect(@breeze.id).to eq resource_response['id']
+          expect(@breeze.title).to eq resource_response['title']
+        end
+      end
+    end
+    context 'with an errored response' do
       before(:each) do
         root_request
         @create_request = WebMock.stub_request(:post, "#{Breeze.api_root}/breezes")
                           .with(headers: { 'Http-Authorization' => Breeze.api_token })
-                          .to_return(status: 201, body: resource_response)
-        @params = { title: 'What a nice breeze.' }
-        @breeze = Breeze.create(@params)
+                          .to_return(status: 500)
       end
-      it 'performs the API root request' do
+      it 'returns a Faraday::ClientError error' do
+        expect do
+          Breeze.create {}
+        end.to raise_error(Faraday::ClientError, 'the server responded with status 500')
         expect(root_request).to have_been_requested
-      end
-      it 'performs the create request' do
         expect(@create_request).to have_been_requested
       end
-      it 'returns an item with the correct class' do
-        expect(@breeze).to be_a_kind_of Breeze
-      end
-      it 'returns an item with the correct attributes' do
-        expect(@breeze.id).to eq resource_response['id']
-        expect(@breeze.title).to eq resource_response['title']
-      end
     end
   end
   context 'instance methods' do
@@ -193,6 +240,22 @@ RSpec.describe Braque::Model, type: :model do
         end
       end
 
+      context 'with an errored response' do
+        before(:each) do
+          root_request
+          @save_request = WebMock.stub_request(:patch, "#{Breeze.api_root}/breezes/1")
+                          .with(headers: { 'Http-Authorization' => Breeze.api_token })
+                          .to_return(status: 500)
+        end
+        it 'returns a Faraday::ClientError error' do
+          expect do
+            Breeze.new(id: 1).save(@params)
+          end.to raise_error(Faraday::ClientError, 'the server responded with status 500')
+          expect(root_request).to have_been_requested
+          expect(@save_request).to have_been_requested
+        end
+      end
+
       context 'when overriding resource_find_options' do
         before(:each) do
           class Breeze
@@ -245,6 +308,22 @@ RSpec.describe Braque::Model, type: :model do
         end
       end
 
+      context 'with an errored response' do
+        before(:each) do
+          root_request
+          @destroy_request = WebMock.stub_request(:delete, "#{Breeze.api_root}/breezes/1")
+                             .with(headers: { 'Http-Authorization' => Breeze.api_token })
+                             .to_return(status: 500)
+        end
+        it 'returns a Faraday::ClientError error' do
+          expect do
+            Breeze.new(id: 1).destroy
+          end.to raise_error(Faraday::ClientError, 'the server responded with status 500')
+          expect(root_request).to have_been_requested
+          expect(@destroy_request).to have_been_requested
+        end
+      end
+
       context 'when overriding resource_find_options' do
         before(:each) do
           class Breeze

data/spec/spec_helper.rb

@@ -11,5 +11,5 @@ end
 
 # Require library up front
 require 'braque'
-require 'shoulda/matchers'
 require 'webmock/rspec'
+require 'byebug'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: braque
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Dylan Fareed
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-04 00:00:00.000000000 Z
+date: 2015-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hyperclient
@@ -98,7 +98,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.3
 signing_key: 
 specification_version: 4
 summary: Braque provides a simple interface for interacting with Hypermedia API services