no debug

Author: daveMueller

src/coverlet.core/Abstractions/ISourceRootTranslator.cs

@@ -12,7 +12,5 @@ internal interface ISourceRootTranslator
         string ResolveFilePath(string originalFileName);
         string ResolveDeterministicPath(string originalFileName);
         IReadOnlyList<SourceRootMapping> ResolvePathRoot(string pathRoot);
-
-        Dictionary<string, List<SourceRootMapping>> SourceRootMapping { get; }
     }
 }

src/coverlet.core/Helpers/InstrumentationHelper.cs

@@ -187,7 +187,7 @@ private bool MatchDocumentsWithSources(string module, AssemblySearchType exclude
         {
             if (excludeAssembliesWithoutSources.Equals(AssemblySearchType.MissingAll))
             {
-                bool anyDocumentMatches = MatchDocumentsWithSourcesMissingAll(metadataReader, module);
+                bool anyDocumentMatches = MatchDocumentsWithSourcesMissingAll(metadataReader);
 
                 if (!anyDocumentMatches)
                 {
@@ -211,45 +211,19 @@ private bool MatchDocumentsWithSources(string module, AssemblySearchType exclude
             return true;
         }
 
-        private IEnumerable<(string documentName, bool documentExists)> DocumentSourceMap(MetadataReader metadataReader, string module = "")
+        private IEnumerable<(string documentName, bool documentExists)> DocumentSourceMap(MetadataReader metadataReader)
         {
-            var foo = metadataReader.Documents.Select(docHandle =>
+            return metadataReader.Documents.Select(docHandle =>
             {
                 Document document = metadataReader.GetDocument(docHandle);
                 string docName = _sourceRootTranslator.ResolveFilePath(metadataReader.GetString(document.Name));
                 return (docName, _fileSystem.Exists(docName));
             });
-
-            var baa = metadataReader.Documents.Select(docHandle =>
-            {
-                Document document = metadataReader.GetDocument(docHandle);
-                string docName = _sourceRootTranslator.ResolveFilePath(metadataReader.GetString(document.Name));
-                return (docName, metadataReader.GetString(document.Name));
-            });
-
-            if (module.Contains("coverlet.tests.projectsample.empty"))
-            {
-                //throw new Exception(string.Join(Environment.NewLine,
-                //    baa.Select(x =>
-                //        $"Translator.Name: {x.docName}{Environment.NewLine}MetaData.Name: {x.Item2}{Environment.NewLine}Exists: {_fileSystem.Exists(x.docName)}{Environment.NewLine}SourceRoot: {_sourceRootTranslator.SourceRootMapping.Count}")));
-
-                throw new Exception(string.Join(Environment.NewLine,
-                    new string[]
-                    {
-                        $"Translator.Name: {baa.First().docName}",
-                        $"MetaData.Name: {baa.First().Item2}",
-                        $"Exists: {_fileSystem.Exists(baa.First().docName)}"
-                    }.Concat(_sourceRootTranslator.SourceRootMapping.Values.SelectMany(x => x.Select(y => $"ProjectPath: {y.ProjectPath}{Environment.NewLine}OrigPath: {y.OriginalPath}")))));
-
-            }
-
-
-            return foo;
         }
 
-        private bool MatchDocumentsWithSourcesMissingAll(MetadataReader metadataReader, string module = "")
+        private bool MatchDocumentsWithSourcesMissingAll(MetadataReader metadataReader)
         {
-            return DocumentSourceMap(metadataReader, module).Any(x => x.documentExists);
+            return DocumentSourceMap(metadataReader).Any(x => x.documentExists);
         }
 
         private (bool allDocumentsMatch, string notFoundDocument) MatchDocumentsWithSourcesMissingAny(

src/coverlet.core/Instrumentation/Instrumenter.cs

@@ -127,11 +127,6 @@ public bool CanInstrument()
             }
             catch (Exception ex)
             {
-                if (_module.Contains("coverlet.tests.projectsample.empty"))
-                {
-                    throw new Exception("EXCEPTION", ex);
-                }
-
                 _logger.LogWarning($"Unable to instrument module: '{_module}'\n{ex}");
                 return false;
             }