mongoid-history 0.8.0 → 0.8.1

data/spec/unit/tracker_spec.rb

@@ -1,54 +1,65 @@
 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
+  context 'when included' do
+    before :each do
+      Mongoid::History.tracker_class_name = nil
 
-  it 'should set tracker_class_name when included' do
-    class MyTracker
-      include Mongoid::History::Tracker
+      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
-    expect(Mongoid::History.tracker_class_name).to eq(:my_tracker)
-  end
 
-  it 'should set fields defaults' do
-    class MyTrackerTwo
-      include Mongoid::History::Tracker
+    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
-    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
-
+    before :each do
+      class TrackerOne
         include Mongoid::History::Tracker
       end
 
-      ModelOne = Class.new do
+      class ModelOne
         include Mongoid::Document
         include Mongoid::History::Trackable
+
         store_in collection: :model_ones
-        embeds_many :emb_ones, inverse_class_name: 'EmbOne'
+
+        if Mongoid::Compatibility::Version.mongoid7_or_newer?
+          embeds_many :emb_ones
+        else
+          embeds_many :emb_ones, inverse_class_name: 'EmbOne'
+        end
       end
 
-      EmbOne = Class.new do
+      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.instance_variable_set(:@history_trackable_options, nil)
+      before :each do
         ModelOne.track_history(on: :emb_ones)
         allow(tracker).to receive(:trackable_parent_class) { ModelOne }
       end
@@ -56,23 +67,29 @@ describe Mongoid::History::Tracker do
       let(:tracker) { TrackerOne.new }
 
       describe '#prepare_tracked_edits_for_embeds_many' do
-        before(:each) do
-          tracker.instance_variable_set(:@tracked_edits, nil)
+        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' }] } }
+            {
+              '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' }]
@@ -84,8 +101,12 @@ describe Mongoid::History::Tracker do
 
         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' }] } }
+            {
+              '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
@@ -98,8 +119,12 @@ describe Mongoid::History::Tracker do
 
         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' }] } }
+            {
+              '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
@@ -112,8 +137,12 @@ describe Mongoid::History::Tracker do
 
         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' }] } }
+            {
+              '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' }]
@@ -124,10 +153,14 @@ describe Mongoid::History::Tracker do
 
         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' }] } }
+            {
+              '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' }])
@@ -138,8 +171,12 @@ describe Mongoid::History::Tracker do
 
         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' }] } }
+            {
+              '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
@@ -149,15 +186,5 @@ describe Mongoid::History::Tracker do
         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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongoid-history
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Aaron Qian
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-16 00:00:00.000000000 Z
+date: 2018-06-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: easy_diff