rename methods

Author: tonyhallett

FineCodeCoverageTests/OpenCoverExeArgumentsProvider_Tests.cs

@@ -65,7 +65,7 @@ public void Should_Safely_Include_The_TestDLLFile_In_The_TargetArgs()
 
             var arguments = openCoverExeArgumentsProvider.Provide(mockCoverageProject.Object, "");
 
-            AssertHasEscapedSetting(arguments, $"-targetargs:{CommandLineArguments.EscapeQuotes("testDllFile")}");
+            AssertHasEscapedSetting(arguments, $"-targetargs:{CommandLineArguments.AddEscapeQuotes("testDllFile")}");
         }
 
         [Test]
@@ -78,7 +78,7 @@ public void Should_Safely_Include_The_Project_RunSettingsFile_In_The_TargetArgs_
 
             var arguments = openCoverExeArgumentsProvider.Provide(mockCoverageProject.Object, "");
 
-            AssertHasEscapedSetting(arguments, $"-targetargs:{CommandLineArguments.EscapeQuotes("testDllFile")} /Settings:{CommandLineArguments.EscapeQuotes("runSettingsFile")}");
+            AssertHasEscapedSetting(arguments, $"-targetargs:{CommandLineArguments.AddEscapeQuotes("testDllFile")} /Settings:{CommandLineArguments.AddEscapeQuotes("runSettingsFile")}");
         }
 
         [Test]
@@ -231,7 +231,7 @@ private void AssertHasSetting(List<string> openCoverSettings, string setting)
 
         private void AssertHasEscapedSetting(List<string> openCoverSettings, string setting)
         {
-            AssertHasSetting(openCoverSettings, CommandLineArguments.EscapeArgument(setting));
+            AssertHasSetting(openCoverSettings, CommandLineArguments.AddQuotes(setting));
         }
     }
 }

SharedProject/Core/OpenCover/OpenCoverExeArgumentsProvider.cs

@@ -8,12 +8,12 @@ namespace FineCodeCoverage.Engine.OpenCover
 {
     internal static class CommandLineArguments
     {
-        public static string EscapeArgument(string value)
+        public static string AddQuotes(string value)
         {
             return $@"""{value}""";
         }
 
-        public static string EscapeQuotes(string arg)
+        public static string AddEscapeQuotes(string arg)
         {
             return $@"\""{arg}\""";
         }
@@ -138,18 +138,18 @@ string GetAlternateName(string exclude)
 
         private string GetTargetArgs(ICoverageProject project)
         {
-            var runSettings = !string.IsNullOrWhiteSpace(project.RunSettingsFile) ? $@" /Settings:{CommandLineArguments.EscapeQuotes(project.RunSettingsFile)}" : default;
-            return $@"""-targetargs:{CommandLineArguments.EscapeQuotes(project.TestDllFile)}{runSettings}""";
+            var runSettings = !string.IsNullOrWhiteSpace(project.RunSettingsFile) ? $@" /Settings:{CommandLineArguments.AddEscapeQuotes(project.RunSettingsFile)}" : default;
+            return $@"""-targetargs:{CommandLineArguments.AddEscapeQuotes(project.TestDllFile)}{runSettings}""";
         }
 
         public List<string> Provide(ICoverageProject project,string msTestPlatformExePath)
         {
             var opencoverSettings = new List<string>();
 
-            opencoverSettings.Add(CommandLineArguments.EscapeArgument($"-target:{msTestPlatformExePath}"));
+            opencoverSettings.Add(CommandLineArguments.AddQuotes($"-target:{msTestPlatformExePath}"));
             opencoverSettings.Add(GetTargetArgs(project));
 
-            opencoverSettings.Add(CommandLineArguments.EscapeArgument($"-output:{project.CoverageOutputFile}"));
+            opencoverSettings.Add(CommandLineArguments.AddQuotes($"-output:{project.CoverageOutputFile}"));
 
             AddFilter(project, opencoverSettings);
             AddExcludeByFile(project, opencoverSettings);