mongoid_alize 0.1.0 → 0.2.0

data/spec/mongoid/alize/callbacks/to/many_from_one_spec.rb

@@ -6,16 +6,17 @@ describe Mongoid::Alize::Callbacks::To::ManyFromOne do
   end
 
   def args
-    [Person, :seen_by, [:name, :created_at]]
+    [Person, :seen_by, [:name, :location, :created_at]]
   end
 
-  def new_unit
+  def new_callback
     klass.new(*args)
   end
 
   def sees_fields
     { "_id" => @person.id,
       "name"=> "Bob",
+      "location" => "Paris",
       "created_at"=> @now.to_s(:utc) }
   end
 
@@ -32,20 +33,16 @@ describe Mongoid::Alize::Callbacks::To::ManyFromOne do
       :sees => [@person = Person.create(:name => "Bob",
                                       :created_at => @now = Time.now)])
     @person.seen_by = @head
-    @unit = new_unit
+    @callback = new_callback
   end
 
   describe "#define_callback" do
     before do
-      @unit.send(:define_callback)
+      @callback.send(:define_callback)
     end
 
     def run_callback
-      @person.send(callback_name)
-    end
-
-    def callback_name
-      "denormalize_to_seen_by"
+      @person.send(:_denormalize_to_seen_by)
     end
 
     it "should push the fields to the relation" do
@@ -75,15 +72,11 @@ describe Mongoid::Alize::Callbacks::To::ManyFromOne do
 
   describe "#define_destroy_callback" do
     before do
-      @unit.send(:define_destroy_callback)
+      @callback.send(:define_destroy_callback)
     end
 
     def run_destroy_callback
-      @person.send(destroy_callback_name)
-    end
-
-    def destroy_callback_name
-      "denormalize_destroy_to_seen_by"
+      @person.send(:_denormalize_destroy_to_seen_by)
     end
 
     it "should pull first any existing array entries matching the _id" do

data/spec/mongoid/alize/callbacks/to/one_from_many_spec.rb

@@ -6,16 +6,17 @@ describe Mongoid::Alize::Callbacks::To::OneFromMany do
   end
 
   def args
-    [Person, :heads, [:name, :created_at]]
+    [Person, :heads, [:name, :location, :created_at]]
   end
 
-  def new_unit
+  def new_callback
     klass.new(*args)
   end
 
   before do
     Head.class_eval do
       field :captor_name, :type => String
+      field :captor_location, :type => String
       field :captor_created_at, :type => Time
     end
 
@@ -24,49 +25,45 @@ describe Mongoid::Alize::Callbacks::To::OneFromMany do
                                          :created_at => @now = Time.now))
     @person.heads = [@head]
 
-    @unit = new_unit
+    @callback = new_callback
   end
 
   describe "#define_callback" do
     before do
-      @unit.send(:define_callback)
+      @callback.send(:define_callback)
     end
 
     def run_callback
-      @person.send(callback_name)
-    end
-
-    def callback_name
-      "denormalize_to_heads"
+      @person.send(:_denormalize_to_heads)
     end
 
     it "should push the fields to the relation" do
       @head.captor_name.should be_nil
+      @head.captor_location.should be_nil
       @head.captor_created_at.should be_nil
       run_callback
       @head.captor_name.should == "Bob"
+      @head.captor_location.should == "Paris"
       @head.captor_created_at.to_i.should == @now.to_i
     end
   end
 
   describe "#define_destroy_callback" do
     def run_destroy_callback
-      @person.send(destroy_callback_name)
-    end
-
-    def destroy_callback_name
-      "denormalize_destroy_to_heads"
+      @person.send(:_denormalize_destroy_to_heads)
     end
 
     before do
-      @unit.send(:define_destroy_callback)
+      @callback.send(:define_destroy_callback)
     end
 
     it "should remove the fields from the relation" do
-      @head.captor_name.should be_nil
-      @head.captor_created_at.should be_nil
+      @head.captor_name = "Chuck"
+      @head.captor_location = "Paris"
+      @head.captor_created_at = Time.now
       run_destroy_callback
       @head.captor_name.should be_nil
+      @head.captor_location.should be_nil
       @head.captor_created_at.should be_nil
     end
   end

data/spec/mongoid/alize/callbacks/to/one_from_one_spec.rb

@@ -6,16 +6,17 @@ describe Mongoid::Alize::Callbacks::To::OneFromOne do
   end
 
   def args
-    [Person, :head, [:name, :created_at]]
+    [Person, :head, [:name, :location, :created_at]]
   end
 
-  def new_unit
+  def new_callback
     klass.new(*args)
   end
 
   before do
     Head.class_eval do
       field :person_name, :type => String
+      field :person_location, :type => String
       field :person_created_at, :type => Time
     end
 
@@ -23,27 +24,25 @@ describe Mongoid::Alize::Callbacks::To::OneFromOne do
       :person => @person = Person.create(:name => "Bob",
                                       :created_at => @now = Time.now))
 
-    @unit = new_unit
+    @callback = new_callback
   end
 
   describe "define_callback" do
     before do
-      @unit.send(:define_callback)
+      @callback.send(:define_callback)
     end
 
     def run_callback
-      @person.send(callback_name)
-    end
-
-    def callback_name
-      "denormalize_to_head"
+      @person.send(:_denormalize_to_head)
     end
 
     it "should push the fields to the relation" do
       @head.person_name.should be_nil
+      @head.person_location.should be_nil
       @head.person_created_at.should be_nil
       run_callback
       @head.person_name.should == "Bob"
+      @head.person_location.should == "Paris"
       @head.person_created_at.to_i.should == @now.to_i
     end
 
@@ -55,22 +54,20 @@ describe Mongoid::Alize::Callbacks::To::OneFromOne do
 
   describe "define_destroy_callback" do
     before do
-      @unit.send(:define_destroy_callback)
+      @callback.send(:define_destroy_callback)
     end
 
     def run_destroy_callback
-      @person.send(destroy_callback_name)
-    end
-
-    def destroy_callback_name
-      "denormalize_destroy_to_head"
+      @person.send(:_denormalize_destroy_to_head)
     end
 
     it "should nillify the fields in the relation" do
       @head.person_name = "Chuck"
+      @head.person_location = "Paris"
       @head.person_created_at = Time.now
       run_destroy_callback
       @head.person_name.should be_nil
+      @head.person_location.should be_nil
       @head.person_created_at.should be_nil
     end
 

data/spec/mongoid/alize/from_callback_spec.rb

@@ -0,0 +1,44 @@
+require 'spec_helper'
+
+class Mongoid::Alize::SpecFromCallback < Mongoid::Alize::FromCallback
+  def define_fields
+  end
+
+  def define_callback
+    klass.class_eval <<-CALLBACK
+      def _denormalize_from_person
+      end
+    CALLBACK
+  end
+end
+
+describe Mongoid::Alize::FromCallback do
+  def klass
+    Mongoid::Alize::SpecFromCallback
+  end
+
+  def args
+    [Head, :person, [:name, :location, :created_at]]
+  end
+
+  def new_callback
+    klass.new(*args)
+  end
+
+  before do
+    @callback = new_callback
+  end
+
+  describe "#set_callback" do
+    it "should set a callback on the klass" do
+      mock(@callback.klass).set_callback(:save, :before, "denormalize_from_person")
+      @callback.send(:set_callback)
+    end
+
+    it "should not set the callback if it's already set" do
+      @callback.send(:attach)
+      dont_allow(@callback.klass).set_callback
+      @callback.send(:set_callback)
+    end
+  end
+end

data/spec/mongoid/alize/to_callback_spec.rb

@@ -0,0 +1,74 @@
+require 'spec_helper'
+
+class Mongoid::Alize::SpecToCallback < Mongoid::Alize::ToCallback
+  def define_callback
+    klass.class_eval <<-CALLBACK
+      def _denormalize_to_head
+      end
+    CALLBACK
+  end
+
+  def define_destroy_callback
+    klass.class_eval <<-CALLBACK
+      def _denormalize_destroy_to_head
+      end
+    CALLBACK
+  end
+end
+
+describe Mongoid::Alize::ToCallback do
+  def klass
+    Mongoid::Alize::SpecToCallback
+  end
+
+  def args
+    [Person, :head, [:name, :location, :created_at]]
+  end
+
+  def new_callback
+    klass.new(*args)
+  end
+
+  before do
+    @callback = new_callback
+  end
+
+  describe "#set_callback" do
+    it "should set a callback on the klass" do
+      mock(@callback.klass).set_callback(:save, :after, "denormalize_to_head")
+      @callback.send(:set_callback)
+    end
+
+    it "should not set the callback if it's already set" do
+      @callback.send(:attach)
+      dont_allow(@callback.klass).set_callback
+      @callback.send(:set_callback)
+    end
+  end
+
+  describe "#set_destroy_callback" do
+    it "should set a destroy callback on the klass" do
+      mock(@callback.klass).set_callback(:destroy, :after, "denormalize_destroy_to_head")
+      @callback.send(:set_destroy_callback)
+    end
+
+    it "should not set the destroy callback if it's already set" do
+      @callback.send(:attach)
+      dont_allow(@callback.klass).set_callback
+      @callback.send(:set_destroy_callback)
+    end
+  end
+
+  describe "#alias_destroy_callback" do
+    it "should alias the destroy callback on the klass" do
+      mock(@callback.klass).alias_method("denormalize_destroy_to_head", "_denormalize_destroy_to_head")
+      @callback.send(:alias_destroy_callback)
+    end
+
+    it "should not alias the destroy callback if it's already set" do
+      @callback.send(:attach)
+      dont_allow(@callback.klass).alias_method
+      @callback.send(:alias_destroy_callback)
+    end
+  end
+end

data/spec/mongoid_alize_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe Mongoid::Alize do
   def person_fields
-    [:name]
+    [:name, :location]
   end
 
   def head_fields
@@ -10,8 +10,10 @@ describe Mongoid::Alize do
   end
 
   before do
-    @head = Head.new(:size => @size = 10)
-    @person = Person.new(:name => @name = "Bob")
+    @now = Time.now
+    stub(Time).now { @now }
+    @head = Head.new(:size => @size = 10, created_at: @now)
+    @person = Person.new(:name => @name = "Bob", created_at: @now)
   end
 
   describe "one-to-one" do
@@ -23,14 +25,29 @@ describe Mongoid::Alize do
 
       def assert_head
         @head.person_name.should == @name
+        @head.person_location.should == "Paris"
       end
 
-      it "should pull attributes from person" do
+      it "should pull data from person on create" do
         @head.save!
         assert_head
       end
 
-      it "should push attributes to head" do
+      it "should pull data from a changed person on save" do
+        @head.save!
+        @head.person = Person.create(:name => @name = "Bill")
+        @head.save!
+        assert_head
+      end
+
+      it "should not pull data from an unchanged person on save" do
+        @head.save!
+        @head.person_name = "Cowboy"
+        @head.save!
+        @head.person_name.should == "Cowboy"
+      end
+
+      it "should push data to head" do
         @person.update_attributes!(:name => @name = "Bill")
         assert_head
       end
@@ -39,6 +56,7 @@ describe Mongoid::Alize do
         @head.update_attributes!(:person_name => "Old Gregg")
         @person.destroy
         @head.person_name.should be_nil
+        @head.person_location.should be_nil
       end
     end
 
@@ -52,12 +70,19 @@ describe Mongoid::Alize do
         @person.head_size.should == @size
       end
 
-      it "should pull attributes from head" do
+      it "should pull data from head on create" do
         @person.save!
         assert_person
       end
 
-      it "should push attributes to person" do
+      it "should pull data from head on save" do
+        @person.save!
+        @person.head = Head.create(:size => @size = 18)
+        @person.save!
+        assert_person
+      end
+
+      it "should push data to person" do
         @head.update_attributes!(:size => @size = "20lbs")
         assert_person
       end
@@ -81,17 +106,24 @@ describe Mongoid::Alize do
         @head.captor_name.should == @name
       end
 
-      it "should pull attributes from head" do
+      it "should pull data from captor on create" do
         @head.save!
         assert_captor
       end
 
-      it "should push attributes to person" do
+      it "should pull data from captor on save" do
+        @head.save!
+        @head.captor = Person.create(:name => @name = "Bill")
+        @head.save!
+        assert_captor
+      end
+
+      it "should push data to person" do
         @person.update_attributes!(:name => @name = "Bill")
         assert_captor
       end
 
-      it "should nillify captor fieds when person is destroyed" do
+      it "should nillify captor fields when person is destroyed" do
         @head.update_attributes!(:captor_name => "Old Gregg")
         @person.destroy
         @head.captor_name.should be_nil
@@ -107,15 +139,23 @@ describe Mongoid::Alize do
       def assert_sees
         @head.sees_fields.should == [{
           "_id" => @person.id,
+          "location" => "Paris",
           "name" => @name }]
       end
 
-      it "should pull attributes from sees" do
+      it "should pull data from sees on create" do
+        @head.save!
+        assert_sees
+      end
+
+      it "should pull data from a sees on save" do
+        @head.save!
+        @head.sees = [@person = Person.create(:name => @name = "Bill")]
         @head.save!
         assert_sees
       end
 
-      it "should push attributes to seen_by" do
+      it "should push data to seen_by" do
         @person.update_attributes!(:name => @name = "Bill")
         assert_sees
       end
@@ -134,33 +174,129 @@ describe Mongoid::Alize do
     describe "has_and_belongs_to_many" do
       before do
         Head.send(:alize, :wanted_by, *person_fields)
-        @head.wanted_by = [@person]
-        @person.wants = [@head]
       end
 
       def assert_wanted_by
         @head.wanted_by_fields.should == [{
           "_id" => @person.id,
+          "location" => "Paris",
           "name" => @name }]
       end
 
-      it "should pull attributes from wanted_by" do
+      it "should pull data from wanted_by on create" do
+        @head.wanted_by = [@person]
+        @head.save!
+        assert_wanted_by
+      end
+
+      it "should pull data from wanted_by on save" do
+        @head.save!
+        @person = Person.create(:name => @name = "Bill")
+        @head.wanted_by = [@person]
         @head.save!
         assert_wanted_by
       end
 
-      it "should push attributes to wanted_by" do
+      it "should push data to wants" do
+        @person.wants = [@head]
         @person.update_attributes!(:name => @name = "Bill")
         assert_wanted_by
       end
 
       it "should remove wanted_by_fields entries in head when person is destroyed" do
+        @head.wanted_by = [@person]
         @head.save!
         assert_wanted_by
         @person.destroy
+        @head.reload
         @head.wanted_by_fields.should == []
         @head.reload.wanted_by_fields.should == []
       end
     end
   end
+
+  describe "without specifying fields" do
+    before do
+      Head.send(:alize, :person)
+      @head.person = @person
+    end
+
+    it "should denormalize all non-internal fields" do
+      @head.save!
+      @head.person_name.should == @name
+      @head.person_created_at.to_i.should == @now.to_i
+    end
+  end
+
+  describe "overriding denormalize methods for custom behavior on the from side" do
+    before do
+      class Person
+        def denormalize_update_name_first
+          self.name = "Overrider"
+        end
+      end
+
+      class Head
+        def denormalize_from_person
+          self.person.denormalize_update_name_first
+          _denormalize_from_person
+        end
+      end
+
+      Head.send(:alize, :person, *person_fields)
+      @head.person = @person
+    end
+
+    it "should be possible to define a method before alize and call the alize version within it" do
+      @head.save!
+      @head.person_name.should == "Overrider"
+    end
+  end
+
+  describe "overriding denormalize methods for custom behavior on the to side" do
+    before do
+      class Person
+        def denormalize_to_head
+          self.name = "Overrider"
+          _denormalize_to_head
+        end
+      end
+
+      Head.send(:alize, :person, *person_fields)
+      @head.person = @person
+      @head.save!
+    end
+
+    it "should be possible to define a method before alize and call the alize version within it" do
+      @person.update_attributes!(:name => @name = "Bill")
+      @head.person_name.should == "Overrider"
+    end
+  end
+
+  describe "forcing denormalization" do
+    before do
+      Head.send(:alize, :person, *person_fields)
+      @head.person = @person
+      @head.save!
+    end
+
+    it "should allow using the force flag to force denormalization" do
+      class Head
+        def denormalize_from_person
+          _denormalize_from_person(true)
+        end
+      end
+
+      @head.person_name = "Misty"
+      @head.save!
+      @head.person_name.should == @name
+    end
+
+    it "should allow using the force_denormalization attribute to force denormalization" do
+      @head.person_name = "Misty"
+      @head.force_denormalization = true
+      @head.save!
+      @head.person_name.should == @name
+    end
+  end
 end