deac/couchrest_model
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fixing incorrectly generated document URIs with testing

Browse source
This commit is contained in:
Sam Lown iMac 2009-09-28 21:42:17 +02:00
parent 2f2c507582
commit bb2c7e87ad
2 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/couchrest/core/document.rb
View file

@ -65,7 +65,7 @@ module CouchRest
# Returns the CouchDB uri for the document # Returns the CouchDB uri for the document
def uri(append_rev = false) def uri(append_rev = false)
return nil if new? return nil if new?
couch_uri = "http://#{database.root}/#{CGI.escape(id)}" couch_uri = "#{database.root}/#{CGI.escape(id)}"
if append_rev == true if append_rev == true
couch_uri << "?rev=#{rev}" couch_uri << "?rev=#{rev}"
elsif append_rev.kind_of?(Integer) elsif append_rev.kind_of?(Integer)

8
spec/couchrest/core/document_spec.rb
View file

@ -83,6 +83,14 @@ describe CouchRest::Document do
@doc.id.should == @resp["id"] @doc.id.should == @resp["id"]
@doc.rev.should == @resp["rev"] @doc.rev.should == @resp["rev"]
end end
it "should generate a correct URI" do
@doc.uri.should == "#{@db.root}/#{@doc.id}"
URI.parse(@doc.uri).to_s.should == @doc.uri
end
it "should generate a correct URI with revision" do
@doc.uri(true).should == "#{@db.root}/#{@doc.id}?rev=#{@doc.rev}"
URI.parse(@doc.uri(true)).to_s.should == @doc.uri(true)
end
end end
describe "bulk saving" do describe "bulk saving" do