exclude local methods in excluded methods

Author: tonerdo

src/coverlet.core/Helpers/InstrumentationHelper.cs

@@ -165,6 +165,9 @@ public static bool IsTypeExcluded(string module, string type, string[] filters)
             return false;
         }
 
+        public static bool IsLocalMethod(string method)
+            => new Regex(WildcardToRegex("<*>*__*|*")).IsMatch(method);
+
         public static string[] GetExcludedFiles(string[] rules)
         {
             const string RELATIVE_KEY = nameof(RELATIVE_KEY);

src/coverlet.core/Instrumentation/Instrumenter.cs

@@ -60,11 +60,19 @@ private void InstrumentModule()
             {
                 resolver.AddSearchDirectory(Path.GetDirectoryName(_module));
                 var parameters = new ReaderParameters { ReadSymbols = true, AssemblyResolver = resolver };
+
                 using (var module = ModuleDefinition.ReadModule(stream, parameters))
                 {
-                    foreach (var type in module.GetTypes())
+                    var types = module.GetTypes();
+                    foreach (var type in types)
                     {
-                        InstrumentType(type);
+                        TypeDefinition actualType = type;
+                        if (type.FullName.Contains("/"))
+                            actualType = types.FirstOrDefault(t => t.FullName == type.FullName.Split('/')[0]);
+
+                        if (!actualType.CustomAttributes.Any(IsExcludeAttribute)
+                            && !InstrumentationHelper.IsTypeExcluded(_module, actualType.FullName, _filters))
+                            InstrumentType(type);
                     }
 
                     module.Write(stream);
@@ -74,17 +82,14 @@ private void InstrumentModule()
 
         private void InstrumentType(TypeDefinition type)
         {
-            TypeDefinition actualType = type;
-            if (type.FullName.Contains("/"))
-                actualType = type.Module.GetTypes().FirstOrDefault(t => t.FullName == type.FullName.Split('/')[0]);
-
-            if (actualType.CustomAttributes.Any(IsExcludeAttribute)
-                || InstrumentationHelper.IsTypeExcluded(_module, actualType.FullName, _filters))
-                return;
-
-            foreach (var method in type.Methods)
+            var methods = type.GetMethods();
+            foreach (var method in methods)
             {
-                if (!method.CustomAttributes.Any(IsExcludeAttribute))
+                MethodDefinition actualMethod = method;
+                if (InstrumentationHelper.IsLocalMethod(method.Name))
+                    actualMethod = methods.FirstOrDefault(m => m.Name == method.Name.Split('>')[0].Substring(1)) ?? method;
+
+                if (!actualMethod.CustomAttributes.Any(IsExcludeAttribute))
                     InstrumentMethod(method);
             }
         }