api-resource 0.3.13 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 12a2916af9f6d3f12a73892c74dbcd5be02b39f3
-  data.tar.gz: 5c01f287f09b35fc7f8241978299c5c94b6c3a97
+  metadata.gz: 3e364dd3af6ba5324fbc0815f624e7ebc9854be1
+  data.tar.gz: 691d9b2272b51d7a4e360b63c88f082480e4de9a
 SHA512:
-  metadata.gz: d567f5c08141e89c2f000fadf6571b9b180d1bf2a44d94f442cbbb1df94455f0825fecd09f602fb4307807ce6eed1fce520b127d7ae9f619a39aeda56ff565b3
-  data.tar.gz: e96bdcfc40605cd76e3ef6706497b57dc7e8428d1479c2f65fc4f219a57fb2b35620d871f932c0853af46520af5816e11202675b83b88d3ed3a02ab3de6f0598
+  metadata.gz: cc8f81f3b3ffe088c9a8245aa600fe31c94114bc0e725f80ef26ea6de27b35cac59f90003f04cc7a4de3ede18187b9c26a35ff6a4c950050595aacb233f4fd8f
+  data.tar.gz: 5560d6dddef680a52344dd4b79554fdf2920dcd022c6f32f9cd85fcba125cb0e50067d35454f77344a3c7723719998f0355181b1552a53c01f48f6e4b3825f3e

data/lib/api-resource/resource.rb

@@ -44,15 +44,15 @@ module ApiResource
     end
 
     def self.all_pages(options={})
-      options = { page: 1, per_page: 500 }.merge(options)
+      options  = { page: 1, per_page: 500 }.merge(options)
       per_page = options[:per_page].to_i
-      page = options[:page].to_i
+      page     = options[:page].to_i
       begin
         options[:page] = page
-        result = self.where(options)
-        total_count = result.try :total_count
-        arr = result.to_a
-        count = arr.length
+        result         = self.where(options)
+        total_count    = result.try :total_count
+        arr            = result.to_a
+        count          = arr.length
         break if yield(arr, page, per_page, total_count) || (page - 1) * per_page + count >= total_count
         page += 1
       end while true
@@ -63,6 +63,20 @@ module ApiResource
       create_resource_collection(result)
     end
 
+    def save!
+      if self.id
+        self.class.client(:put, attributes, self.class.resource_name)
+      else
+        result          = self.class.client(:post, attributes, self.class.resource_name)
+        json            = JSON.parse(result)
+        self.attributes = json['data'] || json
+      end
+    end
+
+    def destroy!
+      self.class.client(:delete, id, self.class.resource_name) if self.id
+    end
+
     def attributes
       instance_values.with_indifferent_access
     end

data/lib/api-resource/version.rb

@@ -1,3 +1,3 @@
 module ApiResource
-  VERSION = '0.3.13'
+  VERSION = '0.4.0'
 end

data/spec/resource_spec.rb

@@ -27,10 +27,16 @@ RSpec.describe ApiResource::Resource do
     expect(actual.attributes).to match((expected[:data] || expected).except(*except))
   end
 
-  def req(verb, path, resource)
+  def req(verb, path, resource, status=200)
     stub_request(verb, "#{BlogResource.base_url}#{path}").
         with(headers: { 'Accept' => 'application/json' }).
-        to_return(status: 200, body: resource.to_json)
+        to_return(status: status, body: resource === String ? resource : resource.to_json)
+  end
+
+  before do
+    @expected_resources = { data: [{ id: 1257, title: 'Hello' },
+                                   { id: 33, title: 'World' },
+                                   { id: 38, title: 'Bonjour!' }] }
   end
 
   context '::all_pages' do
@@ -53,11 +59,6 @@ RSpec.describe ApiResource::Resource do
     end
   end
 
-  before do
-    @expected_resources = { data: [{ id: 1257, title: 'Hello' },
-                                   { id: 33, title: 'World' },
-                                   { id: 38, title: 'Bonjour!' }] }
-  end
   context '::find' do
     def check_found_resource(expected_value)
       resource_id = 3
@@ -175,10 +176,49 @@ RSpec.describe ApiResource::Resource do
       end
     end
   end
+
   context 'empty base_url' do
     it 'throws on requests' do
-      class Foo < ApiResource::Resource; end
+      class Foo < ApiResource::Resource;
+      end
       expect(-> { Foo.all }).to raise_exception(StandardError)
     end
   end
+
+  context '#save!' do
+    it 'POSTs data when id is nil' do
+      blog = Blog.new(title: 'hello')
+      expect(blog.id).to be_nil
+      expected_value = { data: { id: 325, title: 'hello' } }
+
+      stub = req(:post, '/blogs', expected_value, 201)
+      blog.save!
+      check_returned_object(Blog, expected_value, blog)
+      expect(stub).to have_been_requested
+    end
+    it 'PUTs data when id is not nil' do
+      blog_id = 525
+      blog = Blog.new(id: blog_id, title: 'hello')
+
+      stub = req(:put, '/blogs', '', 204)
+      blog.save!
+      expect(blog.id).to eq(blog_id)
+      expect(stub).to have_been_requested
+    end
+  end
+
+  context '#destroy!' do
+    it 'DELETEs data when id is not nil' do
+      blog = Blog.new(id: 323, title: 'hello')
+      stub = req(:delete, '/blogs', '', 204)
+      blog.destroy!
+      expect(stub).to have_been_requested
+    end
+    it 'not DELETE data when id is nil' do
+      blog = Blog.new(id: nil, title: 'hello')
+      stub = req(:delete, '/blogs', '', 204)
+      blog.destroy!
+      expect(stub).to_not have_been_requested
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: api-resource
 version: !ruby/object:Gem::Version
-  version: 0.3.13
+  version: 0.4.0
 platform: ruby
 authors:
 - Chaker Nakhli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-03 00:00:00.000000000 Z
+date: 2015-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simple-hmac