Skip to content

Commit 0a7f49a

Browse files
authored
Fix dotnet test when not generating code coverage reports (#80)
#79 added code coverage reports, but broke a basic `dotnet test`. In that case no Cobertura reports are generated and thus the `<ReportGenerator>` task fails. Fix to condition running the report generator on having reports to process.
1 parent 5212cbe commit 0a7f49a

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

build/targets/tests/Tests.targets

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,11 @@
2525
<ItemGroup>
2626
<_CoverageFiles Include="$(_TestCoverageGlob)" />
2727
</ItemGroup>
28-
<ReportGenerator ProjectDirectory="$(MSBuildProjectDirectory)" ReportFiles="@(_CoverageFiles)" TargetDirectory="$(_TestCoverageReportDirectory)" ReportTypes="MarkdownSummaryGithub;Cobertura;HtmlInline" />
28+
<ReportGenerator
29+
Condition=" '@(_CoverageFiles)' != '' "
30+
ProjectDirectory="$(MSBuildProjectDirectory)"
31+
ReportFiles="@(_CoverageFiles)"
32+
TargetDirectory="$(_TestCoverageReportDirectory)"
33+
ReportTypes="MarkdownSummaryGithub;Cobertura;HtmlInline" />
2934
</Target>
3035
</Project>

0 commit comments

Comments
 (0)