mongoid-history 0.4.0 → 0.4.1

data/spec/integration/multi_relation_spec.rb

@@ -1,50 +1,53 @@
-require 'spec_helper'
-
-describe Mongoid::History::Tracker do
-  before :all do
-    class Model
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-      
-      field :name, type: String
-      belongs_to :user, inverse_of: :models
-      has_and_belongs_to_many :external_users, class_name: "User", inverse_of: :external_models
-
-      track_history :on => [:name, :user, :external_user_ids], # track title and body fields only, default is :all
-                  :modifier_field => :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
-                  :modifier_field_inverse_of => nil, # no inverse modifier relationship
-                  :version_field => :version,   # adds "field :version, :type => Integer" to track current version, default is :version
-                  :track_create   =>  false,    # track document creation, default is false
-                  :track_update   =>  true,     # track document updates, default is true
-                  :track_destroy  =>  false    # track document destruction, default is false
-    end
-    
-    class User
-      include Mongoid::Document
-      has_many :models, :dependent => :destroy, inverse_of: :user
-      has_and_belongs_to_many :external_model, class_name: "Model", inverse_of: :external_users
-    end
-  end
-
-  it "should be possible to undo when having multiple relations to modifier class" do
-    user = User.new
-    user.save
-    
-    model = Model.new
-    model.name = "Foo"
-    model.user = user
-    model.save!
-    
-    model.name = "Bar"
-    model.save!
-    
-    model.undo! user
-    model.name.should == "Foo"
-  end
-
-  it "should track foreign key relations" do
-    Model.tracked_field?(:external_user_ids).should be_true
-    Model.tracked_field?(:user).should be_true
-    Model.tracked_field?(:user_id).should be_true
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  before :all do
+    class Model
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name, type: String
+      belongs_to :user, inverse_of: :models
+      has_and_belongs_to_many :external_users, class_name: "User", inverse_of: :external_models
+
+      track_history on: [:name, :user, :external_user_ids], # track title and body fields only, default is :all
+                    modifier_field: :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
+                    modifier_field_inverse_of: nil, # no inverse modifier relationship
+                    version_field: :version,   # adds "field :version, :type => Integer" to track current version, default is :version
+                    track_create: false,    # track document creation, default is false
+                    track_update: true,     # track document updates, default is true
+                    track_destroy: false    # track document destruction, default is false
+    end
+
+    class User
+      include Mongoid::Document
+      has_many :models, dependent: :destroy, inverse_of: :user
+      has_and_belongs_to_many :external_model, class_name: "Model", inverse_of: :external_users
+    end
+  end
+
+  it "should be possible to undo when having multiple relations to modifier class" do
+    user = User.new
+    user.save
+
+    model = Model.new
+    model.name = "Foo"
+    model.user = user
+    model.save!
+
+    model.name = "Bar"
+    model.save!
+
+    model.undo! user
+    model.name.should == "Foo"
+
+    model.redo! user, 1
+    model.name.should == "Bar"
+  end
+
+  it "should track foreign key relations" do
+    Model.tracked_field?(:external_user_ids).should be_true
+    Model.tracked_field?(:user).should be_true
+    Model.tracked_field?(:user_id).should be_true
+  end
+end

data/spec/integration/nested_embedded_documents_spec.rb

@@ -1,77 +1,77 @@
-require 'spec_helper'
-
-describe Mongoid::History::Tracker do
-  before :all do
-    class Model
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-      
-      field :name, type: String
-      belongs_to :user, inverse_of: :models
-      embeds_many :embones
-
-      track_history   :on => :all,       # track title and body fields only, default is :all
-                  :modifier_field => :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
-                  :version_field => :version,   # adds "field :version, :type => Integer" to track current version, default is :version
-                  :track_create   =>  false,    # track document creation, default is false
-                  :track_update   =>  true,     # track document updates, default is true
-                  :track_destroy  =>  false    # track document destruction, default is false
-    end
-    
-    class Embone
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-      
-      field :name
-      embeds_many :embtwos, store_as: :ems
-      embedded_in :model
-      
-      track_history   :on => :all,       # track title and body fields only, default is :all
-                  :modifier_field => :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
-                  :version_field => :version,   # adds "field :version, :type => Integer" to track current version, default is :version
-                  :track_create   =>  false,    # track document creation, default is false
-                  :track_update   =>  true,     # track document updates, default is true
-                  :track_destroy  =>  false,    # track document destruction, default is false
-                  :scope => :model
-    end
-    
-    class Embtwo
-      include Mongoid::Document
-      include Mongoid::History::Trackable
-      
-      field :name
-      embedded_in :embone
-      
-      track_history   :on => :all,       # track title and body fields only, default is :all
-                  :modifier_field => :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
-                  :version_field => :version,   # adds "field :version, :type => Integer" to track current version, default is :version
-                  :track_create   =>  false,    # track document creation, default is false
-                  :track_update   =>  true,     # track document updates, default is true
-                  :track_destroy  =>  false,    # track document destruction, default is false
-                  :scope => :model
-    end
-    
-    class User
-      include Mongoid::Document
-      has_many :models, :dependent => :destroy, inverse_of: :user
-    end
-  end
-
-  it "should be able to track history for nested embedded documents" do
-    user = User.new
-    user.save!
-    
-    model = Model.new({name: "m1name"})
-    model.user = user
-    model.save!
-    embedded1 = model.embones.create({name: "e1name"})
-    embedded2 = embedded1.embtwos.create({name: "e2name"})
-
-    embedded2.name = "a new name"
-    embedded2.save!
-    
-    model.history_tracks.first.undo! user
-    embedded1.reload.name.should == "e1name"
-    embedded2.reload.name.should == "e2name"
-  end
-end
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  before :all do
+    class Model
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name, type: String
+      belongs_to :user, inverse_of: :models
+      embeds_many :embones
+
+      track_history   on: :all,       # track title and body fields only, default is :all
+                      modifier_field: :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
+                      version_field: :version,   # adds "field :version, :type => Integer" to track current version, default is :version
+                      track_create: false,    # track document creation, default is false
+                      track_update: true,     # track document updates, default is true
+                      track_destroy: false    # track document destruction, default is false
+    end
+
+    class Embone
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name
+      embeds_many :embtwos, store_as: :ems
+      embedded_in :model
+
+      track_history   on: :all,       # track title and body fields only, default is :all
+                      modifier_field: :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
+                      version_field: :version,   # adds "field :version, :type => Integer" to track current version, default is :version
+                      track_create: false,    # track document creation, default is false
+                      track_update: true,     # track document updates, default is true
+                      track_destroy: false,    # track document destruction, default is false
+                      scope: :model
+    end
+
+    class Embtwo
+      include Mongoid::Document
+      include Mongoid::History::Trackable
+
+      field :name
+      embedded_in :embone
+
+      track_history   on: :all,       # track title and body fields only, default is :all
+                      modifier_field: :modifier, # adds "referenced_in :modifier" to track who made the change, default is :modifier
+                      version_field: :version,   # adds "field :version, :type => Integer" to track current version, default is :version
+                      track_create: false,    # track document creation, default is false
+                      track_update: true,     # track document updates, default is true
+                      track_destroy: false,    # track document destruction, default is false
+                      scope: :model
+    end
+
+    class User
+      include Mongoid::Document
+      has_many :models, dependent: :destroy, inverse_of: :user
+    end
+  end
+
+  it "should be able to track history for nested embedded documents" do
+    user = User.new
+    user.save!
+
+    model = Model.new(name: "m1name")
+    model.user = user
+    model.save!
+    embedded1 = model.embones.create(name: "e1name")
+    embedded2 = embedded1.embtwos.create(name: "e2name")
+
+    embedded2.name = "a new name"
+    embedded2.save!
+
+    model.history_tracks.first.undo! user
+    embedded1.reload.name.should == "e1name"
+    embedded2.reload.name.should == "e2name"
+  end
+end

data/spec/integration/subclasses_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Mongoid::History::Tracker do
+  before :all do
+    class Element
+      include Mongoid::Document
+      include Mongoid::Timestamps
+      include Mongoid::History::Trackable
+
+      field :body
+
+      track_history on: [:body], track_create: true, track_update: true, track_destroy: true
+    end
+
+    class Prompt < Element
+    end
+  end
+
+  it "tracks subclass create and update" do
+    prompt = Prompt.new
+    expect { prompt.save! }.to change(Tracker, :count).by(1)
+    expect { prompt.update_attributes!(body: "one") }.to change(Tracker, :count).by(1)
+    prompt.undo!
+    prompt.body.should be_blank
+    prompt.redo! nil, 2
+    prompt.body.should == "one"
+    expect { prompt.destroy }.to change(Tracker, :count).by(1)
+  end
+end

data/spec/spec_helper.rb

@@ -1,11 +1,10 @@
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
-
-require 'rubygems'
-require 'bundler/setup'
-
-Bundler.require :default, :test
-
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
-
-require 'mongoid-history'
-
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+
+require 'rubygems'
+require 'bundler/setup'
+
+Bundler.require :default, :test
+
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+
+require 'mongoid-history'

data/spec/support/mongoid.rb

@@ -1,15 +1,11 @@
-Mongoid.configure do |config|
-  config.connect_to('mongoid_history_test')
-end
-
-RSpec.configure do |config|
-  config.before :each do
-    Mongoid.observers = Mongoid::History::Sweeper
-  end
-
-  config.after(:each) do
-    Mongoid.purge!
-  end
-
-  config.backtrace_clean_patterns = [ /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,15 +1,13 @@
-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
-    Mongoid::History.trackable_class_options = nil
-    Thread.current[:mongoid_history_sweeper_controller] = 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
+    Mongoid::History.trackable_class_options = nil
+  end
+end