mongoid-history 0.8.3 → 0.8.5

data/spec/integration/nested_embedded_polymorphic_documents_spec.rb

@@ -1,115 +1,115 @@
-require 'spec_helper'
-
-describe Mongoid::History::Tracker do
-  before :each do
-    class ModelOne
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-
-      field :name, type: String
-      belongs_to :user
-      embeds_one :one_embedded, as: :embedable
-
-      track_history
-    end
-
-    class ModelTwo
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-
-      field :name, type: String
-      belongs_to :user
-      embeds_one :one_embedded, as: :embedable
-
-      track_history
-    end
-
-    class OneEmbedded
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-
-      field :name
-      embeds_many :embedded_twos, store_as: :ems
-      embedded_in :embedable, polymorphic: true
-
-      track_history scope: %i[model_one model_two]
-    end
-
-    class EmbeddedTwo
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-
-      field :name
-      embedded_in :one_embedded
-
-      track_history scope: %i[model_one model_two]
-    end
-
-    class User
-      include Mongoid::Document
-
-      has_many :model_ones
-      has_many :model_twos
-    end
-  end
-
-  after :each do
-    Object.send(:remove_const, :ModelOne)
-    Object.send(:remove_const, :ModelTwo)
-    Object.send(:remove_const, :OneEmbedded)
-    Object.send(:remove_const, :EmbeddedTwo)
-    Object.send(:remove_const, :User)
-  end
-
-  let (:user) { User.create! }
-
-  it 'tracks history for nested embedded documents with polymorphic relations' do
-    user = User.create!
-
-    model_one = user.model_ones.build(name: 'model_one', modifier: user)
-    model_one.save!
-    model_one.build_one_embedded(name: 'model_one_one_embedded', modifier: user).save!
-    expect(model_one.history_tracks.count).to eq(2)
-    expect(model_one.one_embedded.history_tracks.count).to eq(1)
-
-    model_one.reload
-    model_one.one_embedded.update_attributes!(name: 'model_one_embedded_one!')
-    expect(model_one.history_tracks.count).to eq(3)
-    expect(model_one.one_embedded.history_tracks.count).to eq(2)
-    expect(model_one.history_tracks.last.action).to eq('update')
-
-    model_one.build_one_embedded(name: 'Lorem ipsum', modifier: user).save!
-    expect(model_one.history_tracks.count).to eq(4)
-    expect(model_one.one_embedded.history_tracks.count).to eq(1)
-    expect(model_one.one_embedded.history_tracks.last.action).to eq('create')
-    expect(model_one.one_embedded.history_tracks.last.association_chain.last['name']).to eq('one_embedded')
-
-    embedded_one1 = model_one.one_embedded.embedded_twos.create!(name: 'model_one_one_embedded_1', modifier: user)
-    expect(model_one.history_tracks.count).to eq(5)
-    expect(model_one.one_embedded.history_tracks.count).to eq(2)
-    expect(embedded_one1.history_tracks.count).to eq(1)
-
-    model_two = user.model_twos.build(name: 'model_two', modifier: user)
-    model_two.save!
-    model_two.build_one_embedded(name: 'model_two_one_embedded', modifier: user).save!
-    expect(model_two.history_tracks.count).to eq(2)
-    expect(model_two.one_embedded.history_tracks.count).to eq(1)
-
-    model_two.reload
-    model_two.one_embedded.update_attributes!(name: 'model_two_one_embedded!')
-    expect(model_two.history_tracks.count).to eq(3)
-    expect(model_two.one_embedded.history_tracks.count).to eq(2)
-    expect(model_two.history_tracks.last.action).to eq('update')
-
-    model_two.build_one_embedded(name: 'Lorem ipsum', modifier: user).save!
-    expect(model_two.history_tracks.count).to eq(4)
-    expect(model_two.one_embedded.history_tracks.count).to eq(1)
-    expect(model_two.one_embedded.history_tracks.last.action).to eq('create')
-    expect(model_two.one_embedded.history_tracks.last.association_chain.last['name']).to eq('one_embedded')
-
-    embedded_one2 = model_two.one_embedded.embedded_twos.create!(name: 'model_two_one_embedded_1', modifier: user)
-    expect(model_two.history_tracks.count).to eq(5)
-    expect(model_two.one_embedded.history_tracks.count).to eq(2)
-    expect(embedded_one2.history_tracks.count).to eq(1)
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  before :each do
+    class ModelOne
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name, type: String
+      belongs_to :user
+      embeds_one :one_embedded, as: :embedable
+
+      track_history
+    end
+
+    class ModelTwo
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name, type: String
+      belongs_to :user
+      embeds_one :one_embedded, as: :embedable
+
+      track_history
+    end
+
+    class OneEmbedded
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name
+      embeds_many :embedded_twos, store_as: :ems
+      embedded_in :embedable, polymorphic: true
+
+      track_history scope: %i[model_one model_two]
+    end
+
+    class EmbeddedTwo
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name
+      embedded_in :one_embedded
+
+      track_history scope: %i[model_one model_two]
+    end
+
+    class User
+      include Mongoid::Document
+
+      has_many :model_ones
+      has_many :model_twos
+    end
+  end
+
+  after :each do
+    Object.send(:remove_const, :ModelOne)
+    Object.send(:remove_const, :ModelTwo)
+    Object.send(:remove_const, :OneEmbedded)
+    Object.send(:remove_const, :EmbeddedTwo)
+    Object.send(:remove_const, :User)
+  end
+
+  let (:user) { User.create! }
+
+  it 'tracks history for nested embedded documents with polymorphic relations' do
+    user = User.create!
+
+    model_one = user.model_ones.build(name: 'model_one', modifier: user)
+    model_one.save!
+    model_one.build_one_embedded(name: 'model_one_one_embedded', modifier: user).save!
+    expect(model_one.history_tracks.count).to eq(2)
+    expect(model_one.one_embedded.history_tracks.count).to eq(1)
+
+    model_one.reload
+    model_one.one_embedded.update_attributes!(name: 'model_one_embedded_one!')
+    expect(model_one.history_tracks.count).to eq(3)
+    expect(model_one.one_embedded.history_tracks.count).to eq(2)
+    expect(model_one.history_tracks.last.action).to eq('update')
+
+    model_one.build_one_embedded(name: 'Lorem ipsum', modifier: user).save!
+    expect(model_one.history_tracks.count).to eq(4)
+    expect(model_one.one_embedded.history_tracks.count).to eq(1)
+    expect(model_one.one_embedded.history_tracks.last.action).to eq('create')
+    expect(model_one.one_embedded.history_tracks.last.association_chain.last['name']).to eq('one_embedded')
+
+    embedded_one1 = model_one.one_embedded.embedded_twos.create!(name: 'model_one_one_embedded_1', modifier: user)
+    expect(model_one.history_tracks.count).to eq(5)
+    expect(model_one.one_embedded.history_tracks.count).to eq(2)
+    expect(embedded_one1.history_tracks.count).to eq(1)
+
+    model_two = user.model_twos.build(name: 'model_two', modifier: user)
+    model_two.save!
+    model_two.build_one_embedded(name: 'model_two_one_embedded', modifier: user).save!
+    expect(model_two.history_tracks.count).to eq(2)
+    expect(model_two.one_embedded.history_tracks.count).to eq(1)
+
+    model_two.reload
+    model_two.one_embedded.update_attributes!(name: 'model_two_one_embedded!')
+    expect(model_two.history_tracks.count).to eq(3)
+    expect(model_two.one_embedded.history_tracks.count).to eq(2)
+    expect(model_two.history_tracks.last.action).to eq('update')
+
+    model_two.build_one_embedded(name: 'Lorem ipsum', modifier: user).save!
+    expect(model_two.history_tracks.count).to eq(4)
+    expect(model_two.one_embedded.history_tracks.count).to eq(1)
+    expect(model_two.one_embedded.history_tracks.last.action).to eq('create')
+    expect(model_two.one_embedded.history_tracks.last.association_chain.last['name']).to eq('one_embedded')
+
+    embedded_one2 = model_two.one_embedded.embedded_twos.create!(name: 'model_two_one_embedded_1', modifier: user)
+    expect(model_two.history_tracks.count).to eq(5)
+    expect(model_two.one_embedded.history_tracks.count).to eq(2)
+    expect(embedded_one2.history_tracks.count).to eq(1)
+  end
+end

data/spec/integration/subclasses_spec.rb

@@ -1,47 +1,47 @@
-require 'spec_helper'
-
-describe Mongoid::History::Tracker do
-  before :each do
-    class Element
-      include Mongoid::Document
-      include Mongoid::Timestamps
-      include Mongoid::History::Trackable
-
-      track_history
-
-      field :body
-
-      # force preparation of options
-      history_trackable_options
-    end
-
-    class Prompt < Element
-      field :head
-    end
-
-    class User
-      include Mongoid::Document
-    end
-  end
-
-  after :each do
-    Object.send(:remove_const, :Element)
-    Object.send(:remove_const, :Prompt)
-    Object.send(:remove_const, :User)
-  end
-
-  let(:user) { User.create! }
-
-  it 'tracks subclass create and update' do
-    prompt = Prompt.new(modifier: user)
-    expect { prompt.save! }.to change(Tracker, :count).by(1)
-    expect { prompt.update_attributes!(body: 'one', head: 'two') }.to change(Tracker, :count).by(1)
-    prompt.undo! user
-    expect(prompt.body).to be_blank
-    expect(prompt.head).to be_blank
-    prompt.redo! user, 2
-    expect(prompt.body).to eq('one')
-    expect(prompt.head).to eq('two')
-    expect { prompt.destroy }.to change(Tracker, :count).by(1)
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  before :each do
+    class Element
+      include Mongoid::Document
+      include Mongoid::Timestamps
+      include Mongoid::History::Trackable
+
+      track_history
+
+      field :body
+
+      # force preparation of options
+      history_trackable_options
+    end
+
+    class Prompt < Element
+      field :head
+    end
+
+    class User
+      include Mongoid::Document
+    end
+  end
+
+  after :each do
+    Object.send(:remove_const, :Element)
+    Object.send(:remove_const, :Prompt)
+    Object.send(:remove_const, :User)
+  end
+
+  let(:user) { User.create! }
+
+  it 'tracks subclass create and update' do
+    prompt = Prompt.new(modifier: user)
+    expect { prompt.save! }.to change(Tracker, :count).by(1)
+    expect { prompt.update_attributes!(body: 'one', head: 'two') }.to change(Tracker, :count).by(1)
+    prompt.undo! user
+    expect(prompt.body).to be_blank
+    expect(prompt.head).to be_blank
+    prompt.redo! user, 2
+    expect(prompt.body).to eq('one')
+    expect(prompt.head).to eq('two')
+    expect { prompt.destroy }.to change(Tracker, :count).by(1)
+  end
+end

data/spec/integration/track_history_order_spec.rb

@@ -1,84 +1,84 @@
-require 'spec_helper'
-
-describe Mongoid::History::Tracker do
-  context 'when track_history not called' do
-    before :each do
-      class NotModel
-        include Mongoid::Document
-        include Mongoid::History::Trackable
-
-        field :foo
-      end
-    end
-
-    after :each do
-      Object.send(:remove_const, :NotModel)
-    end
-
-    it 'should not track fields' do
-      expect(NotModel.respond_to?(:tracked?)).to be false
-    end
-  end
-
-  context 'boefore field' do
-    before :each do
-      class InsideBeforeModel
-        include Mongoid::Document
-        include Mongoid::History::Trackable
-
-        track_history on: :fields
-
-        field :foo
-      end
-    end
-
-    after :each do
-      Object.send(:remove_const, :InsideBeforeModel)
-    end
-
-    it 'should track fields' do
-      expect(InsideBeforeModel.tracked?(:foo)).to be true
-    end
-  end
-
-  context 'when track_history called inside class and after fields' do
-    before :each do
-      class InsideAfterModel
-        include Mongoid::Document
-        include Mongoid::History::Trackable
-
-        field :foo
-
-        track_history on: :fields
-      end
-    end
-
-    after :each do
-      Object.send(:remove_const, :InsideAfterModel)
-    end
-
-    it 'should track fields' do
-      expect(InsideAfterModel.tracked?(:foo)).to be true
-    end
-  end
-
-  context 'when track_history called outside class' do
-    before :each do
-      class OutsideModel
-        include Mongoid::Document
-        include Mongoid::History::Trackable
-
-        field :foo
-      end
-    end
-
-    after :each do
-      Object.send(:remove_const, :OutsideModel)
-    end
-
-    it 'should track fields' do
-      OutsideModel.track_history on: :fields
-      expect(OutsideModel.tracked?(:foo)).to be true
-    end
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  context 'when track_history not called' do
+    before :each do
+      class NotModel
+        include Mongoid::Document
+        include Mongoid::History::Trackable
+
+        field :foo
+      end
+    end
+
+    after :each do
+      Object.send(:remove_const, :NotModel)
+    end
+
+    it 'should not track fields' do
+      expect(NotModel.respond_to?(:tracked?)).to be false
+    end
+  end
+
+  context 'boefore field' do
+    before :each do
+      class InsideBeforeModel
+        include Mongoid::Document
+        include Mongoid::History::Trackable
+
+        track_history on: :fields
+
+        field :foo
+      end
+    end
+
+    after :each do
+      Object.send(:remove_const, :InsideBeforeModel)
+    end
+
+    it 'should track fields' do
+      expect(InsideBeforeModel.tracked?(:foo)).to be true
+    end
+  end
+
+  context 'when track_history called inside class and after fields' do
+    before :each do
+      class InsideAfterModel
+        include Mongoid::Document
+        include Mongoid::History::Trackable
+
+        field :foo
+
+        track_history on: :fields
+      end
+    end
+
+    after :each do
+      Object.send(:remove_const, :InsideAfterModel)
+    end
+
+    it 'should track fields' do
+      expect(InsideAfterModel.tracked?(:foo)).to be true
+    end
+  end
+
+  context 'when track_history called outside class' do
+    before :each do
+      class OutsideModel
+        include Mongoid::Document
+        include Mongoid::History::Trackable
+
+        field :foo
+      end
+    end
+
+    after :each do
+      Object.send(:remove_const, :OutsideModel)
+    end
+
+    it 'should track fields' do
+      OutsideModel.track_history on: :fields
+      expect(OutsideModel.tracked?(:foo)).to be true
+    end
+  end
+end

data/spec/integration/validation_failure_spec.rb

@@ -1,76 +1,76 @@
-require 'spec_helper'
-
-describe Mongoid::History::Tracker do
-  before :each do
-    class Element
-      include Mongoid::Document
-      include Mongoid::Timestamps
-      include Mongoid::History::Trackable
-
-      field :title
-      field :body
-
-      validates :title, presence: true
-
-      if Mongoid::Compatibility::Version.mongoid7_or_newer?
-        has_many :items, dependent: :restrict_with_exception
-      else
-        has_many :items, dependent: :restrict
-      end
-
-      track_history on: [:body]
-    end
-
-    class Item
-      include Mongoid::Document
-      include Mongoid::Timestamps
-
-      belongs_to :element
-    end
-
-    class Prompt < Element
-    end
-
-    class User
-      include Mongoid::Document
-    end
-  end
-
-  after :each do
-    Object.send(:remove_const, :Element)
-    Object.send(:remove_const, :Item)
-    Object.send(:remove_const, :Prompt)
-    Object.send(:remove_const, :User)
-  end
-
-  let(:user) { User.create! }
-
-  it 'does not track delete when parent class validation fails' do
-    prompt = Prompt.new(title: 'first', modifier: user)
-    expect { prompt.save! }.to change(Tracker, :count).by(1)
-    expect do
-      expect { prompt.update_attributes!(title: nil, body: 'one') }
-        .to raise_error(Mongoid::Errors::Validations)
-    end.to change(Tracker, :count).by(0)
-  end
-
-  it 'does not track delete when parent class restrict dependency fails' do
-    prompt = Prompt.new(title: 'first', modifier: user)
-    prompt.items << Item.new
-    expect { prompt.save! }.to change(Tracker, :count).by(1)
-    expect(prompt.version).to eq(1)
-    expect do
-      expect { prompt.destroy }.to raise_error(Mongoid::Errors::DeleteRestriction)
-    end.to change(Tracker, :count).by(0)
-  end
-
-  it 'does not track delete when restrict dependency fails' do
-    elem = Element.new(title: 'first', modifier: user)
-    elem.items << Item.new
-    expect { elem.save! }.to change(Tracker, :count).by(1)
-    expect(elem.version).to eq(1)
-    expect do
-      expect { elem.destroy }.to raise_error(Mongoid::Errors::DeleteRestriction)
-    end.to change(Tracker, :count).by(0)
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  before :each do
+    class Element
+      include Mongoid::Document
+      include Mongoid::Timestamps
+      include Mongoid::History::Trackable
+
+      field :title
+      field :body
+
+      validates :title, presence: true
+
+      if Mongoid::Compatibility::Version.mongoid7_or_newer?
+        has_many :items, dependent: :restrict_with_exception
+      else
+        has_many :items, dependent: :restrict
+      end
+
+      track_history on: [:body]
+    end
+
+    class Item
+      include Mongoid::Document
+      include Mongoid::Timestamps
+
+      belongs_to :element
+    end
+
+    class Prompt < Element
+    end
+
+    class User
+      include Mongoid::Document
+    end
+  end
+
+  after :each do
+    Object.send(:remove_const, :Element)
+    Object.send(:remove_const, :Item)
+    Object.send(:remove_const, :Prompt)
+    Object.send(:remove_const, :User)
+  end
+
+  let(:user) { User.create! }
+
+  it 'does not track delete when parent class validation fails' do
+    prompt = Prompt.new(title: 'first', modifier: user)
+    expect { prompt.save! }.to change(Tracker, :count).by(1)
+    expect do
+      expect { prompt.update_attributes!(title: nil, body: 'one') }
+        .to raise_error(Mongoid::Errors::Validations)
+    end.to change(Tracker, :count).by(0)
+  end
+
+  it 'does not track delete when parent class restrict dependency fails' do
+    prompt = Prompt.new(title: 'first', modifier: user)
+    prompt.items << Item.new
+    expect { prompt.save! }.to change(Tracker, :count).by(1)
+    expect(prompt.version).to eq(1)
+    expect do
+      expect { prompt.destroy }.to raise_error(Mongoid::Errors::DeleteRestriction)
+    end.to change(Tracker, :count).by(0)
+  end
+
+  it 'does not track delete when restrict dependency fails' do
+    elem = Element.new(title: 'first', modifier: user)
+    elem.items << Item.new
+    expect { elem.save! }.to change(Tracker, :count).by(1)
+    expect(elem.version).to eq(1)
+    expect do
+      expect { elem.destroy }.to raise_error(Mongoid::Errors::DeleteRestriction)
+    end.to change(Tracker, :count).by(0)
+  end
+end