resolve review comments and add DotnetTool tests

Author: Bertk

Directory.Build.props

@@ -13,7 +13,7 @@
     <AnalysisLevel>preview</AnalysisLevel>
     <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
     <LangVersion>preview</LangVersion>
-    <NoWarn>$(NoWarn);NU5105</NoWarn>
+    <NoWarn>$(NoWarn);NU5105;CS1591</NoWarn>
     <RestoreSources>
       https://api.nuget.org/v3/index.json;
     </RestoreSources>
@@ -25,13 +25,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Nerdbank.GitVersioning" Version="3.6.133">
-      <PrivateAssets>all</PrivateAssets>
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
-    </PackageReference>
-    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1">
-      <PrivateAssets>all</PrivateAssets>
-      <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
-    </PackageReference>
+    <PackageReference Include="Nerdbank.GitVersioning" Version="3.6.133" PrivateAssets="all" />
+    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1" PrivateAssets="all" />
   </ItemGroup>
 </Project>

Directory.Build.targets

@@ -25,7 +25,6 @@
     <PackageReference Update="xunit.runner.visualstudio" Version="2.4.1"/>
     <PackageReference Update="Tmds.ExecFunction" Version="0.4.0" />
     <PackageReference Update="System.CommandLine" Version="2.0.0-beta4.22272.1" />
-    <PackageReference Update="System.CommandLine.NamingConventionBinder" Version="2.0.0-beta4.22272.1" />
   </ItemGroup>
 
   <!-- Deterministic build workaround -->

Documentation/GlobalTool.md

@@ -17,16 +17,14 @@ Arguments:
   <ASSEMBLY|DIRECTORY>                  Path to the test assembly or application directory.
 
 Options:
-  -h|--help                             Show help information
-  -v|--version                          Show version information
-  -t|--target                           Path to the test runner application.
+  -t|--target (REQUIRED)                Path to the test runner application.
   -a|--targetargs                       Arguments to be passed to the test runner.
   -o|--output                           Output of the generated coverage report
   -v|--verbosity                        Sets the verbosity level of the command. Allowed values are quiet, minimal, normal, detailed.
-  -f|--format                           Format of the generated coverage report.
+  -f|--format                           Format of the generated coverage report. [default: json]
   --threshold                           Exits with error if the coverage % is below value.
   --threshold-type                      Coverage type to apply the threshold to.
-  --threshold-stat                      Coverage statistic used to enforce the threshold value.
+  --threshold-stat                      Coverage statistic used to enforce the threshold value. [default: Minimum]
   --exclude                             Filter expressions to exclude specific modules and types.
   --include                             Filter expressions to include only specific modules and types.
   --exclude-by-file                     Glob patterns specifying source files to exclude.
@@ -39,6 +37,8 @@ Options:
   --use-source-link                     Specifies whether to use SourceLink URIs in place of file system paths.
   --does-not-return-attribute           Attributes that mark methods that do not return.
   --exclude-assemblies-without-sources  Specifies behaviour of heuristic to ignore assemblies with missing source documents.
+  --version                             Show version information
+  -?, -h, --help                        Show help and usage information
 ```
 
 NB. For a [multiple value] options you have to specify values multiple times i.e.

global.json

@@ -1,6 +1,6 @@
 {
   "sdk": {
-    "version": "6.0.408",
+    "version": "6.0.410",
     "rollForward": "latestMajor"
   }
 }

src/coverlet.collector/DataCollection/CoverageWrapper.cs

@@ -17,6 +17,10 @@ internal class CoverageWrapper : ICoverageWrapper
         /// </summary>
         /// <param name="settings">Coverlet settings</param>
         /// <param name="coverletLogger">Coverlet logger</param>
+        /// <param name="instrumentationHelper"></param>
+        /// <param name="fileSystem"></param>
+        /// <param name="sourceRootTranslator"></param>
+        /// <param name="cecilSymbolHelper"></param>
         /// <returns>Coverage object</returns>
         public Coverage CreateCoverage(CoverletSettings settings, ILogger coverletLogger, IInstrumentationHelper instrumentationHelper, IFileSystem fileSystem, ISourceRootTranslator sourceRootTranslator, ICecilSymbolHelper cecilSymbolHelper)
         {

src/coverlet.collector/coverlet.collector.csproj

@@ -1,4 +1,4 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
     <TargetFramework>netstandard2.0</TargetFramework>
     <AssemblyTitle>coverlet.collector</AssemblyTitle>