mongoid-history 0.8.3 → 0.8.5

data/spec/spec_helper.rb

@@ -1,30 +1,32 @@
-require 'coveralls'
-Coveralls.wear!
-
-$LOAD_PATH.push File.expand_path('../../lib', __FILE__)
-
-require 'active_support/all'
-require 'mongoid'
-require 'request_store'
-
-# Undefine RequestStore so that it may be stubbed in specific tests
-RequestStoreTemp = RequestStore
-Object.send(:remove_const, :RequestStore)
-
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
-
-require 'mongoid/history'
-
-RSpec.configure do |config|
-  config.raise_errors_for_deprecations!
-
-  config.before :all do
-    Mongoid.logger.level = Logger::INFO
-    Mongo::Logger.logger.level = Logger::INFO if Mongoid::Compatibility::Version.mongoid5_or_newer?
-    Mongoid.belongs_to_required_by_default = false if Mongoid::Compatibility::Version.mongoid6?
-  end
-
-  config.before :each do
-    Mongoid::History.reset!
-  end
-end
+require 'coveralls'
+Coveralls.wear!
+
+$LOAD_PATH.push File.expand_path('../../lib', __FILE__)
+
+require 'active_support/all'
+require 'mongoid'
+require 'request_store'
+
+# Undefine RequestStore so that it may be stubbed in specific tests
+RequestStoreTemp = RequestStore
+Object.send(:remove_const, :RequestStore)
+
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+
+require 'mongoid/history'
+
+RSpec.configure do |config|
+  config.raise_errors_for_deprecations!
+
+  config.before :all do
+    Mongoid.logger.level = Logger::INFO
+    Mongo::Logger.logger.level = Logger::INFO if Mongoid::Compatibility::Version.mongoid5_or_newer?
+    Mongoid.belongs_to_required_by_default = false if Mongoid::Compatibility::Version.mongoid6?
+  end
+
+  config.before :each do
+    Mongoid::History.reset!
+  end
+
+  config.include ErrorHelpers
+end

data/spec/support/error_helpers.rb

@@ -0,0 +1,7 @@
+module ErrorHelpers
+  def ignore_errors
+    yield
+  rescue StandardError => e
+    Mongoid.logger.debug "ignored error #{e}"
+  end
+end

data/spec/support/mongoid.rb

@@ -1,11 +1,11 @@
-Mongoid.configure do |config|
-  config.connect_to('mongoid_history_test')
-end
-
-RSpec.configure do |config|
-  config.after(:each) do
-    Mongoid.purge!
-  end
-
-  config.backtrace_exclusion_patterns = [%r{lib\/rspec\/(core|expectations|matchers|mocks)}]
-end
+Mongoid.configure do |config|
+  config.connect_to('mongoid_history_test')
+end
+
+RSpec.configure do |config|
+  config.after(:each) do
+    Mongoid.purge!
+  end
+
+  config.backtrace_exclusion_patterns = [%r{lib\/rspec\/(core|expectations|matchers|mocks)}]
+end

data/spec/support/mongoid_history.rb

@@ -1,12 +1,12 @@
-RSpec.configure do |config|
-  config.before :each do
-    class Tracker
-      include Mongoid::History::Tracker
-    end
-    Mongoid::History.tracker_class_name = 'Tracker'
-    Mongoid::History.modifier_class_name = 'User'
-  end
-  config.after :each do
-    Mongoid::History.tracker_class_name = nil
-  end
-end
+RSpec.configure do |config|
+  config.before :each do
+    class Tracker
+      include Mongoid::History::Tracker
+    end
+    Mongoid::History.tracker_class_name = 'Tracker'
+    Mongoid::History.modifier_class_name = 'User'
+  end
+  config.after :each do
+    Mongoid::History.tracker_class_name = nil
+  end
+end

data/spec/unit/attributes/base_spec.rb

@@ -1,141 +1,141 @@
-require 'spec_helper'
-
-describe Mongoid::History::Attributes::Base do
-  before :each do
-    class ModelOne
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-
-      field :foo
-      field :b, as: :bar
-    end
-
-    class ModelTwo
-      include Mongoid::Document
-
-      field :foo
-      field :goo
-    end
-  end
-
-  after :each do
-    Object.send(:remove_const, :ModelOne)
-    Object.send(:remove_const, :ModelTwo)
-  end
-
-  let(:obj_one) { ModelOne.new }
-  let(:base) { described_class.new(obj_one) }
-  subject { base }
-
-  it { is_expected.to respond_to(:trackable) }
-
-  describe '#initialize' do
-    it { expect(base.instance_variable_get(:@trackable)).to eq obj_one }
-  end
-
-  describe '#trackable_class' do
-    subject { base.send(:trackable_class) }
-    it { is_expected.to eq ModelOne }
-  end
-
-  describe '#aliased_fields' do
-    subject { base.send(:aliased_fields) }
-    it { is_expected.to eq('id' => '_id', 'bar' => 'b') }
-  end
-
-  describe '#changes_method' do
-    before(:each) do
-      ModelOne.track_history changes_method: :my_changes
-    end
-    subject { base.send(:changes_method) }
-    it { is_expected.to eq :my_changes }
-  end
-
-  describe '#changes' do
-    before(:each) do
-      ModelOne.track_history
-      allow(obj_one).to receive(:changes) { { 'foo' => ['Foo', 'Foo-new'] } }
-    end
-    subject { base.send(:changes) }
-    it { is_expected.to eq('foo' => ['Foo', 'Foo-new']) }
-  end
-
-  describe '#format_field' do
-    subject { base.send(:format_field, :bar, 'foo') }
-
-    context 'when formatted via string' do
-      before do
-        ModelOne.track_history format: { bar: '*%s*' }
-      end
-
-      it { is_expected.to eq '*foo*' }
-    end
-
-    context 'when formatted via proc' do
-      before do
-        ModelOne.track_history format: { bar: ->(v) { v * 2 } }
-      end
-
-      it { is_expected.to eq 'foofoo' }
-    end
-
-    context 'when not formatted' do
-      before do
-        ModelOne.track_history
-      end
-
-      it { is_expected.to eq 'foo' }
-    end
-  end
-
-  shared_examples 'formats embedded relation' do |relation_type|
-    let(:model_two) { ModelTwo.new(foo: :bar, goo: :baz) }
-
-    before :each do
-      ModelOne.send(relation_type, :model_two)
-    end
-
-    subject { base.send("format_#{relation_type}_relation", :model_two, model_two.attributes) }
-
-    context 'with permitted attributes' do
-      before do
-        ModelOne.track_history on: { model_two: %i[foo] }
-      end
-
-      it 'should select only permitted attributes' do
-        is_expected.to include('foo' => :bar)
-        is_expected.to_not include('goo')
-      end
-    end
-
-    context 'with attributes formatted via string' do
-      before do
-        ModelOne.track_history on: { model_two: %i[foo] }, format: { model_two: { foo: '&%s&' } }
-      end
-
-      it 'should select obfuscate permitted attributes' do
-        is_expected.to include('foo' => '&bar&')
-        is_expected.to_not include('goo')
-      end
-    end
-
-    context 'with attributes formatted via proc' do
-      before do
-        ModelOne.track_history on: { model_two: %i[foo] }, format: { model_two: { foo: ->(v) { v.to_s * 2 } } }
-      end
-
-      it 'should select obfuscate permitted attributes' do
-        is_expected.to include('foo' => 'barbar')
-        is_expected.to_not include('goo')
-      end
-    end
-  end
-
-  describe '#format_embeds_one_relation' do
-    include_examples 'formats embedded relation', :embeds_one
-  end
-
-  describe '#format_embeds_many_relation' do
-    include_examples 'formats embedded relation', :embeds_many
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Attributes::Base do
+  before :each do
+    class ModelOne
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :foo
+      field :b, as: :bar
+    end
+
+    class ModelTwo
+      include Mongoid::Document
+
+      field :foo
+      field :goo
+    end
+  end
+
+  after :each do
+    Object.send(:remove_const, :ModelOne)
+    Object.send(:remove_const, :ModelTwo)
+  end
+
+  let(:obj_one) { ModelOne.new }
+  let(:base) { described_class.new(obj_one) }
+  subject { base }
+
+  it { is_expected.to respond_to(:trackable) }
+
+  describe '#initialize' do
+    it { expect(base.instance_variable_get(:@trackable)).to eq obj_one }
+  end
+
+  describe '#trackable_class' do
+    subject { base.send(:trackable_class) }
+    it { is_expected.to eq ModelOne }
+  end
+
+  describe '#aliased_fields' do
+    subject { base.send(:aliased_fields) }
+    it { is_expected.to eq('id' => '_id', 'bar' => 'b') }
+  end
+
+  describe '#changes_method' do
+    before(:each) do
+      ModelOne.track_history changes_method: :my_changes
+    end
+    subject { base.send(:changes_method) }
+    it { is_expected.to eq :my_changes }
+  end
+
+  describe '#changes' do
+    before(:each) do
+      ModelOne.track_history
+      allow(obj_one).to receive(:changes) { { 'foo' => ['Foo', 'Foo-new'] } }
+    end
+    subject { base.send(:changes) }
+    it { is_expected.to eq('foo' => ['Foo', 'Foo-new']) }
+  end
+
+  describe '#format_field' do
+    subject { base.send(:format_field, :bar, 'foo') }
+
+    context 'when formatted via string' do
+      before do
+        ModelOne.track_history format: { bar: '*%s*' }
+      end
+
+      it { is_expected.to eq '*foo*' }
+    end
+
+    context 'when formatted via proc' do
+      before do
+        ModelOne.track_history format: { bar: ->(v) { v * 2 } }
+      end
+
+      it { is_expected.to eq 'foofoo' }
+    end
+
+    context 'when not formatted' do
+      before do
+        ModelOne.track_history
+      end
+
+      it { is_expected.to eq 'foo' }
+    end
+  end
+
+  shared_examples 'formats embedded relation' do |relation_type|
+    let(:model_two) { ModelTwo.new(foo: :bar, goo: :baz) }
+
+    before :each do
+      ModelOne.send(relation_type, :model_two)
+    end
+
+    subject { base.send("format_#{relation_type}_relation", :model_two, model_two.attributes) }
+
+    context 'with permitted attributes' do
+      before do
+        ModelOne.track_history on: { model_two: %i[foo] }
+      end
+
+      it 'should select only permitted attributes' do
+        is_expected.to include('foo' => :bar)
+        is_expected.to_not include('goo')
+      end
+    end
+
+    context 'with attributes formatted via string' do
+      before do
+        ModelOne.track_history on: { model_two: %i[foo] }, format: { model_two: { foo: '&%s&' } }
+      end
+
+      it 'should select obfuscate permitted attributes' do
+        is_expected.to include('foo' => '&bar&')
+        is_expected.to_not include('goo')
+      end
+    end
+
+    context 'with attributes formatted via proc' do
+      before do
+        ModelOne.track_history on: { model_two: %i[foo] }, format: { model_two: { foo: ->(v) { v.to_s * 2 } } }
+      end
+
+      it 'should select obfuscate permitted attributes' do
+        is_expected.to include('foo' => 'barbar')
+        is_expected.to_not include('goo')
+      end
+    end
+  end
+
+  describe '#format_embeds_one_relation' do
+    include_examples 'formats embedded relation', :embeds_one
+  end
+
+  describe '#format_embeds_many_relation' do
+    include_examples 'formats embedded relation', :embeds_many
+  end
+end