mongoid-history 0.5.0 → 0.6.0

data/spec/unit/tracker_spec.rb

@@ -1,10 +1,163 @@
 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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mongoid-history
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Aaron Qian
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-18 00:00:00.000000000 Z
+date: 2016-09-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: easy_diff
@@ -68,104 +68,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.34.1
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '='
-      - !ruby/object:Gem::Version
-        version: 0.34.1
-- !ruby/object:Gem::Dependency
-  name: yard
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: gem-release
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: coveralls
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 description: This library tracks historical changes for any document, including embedded
   ones. It achieves this by storing all history tracks in a single collection that
   you define. Embedded documents are referenced by storing an association path, which
@@ -192,12 +94,18 @@ files:
 - ".travis.yml"
 - CHANGELOG.md
 - CONTRIBUTING.md
+- Dangerfile
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - lib/mongoid-history.rb
 - lib/mongoid/history.rb
+- lib/mongoid/history/attributes/base.rb
+- lib/mongoid/history/attributes/create.rb
+- lib/mongoid/history/attributes/destroy.rb
+- lib/mongoid/history/attributes/update.rb
+- lib/mongoid/history/options.rb
 - lib/mongoid/history/trackable.rb
 - lib/mongoid/history/tracker.rb
 - lib/mongoid/history/version.rb
@@ -211,6 +119,17 @@ files:
 - spec/spec_helper.rb
 - spec/support/mongoid.rb
 - spec/support/mongoid_history.rb
+- spec/unit/attributes/base_spec.rb
+- spec/unit/attributes/create_spec.rb
+- spec/unit/attributes/destroy_spec.rb
+- spec/unit/attributes/update_spec.rb
+- spec/unit/embedded_methods_spec.rb
+- spec/unit/history_spec.rb
+- spec/unit/my_instance_methods_spec.rb
+- spec/unit/options_spec.rb
+- spec/unit/singleton_methods_spec.rb
+- spec/unit/store/default_store_spec.rb
+- spec/unit/store/request_store_spec.rb
 - spec/unit/trackable_spec.rb
 - spec/unit/tracker_spec.rb
 homepage: http://github.com/aq1018/mongoid-history
@@ -233,7 +152,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Track and audit, undo and redo changes on Mongoid documents.
@@ -247,6 +166,16 @@ test_files:
 - spec/spec_helper.rb
 - spec/support/mongoid.rb
 - spec/support/mongoid_history.rb
+- spec/unit/attributes/base_spec.rb
+- spec/unit/attributes/create_spec.rb
+- spec/unit/attributes/destroy_spec.rb
+- spec/unit/attributes/update_spec.rb
+- spec/unit/embedded_methods_spec.rb
+- spec/unit/history_spec.rb
+- spec/unit/my_instance_methods_spec.rb
+- spec/unit/options_spec.rb
+- spec/unit/singleton_methods_spec.rb
+- spec/unit/store/default_store_spec.rb
+- spec/unit/store/request_store_spec.rb
 - spec/unit/trackable_spec.rb
 - spec/unit/tracker_spec.rb
-has_rdoc: