From 5d2c3a14a3bfbde2b48d9021a3e58c3f6fe024e7 Mon Sep 17 00:00:00 2001
From: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed, 8 May 2013 10:31:02 -0700
Subject: [PATCH] adding more tests around callback inconsistencies

---
 activesupport/test/callbacks_test.rb | 52 +++++++++++++++++++++++++---
 1 file changed, 47 insertions(+), 5 deletions(-)

diff --git a/activesupport/test/callbacks_test.rb b/activesupport/test/callbacks_test.rb
index 0430d431b8..8df1c5bb7a 100644
--- a/activesupport/test/callbacks_test.rb
+++ b/activesupport/test/callbacks_test.rb
@@ -802,17 +802,18 @@ module CallbacksTest
     end
   end
 
-  class AddCallbackTypeTest < ActiveSupport::TestCase
+  class CallbackTypeTest < ActiveSupport::TestCase
     def build_class(callback, n = 10)
       Class.new {
         include ActiveSupport::Callbacks
         define_callbacks :foo
         n.times { set_callback :foo, callback }
         def run; run_callbacks :foo; end
+        def self.skip(thing); skip_callback :foo, thing; end
       }
     end
 
-    def test_class
+    def test_add_class
       calls = []
       callback = Class.new {
         define_singleton_method(:before) { |o| calls << o }
@@ -821,13 +822,13 @@ module CallbacksTest
       assert_equal 10, calls.length
     end
 
-    def test_lambda
+    def test_add_lambda
       calls = []
       build_class(->(o) { calls << o }).new.run
       assert_equal 10, calls.length
     end
 
-    def test_symbol
+    def test_add_symbol
       calls = []
       klass = build_class(:bar)
       klass.class_eval { define_method(:bar) { calls << klass } }
@@ -835,12 +836,53 @@ module CallbacksTest
       assert_equal 1, calls.length
     end
 
-    def test_string
+    def test_add_eval
       calls = []
       klass = build_class("bar")
       klass.class_eval { define_method(:bar) { calls << klass } }
       klass.new.run
       assert_equal 1, calls.length
     end
+
+    def test_skip_class # removes one at a time
+      calls = []
+      callback = Class.new {
+        define_singleton_method(:before) { |o| calls << o }
+      }
+      klass = build_class(callback)
+      9.downto(0) { |i|
+        klass.skip callback
+        klass.new.run
+        assert_equal i, calls.length
+        calls.clear
+      }
+    end
+
+    def test_skip_lambda # removes nothing
+      calls = []
+      callback = ->(o) { calls << o }
+      klass = build_class(callback)
+      10.times { klass.skip callback }
+      klass.new.run
+      assert_equal 10, calls.length
+    end
+
+    def test_skip_symbol # removes all
+      calls = []
+      klass = build_class(:bar)
+      klass.class_eval { define_method(:bar) { calls << klass } }
+      klass.skip :bar
+      klass.new.run
+      assert_equal 0, calls.length
+    end
+
+    def test_skip_eval # removes nothing
+      calls = []
+      klass = build_class("bar")
+      klass.class_eval { define_method(:bar) { calls << klass } }
+      klass.skip "bar"
+      klass.new.run
+      assert_equal 1, calls.length
+    end
   end
 end