restfully 0.5.8 → 0.5.9

data/CHANGELOG

@@ -1,3 +1,6 @@
+0.5.9
+* fixed bug in #http_methods that made a resource not removable unless force reloading it
+
 0.5.8
 * explicit dependency on 'json' >= 1.2.0
 

data/VERSION

@@ -1 +1 @@
-0.5.8
+0.5.9

data/lib/restfully.rb

@@ -14,7 +14,7 @@ require 'restfully/collection'
 
 
 module Restfully
-  VERSION = "0.5.8"
+  VERSION = "0.5.9"
   class << self
     attr_accessor :adapter
   end

data/lib/restfully/resource.rb

@@ -172,7 +172,7 @@ module Restfully
     #   => ['GET', 'POST']
     # 
     def http_methods
-      reload if executed_requests['GET'].nil? || executed_requests['GET']['headers'].nil?
+      reload if executed_requests['GET'].nil? || executed_requests['GET']['headers'].nil? || executed_requests['GET']['headers'].empty?
       (executed_requests['GET']['headers']['Allow'] || "GET").split(/,\s*/)
     end
     

data/restfully.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{restfully}
-  s.version = "0.5.8"
+  s.version = "0.5.9"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Cyril Rohr"]
-  s.date = %q{2010-03-29}
+  s.date = %q{2010-03-31}
   s.default_executable = %q{restfully}
   s.description = %q{Experimental code for auto-generation of wrappers on top of RESTful APIs that follow HATEOAS principles and provide OPTIONS methods and/or Allow headers.}
   s.email = %q{cyril.rohr@gmail.com}

data/spec/resource_spec.rb

@@ -286,4 +286,35 @@ describe Resource do
       @resource.delete(:query => {:q => 'v'}, :headers => {'Accept' => 'application/json'}).should be_true
     end
   end
+  
+  describe "supported http methodes" do
+    before do
+      @resource = Resource.new(@uri, @session = mock("session", :logger => @logger))
+    end
+    it "should reload if a GET request has not been executed before" do
+      @resource.should_receive(:executed_requests).ordered.and_return({})
+      @resource.should_receive(:reload).ordered
+      @resource.should_receive(:executed_requests).ordered.and_return({
+        'GET' => {'body' => "xxx", "headers" => {'Allow' => 'POST, PUT'}}
+      })
+      @resource.http_methods.should == ['POST','PUT']
+    end
+    it "should reload if the headers are empty" do
+      @resource.should_receive(:executed_requests).exactly(3).ordered.and_return({
+        'GET' => {'body' => "xxx", "headers" => {}}
+      })
+      @resource.should_receive(:reload).ordered
+      @resource.should_receive(:executed_requests).once.ordered.and_return({
+        'GET' => {'body' => "xxx", "headers" => {'Allow' => 'POST, PUT'}}
+      })
+      @resource.http_methods.should == ['POST','PUT']
+    end
+    it "should not reload if the headers are not empty" do
+      @resource.should_receive(:executed_requests).exactly(4).and_return({
+        'GET' => {'body' => "xxx", "headers" => {'Allow' => 'POST, PUT'}}
+      })
+      @resource.should_not_receive(:reload)
+      @resource.http_methods.should == ['POST','PUT']
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: restfully
 version: !ruby/object:Gem::Version 
-  version: 0.5.8
+  version: 0.5.9
 platform: ruby
 authors: 
 - Cyril Rohr
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-03-29 00:00:00 +02:00
+date: 2010-03-31 00:00:00 +02:00
 default_executable: restfully
 dependencies: 
 - !ruby/object:Gem::Dependency