Merge pull request #10 from invenia/redefinition-fixes

Fixes for automatic recompilation of dependent functions

src/Mocking.jl

@@ -181,11 +181,16 @@ macro mock(expr)
     env_var = gensym("env")
     args_var = gensym("args")
 
+    # Note: The fix to Julia issue #265 (PR #17057) introduced changes where no compiled
+    # calls could be made to functions compiled afterwards. Since the `apply` do block
+    # syntax compiles the do block function before evaluating the do "outer" function this
+    # means our patch functions will be compiled after the "inner" function.
+    # Also note that we need to QuoteNode args_var to handle Symbols in args_var.
     result = quote
         local $env_var = Mocking.get_active_env()
         local $args_var = tuple($(args...))
         if Mocking.ismocked($env_var, $func_name, $args_var)
-            $env_var.mod.$func($args_var...)
+            eval(Expr(:call, $env_var.mod.$func, map(QuoteNode, $args_var)...))
         else
             $func($args_var...)
         end

test/closure.jl

@@ -1,6 +1,6 @@
 let magic
     magic(x) = false
-    sentinel = gensym()
+    sentinel = gensym("sentinel")
     @test magic(sentinel) == false
 
     # Getting closers to work means having a function created in the current scope

test/concept.jl

@@ -39,7 +39,7 @@ Mocking.set_active_env(pe)
 @test (@mock multiply(0x2)) == 0x4
 @test (@mock multiply(2//1)) == 4//1
 
-# Use convienient syntax
+# Use convenient syntax
 apply(patches) do
     @test (@mock multiply(2)) == 8
     @test (@mock multiply(0x2)) == 0x6