dynamoid 0.4.1 → 0.5.0

data/spec/dynamoid/document_spec.rb

@@ -87,9 +87,13 @@ describe "Dynamoid::Document" do
     
     Address.first.update_attributes(:city => 'Chicago')
     
-    @address.city.should be_nil
     @address.reload.city.should == 'Chicago'
   end
+
+  it 'reloads document with range key' do
+    tweet = Tweet.create(:tweet_id => 'x', :group => 'abc')
+    tweet.reload.group.should == 'abc'
+  end
   
   it 'has default table options' do
     @address = Address.create

data/spec/dynamoid/fields_spec.rb

@@ -9,34 +9,34 @@ describe "Dynamoid::Fields" do
   it 'declares read attributes' do
     @address.city.should be_nil
   end
-  
+
   it 'declares write attributes' do
-    @address.city = 'Chicago' 
+    @address.city = 'Chicago'
     @address.city.should == 'Chicago'
   end
-  
+
   it 'declares a query attribute' do
     @address.city?.should be_false
-    
+
     @address.city = 'Chicago'
-    
+
     @address.city?.should be_true
   end
-  
+
   it 'automatically declares id' do
     lambda {@address.id}.should_not raise_error
   end
-  
+
   it 'automatically declares and fills in created_at and updated_at' do
     @address.save
-    
+
     @address = @address.reload
     @address.created_at.should_not be_nil
     @address.created_at.class.should == DateTime
     @address.updated_at.should_not be_nil
     @address.updated_at.class.should == DateTime
   end
-  
+
   context 'with a saved address' do
     before do
       @address = Address.create
@@ -46,35 +46,35 @@ describe "Dynamoid::Fields" do
     it 'should write an attribute correctly' do
       @address.write_attribute(:city, 'Chicago')
     end
-  
+
     it 'should write an attribute with an alias' do
       @address[:city] = 'Chicago'
     end
-  
+
     it 'should read a written attribute' do
       @address.write_attribute(:city, 'Chicago')
       @address.read_attribute(:city).should == 'Chicago'
     end
-  
+
     it 'should read a written attribute with the alias' do
       @address.write_attribute(:city, 'Chicago')
       @address[:city].should == 'Chicago'
     end
-  
+
     it 'should update all attributes' do
       @address.expects(:save).once.returns(true)
       @address.update_attributes(:city => 'Chicago')
       @address[:city].should == 'Chicago'
       @address.id.should == @original_id
     end
-  
+
     it 'should update one attribute' do
       @address.expects(:save).once.returns(true)
       @address.update_attribute(:city, 'Chicago')
       @address[:city].should == 'Chicago'
       @address.id.should == @original_id
     end
-    
+
     it 'adds in dirty methods for attributes' do
       @address.city = 'Chicago'
       @address.save
@@ -83,7 +83,7 @@ describe "Dynamoid::Fields" do
 
       @address.city_was.should == 'Chicago'
     end
-  
+
     it 'returns all attributes' do
       Address.attributes.should == {:id=>{:type=>:string}, :created_at=>{:type=>:datetime}, :updated_at=>{:type=>:datetime}, :city=>{:type=>:string}, :options=>{:type=>:serialized}}
     end
@@ -93,5 +93,35 @@ describe "Dynamoid::Fields" do
     Dynamoid.logger.expects(:warn).with(regexp_matches(/city field has a length of 66000/))
     Address.new city: ("Ten chars " * 6_600)
   end
-  
+
+
+  context 'default values for fields' do
+    before do
+      @clazz = Class.new do
+        include Dynamoid::Document
+
+        field :name, :string, :default => 'x'
+        field :uid, :integer, :default => lambda { 42 }
+
+        def self.name
+          'Document'
+        end
+      end
+
+
+      @doc = @clazz.new
+    end
+
+    it 'returns default value' do
+      @doc.name.should eq('x')
+      @doc.uid.should eq(42)
+    end
+
+    it 'should save default value' do
+      @doc.save!
+      @doc.reload.name.should eq('x')
+      @doc.uid.should eq(42)
+    end
+  end
+
 end

data/spec/dynamoid/finders_spec.rb

@@ -8,24 +8,24 @@ describe "Dynamoid::Finders" do
 
   it 'finds an existing address' do
     found = Address.find(@address.id)
-    
+
     found.should == @address
     found.city.should == 'Chicago'
   end
-  
+
   it 'is not a new object' do
     found = Address.find(@address.id)
-    
+
     found.new_record.should_not be_true
   end
-  
+
   it 'returns nil when nothing is found' do
     Address.find('1234').should be_nil
   end
-  
+
   it 'finds multiple ids' do
     @address2 = Address.create(:city => 'Illinois')
-    
+
     Address.find(@address.id, @address2.id).should include @address, @address2
   end
 
@@ -33,12 +33,12 @@ describe "Dynamoid::Finders" do
     Dynamoid::Adapter.expects(:get_item).with { |table_name, key, options| options[:consistent_read] == true }
     Address.find('x', :consistent_read => true)
   end
-  
+
   context 'with users' do
     before do
       User.create_indexes
     end
-    
+
     it 'finds using method_missing for attributes' do
       array = Address.find_by_city('Chicago')
 
@@ -83,7 +83,7 @@ describe "Dynamoid::Finders" do
 
       array.should be_empty
     end
-    
+
     it 'finds using method_missing for a single attribute and no results' do
       @user1 = User.create(:name => 'Josh', :email => 'josh@joshsymonds.com')
       @user2 = User.create(:name => 'Justin', :email => 'justin@joshsymonds.com')
@@ -108,12 +108,28 @@ describe "Dynamoid::Finders" do
 
       array.should == [@user]
     end
-    
+
     it 'should return an empty array when fields exist but nothing is found' do
       array = User.find_all_by_password('Test')
-      
+
       array.should be_empty
     end
 
   end
+
+  context 'find_all' do
+    it 'should return a array of users' do
+      users = (1..10).map { User.create }
+      User.find_all(users.map(&:id)).should eq(users)
+    end
+
+    it 'should return a array of tweets' do
+      tweets = (1..10).map { |i| Tweet.create(:tweet_id => "#{i}", :group => "group_#{i}") }
+      Tweet.find_all(tweets.map { |t| [t.tweet_id, t.group] }).should eq(tweets)
+    end
+
+    it 'should return an empty array' do
+      User.find_all([]).should eq([])
+    end
+  end
 end

data/spec/dynamoid/identity_map_spec.rb

@@ -0,0 +1,45 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe "Dynamoid::IdentityMap" do
+  before :each do
+    Dynamoid::Config.identity_map = true
+  end
+
+  after :each do
+    Dynamoid::Config.identity_map = false
+  end
+
+  context 'object identity' do
+    it 'maintains a single object' do
+      tweet = Tweet.create(:tweet_id => "x", :group => "one")
+      tweet1 = Tweet.where(:tweet_id => "x", :group => "one").first
+      tweet.should equal(tweet1)
+    end
+  end
+
+  context 'cache' do
+    it 'uses cache' do
+      tweet = Tweet.create(:tweet_id => "x", :group => "one")
+      Dynamoid::Adapter.expects(:read).never
+      tweet1 = Tweet.find_by_id("x", :range_key => "one")
+      tweet.should equal(tweet1)
+    end
+
+    it 'clears cache on delete' do
+      tweet = Tweet.create(:tweet_id => "x", :group => "one")
+      tweet.delete
+      Tweet.find_by_id("x", :range_key => "one").should be_nil
+    end
+  end
+
+  context 'clear' do
+    it 'clears the identiy map' do
+      Tweet.create(:tweet_id => "x", :group => "one")
+      Tweet.create(:tweet_id => "x", :group => "two")
+
+      Tweet.identity_map.size.should eq(2)
+      Dynamoid::IdentityMap.clear
+      Tweet.identity_map.size.should eq(0)
+    end
+  end
+end

data/spec/dynamoid/indexes/index_spec.rb

@@ -57,8 +57,6 @@ describe "Dynamoid::Indexes::Index" do
     @user = User.new(:name => 'Josh', :password => 'test123', :created_at => @time)
     @user.name = 'Justin'
     
-    @index.values(@user, true).should == {:hash_value => 'Josh.test123', :range_value => @time.to_f}
-    
     @index.values(@user).should == {:hash_value => 'Justin.test123', :range_value => @time.to_f}
   end
   

data/spec/dynamoid/persistence_spec.rb

@@ -12,7 +12,7 @@ describe "Dynamoid::Persistence" do
       before do
         Dynamoid::Adapter.delete_table(Address.table_name) if Dynamoid::Adapter.list_tables.include?(Address.table_name)
       end
-    
+
       it 'creates a table' do
         Address.create_table(:table_name => Address.table_name)
 
@@ -27,57 +27,57 @@ describe "Dynamoid::Persistence" do
       end
     end
   end
-  
+
   it 'assigns itself an id on save' do
     @address.save
-    
+
     Dynamoid::Adapter.read("dynamoid_tests_addresses", @address.id)[:id].should == @address.id
   end
-  
+
   it 'assigns itself an id on save only if it does not have one' do
     @address.id = 'test123'
     @address.save
-    
+
     Dynamoid::Adapter.read("dynamoid_tests_addresses", 'test123').should_not be_empty
   end
-  
+
   it 'has a table name' do
     Address.table_name.should == 'dynamoid_tests_addresses'
   end
-    
+
   it 'saves indexes along with itself' do
     @user = User.new(:name => 'Josh')
-    
+
     @user.expects(:save_indexes).once.returns(true)
     @user.save
   end
-  
+
   it 'deletes an item completely' do
     @user = User.create(:name => 'Josh')
     @user.destroy
-    
+
     Dynamoid::Adapter.read("dynamoid_tests_users", @user.id).should be_nil
   end
-  
+
   it 'keeps string attributes as strings' do
     @user = User.new(:name => 'Josh')
     @user.send(:dump)[:name].should == 'Josh'
   end
-  
+
   it 'dumps datetime attributes' do
     @user = User.create(:name => 'Josh')
     @user.send(:dump)[:name].should == 'Josh'
   end
-  
+
   it 'dumps integer attributes' do
     @subscription = Subscription.create(:length => 10)
     @subscription.send(:dump)[:length].should == 10
   end
-  
+
   it 'dumps set attributes' do
     @subscription = Subscription.create(:length => 10)
     @magazine = @subscription.magazine.create
-    
+
     @subscription.send(:dump)[:magazine_ids].should == Set[@magazine.id]
   end
 
@@ -94,11 +94,11 @@ describe "Dynamoid::Persistence" do
     hash = {foo: :bar}
     Address.new(options: hash).options.should == hash
   end
-  
+
   it 'loads attributes from a hash' do
     @time = DateTime.now
     @hash = {:name => 'Josh', :created_at => @time.to_f}
-    
+
     User.undump(@hash)[:name].should == 'Josh'
     User.undump(@hash)[:created_at].to_f == @time.to_f
   end
@@ -120,21 +120,57 @@ describe "Dynamoid::Persistence" do
 
     CamelCase.new.save
   end
-  
-  it 'tracks previous changes on save or update' do
-    @address.city = 'Chicago'
-    @address.save
-    
-    @address.city = 'San Francisco'
-    @address.save
-    
-    @address.city_was.should == 'Chicago'
-  end
-  
+
   it 'works with a HashWithIndifferentAccess' do
     hash = ActiveSupport::HashWithIndifferentAccess.new("city" => "Atlanta")
-    
+
     lambda {Address.create(hash)}.should_not raise_error
   end
 
+  context 'update' do
+
+    before :each do
+      @tweet = Tweet.create(:tweet_id => 1, :group => 'abc', :count => 5, :tags => ['db', 'sql'], :user_name => 'john')
+    end
+
+    it 'support add/delete operation on a field' do
+      @tweet.update do |t|
+        t.add(:count => 3)
+        t.delete(:tags => ['db'])
+      end
+
+      @tweet.count.should eq(8)
+      @tweet.tags.to_a.should eq(['sql'])
+    end
+
+    it 'checks the conditions on update' do
+      @tweet.update(:if => { :count => 5 }) do |t|
+        t.add(:count => 3)
+      end.should be_true
+
+      @tweet.count.should eq(8)
+
+      @tweet.update(:if => { :count => 5 }) do |t|
+        t.add(:count => 3)
+      end.should be_false
+
+      @tweet.count.should eq(8)
+
+      expect {
+        @tweet.update!(:if => { :count => 5 }) do |t|
+          t.add(:count => 3)
+        end
+      }.to raise_error(Dynamoid::Errors::ConditionalCheckFailedException)
+    end
+
+  end
+
+  context 'delete' do
+    it 'deletes model with datetime range key' do
+      lambda {
+        msg = Message.create!(:message_id => 1, :time => DateTime.now, :text => "Hell yeah")
+        msg.destroy
+      }.should_not raise_error
+    end
+  end
 end