[dm-is-remixable] Don't use should_receive to see if a hook is executed

* Methods involved in hooks may not always be testable using this method,
  and it is better to test for the side effect of the method rather than
  if a specific method was called.

dm-is-remixable/lib/dm-is-remixable/is/remixable.rb

@@ -405,7 +405,7 @@ def belongs_to_name(class_name)
 
         def clone_hooks(remixable, model)
           # event name is by default :create, :destroy etc
-          remixable.instance_hooks.map do |event_name, hooks|
+          remixable.instance_hooks.each do |event_name, hooks|
             [:after, :before].each do |callback|
               hooks[callback].each do |hook|
                 model.send(callback, event_name, hook[:name])

dm-is-remixable/spec/data/image.rb

@@ -7,9 +7,14 @@ module Image
   property :description,  String
   property :path,         String
 
-  after :save, :run_hook_method
+  after :save, :hook_method
 
-  def run_hook_method
+  def hook_method
+    @hook_method = true
+  end
+
+  def hook_method_called?
+    defined?(@hook_method) ? true : false
   end
 
   # These methods will be available to the class remixing this module

dm-is-remixable/spec/integration/remixable_spec.rb

@@ -135,8 +135,9 @@ def self.test_enhance_2
       article = Article.new
       article.pics = (@pic = ArticleImage.new)
 
-      @pic.should_receive(:run_hook_method).once
-      article.save
+      expect {
+        article.save
+      }.to change(@pic, :hook_method_called?).from(false).to(true)
     end
 
     it "should copy validation contexts from the Remixable Module to the Remixed Model" do