Make ms code coverage the default

Author: tonyhallett

FineCodeCoverageTests/AppOptionsProvider_Tests.cs

@@ -200,6 +200,7 @@ public void Should_Not_Default_Any_Other_AppOptions_Properties()
                 nameof(IAppOptions.ThresholdForCrapScore),
                 nameof(IAppOptions.ThresholdForNPathComplexity),
                 nameof(IAppOptions.ThresholdForCyclomaticComplexity),
+                nameof(IAppOptions.RunMsCodeCoverage),
                 nameof(IAppOptions.NamespacedClasses),
                 nameof(IAppOptions.ShowCoverageInOverviewMargin),
                 nameof(IAppOptions.ShowCoveredInOverviewMargin),

README.md

@@ -27,7 +27,7 @@ When not using Microsoft.TestingPlatform you have added test adapters through nu
 
 Fine Code Coverage provides code coverage using one of 3 different coverage tools. In previous releases there were two coverage tools being utilised, OpenCover and Coverlet that will be referred to as 'old coverage'.  
 Microsoft now provides a free coverage solution that you can choose to use by setting the Visual Studio Fine Code Coverage enumeration option RunMsCodeCoverage. This will probably be the preferred coverage
-tool for most developers. It is currently in Beta.
+tool for most developers.
 
 With the old coverage it was possible for FCC to provide an abstraction over each tool's exclusion / inclusion options. This abstraction does not work for MS code coverage.  
 Thus you will find that there are separate configuration options for Ms coverage vs old coverage and options that are common to the two.
@@ -109,8 +109,6 @@ The old coverage was based upon every test. Ms code coverage is coverage from th
 
 ## How to utilize MS Code Coverage with FCC ?
 
-Firstly you need to change the RunMsCodeCoverage option from No.
-
 Ms code coverage requires a [runsettings](https://docs.microsoft.com/en-us/visualstudio/test/configure-unit-tests-by-using-a-dot-runsettings-file?view=vs-2022) file that is configured appropriately for
 code coverage. This requires that you have the ms code coverage package and have pointed to it with the TestAdaptersPaths element as well as specifying the ms data collector. [Exclusions and inclusions](https://docs.microsoft.com/en-us/visualstudio/test/customizing-code-coverage-analysis?view=vs-2022#include-or-exclude-assemblies-and-members)
 are also specified in the runsettings. I don't think that the documentation is clear enough on how this works so you may want to look at [this issue](https://github.com/microsoft/vstest/issues/3462).
@@ -349,7 +347,7 @@ If you are using option 1) then project and global options will only be used whe
 | DisabledNoCoverage                                                              | Set to false for VS Option Enabled=false to not disable coverage                                                                                                                                                                                                                                                                                               |
 | RunWhenTestsFail                                                                | By default coverage runs when tests fail. Set to false to prevent this. **Cannot be used in conjunction with RunInParallel**                                                                                                                                                                                                                                   |
 | RunWhenTestsExceed                                                              | Specify a value to only run coverage based upon the number of executing tests. **Cannot be used in conjunction with RunInParallel**                                                                                                                                                                                                                            |
-| RunMsCodeCoverage                                                               | Change to IfInRunSettings to only collect with configured runsettings. Yes for runsettings generation.                                                                                                                                                                                                                                                         |
+| RunMsCodeCoverage                                                               | Change to IfInRunSettings to only collect with configured runsettings. Yes (default) for runsettings generation. No to use Coverlet or OpenCover.                                                                                                                                                                                                                                                          |
 | IncludeTestAssembly                                                             | Specifies whether to report code coverage of the test assembly                                                                                                                                                                                                                                                                                                 |
 | IncludeReferencedProjects                                                       | Set to true to add all directly referenced projects to Include.                                                                                                                                                                                                                                                                                                |
 | IncludeAssemblies                                                               | Provide a list of assemblies to include in coverage. The dll name without extension is used for matching.                                                                                                                                                                                                                                                      |

SharedProject/Core/MsTestPlatform/CodeCoverage/MsCodeCoverageRunSettingsService.cs

@@ -121,12 +121,7 @@ public void Initialize(string appDataFolder, IFCCEngine fccEngine, CancellationT
         public async Task<MsCodeCoverageCollectionStatus> IsCollectingAsync(ITestOperation testOperation)
         {
             await InitializeIsCollectingAsync(testOperation);
-            if( runMsCodeCoverage == RunMsCodeCoverage.No)
-            {
-                logger.Log($"See option {nameof(IAppOptions.RunMsCodeCoverage)} for a better ( Beta ) experience.  {FCCGithub.Readme}");
-                reportGeneratorUtil.LogCoverageProcess($"See option {nameof(IAppOptions.RunMsCodeCoverage)} for a better ( Beta ) experience. View readme.");
-            }
-            else
+            if( runMsCodeCoverage != RunMsCodeCoverage.No)
             {
                 await TrySetUpForCollectionAsync(testOperation.SolutionDirectory);
             }

SharedProject/Options/AppOptionsPage.cs

@@ -75,7 +75,7 @@ private static IAppOptionsStorageProvider GetAppOptionsStorageProvider()
         public bool DisabledNoCoverage { get; set; }
 
         [Category(commonRunCategory)]
-        [Description("Specifies whether or not the ms code coverage is used (BETA).  No, IfInRunSettings, Yes")]
+        [Description("Specifies whether or not the ms code coverage is used.  No, IfInRunSettings, Yes ( default )")]
         //[DisplayName("Run Ms Code Coverage)")]
         public RunMsCodeCoverage RunMsCodeCoverage { get; set; }
 

SharedProject/Options/AppOptionsProvider.cs

@@ -60,6 +60,7 @@ private void AddDefaults(IAppOptions appOptions)
             appOptions.ThresholdForCrapScore = 15;
             appOptions.ThresholdForNPathComplexity = 200;
             appOptions.ThresholdForCyclomaticComplexity = 30;
+            appOptions.RunMsCodeCoverage = RunMsCodeCoverage.Yes;
             appOptions.RunSettingsOnly = true;
             appOptions.RunWhenTestsFail = true;
             appOptions.ExcludeByAttribute = new[] { "GeneratedCode" };