RubyGems - mongoid-history - Versions diffs - 0.8.3 → 0.8.5 - Mend

mongoid-history 0.8.3 → 0.8.5

Files changed (60) hide show

checksums.yaml +4 -4
data/.coveralls.yml +1 -1
data/.document +5 -5
data/.github/workflows/test.yml +72 -0
data/.gitignore +46 -46
data/.rspec +2 -2
data/.rubocop.yml +6 -6
data/.rubocop_todo.yml +99 -99
data/CHANGELOG.md +173 -163
data/CONTRIBUTING.md +117 -118
data/Dangerfile +1 -1
data/Gemfile +49 -40
data/LICENSE.txt +20 -20
data/README.md +609 -608
data/RELEASING.md +66 -67
data/Rakefile +24 -24
data/UPGRADING.md +53 -53
data/lib/mongoid/history/attributes/base.rb +72 -72
data/lib/mongoid/history/attributes/create.rb +45 -45
data/lib/mongoid/history/attributes/destroy.rb +34 -34
data/lib/mongoid/history/attributes/update.rb +104 -104
data/lib/mongoid/history/options.rb +177 -177
data/lib/mongoid/history/trackable.rb +588 -583
data/lib/mongoid/history/tracker.rb +247 -247
data/lib/mongoid/history/version.rb +5 -5
data/lib/mongoid/history.rb +77 -77
data/lib/mongoid-history.rb +1 -1
data/mongoid-history.gemspec +25 -25
data/perf/benchmark_modified_attributes_for_create.rb +65 -65
data/perf/gc_suite.rb +21 -21
data/spec/integration/embedded_in_polymorphic_spec.rb +112 -112
data/spec/integration/integration_spec.rb +976 -976
data/spec/integration/multi_relation_spec.rb +47 -47
data/spec/integration/multiple_trackers_spec.rb +68 -68
data/spec/integration/nested_embedded_documents_spec.rb +64 -64
data/spec/integration/nested_embedded_documents_tracked_in_parent_spec.rb +124 -124
data/spec/integration/nested_embedded_polymorphic_documents_spec.rb +115 -115
data/spec/integration/subclasses_spec.rb +47 -47
data/spec/integration/track_history_order_spec.rb +84 -84
data/spec/integration/validation_failure_spec.rb +76 -76
data/spec/spec_helper.rb +32 -30
data/spec/support/error_helpers.rb +7 -0
data/spec/support/mongoid.rb +11 -11
data/spec/support/mongoid_history.rb +12 -12
data/spec/unit/attributes/base_spec.rb +141 -141
data/spec/unit/attributes/create_spec.rb +342 -342
data/spec/unit/attributes/destroy_spec.rb +228 -228
data/spec/unit/attributes/update_spec.rb +342 -342
data/spec/unit/callback_options_spec.rb +165 -165
data/spec/unit/embedded_methods_spec.rb +87 -87
data/spec/unit/history_spec.rb +58 -58
data/spec/unit/my_instance_methods_spec.rb +555 -555
data/spec/unit/options_spec.rb +365 -365
data/spec/unit/singleton_methods_spec.rb +406 -406
data/spec/unit/store/default_store_spec.rb +11 -11
data/spec/unit/store/request_store_spec.rb +13 -13
data/spec/unit/trackable_spec.rb +1057 -987
data/spec/unit/tracker_spec.rb +190 -190
metadata +9 -7
data/.travis.yml +0 -36

data/spec/unit/attributes/create_spec.rb CHANGED Viewed

@@ -1,342 +1,342 @@
-require 'spec_helper'
-describe Mongoid::History::Attributes::Create do
-  before :each do
-    class ModelOne
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-      store_in collection: :model_ones
-      field :foo
-      field :b, as: :bar
-      track_history on: :foo
-    end
-  end
-  after :each do
-    Object.send(:remove_const, :ModelOne)
-  end
-  let(:base) { described_class.new(obj_one) }
-  subject { base }
-  describe '#attributes' do
-    subject { base.attributes }
-    describe 'fields' do
-      let(:obj_one) { ModelOne.new }
-      let(:obj_one) { ModelOne.new(foo: 'Foo', bar: 'Bar') }
-      it { is_expected.to eq('foo' => [nil, 'Foo']) }
-    end
-    describe '#insert_embeds_one_changes' do
-      context 'when untracked relation' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one
-            else
-              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
-            end
-            track_history on: :fields
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            field :em_bar
-            embedded_in :model_one
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
-        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
-        it { is_expected.to eq({}) }
-      end
-      context 'when tracked relation' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one
-            else
-              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
-            end
-            track_history on: :emb_one_one
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            field :em_bar
-            embedded_in :model_one
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
-        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
-        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
-      end
-      context 'when paranoia_field without alias' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one
-            else
-              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
-            end
-            track_history on: :emb_one_one
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            field :em_bar
-            field :removed_at
-            embedded_in :model_one
-            history_settings paranoia_field: :removed_at
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
-        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar', removed_at: Time.now) }
-        it { is_expected.to eq({}) }
-      end
-      context 'when paranoia_field with alias' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one
-            else
-              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
-            end
-            track_history on: :emb_one_one
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            field :em_bar
-            field :rmvt, as: :removed_at
-            embedded_in :model_one
-            history_settings paranoia_field: :removed_at
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
-        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar', removed_at: Time.now) }
-        it { is_expected.to eq({}) }
-      end
-      context 'with permitted attributes' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one
-            else
-              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
-            end
-            track_history on: [{ emb_one_one: :em_bar }]
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            field :em_foo
-            field :em_bar
-            embedded_in :model_one
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
-        let(:emb_obj) { EmbOneOne.new(em_foo: 'Em-Foo', em_bar: 'Em-Bar') }
-        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
-      end
-      context 'when relation with alias' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one, store_as: :eoo
-            else
-              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne', store_as: :eoo
-            end
-            track_history on: :emb_one_one
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            field :em_bar
-            embedded_in :model_one
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
-        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
-        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
-      end
-      context 'when no object' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one, store_as: :eoo
-            else
-              embeds_one :emb_one_one, store_as: :eoo, inverse_class_name: 'EmbOneOne'
-            end
-            track_history on: :emb_one_one
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            field :em_bar
-            embedded_in :model_one
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new }
-        it { is_expected.to eq({}) }
-      end
-      context 'when object not paranoid' do
-        before :each do
-          class ModelTwo
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            store_in collection: :model_twos
-            if Mongoid::Compatibility::Version.mongoid7_or_newer?
-              embeds_one :emb_one_one, store_as: :eoo
-            else
-              embeds_one :emb_one_one, store_as: :eoo, inverse_class_name: 'EmbOneOne'
-            end
-            track_history on: :emb_one_one
-          end
-          class EmbOneOne
-            include Mongoid::Document
-            include Mongoid::History::Trackable
-            field :em_bar
-            field :cancelled_at
-            embedded_in :model_one
-            history_settings paranoia_field: :cancelled_at
-          end
-        end
-        after :each do
-          Object.send(:remove_const, :ModelTwo)
-          Object.send(:remove_const, :EmbOneOne)
-        end
-        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
-        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
-        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
-      end
-    end
-    pending '#insert_embeds_many_changes'
-  end
-end
+require 'spec_helper'
+describe Mongoid::History::Attributes::Create do
+  before :each do
+    class ModelOne
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+      store_in collection: :model_ones
+      field :foo
+      field :b, as: :bar
+      track_history on: :foo
+    end
+  end
+  after :each do
+    Object.send(:remove_const, :ModelOne)
+  end
+  let(:base) { described_class.new(obj_one) }
+  subject { base }
+  describe '#attributes' do
+    subject { base.attributes }
+    describe 'fields' do
+      let(:obj_one) { ModelOne.new }
+      let(:obj_one) { ModelOne.new(foo: 'Foo', bar: 'Bar') }
+      it { is_expected.to eq('foo' => [nil, 'Foo']) }
+    end
+    describe '#insert_embeds_one_changes' do
+      context 'when untracked relation' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one
+            else
+              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
+            end
+            track_history on: :fields
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            field :em_bar
+            embedded_in :model_one
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
+        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
+        it { is_expected.to eq({}) }
+      end
+      context 'when tracked relation' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one
+            else
+              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
+            end
+            track_history on: :emb_one_one
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            field :em_bar
+            embedded_in :model_one
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
+        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
+        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
+      end
+      context 'when paranoia_field without alias' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one
+            else
+              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
+            end
+            track_history on: :emb_one_one
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            field :em_bar
+            field :removed_at
+            embedded_in :model_one
+            history_settings paranoia_field: :removed_at
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
+        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar', removed_at: Time.now) }
+        it { is_expected.to eq({}) }
+      end
+      context 'when paranoia_field with alias' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one
+            else
+              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
+            end
+            track_history on: :emb_one_one
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            field :em_bar
+            field :rmvt, as: :removed_at
+            embedded_in :model_one
+            history_settings paranoia_field: :removed_at
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
+        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar', removed_at: Time.now) }
+        it { is_expected.to eq({}) }
+      end
+      context 'with permitted attributes' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one
+            else
+              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne'
+            end
+            track_history on: [{ emb_one_one: :em_bar }]
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            field :em_foo
+            field :em_bar
+            embedded_in :model_one
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
+        let(:emb_obj) { EmbOneOne.new(em_foo: 'Em-Foo', em_bar: 'Em-Bar') }
+        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
+      end
+      context 'when relation with alias' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one, store_as: :eoo
+            else
+              embeds_one :emb_one_one, inverse_class_name: 'EmbOneOne', store_as: :eoo
+            end
+            track_history on: :emb_one_one
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            field :em_bar
+            embedded_in :model_one
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
+        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
+        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
+      end
+      context 'when no object' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one, store_as: :eoo
+            else
+              embeds_one :emb_one_one, store_as: :eoo, inverse_class_name: 'EmbOneOne'
+            end
+            track_history on: :emb_one_one
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            field :em_bar
+            embedded_in :model_one
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new }
+        it { is_expected.to eq({}) }
+      end
+      context 'when object not paranoid' do
+        before :each do
+          class ModelTwo
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            store_in collection: :model_twos
+            if Mongoid::Compatibility::Version.mongoid7_or_newer?
+              embeds_one :emb_one_one, store_as: :eoo
+            else
+              embeds_one :emb_one_one, store_as: :eoo, inverse_class_name: 'EmbOneOne'
+            end
+            track_history on: :emb_one_one
+          end
+          class EmbOneOne
+            include Mongoid::Document
+            include Mongoid::History::Trackable
+            field :em_bar
+            field :cancelled_at
+            embedded_in :model_one
+            history_settings paranoia_field: :cancelled_at
+          end
+        end
+        after :each do
+          Object.send(:remove_const, :ModelTwo)
+          Object.send(:remove_const, :EmbOneOne)
+        end
+        let(:obj_one) { ModelTwo.new(emb_one_one: emb_obj) }
+        let(:emb_obj) { EmbOneOne.new(em_bar: 'Em-Bar') }
+        it { is_expected.to eq('emb_one_one' => [nil, { '_id' => emb_obj._id, 'em_bar' => 'Em-Bar' }]) }
+      end
+    end
+    pending '#insert_embeds_many_changes'
+  end
+end