Skip to content

Commit 11c15b5

Browse files
add some logs
1 parent f258333 commit 11c15b5

File tree

2 files changed

+18
-10
lines changed

2 files changed

+18
-10
lines changed

build.proj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
</Target>
2626

2727
<Target Name="RunTests" AfterTargets="CopyMSBuildScripts">
28-
<Exec Command="dotnet test &quot;$(MSBuildThisFileDirectory)test\coverlet.core.tests\coverlet.core.tests.csproj&quot; -c $(Configuration) /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:Include=[coverlet.*]*"/>
28+
<Exec Command="dotnet test &quot;$(MSBuildThisFileDirectory)test\coverlet.core.tests\coverlet.core.tests.csproj&quot; -c $(Configuration) /p:CollectCoverage=true /p:CoverletOutputFormat=opencover /p:Include=[coverlet.*]* -verbosity:minimal"/>
2929
</Target>
3030

3131
<Target Name="CreateNuGetPackage" AfterTargets="RunTests" Condition="$(Configuration) == 'Release'">

src/coverlet.core/Coverage.cs

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -32,14 +32,14 @@ public string Identifier
3232
get { return _identifier; }
3333
}
3434

35-
public Coverage(string module,
36-
string[] includeFilters,
37-
string[] includeDirectories,
38-
string[] excludeFilters,
39-
string[] excludedSourceFiles,
40-
string[] excludeAttributes,
41-
bool singleHit,
42-
string mergeWith,
35+
public Coverage(string module,
36+
string[] includeFilters,
37+
string[] includeDirectories,
38+
string[] excludeFilters,
39+
string[] excludedSourceFiles,
40+
string[] excludeAttributes,
41+
bool singleHit,
42+
string mergeWith,
4343
bool useSourceLink,
4444
ILogger logger)
4545
{
@@ -62,14 +62,21 @@ public void PrepareModules()
6262
{
6363
string[] modules = InstrumentationHelper.GetCoverableModules(_module, _includeDirectories);
6464
string[] excludes = InstrumentationHelper.GetExcludedFiles(_excludedSourceFiles);
65+
66+
Array.ForEach(_excludeFilters ?? Array.Empty<string>(), filter => _logger.LogInformation($"Exclude filter '{filter}'"));
67+
Array.ForEach(_includeFilters ?? Array.Empty<string>(), filter => _logger.LogInformation($"Include filter '{filter}'"));
68+
6569
_excludeFilters = _excludeFilters?.Where(f => InstrumentationHelper.IsValidFilterExpression(f)).ToArray();
6670
_includeFilters = _includeFilters?.Where(f => InstrumentationHelper.IsValidFilterExpression(f)).ToArray();
6771

6872
foreach (var module in modules)
6973
{
7074
if (InstrumentationHelper.IsModuleExcluded(module, _excludeFilters) ||
7175
!InstrumentationHelper.IsModuleIncluded(module, _includeFilters))
76+
{
77+
_logger.LogInformation($"Excluded module: '{module}'");
7278
continue;
79+
}
7380

7481
var instrumenter = new Instrumenter(module, _identifier, _excludeFilters, _includeFilters, excludes, _excludeAttributes, _singleHit);
7582
if (instrumenter.CanInstrument())
@@ -81,6 +88,7 @@ public void PrepareModules()
8188
{
8289
var result = instrumenter.Instrument();
8390
_results.Add(result);
91+
_logger.LogInformation($"Instrumented module: '{module}'");
8492
}
8593
catch (Exception ex)
8694
{
@@ -197,7 +205,7 @@ private void CalculateCoverage()
197205
{
198206
if (!File.Exists(result.HitsFilePath))
199207
{
200-
// File not instrumented, or nothing in it called. Warn about this?
208+
_logger.LogWarning($"Hits file:'{result.HitsFilePath}' not found for module: '{result.Module}'");
201209
continue;
202210
}
203211

0 commit comments

Comments
 (0)