Exlcude now exludes kwsorter as well

src/ambiguities.jl

@@ -176,16 +176,11 @@ end
 struct _NoValue end
 
 function getobj(m::Method)
-    ty = try
-        fieldtype(m.sig, 1)
-    catch err
-        @error(
-            "Failed to obtain a function from `Method`.",
-            exception = (err, catch_backtrace())
-        )
-        # If this doesn't work, `Base` internal was probably changed
-        # too much compared to what it is now.  So, bailing out.
-        return _NoValue()
+    signature = Base.unwrap_unionall(m.sig)
+    ty = if is_kwcall(signature)
+        signature.parameters[3]
+    else
+        signature.parameters[1]
     end
     ty = Base.unwrap_unionall(ty)
     if ty <: Function

test/test_ambiguities.jl

@@ -44,7 +44,7 @@ include("preamble.jl")
     check_testcase([PkgWithAmbiguities.f], total - num_ambs_f)
 
     # exclude function and kwsorter
-    check_testcase([PkgWithAmbiguities.g], total - num_ambs_g; broken = true)
+    check_testcase([PkgWithAmbiguities.g], total - num_ambs_g)
 
     # exclude callables and constructors
     check_testcase([PkgWithAmbiguities.SingletonType], total - num_ambs_SingletonType)
@@ -73,8 +73,7 @@ include("preamble.jl")
                 PkgWithAmbiguities.ConcreteType,
                 PkgWithAmbiguities.ConcreteParameterizedType,
             ],
-            0;
-            broken = true,
+            0,
         )
     else
         # exclude everything
@@ -85,8 +84,7 @@ include("preamble.jl")
                 PkgWithAmbiguities.SingletonType,
                 PkgWithAmbiguities.ConcreteType,
             ],
-            0;
-            broken = true,
+            0,
         )
     end