mongoid-history 0.8.0 → 0.8.5

data/spec/unit/tracker_spec.rb

@@ -1,163 +1,190 @@
-require 'spec_helper'
-
-describe Mongoid::History::Tracker do
-  before do
-    @tracker_class_name = Mongoid::History.tracker_class_name
-    Mongoid::History.tracker_class_name = nil
-  end
-
-  it 'should set tracker_class_name when included' do
-    class MyTracker
-      include Mongoid::History::Tracker
-    end
-    expect(Mongoid::History.tracker_class_name).to eq(:my_tracker)
-  end
-
-  it 'should set fields defaults' do
-    class MyTrackerTwo
-      include Mongoid::History::Tracker
-    end
-    expect(MyTrackerTwo.new.association_chain).to eq([])
-    expect(MyTrackerTwo.new.original).to eq({})
-    expect(MyTrackerTwo.new.modified).to eq({})
-  end
-
-  describe '#tracked_edits' do
-    before(:all) do
-      TrackerOne = Class.new do
-        def self.name
-          'TrackerOne'
-        end
-
-        include Mongoid::History::Tracker
-      end
-
-      ModelOne = Class.new do
-        include Mongoid::Document
-        include Mongoid::History::Trackable
-        store_in collection: :model_ones
-        embeds_many :emb_ones, inverse_class_name: 'EmbOne'
-      end
-
-      EmbOne = Class.new do
-        include Mongoid::Document
-        field :em_foo
-        embedded_in :model_one
-      end
-    end
-
-    context 'when embeds_many' do
-      before(:each) do
-        ModelOne.instance_variable_set(:@history_trackable_options, nil)
-        ModelOne.track_history(on: :emb_ones)
-        allow(tracker).to receive(:trackable_parent_class) { ModelOne }
-      end
-
-      let(:tracker) { TrackerOne.new }
-
-      describe '#prepare_tracked_edits_for_embeds_many' do
-        before(:each) do
-          tracker.instance_variable_set(:@tracked_edits, nil)
-          allow(tracker).to receive(:tracked_changes) { changes }
-        end
-        let(:emb_one) { EmbOne.new }
-        let(:emb_one_2) { EmbOne.new }
-        let(:emb_one_3) { EmbOne.new }
-        let(:changes) { {} }
-        subject { tracker.tracked_edits['embeds_many']['emb_ones'] }
-
-        context 'when all values present' do
-          let(:changes) do
-            { 'emb_ones' => { from: [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' },
-                                     { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }],
-                              to: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new' },
-                                   { '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }] } }
-          end
-          it 'should include :add, :remove, and :modify' do
-            expect(subject['add']).to eq [{ '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }]
-            expect(subject['remove']).to eq [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' }]
-            expect(subject['modify'].size).to eq 1
-            expect(subject['modify'][0]['from']).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2')
-            expect(subject['modify'][0]['to']).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new')
-          end
-        end
-
-        context 'when value :from blank' do
-          let(:changes) do
-            { 'emb_ones' => { to: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new' },
-                                   { '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }] } }
-          end
-          it 'should include :add' do
-            expect(subject['add'].size).to eq 2
-            expect(subject['add'][0]).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new')
-            expect(subject['add'][1]).to eq('_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3')
-            expect(subject['remove']).to be_nil
-            expect(subject['modify']).to be_nil
-          end
-        end
-
-        context 'when value :to blank' do
-          let(:changes) do
-            { 'emb_ones' => { from: [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' },
-                                     { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }] } }
-          end
-          it 'should include :remove' do
-            expect(subject['add']).to be_nil
-            expect(subject['modify']).to be_nil
-            expect(subject['remove'].size).to eq 2
-            expect(subject['remove'][0]).to eq('_id' => emb_one._id, 'em_foo' => 'Em-Foo')
-            expect(subject['remove'][1]).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2')
-          end
-        end
-
-        context 'when no id common in :from and :to' do
-          let(:changes) do
-            { 'emb_ones' => { from: [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' }],
-                              to: [{ '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }] } }
-          end
-          it 'should include :add, and :remove' do
-            expect(subject['add']).to eq [{ '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }]
-            expect(subject['modify']).to be_nil
-            expect(subject['remove']).to eq [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' }]
-          end
-        end
-
-        context 'when _id attribute not set' do
-          let(:changes) do
-            { 'emb_ones' => { from: [{ 'em_foo' => 'Em-Foo' },
-                                     { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }],
-                              to: [{ 'em_foo' => 'Em-Foo-2-new' },
-                                   { 'em_foo' => 'Em-Foo-3' }] } }
-          end
-          it 'should include :add, and :remove' do
-            expect(subject['add']).to eq([{ 'em_foo' => 'Em-Foo-2-new' }, { 'em_foo' => 'Em-Foo-3' }])
-            expect(subject['modify']).to be_nil
-            expect(subject['remove']).to eq [{ 'em_foo' => 'Em-Foo' }, { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }]
-          end
-        end
-
-        context 'when no change in an object' do
-          let(:changes) do
-            { 'emb_ones' => { from: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }],
-                              to: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }] } }
-          end
-          it 'should include not :add, :remove, and :modify' do
-            expect(subject['add']).to be_nil
-            expect(subject['modify']).to be_nil
-            expect(subject['remove']).to be_nil
-          end
-        end
-      end
-    end
-
-    after(:all) do
-      Object.send(:remove_const, :TrackerOne)
-      Object.send(:remove_const, :ModelOne)
-      Object.send(:remove_const, :EmbOne)
-    end
-  end
-
-  after do
-    Mongoid::History.tracker_class_name = @tracker_class_name
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  context 'when included' do
+    before :each do
+      Mongoid::History.tracker_class_name = nil
+
+      class MyTracker
+        include Mongoid::History::Tracker
+      end
+    end
+
+    after :each do
+      Object.send(:remove_const, :MyTracker)
+    end
+
+    it 'should set tracker_class_name when included' do
+      expect(Mongoid::History.tracker_class_name).to eq(:my_tracker)
+    end
+
+    it 'should set fields defaults' do
+      expect(MyTracker.new.association_chain).to eq([])
+      expect(MyTracker.new.original).to eq({})
+      expect(MyTracker.new.modified).to eq({})
+    end
+  end
+
+  describe '#tracked_edits' do
+    before :each do
+      class TrackerOne
+        include Mongoid::History::Tracker
+      end
+
+      class ModelOne
+        include Mongoid::Document
+        include Mongoid::History::Trackable
+
+        store_in collection: :model_ones
+
+        if Mongoid::Compatibility::Version.mongoid7_or_newer?
+          embeds_many :emb_ones
+        else
+          embeds_many :emb_ones, inverse_class_name: 'EmbOne'
+        end
+      end
+
+      class EmbOne
+        include Mongoid::Document
+
+        field :em_foo
+        embedded_in :model_one
+      end
+    end
+
+    after :each do
+      Object.send(:remove_const, :TrackerOne)
+      Object.send(:remove_const, :ModelOne)
+      Object.send(:remove_const, :EmbOne)
+    end
+
+    context 'when embeds_many' do
+      before :each do
+        ModelOne.track_history(on: :emb_ones)
+        allow(tracker).to receive(:trackable_parent_class) { ModelOne }
+      end
+
+      let(:tracker) { TrackerOne.new }
+
+      describe '#prepare_tracked_edits_for_embeds_many' do
+        before :each do
+          allow(tracker).to receive(:tracked_changes) { changes }
+        end
+
+        let(:emb_one) { EmbOne.new }
+        let(:emb_one_2) { EmbOne.new }
+        let(:emb_one_3) { EmbOne.new }
+        let(:changes) { {} }
+
+        subject { tracker.tracked_edits['embeds_many']['emb_ones'] }
+
+        context 'when all values present' do
+          let(:changes) do
+            {
+              'emb_ones' => {
+                from: [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' },
+                       { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }],
+                to: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new' },
+                     { '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }]
+              }
+            }
+          end
+
+          it 'should include :add, :remove, and :modify' do
+            expect(subject['add']).to eq [{ '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }]
+            expect(subject['remove']).to eq [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' }]
+            expect(subject['modify'].size).to eq 1
+            expect(subject['modify'][0]['from']).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2')
+            expect(subject['modify'][0]['to']).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new')
+          end
+        end
+
+        context 'when value :from blank' do
+          let(:changes) do
+            {
+              'emb_ones' => {
+                to: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new' },
+                     { '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }]
+              }
+            }
+          end
+          it 'should include :add' do
+            expect(subject['add'].size).to eq 2
+            expect(subject['add'][0]).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2-new')
+            expect(subject['add'][1]).to eq('_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3')
+            expect(subject['remove']).to be_nil
+            expect(subject['modify']).to be_nil
+          end
+        end
+
+        context 'when value :to blank' do
+          let(:changes) do
+            {
+              'emb_ones' => {
+                from: [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' },
+                       { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }]
+              }
+            }
+          end
+          it 'should include :remove' do
+            expect(subject['add']).to be_nil
+            expect(subject['modify']).to be_nil
+            expect(subject['remove'].size).to eq 2
+            expect(subject['remove'][0]).to eq('_id' => emb_one._id, 'em_foo' => 'Em-Foo')
+            expect(subject['remove'][1]).to eq('_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2')
+          end
+        end
+
+        context 'when no id common in :from and :to' do
+          let(:changes) do
+            {
+              'emb_ones' => {
+                from: [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' }],
+                to: [{ '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }]
+              }
+            }
+          end
+          it 'should include :add, and :remove' do
+            expect(subject['add']).to eq [{ '_id' => emb_one_3._id, 'em_foo' => 'Em-Foo-3' }]
+            expect(subject['modify']).to be_nil
+            expect(subject['remove']).to eq [{ '_id' => emb_one._id, 'em_foo' => 'Em-Foo' }]
+          end
+        end
+
+        context 'when _id attribute not set' do
+          let(:changes) do
+            {
+              'emb_ones' => {
+                from: [{ 'em_foo' => 'Em-Foo' },
+                       { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }],
+                to: [{ 'em_foo' => 'Em-Foo-2-new' },
+                     { 'em_foo' => 'Em-Foo-3' }]
+              }
+            }
+          end
+          it 'should include :add, and :remove' do
+            expect(subject['add']).to eq([{ 'em_foo' => 'Em-Foo-2-new' }, { 'em_foo' => 'Em-Foo-3' }])
+            expect(subject['modify']).to be_nil
+            expect(subject['remove']).to eq [{ 'em_foo' => 'Em-Foo' }, { '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }]
+          end
+        end
+
+        context 'when no change in an object' do
+          let(:changes) do
+            {
+              'emb_ones' => {
+                from: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }],
+                to: [{ '_id' => emb_one_2._id, 'em_foo' => 'Em-Foo-2' }]
+              }
+            }
+          end
+          it 'should include not :add, :remove, and :modify' do
+            expect(subject['add']).to be_nil
+            expect(subject['modify']).to be_nil
+            expect(subject['remove']).to be_nil
+          end
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,16 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: mongoid-history
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.5
 platform: ruby
 authors:
 - Aaron Qian
 - Justin Grimes
 - Daniel Doubrovkine
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-01-16 00:00:00.000000000 Z
+date: 2021-09-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: easy_diff
@@ -87,11 +87,11 @@ extra_rdoc_files: []
 files:
 - ".coveralls.yml"
 - ".document"
+- ".github/workflows/test.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
-- ".travis.yml"
 - CHANGELOG.md
 - CONTRIBUTING.md
 - Dangerfile
@@ -112,16 +112,20 @@ files:
 - lib/mongoid/history/tracker.rb
 - lib/mongoid/history/version.rb
 - mongoid-history.gemspec
+- perf/benchmark_modified_attributes_for_create.rb
+- perf/gc_suite.rb
 - spec/integration/embedded_in_polymorphic_spec.rb
 - spec/integration/integration_spec.rb
 - spec/integration/multi_relation_spec.rb
 - spec/integration/multiple_trackers_spec.rb
 - spec/integration/nested_embedded_documents_spec.rb
+- spec/integration/nested_embedded_documents_tracked_in_parent_spec.rb
 - spec/integration/nested_embedded_polymorphic_documents_spec.rb
 - spec/integration/subclasses_spec.rb
 - spec/integration/track_history_order_spec.rb
 - spec/integration/validation_failure_spec.rb
 - spec/spec_helper.rb
+- spec/support/error_helpers.rb
 - spec/support/mongoid.rb
 - spec/support/mongoid_history.rb
 - spec/unit/attributes/base_spec.rb
@@ -142,7 +146,7 @@ homepage: http://github.com/mongoid/mongoid-history
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -157,9 +161,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.13
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: Track and audit, undo and redo changes on Mongoid documents.
 test_files:
@@ -168,11 +171,13 @@ test_files:
 - spec/integration/multi_relation_spec.rb
 - spec/integration/multiple_trackers_spec.rb
 - spec/integration/nested_embedded_documents_spec.rb
+- spec/integration/nested_embedded_documents_tracked_in_parent_spec.rb
 - spec/integration/nested_embedded_polymorphic_documents_spec.rb
 - spec/integration/subclasses_spec.rb
 - spec/integration/track_history_order_spec.rb
 - spec/integration/validation_failure_spec.rb
 - spec/spec_helper.rb
+- spec/support/error_helpers.rb
 - spec/support/mongoid.rb
 - spec/support/mongoid_history.rb
 - spec/unit/attributes/base_spec.rb

data/.travis.yml

@@ -1,35 +0,0 @@
-
-sudo: false
-
-language: ruby
-
-cache: bundler
-
-services: mongodb
-
-env:
-  - MONGOID_VERSION=3
-  - MONGOID_VERSION=4
-  - MONGOID_VERSION=5
-  - MONGOID_VERSION=6
-  - MONGOID_VERSION=HEAD
-
-rvm:
-  - 2.3.1
-
-before_install:
-  - gem update bundler
-
-before_script:
-  - bundle exec danger
-
-addons:
-  apt:
-    sources:
-      - mongodb-3.2-precise
-    packages:
-      - mongodb-org-server
-
-matrix:
-  allow_failures:
-    - env: MONGOID_VERSION=HEAD