toystore 0.8.3 → 0.9.0

data/spec/toy/callbacks_spec.rb

@@ -24,7 +24,7 @@ module CallbackHelper
 end
 
 describe Toy::Callbacks do
-  uses_constants('Game', 'Move')
+  uses_constants('Game')
 
   context "regular" do
     before do
@@ -50,48 +50,4 @@ describe Toy::Callbacks do
       doc.history.should == [:before_destroy, :after_destroy]
     end
   end
-
-  context "embedded" do
-    before do
-      Game.embedded_list(:moves)
-      Move.send(:include, CallbackHelper)
-
-      @move = Move.new
-      @game = Game.create(:moves => [@move])
-    end
-
-    it "runs callbacks for save of parent" do
-      @move.history.should == [:before_save, :before_create, :after_create, :after_save]
-    end
-
-    it "runs callbacks for update of parent" do
-      @move.clear_history
-      @game.save
-      @move.history.should == [:before_save, :before_update, :after_update, :after_save]
-    end
-
-    it "runs callbacks for destroy of parent" do
-      @move.clear_history
-      @game.destroy
-      @move.history.should == [:before_destroy, :after_destroy]
-    end
-
-    it "does not attempt to run callback defined on parent that is not defined on embedded" do
-      Game.define_callbacks :win
-      @move.clear_history
-
-      lambda do
-        @game.run_callbacks(:win)
-        @move.history.should be_empty
-      end.should_not raise_error
-    end
-
-    it "runs create callback when saving new embbeded doc on existing parent" do
-      @game.save
-      move = Move.new
-      @game.moves << move
-      @game.save
-      move.history.should == [:before_save, :before_create, :after_create, :after_save]
-    end
-  end
 end

data/spec/toy/cloneable_spec.rb

@@ -0,0 +1,47 @@
+require 'helper'
+
+describe Toy::Cloneable do
+  uses_objects('User')
+
+  before do
+    User.attribute(:name, String)
+    User.attribute(:skills, Array)
+
+    @user = User.new({
+      :name   => 'John',
+      :skills => ['looking awesome', 'growing beards'],
+    })
+  end
+  let(:user)  { @user }
+
+  describe "#clone" do
+    it "clones the @attributes hash" do
+      user.clone.instance_variable_get("@attributes").should_not equal(user.instance_variable_get("@attributes"))
+    end
+
+    it "copies the attributes" do
+      user.clone.tap do |clone|
+        clone.name.should == user.name
+        clone.skills.should == user.skills
+      end
+    end
+
+    it "clones duplicable attributes" do
+      user.clone.skills.should_not equal(user.skills)
+    end
+
+    it "regenerates id" do
+      user.clone.tap do |clone|
+        clone.id.should_not be_nil
+        clone.id.should_not == user.id
+      end
+    end
+
+    it "nullifies defined instance variables" do
+      user.instance_variable_set("@foo", true)
+      user.clone.tap do |clone|
+        clone.instance_variable_get("@foo").should be_nil
+      end
+    end
+  end
+end

data/spec/toy/dirty_spec.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
 describe Toy::Dirty do
-  uses_constants('User')
+  uses_objects('User')
 
   before do
     User.attribute(:name, String)
@@ -29,49 +29,11 @@ describe Toy::Dirty do
   end
 
   it "knows when attribute did not change" do
-    user = User.create(:name => 'Geoffrey')
-    user.name = 'Geoffrey'
+    user = User.new
+    user.name = nil
     user.should_not be_changed
   end
 
-  describe "#save" do
-    before      { @user = User.create(:name => 'Geoffrey') }
-    let(:user)  { @user }
-
-    it "clears changes" do
-      user.name = 'John'
-      user.should be_changed
-      user.save
-      user.should_not be_changed
-    end
-
-    it "sets previous changes" do
-      user.previous_changes.should == {'name' => [nil, 'Geoffrey']}
-    end
-  end
-
-  it "does not have changes when loaded from database" do
-    user = User.create
-    loaded = User.get(user.id)
-    loaded.should_not be_changed
-  end
-
-  describe "#reload" do
-    before      { @user = User.create(:name => 'John') }
-    let(:user)  { @user }
-
-    it "clears changes" do
-      user.name = 'Steve'
-      user.reload
-      user.should_not be_changed
-    end
-
-    it "clears previously changed" do
-      user.reload
-      user.previous_changes.should be_empty
-    end
-  end
-
   it "has attribute changed? method" do
     user = User.new
     user.should_not be_name_changed
@@ -80,20 +42,26 @@ describe Toy::Dirty do
   end
 
   it "has attribute was method" do
-    user = User.create(:name => 'John')
+    user = User.new(:name => 'John')
     user.name = 'Steve'
     user.name_was.should == 'John'
   end
 
   it "has attribute change method" do
-    user = User.create(:name => 'John')
+    user = User.new(:name => 'John')
     user.name = 'Steve'
     user.name_change.should == ['John', 'Steve']
   end
 
   it "has attribute will change! method" do
-    user = User.create
+    user = User.new
     user.name_will_change!
     user.should be_changed
   end
+
+  describe "#clone" do
+    it "has no changes" do
+      User.new.clone.should_not be_changed
+    end
+  end
 end

data/spec/toy/dirty_store_spec.rb

@@ -0,0 +1,47 @@
+require 'helper'
+
+describe Toy::DirtyStore do
+  uses_constants('User')
+
+  before do
+    User.attribute(:name, String)
+  end
+
+  it "does not have changes when loaded from database" do
+    user = User.create
+    loaded = User.get(user.id)
+    loaded.should_not be_changed
+  end
+
+  describe "#reload" do
+    before      { @user = User.create(:name => 'John') }
+    let(:user)  { @user }
+
+    it "clears changes" do
+      user.name = 'Steve'
+      user.reload
+      user.should_not be_changed
+    end
+
+    it "clears previously changed" do
+      user.reload
+      user.previous_changes.should be_empty
+    end
+  end
+
+  describe "#save" do
+    before      { @user = User.create(:name => 'Geoffrey') }
+    let(:user)  { @user }
+
+    it "clears changes" do
+      user.name = 'John'
+      user.should be_changed
+      user.save
+      user.should_not be_changed
+    end
+
+    it "sets previous changes" do
+      user.previous_changes.should == {'name' => [nil, 'Geoffrey']}
+    end
+  end
+end

data/spec/toy/equality_spec.rb

@@ -1,7 +1,7 @@
 require 'helper'
 
 describe Toy::Equality do
-  uses_constants('User', 'Game', 'Person')
+  uses_objects('User', 'Person')
 
   describe "#eql?" do
     it "returns true if same class and id" do
@@ -15,32 +15,18 @@ describe Toy::Equality do
     it "returns false if different id" do
       User.new(:id => 1).should_not eql(User.new(:id => 2))
     end
-
-    it "returns true if reference and target is same class and id" do
-      Game.reference(:user)
-      user = User.create
-      game = Game.create(:user => user)
-      user.should eql(game.user)
-    end
   end
 
   describe "equal?" do
     it "returns true if same object" do
-      user = User.create(:id => 1)
+      user = User.new(:id => 1)
       user.should equal(user)
     end
 
-    it "returns true if same object through proxy" do
-      Game.reference(:user)
-      user = User.create
-      game = Game.create(:user => user)
-
-      user.should equal(game.user)
-      game.user.should equal(user)
-    end
-
     it "returns false if not same object" do
-
+      user = User.new
+      other_user = User.new
+      user.should_not equal(other_user)
     end
   end
 end

data/spec/toy/extensions/boolean_spec.rb

@@ -13,11 +13,13 @@ describe "Boolean.to_store" do
     Boolean.to_store('true').should be_true
     Boolean.to_store('t').should be_true
     Boolean.to_store('1').should be_true
+    Boolean.to_store('on').should be_true
     Boolean.to_store(1).should be_true
 
     Boolean.to_store('false').should be_false
     Boolean.to_store('f').should be_false
     Boolean.to_store('0').should be_false
+    Boolean.to_store('off').should be_false
     Boolean.to_store(0).should be_false
   end
 

data/spec/toy/identity/uuid_key_factory_spec.rb

@@ -3,11 +3,11 @@ require 'helper'
 describe Toy::Identity::UUIDKeyFactory do
   uses_constants('User')
 
-  it "should use String as store_type" do
+  it "should use String as key_type" do
     Toy::Identity::UUIDKeyFactory.new.key_type.should be(String)
   end
 
-  it "should use uuid for next key" do
+  it "should use uuid for next_key" do
     Toy::Identity::UUIDKeyFactory.new.next_key(nil).length.should == 36
   end
 

data/spec/toy/identity_map_spec.rb

@@ -4,17 +4,12 @@ describe Toy::IdentityMap do
   uses_constants('User', 'Skill')
 
   before do
-    Toy.identity_map.clear
+    Toy::IdentityMap.enabled = true
+    Toy::IdentityMap.clear
   end
 
-  describe ".identity_map" do
-    it "defaults to hash" do
-      User.identity_map.should == {}
-    end
-
-    it "memoizes" do
-      User.identity_map.should equal(User.identity_map)
-    end
+  after do
+    Toy::IdentityMap.enabled = false
   end
 
   it "adds to map on save" do
@@ -45,7 +40,7 @@ describe Toy::IdentityMap do
   describe ".get" do
     it "adds to map if not in map" do
       user = User.create
-      user.identity_map.clear
+      Toy::IdentityMap.clear
       user.should_not be_in_identity_map
       user = User.get(user.id)
       user.should be_in_identity_map
@@ -60,7 +55,7 @@ describe Toy::IdentityMap do
     it "does not query if in map" do
       user = User.create
       user.should be_in_identity_map
-      user.store.should_not_receive(:read)
+      user.adapter.should_not_receive(:read)
       User.get(user.id).should equal(user)
     end
   end
@@ -69,21 +64,21 @@ describe Toy::IdentityMap do
     it "forces new query each time and skips the identity map" do
       user = User.create
       user.should be_in_identity_map
-      User.store.should_receive(:read).with(user.id).and_return({})
+      User.adapter.should_receive(:read).with(user.id).and_return({})
       user.reload
     end
   end
 
   describe "identity map off" do
     it "does not add to map on save" do
-      User.identity_map_off
+      Toy::IdentityMap.enabled = false
       user = User.new
       user.save!
       user.should_not be_in_identity_map
     end
 
     it "does not add to map on load" do
-      User.identity_map_off
+      Toy::IdentityMap.enabled = false
       user = User.load('1', 'name' => 'John')
       user.should_not be_in_identity_map
     end
@@ -91,7 +86,7 @@ describe Toy::IdentityMap do
     it "does not remove from map on delete" do
       user = User.create
       user.should be_in_identity_map
-      User.identity_map_off
+      Toy::IdentityMap.enabled = false
       user.delete
       user.should be_in_identity_map
     end
@@ -99,14 +94,14 @@ describe Toy::IdentityMap do
     it "does not remove from map on destroy" do
       user = User.create
       user.should be_in_identity_map
-      User.identity_map_off
+      Toy::IdentityMap.enabled = false
       user.destroy
       user.should be_in_identity_map
     end
 
     describe ".get" do
       it "does not add to map if not in map" do
-        User.identity_map_off
+        Toy::IdentityMap.enabled = false
         user = User.create
         user.should_not be_in_identity_map
         user = User.get(user.id)
@@ -116,35 +111,48 @@ describe Toy::IdentityMap do
       it "does not load from map if in map" do
         user = User.create
         user.should be_in_identity_map
-        User.identity_map_off
-        user.store.should_receive(:read).with(user.id).and_return(user.persisted_attributes)
+        Toy::IdentityMap.enabled = false
+        user.adapter.should_receive(:read).with(user.id).and_return(user.persisted_attributes)
         User.get(user.id)
       end
     end
   end
 
-  describe ".without_identity_map" do
-    describe "with identity map off" do
-      it "turns identity map off, yields, and returns it to previous state" do
-        User.identity_map_off
-        User.should be_identity_map_off
-        User.without_identity_map do
-          user = User.create
-          user.should_not be_in_identity_map
-        end
-        User.should be_identity_map_off
+  describe ".without" do
+    before do
+      @user = User.create
+      Toy::IdentityMap.clear
+    end
+    let(:user) { @user }
+
+    it "skips the map" do
+      Toy::IdentityMap.without do
+        User.get(user.id).should_not equal(user.id)
+      end
+    end
+  end
+
+  describe ".use" do
+    before do
+      @user = User.create
+      Toy::IdentityMap.clear
+    end
+    let(:user) { @user }
+
+    it "uses the map" do
+      Toy::IdentityMap.enabled = false
+      Toy::IdentityMap.use do
+        User.get(user.id).should equal(User.get(user.id))
       end
     end
 
-    describe "with identity map on" do
-      it "turns identity map off, yields, and returns it to previous state" do
-        User.should be_identity_map_on
-        User.without_identity_map do
-          user = User.create
-          user.should_not be_in_identity_map
-        end
-        User.should be_identity_map_on
+    it "clears the map" do
+      Toy::IdentityMap.enabled = false
+      Toy::IdentityMap.repository['hello'] = 'world'
+      Toy::IdentityMap.use do
+        User.get(user.id)
       end
+      Toy::IdentityMap.repository.should be_empty
     end
   end
 end