couch_potato 1.6.4 → 1.9.0

data/spec/unit/attributes_spec.rb

@@ -10,7 +10,8 @@ class Plant
   include CouchPotato::Persistence
   property :leaf_count
   property :typed_leaf_count, type: Fixnum
-  property :typed_leaf_size,  type: Float
+  property :integer_something, type: Integer
+  property :typed_leaf_size, type: Float
   property :branch, type: Branch
 end
 
@@ -40,6 +41,7 @@ describe 'attributes' do
       plant = Plant.new(leaf_count: 1)
 
       expect(plant.attributes).to eq('leaf_count' => 1, 'created_at' => nil,
+        'integer_something' => nil,
         'updated_at' => nil, 'typed_leaf_count' => nil,
         'typed_leaf_size' => nil, 'branch' => nil)
     end
@@ -109,6 +111,57 @@ describe 'attributes' do
       end
     end
 
+    describe 'integer' do
+      it 'rounds a float to a fixnum' do
+        @plant.integer_something = 4.5
+
+        expect(@plant.integer_something).to eq(5)
+      end
+
+      it 'converts a string into a fixnum' do
+        @plant.integer_something = '4'
+
+        expect(@plant.integer_something).to eq(4)
+      end
+
+      it 'converts a string into a negative fixnum' do
+        @plant.integer_something = '-4'
+
+        expect(@plant.integer_something).to eq(-4)
+      end
+
+      it 'leaves a fixnum as is' do
+        @plant.integer_something = 4
+
+        expect(@plant.integer_something).to eq(4)
+      end
+
+      it 'leaves nil as is' do
+        @plant.integer_something = nil
+
+        expect(@plant.integer_something).to be_nil
+      end
+
+      it 'sets the attributes to zero if a string given' do
+        @plant.integer_something = 'x'
+
+        expect(@plant.integer_something).to eq(0)
+      end
+
+      it 'parses numbers out of a string' do
+        @plant.integer_something = 'x123'
+
+        expect(@plant.integer_something).to eq(123)
+      end
+
+      it 'sets the attributes to nil if given a blank string' do
+        @plant.integer_something = ''
+
+        expect(@plant.integer_something).to be_nil
+      end
+    end
+
+
     describe 'fixnum' do
       it 'rounds a float to a fixnum' do
         @plant.typed_leaf_count = 4.5

data/spec/unit/caching_spec.rb

@@ -0,0 +1,105 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe 'database caching' do
+  let(:couchrest_db) do
+    double(:couchrest_db, info: double(:info),
+    root: '', get: double.as_null_object)
+  end
+
+  let(:db) do
+    CouchPotato::Database.new(couchrest_db).tap do |db|
+      db.cache = cache
+    end
+  end
+
+  let(:cache) do
+    {}
+  end
+
+  it 'gets an object from the cache the 2nd time via #load_documemt' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(1).times
+
+    db.load_document '1'
+    db.load_document '1'
+  end
+
+  it 'gets an object from the cache the 2nd time via #load' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(1).times
+
+    db.load '1'
+    db.load '1'
+  end
+
+  it 'gets an object from the cache the 2nd time via #load!' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(1).times
+
+    db.load! '1'
+    db.load! '1'
+  end
+
+  it 'returns the correct object' do
+    doc = double(:doc, 'database=': nil)
+    allow(couchrest_db).to receive_messages(get: doc)
+
+    db.load_document '1'
+    expect(db.load_document('1')).to eql(doc)
+  end
+
+  it 'does not cache bulk loads' do
+    allow(couchrest_db).to receive_messages(bulk_load: {'rows' => []})
+    expect(couchrest_db).to receive(:bulk_load).with(['1']).exactly(2).times
+
+    db.load_document ['1']
+    db.load_document ['1']
+  end
+
+  it 'clears the cache when destroying a document via #destroy_document' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(2).times
+
+    db.load_document '1'
+    db.destroy_document double.as_null_object
+    db.load_document '1'
+  end
+
+  it 'clears the cache when destroying a document via #destroy' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(2).times
+
+    db.load_document '1'
+    db.destroy double.as_null_object
+    db.load_document '1'
+  end
+
+  it 'clears the cache when updating a document via #save_document' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(2).times
+
+    db.load_document '1'
+    db.save_document double.as_null_object
+    db.load_document '1'
+  end
+
+  it 'clears the cache when updating a document via #save_document!' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(2).times
+
+    db.load_document '1'
+    db.save_document! double.as_null_object
+    db.load_document '1'
+  end
+
+  it 'clears the cache when updating a document via #save' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(2).times
+
+    db.load_document '1'
+    db.save double.as_null_object
+    db.load_document '1'
+  end
+
+  it 'clears the cache when updating a document via #save!' do
+    expect(couchrest_db).to receive(:get).with('1').exactly(2).times
+
+    db.load_document '1'
+    db.save! double.as_null_object
+    db.load_document '1'
+  end
+end

data/spec/unit/couch_potato_spec.rb

@@ -1,6 +1,62 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
-describe CouchPotato, 'full_url_to_database' do
+describe CouchPotato, '.configure' do
+  after(:example) do
+    # reset defaults
+    CouchPotato::Config.database_name = nil
+    CouchPotato::Config.split_design_documents_per_view = false
+    CouchPotato::Config.digest_view_names = false
+    CouchPotato::Config.default_language = :javascript
+    CouchPotato::Config.database_host = 'http://127.0.0.1:5984'
+    CouchPotato::Config.additional_databases = {}
+  end
+
+  it 'sets the database name when config is a string' do
+    CouchPotato.configure('testdb')
+
+    expect(CouchPotato::Config.database_name).to eq('testdb')
+  end
+
+  it 'does not override database_host if not given' do
+    CouchPotato.configure(
+      database: 'testdb'
+    )
+
+    expect(CouchPotato::Config.database_host).to eq('http://127.0.0.1:5984')
+  end
+
+  it 'sets the given config options' do
+    CouchPotato.configure(
+      database: 'testdb',
+      database_host: 'http://10.0.0.1:2000',
+      additional_databases: {
+        test2: 'test2_db'
+      },
+      split_design_documents_per_view: true,
+      digest_view_names: true,
+      default_language: 'erlang'
+    )
+
+    expect(CouchPotato::Config.database_name).to eq('testdb')
+    expect(CouchPotato::Config.split_design_documents_per_view).to eq(true)
+    expect(CouchPotato::Config.digest_view_names).to eq(true)
+    expect(CouchPotato::Config.default_language).to eq('erlang')
+    expect(CouchPotato::Config.database_host).to eq('http://10.0.0.1:2000')
+    expect(CouchPotato::Config.additional_databases).to eq('test2' => 'test2_db')
+  end
+
+  it 'works with string keys' do
+    CouchPotato.configure(
+      'database' => 'testdb'
+    )
+
+    expect(CouchPotato::Config.database_name).to eq('testdb')
+  end
+end
+
+describe CouchPotato, '.full_url_to_database' do
   before(:each) do
     @original_database_name = CouchPotato::Config.database_name
   end
@@ -8,12 +64,12 @@ describe CouchPotato, 'full_url_to_database' do
     CouchPotato::Config.database_name = @original_database_name
   end
 
-  it "should add the default localhost and port if only a name is set" do
+  it 'should add the default localhost and port if only a name is set' do
     CouchPotato::Config.database_name = 'test'
     expect(CouchPotato.full_url_to_database).to eq('http://127.0.0.1:5984/test')
   end
 
-  it "should return the set url" do
+  it 'should return the set url' do
     CouchPotato::Config.database_name = 'http://db.local/test'
     expect(CouchPotato.full_url_to_database).to eq('http://db.local/test')
   end
@@ -21,13 +77,22 @@ end
 
 describe CouchPotato, 'use' do
   it 'should return the db object' do
-    db = CouchPotato.use("testdb")
+    db = CouchPotato.use('testdb')
     expect(db.couchrest_database.root.to_s).to eq('http://127.0.0.1:5984/testdb')
   end
+
+  it 'returns a db from the additional_databases pool' do
+    CouchPotato::Config.database_host = 'http://127.0.0.1:5984'
+    CouchPotato::Config.additional_databases = { '1' => 'db1', '2' => 'db2' }
+
+    db = CouchPotato.use('2')
+
+    expect(db.couchrest_database.root.to_s).to eq('http://127.0.0.1:5984/db2')
+  end
 end
 
 describe CouchPotato, '.models' do
-  it "returns all classes that have implemented CouchPotato::Persistence" do
+  it 'returns all classes that have implemented CouchPotato::Persistence' do
     clazz = Class.new
     clazz.send(:include, CouchPotato::Persistence)
 

data/spec/unit/create_spec.rb

@@ -39,7 +39,8 @@ describe "create" do
   describe "fails" do
     before(:each) do
       @comment = Comment.new
-      CouchPotato::Database.new(double('database', :info => nil)).save_document(@comment)
+      @db = CouchPotato::Database.new(double('database', :info => nil))
+      @db.save_document(@comment)
     end
 
     it "should not assign an id" do
@@ -58,11 +59,11 @@ describe "create" do
       expect(@comment.updated_at).to be_nil
     end
     
-    describe "with bank" do
+    describe "with !" do
       it "should raise an exception" do
         expect {
-          @comment.save!
-        }.to raise_error
+          @db.save! @comment
+        }.to raise_error(CouchPotato::Database::ValidationsFailedError)
       end
     end
   end