track_changes 0.5.0 → 0.5.1

data/test/rails_root/script/console

@@ -1,3 +1,3 @@
-#!/usr/bin/env ruby
-require File.expand_path('../../config/boot',  __FILE__)
-require 'commands/console'
+#!/usr/bin/env ruby
+require File.expand_path('../../config/boot',  __FILE__)
+require 'commands/console'

data/test/rails_root/script/generate

@@ -1,3 +1,3 @@
-#!/usr/bin/env ruby
-require File.expand_path('../../config/boot',  __FILE__)
-require 'commands/generate'
+#!/usr/bin/env ruby
+require File.expand_path('../../config/boot',  __FILE__)
+require 'commands/generate'

data/test/result_test.rb

@@ -1,35 +1,35 @@
-require File.dirname(__FILE__) + '/test_helper'
-
-class ResultTest < Test::Unit::TestCase
-  context "a Result instance" do
-    setup do
-      @controller = mock()
-      @changes = { "attr" => ["old", "new"] }
-
-      @filter = TrackChanges::Result.new(:model, @controller, @changes)
-    end
-
-    should "define given symbol as method" do
-      @controller.expects(:instance_variable_get).with() {|v| v == "@model"}.returns(:ivar_result)
-      assert_equal :ivar_result, @filter.model
-    end
-
-    should "send method_missing calls to controller" do
-      @controller.expects(:bogus_method).returns(:got_it)
-      assert_equal :got_it, @filter.bogus_method
-    end
-
-    should "get @current_user from controller" do
-      @controller.expects(:instance_variable_get).with() {|v| v == "@current_user"}.returns(:ivar_result)
-      assert_equal :ivar_result, @filter.current_user
-    end
-
-    should "return changes" do
-      assert_equal @changes, @filter.changes
-    end
-
-    should "return controller" do
-      assert_equal @controller, @filter.controller
-    end
-  end
-end
+require File.dirname(__FILE__) + '/test_helper'
+
+class ResultTest < Test::Unit::TestCase
+  context "a Result instance" do
+    setup do
+      @controller = mock()
+      @changes = { "attr" => ["old", "new"] }
+
+      @filter = TrackChanges::Result.new(:model, @controller, @changes)
+    end
+
+    should "define given symbol as method" do
+      @controller.expects(:instance_variable_get).with() {|v| v == "@model"}.returns(:ivar_result)
+      assert_equal :ivar_result, @filter.model
+    end
+
+    should "send method_missing calls to controller" do
+      @controller.expects(:bogus_method).returns(:got_it)
+      assert_equal :got_it, @filter.bogus_method
+    end
+
+    should "get @current_user from controller" do
+      @controller.expects(:instance_variable_get).with() {|v| v == "@current_user"}.returns(:ivar_result)
+      assert_equal :ivar_result, @filter.current_user
+    end
+
+    should "return changes" do
+      assert_equal @changes, @filter.changes
+    end
+
+    should "return controller" do
+      assert_equal @controller, @filter.controller
+    end
+  end
+end

data/test/test_helper.rb

@@ -1,29 +1,29 @@
-# Copied from the shoulda library
-require 'fileutils'
-
-# Load the environment
-ENV['RAILS_ENV'] = 'test'
-
-rails_root = File.dirname(__FILE__) + '/rails_root'
-
-require "#{rails_root}/config/environment.rb"
-
-# Load the testing framework
-require 'test_help'
-silence_warnings { RAILS_ENV = ENV['RAILS_ENV'] }
-
-# Run the migrations
-ActiveRecord::Migration.verbose = false
-ActiveRecord::Migrator.migrate("#{RAILS_ROOT}/db/migrate")
-
-# Setup the fixtures path
-ActiveSupport::TestCase.fixture_path =
-  File.join(File.dirname(__FILE__), "fixtures")
-
-class ActiveSupport::TestCase #:nodoc:
-  self.use_transactional_fixtures = false
-  self.use_instantiated_fixtures  = false
-end
-
-require 'track_changes'
-require 'shoulda'
+# Copied from the shoulda library
+require 'fileutils'
+
+# Load the environment
+ENV['RAILS_ENV'] = 'test'
+
+rails_root = File.dirname(__FILE__) + '/rails_root'
+
+require "#{rails_root}/config/environment.rb"
+
+# Load the testing framework
+require 'test_help'
+silence_warnings { RAILS_ENV = ENV['RAILS_ENV'] }
+
+# Run the migrations
+ActiveRecord::Migration.verbose = false
+ActiveRecord::Migrator.migrate("#{RAILS_ROOT}/db/migrate")
+
+# Setup the fixtures path
+ActiveSupport::TestCase.fixture_path =
+  File.join(File.dirname(__FILE__), "fixtures")
+
+class ActiveSupport::TestCase #:nodoc:
+  self.use_transactional_fixtures = false
+  self.use_instantiated_fixtures  = false
+end
+
+require 'track_changes'
+require 'shoulda'

data/test/track_changes/audit_filter_test.rb

@@ -1,154 +1,154 @@
-require 'test_helper'
-
-module TrackChanges
-  class AuditFilterTest < Test::Unit::TestCase
-    should "accept a single model" do
-      assert TrackChanges::AuditFilter.new(:model)
-    end
-
-    should "accept multiple models" do
-      assert TrackChanges::AuditFilter.new(:model_a, :model_b)
-    end
-
-    should "respond to #before" do
-      assert TrackChanges::AuditFilter.new.respond_to?(:before)
-    end
-
-    should "respond to #after" do
-      assert TrackChanges::AuditFilter.new.respond_to?(:after)
-    end
-
-    should "create an audit when before and after called" do
-      expected_change_set = {"value" => ["old", "new"]}
-      expected_user       = "current user"
-
-      audit = mock('audit')
-
-      model = mock('model')
-      model.stubs(:attributes=).returns({})
-      model.stubs(:changes).returns(expected_change_set)
-
-      modified_model = mock('modified model')
-      modified_model.stubs(:changed?).returns(false)
-      modified_model.stubs(:attributes).returns({})
-      modified_model.stubs(:audits).returns(audit)
-
-      controller = mock('controller')
-      controller.stubs(:instance_variable_get).returns(model, modified_model)
-      controller.stubs(:current_user).returns(expected_user)
-
-      audit_filter = AuditFilter.new(:model)
-      audit_filter.before(controller)
-
-      audit.expects(:create!).with(has_entries(:user => expected_user, :change_set => expected_change_set)).returns(true)
-      audit_filter.after(controller)
-    end
-
-    should "ignore nil models" do
-      controller = mock('controller')
-      controller.expects(:instance_variable_get).returns(nil)
-
-      audit_filter = AuditFilter.new(:model)
-      assert_nothing_raised do
-        audit_filter.before(controller)
-        audit_filter.after(controller)
-      end
-    end
-
-    should "respect ignore_nil option" do
-      controller = mock('controller')
-      controller.stubs(:instance_variable_get).returns(nil)
-
-      audit_filter = AuditFilter.new(:model, :ignore_nil => false)
-      assert_raises TypeError do
-        audit_filter.before(controller)
-        audit_filter.after(controller)
-      end
-
-      audit_filter = AuditFilter.new(:model, :ignore_nil => true)
-      assert_nothing_raised do
-        audit_filter.before(controller)
-        audit_filter.after(controller)
-      end
-    end
-
-    should "respect audit_accessor option" do
-      change = mock('change')
-      change.stubs(:create!).returns(true)
-
-      model = mock('model')
-      model.stubs(:attributes=).returns({})
-      model.stubs(:changes).returns({:not => :empty})
-
-      modified_model = mock('modified model')
-      modified_model.stubs(:changed?).returns(false)
-      modified_model.stubs(:attributes).returns({})
-      modified_model.stubs(:changes).returns(change)
-
-      controller = mock('controller')
-      controller.stubs(:instance_variable_get).returns(model, modified_model)
-      controller.stubs(:current_user).returns("user")
-
-      audit_filter = AuditFilter.new(:model, :audit_accessor => :changes)
-      audit_filter.before(controller)
-
-      # Test call to :changes
-      audit_filter.after(controller)
-
-      audit = mock('audit')
-      audit.stubs(:create!).returns(true)
-      modified_model.stubs(:audits).returns(audit)
-
-      # Test call to :audits
-      audit_filter.after(controller)
-    end
-
-    should "respect :current_user option with :user" do
-      audit = mock('audit')
-      audit.stubs(:create!).returns(true)
-
-      model = mock('model')
-      model.stubs(:attributes=).returns({})
-      model.stubs(:changes).returns({:not => :empty})
-
-      modified_model = mock('modified model')
-      modified_model.stubs(:changed?).returns(false)
-      modified_model.stubs(:attributes).returns({})
-      modified_model.stubs(:audits).returns(audit)
-
-      controller = mock('controller')
-      controller.stubs(:instance_variable_get).returns(model, modified_model)
-      controller.stubs(:user).returns("configured user")
-
-      audit_filter = AuditFilter.new(:model, :current_user => :user)
-      audit_filter.before(controller)
-
-      # Test call to :user
-      audit_filter.after(controller)
-    end
-
-    should "respect :current_user option with :current_user" do
-      audit = mock('audit')
-      audit.stubs(:create!).returns(true)
-
-      model = mock('model')
-      model.stubs(:attributes=).returns({})
-      model.stubs(:changes).returns({:not => :empty})
-
-      modified_model = mock('modified model')
-      modified_model.stubs(:changed?).returns(false)
-      modified_model.stubs(:attributes).returns({})
-      modified_model.stubs(:audits).returns(audit)
-
-      controller = mock('controller')
-      controller.stubs(:instance_variable_get).returns(model, modified_model)
-      controller.stubs(:current_user).returns("configured user")
-
-      audit_filter = AuditFilter.new(:model, :current_user => :current_user)
-      audit_filter.before(controller)
-
-      # Test call to :current_user
-      audit_filter.after(controller)
-    end
-  end
-end
+require 'test_helper'
+
+module TrackChanges
+  class AuditFilterTest < Test::Unit::TestCase
+    should "accept a single model" do
+      assert TrackChanges::AuditFilter.new(:model)
+    end
+
+    should "accept multiple models" do
+      assert TrackChanges::AuditFilter.new(:model_a, :model_b)
+    end
+
+    should "respond to #before" do
+      assert TrackChanges::AuditFilter.new.respond_to?(:before)
+    end
+
+    should "respond to #after" do
+      assert TrackChanges::AuditFilter.new.respond_to?(:after)
+    end
+
+    should "create an audit when before and after called" do
+      expected_change_set = {"value" => ["old", "new"]}
+      expected_user       = "current user"
+
+      audit = mock('audit')
+
+      model = mock('model')
+      model.stubs(:attributes=).returns({})
+      model.stubs(:changes).returns(expected_change_set)
+
+      modified_model = mock('modified model')
+      modified_model.stubs(:changed?).returns(false)
+      modified_model.stubs(:attributes).returns({})
+      modified_model.stubs(:audits).returns(audit)
+
+      controller = mock('controller')
+      controller.stubs(:instance_variable_get).returns(model, modified_model)
+      controller.stubs(:current_user).returns(expected_user)
+
+      audit_filter = AuditFilter.new(:model)
+      audit_filter.before(controller)
+
+      audit.expects(:create!).with(has_entries(:user => expected_user, :change_set => expected_change_set)).returns(true)
+      audit_filter.after(controller)
+    end
+
+    should "ignore nil models" do
+      controller = mock('controller')
+      controller.expects(:instance_variable_get).returns(nil)
+
+      audit_filter = AuditFilter.new(:model)
+      assert_nothing_raised do
+        audit_filter.before(controller)
+        audit_filter.after(controller)
+      end
+    end
+
+    should "respect ignore_nil option" do
+      controller = mock('controller')
+      controller.stubs(:instance_variable_get).returns(nil)
+
+      audit_filter = AuditFilter.new(:model, :ignore_nil => false)
+      assert_raises TypeError do
+        audit_filter.before(controller)
+        audit_filter.after(controller)
+      end
+
+      audit_filter = AuditFilter.new(:model, :ignore_nil => true)
+      assert_nothing_raised do
+        audit_filter.before(controller)
+        audit_filter.after(controller)
+      end
+    end
+
+    should "respect audit_accessor option" do
+      change = mock('change')
+      change.stubs(:create!).returns(true)
+
+      model = mock('model')
+      model.stubs(:attributes=).returns({})
+      model.stubs(:changes).returns({:not => :empty})
+
+      modified_model = mock('modified model')
+      modified_model.stubs(:changed?).returns(false)
+      modified_model.stubs(:attributes).returns({})
+      modified_model.stubs(:changes).returns(change)
+
+      controller = mock('controller')
+      controller.stubs(:instance_variable_get).returns(model, modified_model)
+      controller.stubs(:current_user).returns("user")
+
+      audit_filter = AuditFilter.new(:model, :audit_accessor => :changes)
+      audit_filter.before(controller)
+
+      # Test call to :changes
+      audit_filter.after(controller)
+
+      audit = mock('audit')
+      audit.stubs(:create!).returns(true)
+      modified_model.stubs(:audits).returns(audit)
+
+      # Test call to :audits
+      audit_filter.after(controller)
+    end
+
+    should "respect :current_user option with :user" do
+      audit = mock('audit')
+      audit.stubs(:create!).returns(true)
+
+      model = mock('model')
+      model.stubs(:attributes=).returns({})
+      model.stubs(:changes).returns({:not => :empty})
+
+      modified_model = mock('modified model')
+      modified_model.stubs(:changed?).returns(false)
+      modified_model.stubs(:attributes).returns({})
+      modified_model.stubs(:audits).returns(audit)
+
+      controller = mock('controller')
+      controller.stubs(:instance_variable_get).returns(model, modified_model)
+      controller.stubs(:user).returns("configured user")
+
+      audit_filter = AuditFilter.new(:model, :current_user => :user)
+      audit_filter.before(controller)
+
+      # Test call to :user
+      audit_filter.after(controller)
+    end
+
+    should "respect :current_user option with :current_user" do
+      audit = mock('audit')
+      audit.stubs(:create!).returns(true)
+
+      model = mock('model')
+      model.stubs(:attributes=).returns({})
+      model.stubs(:changes).returns({:not => :empty})
+
+      modified_model = mock('modified model')
+      modified_model.stubs(:changed?).returns(false)
+      modified_model.stubs(:attributes).returns({})
+      modified_model.stubs(:audits).returns(audit)
+
+      controller = mock('controller')
+      controller.stubs(:instance_variable_get).returns(model, modified_model)
+      controller.stubs(:current_user).returns("configured user")
+
+      audit_filter = AuditFilter.new(:model, :current_user => :current_user)
+      audit_filter.before(controller)
+
+      # Test call to :current_user
+      audit_filter.after(controller)
+    end
+  end
+end

data/test/track_changes/configuration_test.rb

@@ -1,36 +1,36 @@
-require 'test_helper'
-
-module TrackChanges
-  class ConfigurationTest < ActiveSupport::TestCase
-    def setup
-      @configuration = TrackChanges::Configuration.new
-    end
-
-    should "have audit_association accessor" do
-      assert @configuration.respond_to?(:audit_association)
-      assert @configuration.respond_to?(:audit_association=)
-    end
-
-    should "have current_user accessor" do
-      assert @configuration.respond_to?(:current_user)
-      assert @configuration.respond_to?(:current_user=)
-    end
-
-    should "have ignore_nil accessor" do
-      assert @configuration.respond_to?(:ignore_nil)
-      assert @configuration.respond_to?(:ignore_nil=)
-    end
-
-    should "set default audit_association to Audit" do
-      assert_equal :audits, @configuration.audit_association
-    end
-
-    should "set default current user association" do
-      assert_equal :current_user, @configuration.current_user
-    end
-
-    should "set default ignore_nil" do
-      assert_equal true, @configuration.ignore_nil
-    end
-  end
-end
+require 'test_helper'
+
+module TrackChanges
+  class ConfigurationTest < ActiveSupport::TestCase
+    def setup
+      @configuration = TrackChanges::Configuration.new
+    end
+
+    should "have audit_association accessor" do
+      assert @configuration.respond_to?(:audit_association)
+      assert @configuration.respond_to?(:audit_association=)
+    end
+
+    should "have current_user accessor" do
+      assert @configuration.respond_to?(:current_user)
+      assert @configuration.respond_to?(:current_user=)
+    end
+
+    should "have ignore_nil accessor" do
+      assert @configuration.respond_to?(:ignore_nil)
+      assert @configuration.respond_to?(:ignore_nil=)
+    end
+
+    should "set default audit_association to Audit" do
+      assert_equal :audits, @configuration.audit_association
+    end
+
+    should "set default current user association" do
+      assert_equal :current_user, @configuration.current_user
+    end
+
+    should "set default ignore_nil" do
+      assert_equal true, @configuration.ignore_nil
+    end
+  end
+end

data/test/track_changes/initializer_test.rb

@@ -1,37 +1,37 @@
-require 'test_helper'
-
-module TrackChanges
-  class InitializerTest < ActiveSupport::TestCase
-    should "be a singleton" do
-      first_instance = Initializer.instance
-      second_instance = Initializer.instance
-      assert_equal first_instance, second_instance
-    end
-
-    should "have an accessor for configuration" do
-      initializer = Initializer.instance
-
-      assert initializer.respond_to?(:configuration)
-    end
-
-    should "yield configuration on call to instance" do
-      block_executed = false
-      initializer = Initializer.instance do |configuration|
-        block_executed = true
-        assert configuration.kind_of?(TrackChanges::Configuration)
-      end
-
-      assert block_executed, "block wasn't executed"
-    end
-
-    should "set global configuration" do
-      initializer = Initializer.instance do |configuration|
-        configuration.audit_association = :test_audit_association
-      end
-
-      initializer = Initializer.instance
-      configuration = initializer.configuration
-      assert_equal :test_audit_association, configuration.audit_association
-    end
-  end
-end
+require 'test_helper'
+
+module TrackChanges
+  class InitializerTest < ActiveSupport::TestCase
+    should "be a singleton" do
+      first_instance = Initializer.instance
+      second_instance = Initializer.instance
+      assert_equal first_instance, second_instance
+    end
+
+    should "have an accessor for configuration" do
+      initializer = Initializer.instance
+
+      assert initializer.respond_to?(:configuration)
+    end
+
+    should "yield configuration on call to instance" do
+      block_executed = false
+      initializer = Initializer.instance do |configuration|
+        block_executed = true
+        assert configuration.kind_of?(TrackChanges::Configuration)
+      end
+
+      assert block_executed, "block wasn't executed"
+    end
+
+    should "set global configuration" do
+      initializer = Initializer.instance do |configuration|
+        configuration.audit_association = :test_audit_association
+      end
+
+      initializer = Initializer.instance
+      configuration = initializer.configuration
+      assert_equal :test_audit_association, configuration.audit_association
+    end
+  end
+end