CHEF-222: shift the couchdb detection into view_uri

Author: AJ Christensen

chef/lib/chef/couchdb.rb

@@ -29,7 +29,6 @@ class CouchDB
     def initialize(url=nil)
       url ||= Chef::Config[:couchdb_url]
       @rest = Chef::REST.new(url)
-      Chef::Config[:couchdb_version] ||= @rest.run_request(:GET, URI.parse(@rest.url + "/"), false, 10, false)["version"].gsub(/-.+/,"").to_f
     end
 
     def create_db
@@ -152,6 +151,7 @@ def safe_name(name)
     end
 
     def view_uri(design, view)
+      Chef::Config[:couchdb_version] ||= @rest.run_request(:GET, URI.parse(@rest.url + "/"), false, 10, false)["version"].gsub(/-.+/,"").to_f
       case Chef::Config[:couchdb_version]
       when 0.9
         "#{Chef::Config[:couchdb_database]}/_design/#{design}/_view/#{view}"

chef/spec/unit/couchdb_spec.rb

@@ -36,10 +36,6 @@
     Chef::REST.should_receive(:new).with("http://monkeypants")
     Chef::CouchDB.new("http://monkeypants")
   end  
-  
-  it "should parse the CouchDB version number" do
-    
-  end
 end
 
 describe Chef::CouchDB, "create_db" do
@@ -271,6 +267,14 @@ def do_delete(rev=nil)
     Chef::Config.stub!(:[]).with(:couchdb_database).and_return("chef")
   end
 
+  describe "when the couchdb version is unknown" do
+    it "should set the couchdb version appropriately" do
+      Chef::Config.stub!(:[]).with(:couchdb_version).and_return(nil)
+      Chef::Config.should_receive(:couchdb_version).and_return(true)
+      @couchdb.view_uri("nodes", "all")
+    end
+  end
+  
   describe "on couchdb 0.8" do
     before do
       Chef::Config.stub!(:[]).with(:couchdb_version).and_return(0.8)