fixup issues with no nodebug

src/breakpoints.jl

@@ -27,7 +27,7 @@ end
 
 function set_nodebug!(the_rules::BreakpointRules, arg::T) where T
     T !== Method || throw(ArgumentError("Disabling instrumentation per method, is not supported."))
-    return push!(the_rules.breakon_rules, Rule(arg))
+    return push!(the_rules.no_instrument_rules, Rule(arg))
 end
 
 for (name, list) in ((:breakpoint, :breakon_rules), (:nodebug, :no_instrument_rules))

src/method_utils.jl

@@ -5,7 +5,10 @@ Returns the module associated with a method.
 Note: this is not the module where it was defined (m.module)
 but rather the module for which it's function was defined.
 """
-moduleof(m) = functiontypeof(m).name.module
+moduleof(m::Method) = moduleof(functiontypeof(m))
+moduleof(sig::DataType) = sig.name.module
+moduleof(sig::UnionAll) = moduleof(sig.body)
+
 functiontypeof(m::Method) = parameter_typeof(m.sig)[1]
 parameter_typeof(sig::UnionAll) = parameter_typeof(sig.body)
 parameter_typeof(sig::DataType) = sig.parameters

test/test_method_utils.jl

@@ -16,6 +16,11 @@ Base.eps(::typeof(moduleof)) = "dummy"
     for meth in methods(eps)
         @test moduleof(meth) == Base
     end
+
+
+    for meth in methods(Vector)  # this is a UnionAll
+        @test moduleof(meth) == Core
+    end
 end
 
 

test/test_ui.jl

@@ -31,6 +31,15 @@ clear_breakpoints!(); clear_nodebugs!()
 end
 
 ###################################################
+clear_breakpoints!(); clear_nodebugs!()
+@testset "No breakpoints, With no instrumenting of Base" begin
+    make_readline_patch([])
+    record = make_recording_breakpoint_hit_patch()
+    set_nodebug!(Base)
+    @iron_debug eg1()
+    @test record == []
+end
+###########################
 
 clear_breakpoints!(); clear_nodebugs!()
 @testset "breakpoint by file and linenum" begin