mongoid 3.1.6 → 4.0.2

data/spec/mongoid/relations/touchable_spec.rb

@@ -0,0 +1,333 @@
+require "spec_helper"
+
+describe Mongoid::Relations::Touchable do
+
+  describe "#touch" do
+
+    context "when the document is embedded" do
+
+      let(:band) do
+        Band.create(name: "Placebo")
+      end
+
+      let(:label) do
+        band.create_label(name: "Mute", updated_at: 10.days.ago)
+      end
+
+      before do
+        label.touch
+      end
+
+      it "updates the updated_at timestamp" do
+        expect(label.updated_at).to be_within(1).of(Time.now)
+      end
+
+      it "persists the changes" do
+        expect(label.reload.updated_at).to be_within(1).of(Time.now)
+      end
+    end
+
+    context "when no relations have touch options" do
+
+      context "when no updated at is defined" do
+
+        let(:person) do
+          Person.create
+        end
+
+        context "when no attribute is provided" do
+
+          let!(:touched) do
+            person.touch
+          end
+
+          it "returns true" do
+            expect(touched).to be true
+          end
+
+          it "does not set the updated at field" do
+            expect(person[:updated_at]).to be_nil
+          end
+        end
+
+        context "when an attribute is provided" do
+
+          let!(:touched) do
+            person.touch(:lunch_time)
+          end
+
+          it "sets the attribute to the current time" do
+            expect(person.lunch_time).to be_within(5).of(Time.now)
+          end
+
+          it "persists the change" do
+            expect(person.reload.lunch_time).to be_within(5).of(Time.now)
+          end
+
+          it "returns true" do
+            expect(touched).to be true
+          end
+        end
+
+        context "when an attribute alias is provided" do
+
+          let!(:touched) do
+            person.touch(:aliased_timestamp)
+          end
+
+          it "sets the attribute to the current time" do
+            expect(person.aliased_timestamp).to be_within(5).of(Time.now)
+          end
+
+          it "persists the change" do
+            expect(person.reload.aliased_timestamp).to be_within(5).of(Time.now)
+          end
+
+          it "returns true" do
+            expect(touched).to be true
+          end
+        end
+      end
+
+      context "when an updated at is defined" do
+
+        let!(:agent) do
+          Agent.create(updated_at: 2.days.ago)
+        end
+
+        context "when no attribute is provided" do
+
+          let!(:touched) do
+            agent.touch
+          end
+
+          it "sets the updated at to the current time" do
+            expect(agent.updated_at).to be_within(5).of(Time.now)
+          end
+
+          it "persists the change" do
+            expect(agent.reload.updated_at).to be_within(5).of(Time.now)
+          end
+
+          it "returns true" do
+            expect(touched).to be true
+          end
+
+          it "keeps changes for next callback" do
+            expect(agent.changes).to_not be_empty
+          end
+        end
+
+        context "when an attribute is provided" do
+
+          let!(:touched) do
+            agent.touch(:dob)
+          end
+
+          it "sets the updated at to the current time" do
+            expect(agent.updated_at).to be_within(5).of(Time.now)
+          end
+
+          it "sets the attribute to the current time" do
+            expect(agent.dob).to be_within(5).of(Time.now)
+          end
+
+          it "sets both attributes to the exact same time" do
+            expect(agent.updated_at).to eq(agent.dob)
+          end
+
+          it "persists the updated at change" do
+            expect(agent.reload.updated_at).to be_within(5).of(Time.now)
+          end
+
+          it "persists the attribute change" do
+            expect(agent.reload.dob).to be_within(5).of(Time.now)
+          end
+
+          it "returns true" do
+            expect(touched).to be true
+          end
+
+          it "keeps changes for next callback" do
+            expect(agent.changes).to_not be_empty
+          end
+        end
+      end
+
+      context "when record is new" do
+
+        let(:agent) do
+          Agent.new(updated_at: 2.days.ago)
+        end
+
+        context "when no attribute is provided" do
+
+          let!(:touched) do
+            agent.touch
+          end
+
+          it "returns false" do
+            expect(touched).to be false
+          end
+        end
+
+        context "when an attribute is provided" do
+
+          let!(:touched) do
+            agent.touch(:dob)
+          end
+
+          it "returns false" do
+            expect(touched).to be false
+          end
+        end
+      end
+
+      context "when creating the child" do
+
+        let(:time) do
+          Time.utc(2012, 4, 3, 12)
+        end
+
+        let(:jar) do
+          Jar.new(_id: 1, updated_at: time).tap do |jar|
+            jar.save!
+          end
+        end
+
+        let!(:cookie) do
+          jar.cookies.create!(updated_at: time)
+        end
+
+        it "does not touch the parent" do
+          expect(jar.updated_at).to eq(time)
+        end
+      end
+    end
+
+    context "when relations have touch options" do
+
+      context "when the relation is a parent of an embedded doc" do
+
+        let(:page) do
+          WikiPage.create(title: "test")
+        end
+
+        let!(:edit) do
+          page.edits.create
+        end
+
+        before do
+          page.unset(:updated_at)
+          edit.touch
+        end
+
+        it "touches the parent document" do
+          expect(page.updated_at).to be_within(5).of(Time.now)
+        end
+      end
+
+      context "when the relation is nil" do
+
+        let!(:agent) do
+          Agent.create
+        end
+
+        context "when the relation autobuilds" do
+
+          let!(:touched) do
+            agent.touch
+          end
+
+          it "does nothing to the relation" do
+            expect(agent.instance_variable_get(:@agency)).to be_nil
+          end
+        end
+      end
+
+      context "when the relation is not nil" do
+
+        let!(:agent) do
+          Agent.create
+        end
+
+        let!(:agency) do
+          agent.create_agency.tap do |a|
+            a.unset(:updated_at)
+          end
+        end
+
+        let!(:touched) do
+          agent.touch
+        end
+
+        it "sets the parent updated at to the current time" do
+          expect(agency.updated_at).to be_within(5).of(Time.now)
+        end
+
+        it "persists the change" do
+          expect(agency.reload.updated_at).to be_within(5).of(Time.now)
+        end
+      end
+
+      context "when creating the child" do
+
+        let!(:agency) do
+          Agency.create
+        end
+
+        let!(:updated) do
+          agency.updated_at
+        end
+
+        let!(:agent) do
+          agency.agents.create
+        end
+
+        it "updates the parent's updated at" do
+          expect(agency.updated_at).to_not eq(updated)
+        end
+      end
+
+      context "when modifying the child" do
+
+        let!(:agency) do
+          Agency.create
+        end
+
+        let!(:agent) do
+          agency.agents.create(number: '1')
+        end
+
+        it "updates the parent's updated at" do
+          expect {
+            agent.update_attributes(number: '2')
+          }.to change { agency.updated_at }
+        end
+      end
+
+      context "when destroying the child" do
+
+        let!(:agency) do
+          Agency.create
+        end
+
+        let!(:agent) do
+          agency.agents.create
+        end
+
+        let!(:updated) do
+          agency.updated_at
+        end
+
+        before do
+          agent.destroy
+        end
+
+        it "updates the parent's updated at" do
+          expect(agency.updated_at).to_not eq(updated)
+        end
+      end
+    end
+  end
+end

data/spec/mongoid/relations_spec.rb

@@ -5,9 +5,10 @@ describe Mongoid::Relations do
   before(:all) do
     Person.field(
       :_id,
-      type: Moped::BSON::ObjectId,
+      type: BSON::ObjectId,
       pre_processed: true,
-      default: ->{ Moped::BSON::ObjectId.new }
+      default: ->{ BSON::ObjectId.new },
+      overwrite: true
     )
   end
 
@@ -28,7 +29,7 @@ describe Mongoid::Relations do
       end
 
       it "returns true" do
-        document.should be_embedded
+        expect(document).to be_embedded
       end
     end
 
@@ -37,14 +38,14 @@ describe Mongoid::Relations do
       context "when the document is embedded in" do
 
         it "returns true" do
-          document.should be_embedded
+          expect(document).to be_embedded
         end
       end
 
       context "when the document class is not embedded in" do
 
         it "returns false" do
-          person.should_not be_embedded
+          expect(person).to_not be_embedded
         end
       end
     end
@@ -54,7 +55,7 @@ describe Mongoid::Relations do
       context "when the document has no parent" do
 
         it "returns false" do
-          Item.should_not be_embedded
+          expect(Item).to_not be_embedded
         end
       end
     end
@@ -64,7 +65,7 @@ describe Mongoid::Relations do
       context "when the document has a parent" do
 
         it "returns true" do
-          SubItem.should be_embedded
+          expect(SubItem).to be_embedded
         end
       end
     end
@@ -83,7 +84,7 @@ describe Mongoid::Relations do
       end
 
       it "returns true" do
-        address.should be_an_embedded_many
+        expect(address).to be_an_embedded_many
       end
     end
 
@@ -94,7 +95,7 @@ describe Mongoid::Relations do
       end
 
       it "returns false" do
-        name.should_not be_an_embedded_many
+        expect(name).to_not be_an_embedded_many
       end
     end
   end
@@ -112,7 +113,7 @@ describe Mongoid::Relations do
       end
 
       it "returns true" do
-        name.should be_an_embedded_one
+        expect(name).to be_an_embedded_one
       end
     end
 
@@ -123,7 +124,7 @@ describe Mongoid::Relations do
       end
 
       it "returns false" do
-        address.should_not be_an_embedded_one
+        expect(address).to_not be_an_embedded_one
       end
     end
   end
@@ -141,7 +142,7 @@ describe Mongoid::Relations do
       end
 
       it "returns true" do
-        post.should be_a_referenced_many
+        expect(post).to be_a_referenced_many
       end
     end
 
@@ -152,7 +153,7 @@ describe Mongoid::Relations do
       end
 
       it "returns false" do
-        game.should_not be_a_referenced_many
+        expect(game).to_not be_a_referenced_many
       end
     end
   end
@@ -170,7 +171,7 @@ describe Mongoid::Relations do
       end
 
       it "returns true" do
-        game.should be_a_referenced_one
+        expect(game).to be_a_referenced_one
       end
     end
 
@@ -181,7 +182,7 @@ describe Mongoid::Relations do
       end
 
       it "returns false" do
-        post.should_not be_a_referenced_one
+        expect(post).to_not be_a_referenced_one
       end
     end
   end