mongoid 3.1.4 → 3.1.5

data/spec/mongoid/persistence/atomic/sets_spec.rb

@@ -0,0 +1,154 @@
+require "spec_helper"
+
+describe Mongoid::Persistence::Atomic::Sets do
+
+  describe "#set" do
+
+    let(:person) do
+      Person.create(age: 100, pets: true)
+    end
+
+    let(:reloaded) do
+      person.reload
+    end
+
+    context "when setting a field on an embedded document" do
+
+      let(:address) do
+        person.addresses.create(street: "Tauentzienstr", number: 5)
+      end
+
+      let!(:set) do
+        address.set(:number, 5)
+      end
+
+      it "sets the provided value" do
+        set.should eq(5)
+      end
+
+      it "persists the change" do
+        reloaded.addresses.first.number.should eq(5)
+      end
+    end
+
+    context "when setting a field with a value" do
+
+      let!(:set) do
+        person.set(:age, 2)
+      end
+
+      it "sets the provided value" do
+        person.age.should eq(2)
+      end
+
+      it "returns the new value" do
+        set.should eq(2)
+      end
+
+      it "persists the changes" do
+        reloaded.age.should eq(2)
+      end
+
+      it "resets the dirty attributes" do
+        person.changes["age"].should be_nil
+      end
+    end
+
+    context "when setting a field with a value that must be cast" do
+
+      let(:date_time) do
+        DateTime.new(2012, 1, 2)
+      end
+
+      let!(:set) do
+        person.set(:lunch_time, date_time)
+      end
+
+      it "sets the provided value" do
+        person.lunch_time.should eq(date_time)
+      end
+
+      it "returns the new value" do
+        set.should eq(date_time)
+      end
+
+      it "persists the changes" do
+        reloaded.lunch_time.should eq(date_time)
+      end
+
+      it "resets the dirty attributes" do
+        person.changes["lunch_time"].should be_nil
+      end
+    end
+
+    context "when setting a field to false" do
+
+      let!(:set) do
+        person.set(:pets, false)
+      end
+
+      it "sets the provided value" do
+        person.pets.should be_false
+      end
+
+      it "returns the new value" do
+        set.should be_false
+      end
+
+      it "persists the changes" do
+        reloaded.pets.should be_false
+      end
+
+      it "resets the dirty attributes" do
+        person.changes["pets"].should be_nil
+      end
+
+    end
+
+    context "when setting a nil field" do
+
+      let!(:set) do
+        person.set(:score, 2)
+      end
+
+      it "sets the value to the provided number" do
+        person.score.should eq(2)
+      end
+
+      it "returns the new value" do
+        set.should eq(2)
+      end
+
+      it "persists the changes" do
+        reloaded.score.should eq(2)
+      end
+
+      it "resets the dirty attributes" do
+        person.changes["score"].should be_nil
+      end
+    end
+
+    context "when setting a non existant field" do
+
+      let!(:set) do
+        person.set(:high_score, 5)
+      end
+
+      it "sets the value to the provided number" do
+        person.high_score.should eq(5)
+      end
+
+      it "returns the new value" do
+        set.should eq(5)
+      end
+
+      it "persists the changes" do
+        reloaded.high_score.should eq(5)
+      end
+
+      it "resets the dirty attributes" do
+        person.changes["high_score"].should be_nil
+      end
+    end
+  end
+end

data/spec/mongoid/persistence/atomic/unset_spec.rb

@@ -0,0 +1,65 @@
+require "spec_helper"
+
+describe Mongoid::Persistence::Atomic::Unset do
+
+  describe "#persist" do
+
+    context "when unsetting a field" do
+
+      let(:person) do
+        Person.create(age: 100)
+      end
+
+      let!(:removed) do
+        person.unset(:age)
+      end
+
+      it "removes the field" do
+        person.age.should be_nil
+      end
+
+      it "resets the dirty attributes" do
+        person.changes["age"].should be_nil
+      end
+
+      it "returns nil" do
+        removed.should be_nil
+      end
+    end
+
+
+    [[ :age, :score, { safe: true }], [ :age, :score ], [ [:age, :score ]]].each do |args|
+
+      context "when unsetting multiple fields using #{args}" do
+
+        let(:person) do
+          Person.create(age: 100, score: 2)
+        end
+
+        let!(:removed) do
+          person.unset *(args)
+        end
+
+        it "removes age field" do
+          person.age.should be_nil
+        end
+
+        it "removes score field" do
+          person.score.should be_nil
+        end
+
+        it "resets the age dirty attribute" do
+          person.changes["age"].should be_nil
+        end
+
+        it "resets the score dirty attribute" do
+          person.changes["score"].should be_nil
+        end
+
+        it "returns nil" do
+          removed.should be_nil
+        end
+      end
+    end
+  end
+end

data/spec/mongoid/persistence/atomic_spec.rb

@@ -0,0 +1,216 @@
+require "spec_helper"
+
+describe Mongoid::Persistence::Atomic do
+
+  context "when using aliased field names" do
+
+    describe "#add_to_set" do
+
+      let(:person) do
+        Person.create(array: [ "test" ])
+      end
+
+      before do
+        person.add_to_set(:array, "testy")
+      end
+
+      it "adds to the aliased field" do
+        person.array.should eq([ "test", "testy" ])
+      end
+
+      it "persists the change" do
+        person.reload.array.should eq([ "test", "testy" ])
+      end
+    end
+
+    describe "#bit" do
+
+      let(:person) do
+        Person.create(inte: 60)
+      end
+
+      before do
+        person.bit(:inte, { and: 13 })
+      end
+
+      it "performs the bitwise operation" do
+        person.inte.should eq(12)
+      end
+
+      it "persists the changes" do
+        person.reload.inte.should eq(12)
+      end
+    end
+
+    describe "#inc" do
+
+      let(:person) do
+        Person.create(inte: 5)
+      end
+
+      before do
+        person.inc(:inte, 1)
+      end
+
+      it "increments the aliased field" do
+        person.inte.should eq(6)
+      end
+
+      it "persists the change" do
+        person.reload.inte.should eq(6)
+      end
+    end
+
+    describe "#pop" do
+
+      let(:person) do
+        Person.create(array: [ "test1", "test2" ])
+      end
+
+      before do
+        person.pop(:array, 1)
+      end
+
+      it "removes from the aliased field" do
+        person.array.should eq([ "test1" ])
+      end
+
+      it "persists the change" do
+        person.reload.array.should eq([ "test1" ])
+      end
+    end
+
+    describe "#pull" do
+
+      let(:person) do
+        Person.create(array: [ "test1", "test2" ])
+      end
+
+      before do
+        person.pull(:array, "test1")
+      end
+
+      it "removes from the aliased field" do
+        person.array.should eq([ "test2" ])
+      end
+
+      it "persists the change" do
+        person.reload.array.should eq([ "test2" ])
+      end
+    end
+
+    describe "#pull_all" do
+
+      let(:person) do
+        Person.create(array: [ "test1", "test2" ])
+      end
+
+      before do
+        person.pull_all(:array, [ "test1", "test2" ])
+      end
+
+      it "removes from the aliased field" do
+        person.array.should be_empty
+      end
+
+      it "persists the change" do
+        person.reload.array.should be_empty
+      end
+    end
+
+    describe "#push" do
+
+      let(:person) do
+        Person.create(array: [ "test" ])
+      end
+
+      before do
+        person.push(:array, "testy")
+      end
+
+      it "adds to the aliased field" do
+        person.array.should eq([ "test", "testy" ])
+      end
+
+      it "persists the change" do
+        person.reload.array.should eq([ "test", "testy" ])
+      end
+    end
+
+    describe "#push_all" do
+
+      let(:person) do
+        Person.create(array: [ "test" ])
+      end
+
+      before do
+        person.push_all(:array, [ "testy", "test2" ])
+      end
+
+      it "adds to the aliased field" do
+        person.array.should eq([ "test", "testy", "test2" ])
+      end
+
+      it "persists the change" do
+        person.reload.array.should eq([ "test", "testy", "test2" ])
+      end
+    end
+
+    describe "#rename" do
+
+      let(:person) do
+        Person.create(inte: 5)
+      end
+
+      before do
+        person.rename(:inte, :integer)
+      end
+
+      it "renames the aliased field" do
+        person.integer.should eq(5)
+      end
+
+      it "persists the change" do
+        person.reload.integer.should eq(5)
+      end
+    end
+
+    describe "#set" do
+
+      let(:person) do
+        Person.create(inte: 5)
+      end
+
+      before do
+        person.set(:inte, 8)
+      end
+
+      it "sets the aliased field" do
+        person.inte.should eq(8)
+      end
+
+      it "persists the change" do
+        person.reload.inte.should eq(8)
+      end
+    end
+
+    describe "#unset" do
+
+      let(:person) do
+        Person.create(inte: 5)
+      end
+
+      before do
+        person.unset(:inte)
+      end
+
+      it "unsets the aliased field" do
+        person.inte.should be_nil
+      end
+
+      it "persists the change" do
+        person.reload.inte.should be_nil
+      end
+    end
+  end
+end

data/spec/mongoid/persistence/operations/embedded/insert_spec.rb

@@ -0,0 +1,191 @@
+require "spec_helper"
+
+describe Mongoid::Persistence::Operations::Embedded::Insert do
+
+  let(:document) do
+    Patient.new(title: "Mr")
+  end
+
+  let(:address) do
+    Address.new(street: "Oxford St")
+  end
+
+  let(:collection) do
+    stub("collection")
+  end
+
+  let(:email) do
+    Email.new(address: "test@example.com")
+  end
+
+  before do
+    document.stub(:collection).and_return(collection)
+  end
+
+  let(:query) do
+    stub
+  end
+
+  before(:all) do
+    Mongoid.identity_map_enabled = true
+  end
+
+  after(:all) do
+    Mongoid.identity_map_enabled = false
+  end
+
+  describe "#persist" do
+
+    context "when the insert succeeded" do
+
+      let(:person) do
+        Person.create
+      end
+
+      let(:address) do
+        person.addresses.create(street: "Hobrechtstr")
+      end
+
+      let(:in_map) do
+        Mongoid::IdentityMap.get(Address, address.id)
+      end
+
+      it "does not put the document in the identity map" do
+        in_map.should be_nil
+      end
+    end
+
+    context "when no parent is set" do
+
+      let(:address) do
+        Address.new
+      end
+
+      let(:insert) do
+        described_class.new(address)
+      end
+
+      it "raises an error" do
+        expect {
+          insert.persist
+        }.to raise_error(Mongoid::Errors::NoParent)
+      end
+    end
+
+    def root_insert_expectation
+      ->{
+        collection.should_receive(:insert).with(
+          document.raw_attributes
+        ).and_return("Object")
+      }
+    end
+
+    def root_push_expectation
+      ->{
+        collection.should_receive(:find).with({ "_id" => document.id }).and_return(query)
+        query.should_receive(:update).with(
+          { "$push" => { "addresses" => address.raw_attributes }}
+        ).and_return("Object")
+      }
+    end
+
+    def root_set_expectation
+      ->{
+        collection.should_receive(:find).with({ "_id" => document.id }).and_return(query)
+        query.should_receive(:update).with(
+          { "$set" => { "email" => email.raw_attributes } }
+        ).and_return("Object")
+      }
+    end
+
+    context "when the embedded document is an embeds_one" do
+
+      before do
+        document.email = email
+      end
+
+      context "when the parent is new" do
+
+        let(:insert) do
+          described_class.new(email)
+        end
+
+        it "notifies its changes to parent and inserts the parent" do
+          root_insert_expectation.call
+          insert.persist.should eq(email)
+        end
+      end
+
+      context "when the parent is not new" do
+
+        let(:insert) do
+          described_class.new(email)
+        end
+
+        before do
+          document.instance_variable_set(:@new_record, false)
+        end
+
+        it "performs an in place $set on the embedded document" do
+          root_set_expectation.call
+          insert.persist.should eq(email)
+        end
+      end
+    end
+
+    context "when the embedded document is an embeds_many" do
+
+      before do
+        document.addresses << address
+      end
+
+      context "when the parent is new" do
+
+        let(:insert) do
+          described_class.new(address)
+        end
+
+        it "notifies its changes to the parent and inserts the parent" do
+          root_insert_expectation.call
+          insert.persist.should eq(address)
+        end
+
+        it "sets new_record to false" do
+          root_insert_expectation.call
+          insert.persist.new_record?.should be_false
+        end
+      end
+
+      context "when the parent is not new" do
+
+        let(:insert) do
+          described_class.new(address)
+        end
+
+        before do
+          document.instance_variable_set(:@new_record, false)
+        end
+
+        it "performs a $push on the embedded array" do
+          root_push_expectation.call
+          insert.persist.should eq(address)
+        end
+
+        context "when we add the parent to the child" do
+
+          let(:other_address) do
+            document.addresses.build(street: "Oxford St")
+          end
+
+          it "performs a $push on the embedded array" do
+            collection.should_receive(:find).with({ "_id" => document.id }).and_return(query)
+            query.should_receive(:update).with(
+              { "$push" => { "addresses" => other_address.raw_attributes } }
+            ).and_return("Object")
+            described_class.new(other_address).persist.should eq(other_address)
+          end
+        end
+      end
+    end
+  end
+end