dupe 1.0.1 → 1.1.0.rc1

data/lib/dupe/active_resource_extensions.rb

@@ -23,7 +23,7 @@ module ActiveResource #:nodoc:
       if ActiveResource::VERSION::MAJOR == 3 && ActiveResource::VERSION::MINOR >= 1
         response
       else
-        format.decode(response.body)
+        Dupe.format.decode(response.body)
       end
     end
 
@@ -34,20 +34,19 @@ module ActiveResource #:nodoc:
       # if the request threw an exception
       rescue
         unless body.blank?
-          resource_hash = Hash.from_xml(body)
-          resource_hash = resource_hash[resource_hash.keys.first]
+          resource_hash = Dupe.format.decode(body)      
         end
         resource_hash = {} unless resource_hash.kind_of?(Hash)
         begin
           mocked_response, new_path = Dupe.network.request(:post, path, resource_hash)
           error = false
         rescue Dupe::UnprocessableEntity => e
-          mocked_response = {:error => e.message.to_s}.to_xml(:root => 'errors')
+          mocked_response = Dupe.format.encode( {:error => e.message.to_s}, :root => 'errors')
           error = true
         end
         ActiveResource::HttpMock.respond_to do |mock|
           if error
-            mock.post(path, {}, mocked_response, 422, "Content-Type" => 'application/xml')
+            mock.post(path, {}, mocked_response, 422, "Content-Type" => Dupe.format.mime_type)
           else
             mock.post(path, {}, mocked_response, 201, "Location" => new_path)
           end
@@ -65,8 +64,7 @@ module ActiveResource #:nodoc:
       # if the request threw an exception
       rescue
         unless body.blank?
-          resource_hash = Hash.from_xml(body)
-          resource_hash = resource_hash[resource_hash.keys.first]
+          resource_hash = Dupe.format.decode(body)
         end
         resource_hash = {} unless resource_hash.kind_of?(Hash)
         resource_hash.symbolize_keys!
@@ -75,12 +73,12 @@ module ActiveResource #:nodoc:
           error = false
           mocked_response, path = Dupe.network.request(:put, path, resource_hash)
         rescue Dupe::UnprocessableEntity => e
-          mocked_response = {:error => e.message.to_s}.to_xml(:root => 'errors')
+          mocked_response = Dupe.format.encode( {:error => e.message.to_s}, :root => 'errors' )
           error = true
         end
         ActiveResource::HttpMock.respond_to do |mock|
           if error
-            mock.put(path, {}, mocked_response, 422, "Content-Type" => 'application/xml')
+            mock.put(path, {}, mocked_response, 422, "Content-Type" => Dupe.format.mime_type)
           else
             mock.put(path, {}, mocked_response, 204)
           end

data/lib/dupe/dupe.rb

@@ -11,7 +11,10 @@ class Dupe
     # set this to "true" if you want Dupe to spit out mocked requests
     # after each of your cucumber scenario's run
     attr_accessor :debug
-    
+
+    # Get the format to use (default is ActiveResource::Base.format)
+    attr_accessor :format
+
     # Suppose we're creating a 'book' resource. Perhaps our app assumes every book has a title, so let's define a book resource
     # that specifies just that:
     # 
@@ -171,27 +174,27 @@ class Dupe
         mocks = %{
           network.define_service_mock(
             :get, 
-            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}\\.xml$}, 
+            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}\\.(?:xml|json)$}, 
             proc { Dupe.find(:#{model_name.to_s.pluralize}) }
           )
           network.define_service_mock(
             :get, 
-            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}/(\\d+)\\.xml$}, 
+            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}/(\\d+)\\.(?:xml|json)$}, 
             proc {|id| Dupe.find(:#{model_name}) {|resource| resource.id == id.to_i}}
           )
           network.define_service_mock(
             :post, 
-            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}\\.xml$}, 
+            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}\\.(?:xml|json)$}, 
             proc { |post_body| Dupe.create(:#{model_name.to_s}, post_body) }
           )
           network.define_service_mock(
             :put,
-            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}/(\\d+)\\.xml$}, 
+            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}/(\\d+)\\.(?:xml|json)$}, 
             proc { |id, put_data| Dupe.find(:#{model_name.to_s}) {|resource| resource.id == id.to_i}.merge!(put_data) }
           )
           network.define_service_mock(
             :delete,
-            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}/(\\d+)\\.xml$}, 
+            %r{^#{model_name.to_s.titleize.constantize.prefix rescue '/'}#{model_name.to_s.pluralize}/(\\d+)\\.(?:xml|json)$}, 
             proc { |id| Dupe.delete(:#{model_name.to_s}) {|resource| resource.id == id.to_i} }
           )
         }
@@ -460,7 +463,10 @@ class Dupe
       @debug ||= false
     end
     
-    
+    # Get the current format, defaulting to ActiveResource's configured formatter
+    def format
+      @format ||= ActiveResource::Base.format
+    end   
     
     private
     def build_conditions(conditions)

data/lib/dupe/hash_pruner.rb

@@ -31,7 +31,11 @@ class HashPruner
 end
 
 class Hash 
-  def to_xml_safe(options={})
-    HashPruner.prune(self).to_xml(options)  
+  def make_safe
+    HashPruner.prune(self)
+  end
+
+  def self.from_json(s)
+    ActiveSupport::JSON.decode(s)
   end
 end

data/lib/dupe/mock.rb

@@ -28,6 +28,7 @@ class Dupe
       
         grouped_results = url_pattern.match(url)[1..-1]
         grouped_results << body if body
+
         resp = @response.call *grouped_results
         process_response(resp, url)
       end
@@ -49,13 +50,15 @@ class Dupe
             raise ResourceNotFoundError, "Failed with 404: the request '#{url}' returned nil." 
             
           when Dupe::Database::Record
-            resp = resp.to_xml_safe(:root => resp.__model__.name.to_s)
+            resp = Dupe.format.encode( resp.make_safe, :root => resp.__model__.name.to_s )
 
           when Array
             if resp.empty?
-              resp = [].to_xml :root => 'results'
+              resp = Dupe.format.encode( [], :root => 'results' )
             else
-              resp = resp.map {|r| HashPruner.prune(r)}.to_xml(:root => resp.first.__model__.name.to_s.pluralize)
+              resp = Dupe.format.encode( 
+                resp.map {|r| HashPruner.prune(r)},
+                :root => resp.first.__model__.name.to_s.pluralize )
             end
         end
         Dupe.network.log.add_request :get, url, resp
@@ -69,7 +72,7 @@ class Dupe
   class Network
     class PostMock < Mock #:nodoc:
       
-      # returns a tuple representing the xml of the processed entity, plus the url to the entity. 
+      # returns a tuple representing the encoded form of the processed entity, plus the url to the entity. 
       def process_response(resp, url)
         case resp
           
@@ -77,8 +80,8 @@ class Dupe
             raise StandardError, "Failed with 500: the request '#{url}' returned nil." 
           
           when Dupe::Database::Record
-            new_path = "/#{resp.__model__.name.to_s.pluralize}/#{resp.id}.xml"
-            resp = resp.to_xml_safe(:root => resp.__model__.name.to_s)
+            new_path = "/#{resp.__model__.name.to_s.pluralize}/#{resp.id}.#{Dupe.format.extension}"
+            resp = Dupe.format.encode( resp.make_safe, :root => resp.__model__.name.to_s)
             Dupe.network.log.add_request :post, url, resp
             return resp, new_path
           
@@ -94,7 +97,7 @@ class Dupe
   class Network
     class PutMock < Mock #:nodoc:
       
-      # returns a tuple representing the xml of the processed entity, plus the url to the entity. 
+      # returns a tuple representing the encoded form of the processed entity, plus the url to the entity. 
       def process_response(resp, url)
         case resp
           

data/spec/lib_specs/active_resource_extensions_spec.rb

@@ -1,189 +1,376 @@
 require 'spec_helper'
 
 describe ActiveResource::Connection do
-  before do
-    Dupe.reset
-  end
-
-  describe "#get" do
+  describe "using xml " do
     before do
-      @book = Dupe.create :book, :title => 'Rooby', :label => 'rooby'
-      class Book < ActiveResource::Base
-        self.site   = 'http://www.example.com'
-        self.format = :xml
-      end
+      Dupe.reset
+      Dupe.format = ActiveResource::Formats::XmlFormat
     end
 
-    it "should pass a request off to the Dupe network if the original request failed" do
-      Dupe.network.should_receive(:request).with(:get, '/books.xml').once.and_return(Dupe.find(:books).to_xml(:root => 'books'))
-      books = Book.find(:all)
-    end
+    describe "#get" do
+      before do
+        @book = Dupe.create :book, :title => 'Rooby', :label => 'rooby'
+        class Book < ActiveResource::Base
+          self.site   = 'http://www.example.com'
+          self.format = :xml
+        end
+      end
 
-    it "should parse the xml and turn the result into active resource objects" do
-      books = Book.find(:all)
-      books.length.should == 1
-      books.first.id.should == 1
-      books.first.title.should == 'Rooby'
-      books.first.label.should == 'rooby'
-    end
-  end
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:get, '/books.xml').once.and_return(Dupe.find(:books).to_xml(:root => 'books'))
+        books = Book.find(:all)
+      end
 
-  describe "#post" do
-    before do
-      @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
-      @book.delete(:id)
-      class Book < ActiveResource::Base
-        self.site = 'http://www.example.com'
+      it "should parse the xml and turn the result into active resource objects" do
+        books = Book.find(:all)
+        books.length.should == 1
+        books.first.id.should == 1
+        books.first.title.should == 'Rooby'
+        books.first.label.should == 'rooby'
       end
     end
 
-    it "should pass a request off to the Dupe network if the original request failed" do
-      Dupe.network.should_receive(:request).with(:post, '/books.xml', Hash.from_xml(@book.to_xml(:root => 'book'))["book"] ).once
-      book = Book.create({:label => 'rooby', :title => 'Rooby'})
-    end
+    describe "#post" do
+      before do
+        @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
+        @book.delete(:id)
+        class Book < ActiveResource::Base
+          self.site = 'http://www.example.com'
+        end
+      end
 
-    it "should parse the xml and turn the result into active resource objects" do
-      book = Book.create({:label => 'rooby', :title => 'Rooby'})
-      book.id.should == 2
-      book.title.should == 'Rooby'
-      book.label.should == 'rooby'
-    end
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:post, '/books.xml', Hash.from_xml(@book.to_xml(:root => 'book'))["book"] ).once
+        book = Book.create({:label => 'rooby', :title => 'Rooby'})
+      end
 
-    it "should make ActiveResource throw an unprocessable entity exception if our Post mock throws a Dupe::UnprocessableEntity exception" do
-      Post %r{/books\.xml} do |post_data|
-        raise Dupe::UnprocessableEntity.new(:title => "must be present.") unless post_data["title"]
-        Dupe.create :book, post_data
+      it "should parse the xml and turn the result into active resource objects" do
+        book = Book.create({:label => 'rooby', :title => 'Rooby'})
+        book.id.should == 2
+        book.title.should == 'Rooby'
+        book.label.should == 'rooby'
       end
 
-      b = Book.create
-      b.new?.should be_true
-      b.errors.should_not be_empty
-      b = Book.create(:title => "hello")
-      b.new?.should be_false
-      b.errors.should be_empty
-    end
+      it "should make ActiveResource throw an unprocessable entity exception if our Post mock throws a Dupe::UnprocessableEntity exception" do
+        Post %r{/books\.xml} do |post_data|
+          raise Dupe::UnprocessableEntity.new(:title => "must be present.") unless post_data["title"]
+          Dupe.create :book, post_data
+        end
 
-    it "should handle request with blank body" do
-      class SubscribableBook < ActiveResource::Base
-        self.site = 'http://www.example.com'
-        self.format = :xml
+        b = Book.create
+        b.new?.should be_true
+        b.errors.should_not be_empty
+        b = Book.create(:title => "hello")
+        b.new?.should be_false
+        b.errors.should be_empty
+      end
 
-        def self.send_update_emails
-          post(:send_update_emails)
+      it "should handle request with blank body" do
+        class SubscribableBook < ActiveResource::Base
+          self.site = 'http://www.example.com'
+          self.format = :xml
+
+          def self.send_update_emails
+            post(:send_update_emails)
+          end
         end
-      end
 
-      Post %r{/subscribable_books/send_update_emails\.xml} do |post_data|
-        Dupe.create :email, post_data
-      end
+        Post %r{/subscribable_books/send_update_emails\.xml} do |post_data|
+          Dupe.create :email, post_data
+        end
 
-      response = SubscribableBook.send_update_emails
-      response.code.should == 201
+        response = SubscribableBook.send_update_emails
+        response.code.should == 201
+      end
     end
-  end
 
-  describe "#put" do
-    before do
-      @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
-      class Book < ActiveResource::Base
-        self.site = 'http://www.example.com'
+    describe "#put" do
+      before do
+        @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
+        class Book < ActiveResource::Base
+          self.site = 'http://www.example.com'
+        end
+        @ar_book = Book.find(1)
       end
-      @ar_book = Book.find(1)
-    end
 
-    it "should pass a request off to the Dupe network if the original request failed" do
-      Dupe.network.should_receive(:request).with(:put, '/books/1.xml', Hash.from_xml(@book.merge(:title => "Rails!").to_xml(:root => 'book'))["book"].symbolize_keys!).once.and_return([nil, '/books/1.xml'])
-      @ar_book.title = 'Rails!'
-      @ar_book.save
-    end
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:put, '/books/1.xml', Hash.from_xml(@book.merge(:title => "Rails!").to_xml(:root => 'book'))["book"].symbolize_keys!).once.and_return([nil, '/books/1.xml'])
+        @ar_book.title = 'Rails!'
+        @ar_book.save
+      end
 
-    context "put methods that return HTTP 204" do
-      before(:each) do
-        class ExpirableBook < ActiveResource::Base
-          self.site   = 'http://www.example.com'
-          self.format = :xml
-          attr_accessor :state
+      context "put methods that return HTTP 204" do
+        before(:each) do
+          class ExpirableBook < ActiveResource::Base
+            self.site   = 'http://www.example.com'
+            self.format = :xml
+            attr_accessor :state
+
+            def expire_copyrights!
+              put(:expire)
+            end
+          end
 
-          def expire_copyrights!
-            put(:expire)
+          Put %r{/expirable_books/(\d)+/expire.xml} do |id, body|
+            Dupe.find(:expirable_book) { |eb| eb.id == id.to_i }.tap { |book|
+              book.state = 'expired'
+            }
           end
+
+          @e = Dupe.create :expirable_book, :title => 'Impermanence', :state => 'active'
         end
 
-        Put %r{/expirable_books/(\d)+/expire.xml} do |id, body|
-          Dupe.find(:expirable_book) { |eb| eb.id == id.to_i }.tap { |book|
-            book.state = 'expired'
-          }
+        it "should handle no-content responses" do
+          response = ExpirableBook.find(@e.id).expire_copyrights!
+          response.body.should be_blank
+          response.code.to_s.should == "204"
         end
+      end
 
-        @e = Dupe.create :expirable_book, :title => 'Impermanence', :state => 'active'
+      it "should parse the xml and turn the result into active resource objects" do
+        @book.title.should == "Rooby"
+        @ar_book.title = "Rails!"
+        @ar_book.save
+        @ar_book.new?.should == false
+        @ar_book.valid?.should == true
+        @ar_book.id.should == 1
+        @ar_book.label.should == "rooby"
+        @book.title.should == "Rails!"
+        @book.id.should == 1
+        @book.label.should == 'rooby'
       end
 
-      it "should handle no-content responses" do
-        response = ExpirableBook.find(@e.id).expire_copyrights!
-        response.body.should be_blank
-        response.code.to_s.should == "204"
+      it "should make ActiveResource throw an unprocessable entity exception if our Put mock throws a Dupe::UnprocessableEntity exception" do
+        Put %r{/books/(\d+)\.xml} do |id, put_data|
+          raise Dupe::UnprocessableEntity.new(:title => " must be present.") unless put_data[:title]
+          Dupe.find(:book) {|b| b.id == id.to_i}.merge!(put_data)
+        end
+
+        @ar_book.title = nil
+        @ar_book.save.should == false
+        @ar_book.errors[:base].should_not be_empty
+
+        @ar_book.title = "Rails!"
+        @ar_book.save.should == true
+        # the following line should be true, were it not for a bug in active_resource 2.3.3 - 2.3.5
+        # i reported the bug here: https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/4169-activeresourcebasesave-put-doesnt-clear-out-errors
+        # @ar_book.errors.should be_empty
       end
     end
 
-    it "should parse the xml and turn the result into active resource objects" do
-      @book.title.should == "Rooby"
-      @ar_book.title = "Rails!"
-      @ar_book.save
-      @ar_book.new?.should == false
-      @ar_book.valid?.should == true
-      @ar_book.id.should == 1
-      @ar_book.label.should == "rooby"
-      @book.title.should == "Rails!"
-      @book.id.should == 1
-      @book.label.should == 'rooby'
-    end
+    describe "#delete" do
+      before do
+        @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
+        class Book < ActiveResource::Base
+          self.site   = 'http://www.example.com'
+          self.format = :xml
+        end
+        @ar_book = Book.find(1)
+      end
 
-    it "should make ActiveResource throw an unprocessable entity exception if our Put mock throws a Dupe::UnprocessableEntity exception" do
-      Put %r{/books/(\d+)\.xml} do |id, put_data|
-        raise Dupe::UnprocessableEntity.new(:title => " must be present.") unless put_data[:title]
-        Dupe.find(:book) {|b| b.id == id.to_i}.merge!(put_data)
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:delete, '/books/1.xml').once
+        @ar_book.destroy
       end
 
-      @ar_book.title = nil
-      @ar_book.save.should == false
-      @ar_book.errors[:base].should_not be_empty
+      it "trigger a Dupe.delete to delete the mocked resource from the duped database" do
+        Dupe.find(:books).length.should == 1
+        @ar_book.destroy
+        Dupe.find(:books).length.should == 0
+      end
 
-      @ar_book.title = "Rails!"
-      @ar_book.save.should == true
-      # the following line should be true, were it not for a bug in active_resource 2.3.3 - 2.3.5
-      # i reported the bug here: https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/4169-activeresourcebasesave-put-doesnt-clear-out-errors
-      # @ar_book.errors.should be_empty
+      it "should allow you to override the default DELETE intercept mock" do
+        Delete %r{/books/(\d+)\.xml} do |id|
+          raise StandardError, "Testing Delete override"
+        end
+
+        proc {@ar_book.destroy}.should raise_error(StandardError, "Testing Delete override")
+      end
     end
   end
 
-  describe "#delete" do
+  describe "using json " do
     before do
-      @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
-      class Book < ActiveResource::Base
-        self.site   = 'http://www.example.com'
-        self.format = :xml
+      Dupe.reset
+      Dupe.format = ActiveResource::Formats::JsonFormat
+    end
+
+    describe "#get" do
+      before do
+        @book = Dupe.create :book, :title => 'Rooby', :label => 'rooby'
+        class Book < ActiveResource::Base
+          self.site   = 'http://www.example.com'
+          self.format = :json
+        end
+      end
+
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:get, '/books.json').once.and_return(Dupe.find(:books).to_json(:root => 'books'))
+        books = Book.find(:all)
+      end
+
+      it "should parse the json and turn the result into active resource objects" do
+        books = Book.find(:all)
+        books.length.should == 1
+        books.first.id.should == 1
+        books.first.title.should == 'Rooby'
+        books.first.label.should == 'rooby'
       end
-      @ar_book = Book.find(1)
     end
 
-    it "should pass a request off to the Dupe network if the original request failed" do
-      Dupe.network.should_receive(:request).with(:delete, '/books/1.xml').once
-      @ar_book.destroy
+    describe "#post" do
+      before do
+        @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
+        @book.delete(:id)
+        class Book < ActiveResource::Base
+          self.site = 'http://www.example.com'
+        end
+      end
+
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:post, '/books.json', Hash.from_json(@book.to_json(:root => 'book')) ).once
+        book = Book.create({:label => 'rooby', :title => 'Rooby'})
+      end
+
+      it "should parse the json and turn the result into active resource objects" do
+        book = Book.create({:label => 'rooby', :title => 'Rooby'})
+        book.id.should == 2
+        book.title.should == 'Rooby'
+        book.label.should == 'rooby'
+      end
+
+      it "should make ActiveResource throw an unprocessable entity exception if our Post mock throws a Dupe::UnprocessableEntity exception" do
+        Post %r{/books\.json} do |post_data|
+          raise Dupe::UnprocessableEntity.new(:title => "must be present.") unless post_data["title"]
+          Dupe.create :book, post_data
+        end
+
+        b = Book.create
+        b.new?.should be_true
+        b.should_not be_empty
+      end
+
+      it "should handle request with blank body" do
+        class SubscribableBook < ActiveResource::Base
+          self.site = 'http://www.example.com'
+          self.format = :json
+
+          def self.send_update_emails
+            post(:send_update_emails)
+          end
+        end
+
+        Post %r{/subscribable_books/send_update_emails\.json} do |post_data|
+          Dupe.create :email, post_data
+        end
+
+        response = SubscribableBook.send_update_emails
+        response.code.should == 201
+      end
     end
 
-    it "trigger a Dupe.delete to delete the mocked resource from the duped database" do
-      Dupe.find(:books).length.should == 1
-      @ar_book.destroy
-      Dupe.find(:books).length.should == 0
+    describe "#put" do
+      before do
+        @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
+        class Book < ActiveResource::Base
+          self.site = 'http://www.example.com'
+        end
+        @ar_book = Book.find(1)
+      end
+
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:put, '/books/1.json', Hash.from_json(@book.merge(:title => "Rails!").to_json(:root => 'book')).symbolize_keys!).once.and_return([nil, '/books/1.json'])
+        @ar_book.title = 'Rails!'
+        @ar_book.save
+      end
+
+      context "put methods that return HTTP 204" do
+        before(:each) do
+          class ExpirableBook < ActiveResource::Base
+            self.site   = 'http://www.example.com'
+            self.format = :json
+            attr_accessor :state
+
+            def expire_copyrights!
+              put(:expire)
+            end
+          end
+
+          Put %r{/expirable_books/(\d)+/expire.json} do |id, body|
+            Dupe.find(:expirable_book) { |eb| eb.id == id.to_i }.tap { |book|
+              book.state = 'expired'
+            }
+          end
+
+          @e = Dupe.create :expirable_book, :title => 'Impermanence', :state => 'active'
+        end
+
+        it "should handle no-content responses" do
+          response = ExpirableBook.find(@e.id).expire_copyrights!
+          response.body.should be_blank
+          response.code.to_s.should == "204"
+        end
+      end
+
+      it "should parse the json and turn the result into active resource objects" do
+        @book.title.should == "Rooby"
+        @ar_book.title = "Rails!"
+        @ar_book.save
+        @ar_book.new?.should == false
+        @ar_book.valid?.should == true
+        @ar_book.id.should == 1
+        @ar_book.label.should == "rooby"
+        @book.title.should == "Rails!"
+        @book.id.should == 1
+        @book.label.should == 'rooby'
+      end
+
+      it "should make ActiveResource throw an unprocessable entity exception if our Put mock throws a Dupe::UnprocessableEntity exception" do
+        Put %r{/books/(\d+)\.json} do |id, put_data|
+          raise Dupe::UnprocessableEntity.new(:title => " must be present.") unless put_data[:title]
+          Dupe.find(:book) {|b| b.id == id.to_i}.merge!(put_data)
+        end
+
+        @ar_book.title = nil
+        @ar_book.save.should == false
+        @ar_book.should_not be_empty
+
+        @ar_book.title = "Rails!"
+        @ar_book.save.should == true
+        # the following line should be true, were it not for a bug in active_resource 2.3.3 - 2.3.5
+        # i reported the bug here: https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/4169-activeresourcebasesave-put-doesnt-clear-out-errors
+        # @ar_book.errors.should be_empty
+      end
     end
 
-    it "should allow you to override the default DELETE intercept mock" do
-      Delete %r{/books/(\d+)\.xml} do |id|
-        raise StandardError, "Testing Delete override"
+    describe "#delete" do
+      before do
+        @book = Dupe.create :book, :label => 'rooby', :title => 'Rooby'
+        class Book < ActiveResource::Base
+          self.site   = 'http://www.example.com'
+          self.format = :json
+        end
+        @ar_book = Book.find(1)
+      end
+
+      it "should pass a request off to the Dupe network if the original request failed" do
+        Dupe.network.should_receive(:request).with(:delete, '/books/1.json').once
+        @ar_book.destroy
       end
 
-      proc {@ar_book.destroy}.should raise_error(StandardError, "Testing Delete override")
+      it "trigger a Dupe.delete to delete the mocked resource from the duped database" do
+        Dupe.find(:books).length.should == 1
+        @ar_book.destroy
+        Dupe.find(:books).length.should == 0
+      end
+
+      it "should allow you to override the default DELETE intercept mock" do
+        Delete %r{/books/(\d+)\.json} do |id|
+          raise StandardError, "Testing Delete override"
+        end
+
+        proc {@ar_book.destroy}.should raise_error(StandardError, "Testing Delete override")
+      end
     end
   end
-
 end