Skip to content

Commit

Permalink
Dont publish html files for native coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
vinayakmsft committed Nov 29, 2024
1 parent 7ae9eb1 commit c0cc1a8
Showing 1 changed file with 14 additions and 2 deletions.
16 changes: 14 additions & 2 deletions src/CoveragePublisher/CoverageProcessor.cs
Original file line number Diff line number Diff line change
Expand Up @@ -93,11 +93,23 @@ public async Task ParseAndPublishCoverage(PublisherConfiguration config, Cancell

else
{
using (new SimpleTimer("CoverageProcesser", "PublishHTMLReport", _telemetry))
foreach (string nativeCoverageFile in config.CoverageFiles)
{
await _publisher.PublishHTMLReport(config.ReportDirectory, token);
if (!(nativeCoverageFile.EndsWith(Constants.CoverageConstants.CoverageBufferFileExtension) ||
nativeCoverageFile.EndsWith(Constants.CoverageConstants.CoverageFileExtension) ||
nativeCoverageFile.EndsWith(Constants.CoverageConstants.CoverageBFileExtension) ||
nativeCoverageFile.EndsWith(Constants.CoverageConstants.CoverageJsonFileExtension) ||
nativeCoverageFile.EndsWith(Constants.CoverageConstants.CoverageXFileExtension)))
{
using (new SimpleTimer("CoverageProcesser", "PublishHTMLReport", _telemetry))
{
await _publisher.PublishHTMLReport(config.ReportDirectory, token);
}
}

}
}

}
}
// Only catastrophic failures should trickle down to these catch blocks
Expand Down

0 comments on commit c0cc1a8

Please sign in to comment.