couch_potato 1.0.0 → 1.0.1

data/.travis.yml

@@ -12,4 +12,4 @@ before_script:
   - sudo /etc/init.d/couchdb restart
 matrix:
   allow_failures:
-    - rvm: jruby
+    - rvm: jruby-19mode

data/CHANGES.md

@@ -1,5 +1,9 @@
 ## Changes
 
+### 1.0.1
+
+* fixes error when bulk loaded document does not respond to database= (Alexander Lang)
+
 ### 1.0.0
 
 * adds `reload` method (Alexander Lang)

data/README.md

@@ -27,7 +27,7 @@ Lastly Couch Potato aims to provide a seamless integration with Ruby on Rails, e
 
 ### Supported Environments
 
-* Ruby 1.9.3
+* Ruby 1.9.3, 2.0, Rubinius
 * CouchDB 1.2.0
 * ActiveSupport 3.2, 4.0
 

data/lib/couch_potato/database.rb

@@ -144,7 +144,9 @@ module CouchPotato
     def bulk_load(ids)
       response = couchrest_database.bulk_load ids
       docs = response['rows'].map{|row| row["doc"]}.compact
-      docs.each{|doc| doc.database = self}
+      docs.each{|doc|
+        doc.database = self if doc.respond_to?(:database=)
+      }
     end
 
     def create_document(document, validate)

data/lib/couch_potato/version.rb

@@ -1,3 +1,3 @@
 module CouchPotato
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/spec/unit/database_spec.rb

@@ -46,7 +46,7 @@ end
 
 describe CouchPotato::Database, 'load' do
 
-  let(:couchrest_db) { stub('couchrest db', :info => nil) }
+  let(:couchrest_db) { stub('couchrest db', :info => nil).as_null_object }
   let(:db) { CouchPotato::Database.new couchrest_db }
 
   it "should raise an exception if nil given" do
@@ -69,7 +69,6 @@ describe CouchPotato::Database, 'load' do
   end
 
   context "when several ids given" do
-
     let(:doc1) { DbTestUser.new }
     let(:doc2) { DbTestUser.new }
     let(:response) do
@@ -77,7 +76,7 @@ describe CouchPotato::Database, 'load' do
     end
 
     before(:each) do
-      couchrest_db.stub(:bulk_load).and_return response
+      couchrest_db.stub(:bulk_load) { response }
     end
 
     it "requests the couchrest bulk method" do
@@ -94,6 +93,18 @@ describe CouchPotato::Database, 'load' do
         doc.database.should eql(db)
       end
     end
+
+    it 'does not write itself to a document that has no database= method' do
+      doc1 = stub(:doc1)
+      doc1.stub(:respond_to?).with(:database=) { false }
+      couchrest_db.stub(:bulk_load) do
+        {"rows" => [{'doc' => doc1}]}
+      end
+
+      doc1.should_not_receive(:database=)
+
+      db.load(['1'])
+    end
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: couch_potato
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-26 00:00:00.000000000 Z
+date: 2013-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json