mongoid-history 0.6.1 → 0.7.0

data/spec/unit/singleton_methods_spec.rb

@@ -31,7 +31,7 @@ describe Mongoid::History::Trackable do
         embedded_in :my_trackable_model
       end
 
-      MyTrackableModel.track_history(on: [:foo, :my_embed_one_model, :my_embed_many_models, :my_dynamic_field])
+      MyTrackableModel.track_history(on: %i[foo my_embed_one_model my_embed_many_models my_dynamic_field])
     end
 
     describe '#tracked?' do
@@ -143,7 +143,7 @@ describe Mongoid::History::Trackable do
 
     describe '#tracked_fields' do
       it 'should include fields and dynamic fields' do
-        expect(MyTrackableModel.tracked_fields).to eq %w(foo my_dynamic_field)
+        expect(MyTrackableModel.tracked_fields).to eq %w[foo my_dynamic_field]
       end
     end
 
@@ -199,17 +199,17 @@ describe Mongoid::History::Trackable do
 
       context 'when relation tracked' do
         before(:each) { ModelOne.track_history(on: :emb_one) }
-        it { expect(ModelOne.tracked_embeds_one_attributes('emb_one')).to eq %w(_id em_foo em_bar) }
+        it { expect(ModelOne.tracked_embeds_one_attributes('emb_one')).to eq %w[_id em_foo em_bar] }
       end
 
       context 'when relation tracked with alias' do
         before(:each) { ModelOne.track_history(on: :emb_two) }
-        it { expect(ModelOne.tracked_embeds_one_attributes('emb_two')).to eq %w(_id em_bar) }
+        it { expect(ModelOne.tracked_embeds_one_attributes('emb_two')).to eq %w[_id em_bar] }
       end
 
       context 'when relation tracked with attributes' do
         before(:each) { ModelOne.track_history(on: { emb_one: :em_foo }) }
-        it { expect(ModelOne.tracked_embeds_one_attributes('emb_one')).to eq %w(_id em_foo) }
+        it { expect(ModelOne.tracked_embeds_one_attributes('emb_one')).to eq %w[_id em_foo] }
       end
 
       context 'when relation not tracked' do
@@ -268,17 +268,17 @@ describe Mongoid::History::Trackable do
 
       context 'when relation tracked' do
         before(:each) { ModelOne.track_history(on: :emb_ones) }
-        it { expect(ModelOne.tracked_embeds_many_attributes('emb_ones')).to eq %w(_id em_foo em_bar) }
+        it { expect(ModelOne.tracked_embeds_many_attributes('emb_ones')).to eq %w[_id em_foo em_bar] }
       end
 
       context 'when relation tracked with alias' do
         before(:each) { ModelOne.track_history(on: :emb_twos) }
-        it { expect(ModelOne.tracked_embeds_many_attributes('emb_twos')).to eq %w(_id em_bar) }
+        it { expect(ModelOne.tracked_embeds_many_attributes('emb_twos')).to eq %w[_id em_bar] }
       end
 
       context 'when relation tracked with attributes' do
         before(:each) { ModelOne.track_history(on: { emb_ones: :em_foo }) }
-        it { expect(ModelOne.tracked_embeds_many_attributes('emb_ones')).to eq %w(_id em_foo) }
+        it { expect(ModelOne.tracked_embeds_many_attributes('emb_ones')).to eq %w[_id em_foo] }
       end
 
       context 'when relation not tracked' do
@@ -309,12 +309,12 @@ describe Mongoid::History::Trackable do
 
     describe '#clear_trackable_memoization' do
       before do
-        MyTrackableModel.instance_variable_set(:@reserved_tracked_fields, %w(_id _type))
-        MyTrackableModel.instance_variable_set(:@history_trackable_options, on: %w(fields))
+        MyTrackableModel.instance_variable_set(:@reserved_tracked_fields, %w[_id _type])
+        MyTrackableModel.instance_variable_set(:@history_trackable_options, on: %w[fields])
         MyTrackableModel.instance_variable_set(:@trackable_settings, paranoia_field: 'deleted_at')
-        MyTrackableModel.instance_variable_set(:@tracked_fields, %w(foo))
-        MyTrackableModel.instance_variable_set(:@tracked_embeds_one, %w(my_embed_one_model))
-        MyTrackableModel.instance_variable_set(:@tracked_embeds_many, %w(my_embed_many_models))
+        MyTrackableModel.instance_variable_set(:@tracked_fields, %w[foo])
+        MyTrackableModel.instance_variable_set(:@tracked_embeds_one, %w[my_embed_one_model])
+        MyTrackableModel.instance_variable_set(:@tracked_embeds_many, %w[my_embed_many_models])
         MyTrackableModel.clear_trackable_memoization
       end
 

data/spec/unit/trackable_spec.rb

@@ -34,8 +34,8 @@ describe Mongoid::History::Trackable do
     end
     before(:each) { Mongoid::History.trackable_class_options = @persisted_history_options }
     let(:expected_option) do
-      { on: %i(foo),
-        except: %w(created_at updated_at),
+      { on: %i[foo],
+        except: %w[created_at updated_at],
         tracker_class_name: nil,
         modifier_field: :modifier,
         version_field: :version,
@@ -44,13 +44,13 @@ describe Mongoid::History::Trackable do
         track_create: false,
         track_update: true,
         track_destroy: false,
-        fields: %w(foo),
+        fields: %w[foo],
         relations: { embeds_one: {}, embeds_many: {} },
         dynamic: [],
         format: {} }
     end
     let(:regular_fields) { ['foo'] }
-    let(:reserved_fields) { %w(_id version modifier_id) }
+    let(:reserved_fields) { %w[_id version modifier_id] }
 
     it 'should have default options' do
       expect(Mongoid::History.trackable_class_options[:my_model]).to eq(expected_option)
@@ -217,7 +217,7 @@ describe Mongoid::History::Trackable do
           it 'should be rescued if an exception occurs' do
             begin
               MyModel.disable_tracking do
-                fail 'exception'
+                raise 'exception'
               end
             rescue
             end
@@ -260,7 +260,7 @@ describe Mongoid::History::Trackable do
             Mongoid::History.disable do
               begin
                 MyModel.disable_tracking do
-                  fail 'exception'
+                  raise 'exception'
                 end
               rescue
               end
@@ -289,7 +289,7 @@ describe Mongoid::History::Trackable do
           begin
             Mongoid::History.disable do
               MyModel.disable_tracking do
-                fail 'exception'
+                raise 'exception'
               end
             end
           rescue
@@ -583,4 +583,63 @@ describe Mongoid::History::Trackable do
       Object.send(:remove_const, :MyTrackerClass)
     end
   end
+
+  describe '#track_update' do
+    before :all do
+      MyModel.track_history(on: :foo, track_update: true)
+      @persisted_history_options = Mongoid::History.trackable_class_options
+    end
+    before(:each) { Mongoid::History.trackable_class_options = @persisted_history_options }
+    let(:m) { MyModel.create!(foo: 'bar') }
+
+    it 'should create history' do
+      expect { m.update_attributes!(foo: 'bar2') }.to change(Tracker, :count).by(1)
+    end
+
+    it 'should not create history when error raised' do
+      expect(m).to receive(:update_attributes!).and_raise(StandardError)
+      expect do
+        expect { m.update_attributes!(foo: 'bar2') }.to raise_error(StandardError)
+      end.to change(Tracker, :count).by(0)
+    end
+  end
+
+  describe '#track_destroy' do
+    before :all do
+      MyModel.track_history(on: :foo, track_destroy: true)
+      @persisted_history_options = Mongoid::History.trackable_class_options
+    end
+    before(:each) { Mongoid::History.trackable_class_options = @persisted_history_options }
+    let(:m) { MyModel.create!(foo: 'bar') }
+
+    it 'should create history' do
+      expect { m.destroy }.to change(Tracker, :count).by(1)
+    end
+
+    it 'should not create history when error raised' do
+      expect(m).to receive(:destroy).and_raise(StandardError)
+      expect do
+        expect { m.destroy }.to raise_error(StandardError)
+      end.to change(Tracker, :count).by(0)
+    end
+  end
+
+  describe '#track_create' do
+    before :all do
+      MyModel.track_history(on: :foo, track_create: true)
+      @persisted_history_options = Mongoid::History.trackable_class_options
+    end
+    before(:each) { Mongoid::History.trackable_class_options = @persisted_history_options }
+
+    it 'should create history' do
+      expect { MyModel.create!(foo: 'bar') }.to change(Tracker, :count).by(1)
+    end
+
+    it 'should not create history when error raised' do
+      expect(MyModel).to receive(:create!).and_raise(StandardError)
+      expect do
+        expect { MyModel.create!(foo: 'bar') }.to raise_error(StandardError)
+      end.to change(Tracker, :count).by(0)
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongoid-history
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.7.0
 platform: ruby
 authors:
 - Aaron Qian
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-04 00:00:00.000000000 Z
+date: 2017-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: easy_diff
@@ -98,7 +98,9 @@ files:
 - Gemfile
 - LICENSE.txt
 - README.md
+- RELEASING.md
 - Rakefile
+- UPGRADING.md
 - lib/mongoid-history.rb
 - lib/mongoid/history.rb
 - lib/mongoid/history/attributes/base.rb
@@ -113,9 +115,11 @@ files:
 - 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_polymorphic_documents_spec.rb
 - spec/integration/subclasses_spec.rb
+- spec/integration/validation_failure_spec.rb
 - spec/spec_helper.rb
 - spec/support/mongoid.rb
 - spec/support/mongoid_history.rb
@@ -152,7 +156,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.12
 signing_key: 
 specification_version: 4
 summary: Track and audit, undo and redo changes on Mongoid documents.
@@ -160,9 +164,11 @@ test_files:
 - 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_polymorphic_documents_spec.rb
 - spec/integration/subclasses_spec.rb
+- spec/integration/validation_failure_spec.rb
 - spec/spec_helper.rb
 - spec/support/mongoid.rb
 - spec/support/mongoid_history.rb