harpy 0.1.1 → 0.1.2

data/lib/harpy/resource.rb

@@ -122,6 +122,13 @@ module Harpy
         end
       end
       
+      def destroy
+        raise Harpy::UrlRequired unless url
+        _run_destroy_callbacks do
+          process_response Harpy.client.delete(url), :destroy
+        end
+      end
+      
       def link(rel)
         link = (@attrs["link"]||[]).detect{|l| l["rel"]==rel.to_s}
         link["href"] if link
@@ -184,7 +191,7 @@ module Harpy
           Harpy.client.invalid_code response
         end
       end
-
+      
       def method_missing(method, *args)
         if persisted? && !@attrs.has_key?(method.to_s)
           super

data/lib/harpy/version.rb

@@ -1,3 +1,3 @@
 module Harpy
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

data/spec/harpy/resource_spec.rb

@@ -27,6 +27,7 @@ module Harpy
       before_save :callback_before_save
       before_create :callback_before_create
       before_update :callback_before_update
+      before_destroy :callback_before_destroy
       attr_reader :callbacks
       def initialize(*args)
         @callbacks = []
@@ -44,6 +45,9 @@ module Harpy
       def callback_before_update
         @callbacks << :update
       end
+      def callback_before_destroy
+        @callbacks << :destroy
+      end
     end
   end
 end
@@ -545,4 +549,78 @@ describe "class including Harpy::Resource" do
       end
     end
   end
+  describe "#destroy" do
+    subject{ Harpy::Spec::User.new "company_name" => "Stark Enterprises" }
+    context "when link to self is missing" do
+      it "raises Harpy::UrlRequired" do
+        lambda{ subject.destroy }.should raise_error Harpy::UrlRequired
+        subject.callbacks.should =~ []
+      end
+    end
+    context "when link to self is present" do
+      let(:url) { "http://localhost/user/1" }
+      subject do
+        Harpy::Spec::User.new({
+          "urn" => "urn:harpy:user:1", 
+          "company_name" => "Stark Enterprises", 
+          "link" => [{"rel" => "self", "href" => url}],
+        })
+      end
+      [200, 201, 302].each do |response_code|
+        it "is true and merges response attributes on #{response_code}" do
+          response = Typhoeus::Response.new :code => response_code, :body => <<-eos
+            {
+              "firstname": "Anthony",
+              "urn": "urn:harpy:user:1",
+              "link": [
+                {"rel": "self", "href": "#{url}/1"}
+              ]
+            }
+          eos
+          Harpy.client.should_receive(:delete).with(url).and_return response
+          subject.destroy.should be_true
+          subject.callbacks.should =~ [:destroy]
+          subject.firstname.should == "Anthony"
+          subject.company_name.should == "Stark Enterprises"
+        end
+      end
+      it "is true but doesn't touch attributes on 204" do
+        response = Typhoeus::Response.new :code => 204
+        Harpy.client.should_receive(:delete).with(url).and_return response
+        subject.destroy.should be_true
+        subject.callbacks.should =~ [:destroy]
+        subject.company_name.should == "Stark Enterprises"
+      end
+      it "raises Harpy::Unauthorized on 401" do
+        response = Typhoeus::Response.new :code => 401
+        Harpy.client.should_receive(:delete).with(url).and_return response
+        lambda { subject.destroy }.should raise_error Harpy::Unauthorized, "Server returned a 401 response code"
+        subject.callbacks.should =~ [:destroy]
+      end
+      it "is false and fills in errors on 422" do
+        response = Typhoeus::Response.new :code => 422, :body => <<-eos
+          {
+            "firstname": "Anthony",
+            "errors": {
+              "lastname": ["can't be blank", "must be unique"]
+            }
+          }
+        eos
+        Harpy.client.should_receive(:delete).with(url).and_return response
+        subject.destroy.should be_false
+        subject.callbacks.should =~ [:destroy]
+        subject.should have(1).error
+        subject.errors[:lastname].should =~ ["can't be blank", "must be unique"]
+        lambda { subject.firstname }.should raise_error NoMethodError
+        subject.company_name.should == "Stark Enterprises"
+      end
+      it "delegates other response codes to client" do
+        response = Typhoeus::Response.new :code => 500
+        Harpy.client.should_receive(:delete).with(url).and_return response
+        Harpy.client.should_receive(:invalid_code).with(response)
+        subject.destroy
+        subject.callbacks.should =~ [:destroy]
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: harpy
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -10,12 +10,12 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-06-16 00:00:00.000000000 +02:00
+date: 2011-06-28 00:00:00.000000000 +02:00
 default_executable: 
 dependencies:
 - !ruby/object:Gem::Dependency
   name: typhoeus
-  requirement: &2165828200 !ruby/object:Gem::Requirement
+  requirement: &2168462220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -23,10 +23,10 @@ dependencies:
         version: 0.2.4
   type: :runtime
   prerelease: false
-  version_requirements: *2165828200
+  version_requirements: *2168462220
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &2165827680 !ruby/object:Gem::Requirement
+  requirement: &2168461700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -34,10 +34,10 @@ dependencies:
         version: 3.0.0
   type: :runtime
   prerelease: false
-  version_requirements: *2165827680
+  version_requirements: *2168461700
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &2165827200 !ruby/object:Gem::Requirement
+  requirement: &2168461220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -45,10 +45,10 @@ dependencies:
         version: 3.0.0
   type: :runtime
   prerelease: false
-  version_requirements: *2165827200
+  version_requirements: *2168461220
 - !ruby/object:Gem::Dependency
   name: hash-deep-merge
-  requirement: &2165826720 !ruby/object:Gem::Requirement
+  requirement: &2168460740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -56,10 +56,10 @@ dependencies:
         version: 0.1.1
   type: :runtime
   prerelease: false
-  version_requirements: *2165826720
+  version_requirements: *2168460740
 - !ruby/object:Gem::Dependency
   name: yajl-ruby
-  requirement: &2165826240 !ruby/object:Gem::Requirement
+  requirement: &2168460260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -67,10 +67,10 @@ dependencies:
         version: 0.8.2
   type: :runtime
   prerelease: false
-  version_requirements: *2165826240
+  version_requirements: *2168460260
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2165825760 !ruby/object:Gem::Requirement
+  requirement: &2168459780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -78,10 +78,10 @@ dependencies:
         version: 0.8.7
   type: :development
   prerelease: false
-  version_requirements: *2165825760
+  version_requirements: *2168459780
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2156256500 !ruby/object:Gem::Requirement
+  requirement: &2168459300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -89,10 +89,10 @@ dependencies:
         version: 2.6.0
   type: :development
   prerelease: false
-  version_requirements: *2156256500
+  version_requirements: *2168459300
 - !ruby/object:Gem::Dependency
   name: rocco
-  requirement: &2156256020 !ruby/object:Gem::Requirement
+  requirement: &2168458820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -100,7 +100,7 @@ dependencies:
         version: '0.7'
   type: :development
   prerelease: false
-  version_requirements: *2156256020
+  version_requirements: *2168458820
 description: Client for REST API with HATEOAS
 email:
 - joseph.halter@thetalentbox.com