MAINT: Use dotnet-coverage instead of Coverlet for coverage collection (

Azure#10809)
geekswagg · May 25, 2023 · eb63729 · eb63729
1 parent 0fa500c
commit eb63729
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 16 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -532,7 +532,7 @@ jobs:
  uses: actions/setup-dotnet@v3
 
  - name: Run Tests
- run: dotnet test --filter '${{ matrix.config.filter}}' --configuration release --logger trx --blame --collect:"XPlat Code Coverage" --settings ./.github/workflows/codecov.runsettings --results-directory ./TestResults/
+ run: dotnet test --filter '${{ matrix.config.filter}}' --configuration release --settings ./.github/workflows/codecov.runsettings
 
  - name: Upload Test Results
  uses: actions/upload-artifact@v3

diff --git a/.github/workflows/codecov.runsettings b/.github/workflows/codecov.runsettings
@@ -3,28 +3,35 @@
  <RunConfiguration>
  <!-- Break builds when no tests are discovered. -->
  <TreatNoTestsAsError>false</TreatNoTestsAsError>
+ <ResultsDirectory>../../TestResults</ResultsDirectory>
  </RunConfiguration>
  <DataCollectionRunSettings>
  <DataCollectors>
- <!-- Settings are documented at https://github.com/coverlet-coverage/coverlet/blob/master/Documentation/VSTestIntegration.md#advanced-options-supported-via-runsettings -->
- <DataCollector friendlyName="XPlat Code Coverage">
+ <!-- "Code Coverage" settings documented in https://learn.microsoft.com/en-us/visualstudio/test/customizing-code-coverage-analysis?view=vs-2022 -->
+ <DataCollector friendlyName="Code Coverage">
  <Configuration>
  <Format>cobertura</Format>
- <!--
- By default coverlet includes all assemblies that are referenced by test assemblies.
- In our case, this includes other test assemblies, which makes the coverage results
- inconsistent. If we want to exclude all test assemblies in the future, we should
- use the Exclude setting instead.
- -->
- <IncludeTestAssembly>true</IncludeTestAssembly>
+ <CodeCoverage>
+ <ModulePaths>
+  <Exclude>
+  <ModulePath>.*moq.dll$</ModulePath>
+  </Exclude>
+  </ModulePaths>
+ </CodeCoverage>
  </Configuration>
  </DataCollector>
  </DataCollectors>
  </DataCollectionRunSettings>
- <!-- <MSTest>
- <Parallelize>
- <Workers>4</Workers>
- <Scope>ClassLevel</Scope>
- </Parallelize>
- </MSTest> -->
+ <LoggerRunSettings>
+ <Loggers>
+ <Logger friendlyName="console" enabled="True">
+ <Configuration>
+ <Verbosity>quiet</Verbosity>
+ </Configuration>
+ </Logger>
+ <Logger friendlyName="trx" enabled="True" />
+ <!-- Enables blame -->
+ <Logger friendlyName="blame" enabled="True" />
+ </Loggers>
+ </LoggerRunSettings>
 </RunSettings>