make tests pass

lib/captain_hook.rb

@@ -29,15 +29,9 @@ def run_around_hooks(method, *args, **kwargs, &block)
 
       instance = self
 
-      hook_proc = proc {
+      proc {
         run_hook(method, hook_configuration, chain, instance, *args, **kwargs)
       }
-
-      next hook_proc if hook_configuration.methods.empty?
-
-      next chain if hook_configuration.skip?(method, args, kwargs)
-
-      hook_proc
     end.call
   end
 
@@ -169,7 +163,7 @@ def decorate_method!(method_name)
 
       original_method_name = :"#{method_name}__without_hooks"
 
-      alias_method original_method_name, method_name
+      alias_method original_method_name, method_name unless method_defined?(original_method_name)
 
       # We decorate the method with the before, after and around hooks
       define_method(method_name) do |*args, **kwargs|

spec/captain_hook_spec.rb

@@ -47,9 +47,7 @@ class ResourceWithHooks
   hook :before,
        hook: BeforeAllHook.new,
        exclude: [:serve],
-       skip_when: lambda { |_args, kwargs|
-         !kwargs[:dto]
-       }
+       skip_when: ->(_args, kwargs) { !kwargs[:dto] }
 
   hook :after, methods: %i[prepare invented_one], hook: PrepareHook.new
 
@@ -95,8 +93,8 @@ def policy_context
 class ResourceChildWithHooks < ResourceWithHooks
   def foo(dto:); end
 
-  def prepare
-    super
+  def prepare(dto:)
+    super(dto: dto)
     "servig food"
   end
 end
@@ -151,12 +149,11 @@ def prepare
     subject { ResourceChildWithHooks.new }
 
     it do
-      expect_any_instance_of(BeforeAllHook).to receive(:call).once
-
+      expect_any_instance_of(BeforeAllHook).to receive(:call).twice
       subject.foo(dto: "fooing")
 
       expect_any_instance_of(BeforeAllHook).to receive(:call).once
-      subject.prepare
+      subject.prepare(dto: "foo")
     end
   end
 end