Run dotnet format whitespace (#1085)

* Run dotnet format whitespace Co-authored-by: Medeni Baykal <433724+Haplois@users.noreply.github.com>
microsoft · Apr 26, 2022 · bf1f427 · bf1f427
1 parent 9437f21
commit bf1f427
Show file tree

Hide file tree

Showing 27 changed files with 167 additions and 167 deletions.
diff --git a/src/Adapter/PlatformServices.Desktop.Legacy/Data/CsvDataConnection.cs b/src/Adapter/PlatformServices.Desktop.Legacy/Data/CsvDataConnection.cs
@@ -70,7 +70,7 @@ public override List<string> GetColumns(string tableName)
             }
             catch (Exception exception)
             {
-               EqtTrace.ErrorIf(EqtTrace.IsErrorEnabled, exception.Message + " for CSV data source " + this.fileName);
+                EqtTrace.ErrorIf(EqtTrace.IsErrorEnabled, exception.Message + " for CSV data source " + this.fileName);
             }
 
             return null;

diff --git a/src/Adapter/PlatformServices.Desktop.Legacy/Services/DesktopTestSourceHost.cs b/src/Adapter/PlatformServices.Desktop.Legacy/Services/DesktopTestSourceHost.cs
@@ -283,65 +283,65 @@ internal virtual List<string> GetResolutionPaths(string sourceFileName, bool isP
             return resolutionPaths;
         }
 
-    internal virtual string GetTargetFrameworkVersionString(string sourceFileName)
-    {
-        return AppDomainUtilities.GetTargetFrameworkVersionString(sourceFileName);
-    }
-
-    private string GetConfigFileForTestSource(string sourceFileName)
-    {
-        return new DeploymentUtility().GetConfigFile(sourceFileName);
-    }
-
-    /// <summary>
-    /// Sets context required for running tests.
-    /// </summary>
-    /// <param name="source">
-    /// source parameter used for setting context
-    /// </param>
-    private void SetContext(string source)
-    {
-        if (string.IsNullOrEmpty(source))
+        internal virtual string GetTargetFrameworkVersionString(string sourceFileName)
         {
-            return;
+            return AppDomainUtilities.GetTargetFrameworkVersionString(sourceFileName);
         }
 
-        Exception setWorkingDirectoryException = null;
-        this.currentDirectory = Environment.CurrentDirectory;
-
-        try
-        {
-            Environment.CurrentDirectory = Path.GetDirectoryName(source);
-            EqtTrace.Info("MSTestExecutor: Changed the working directory to {0}", Environment.CurrentDirectory);
-        }
-        catch (IOException ex)
-        {
-            setWorkingDirectoryException = ex;
-        }
-        catch (System.Security.SecurityException ex)
+        private string GetConfigFileForTestSource(string sourceFileName)
         {
-            setWorkingDirectoryException = ex;
+            return new DeploymentUtility().GetConfigFile(sourceFileName);
         }
 
-        if (setWorkingDirectoryException != null)
+        /// <summary>
+        /// Sets context required for running tests.
+        /// </summary>
+        /// <param name="source">
+        /// source parameter used for setting context
+        /// </param>
+        private void SetContext(string source)
         {
-            EqtTrace.Error("MSTestExecutor.SetWorkingDirectory: Failed to set the working directory to '{0}'. {1}", Path.GetDirectoryName(source), setWorkingDirectoryException);
+            if (string.IsNullOrEmpty(source))
+            {
+                return;
+            }
+
+            Exception setWorkingDirectoryException = null;
+            this.currentDirectory = Environment.CurrentDirectory;
+
+            try
+            {
+                Environment.CurrentDirectory = Path.GetDirectoryName(source);
+                EqtTrace.Info("MSTestExecutor: Changed the working directory to {0}", Environment.CurrentDirectory);
+            }
+            catch (IOException ex)
+            {
+                setWorkingDirectoryException = ex;
+            }
+            catch (System.Security.SecurityException ex)
+            {
+                setWorkingDirectoryException = ex;
+            }
+
+            if (setWorkingDirectoryException != null)
+            {
+                EqtTrace.Error("MSTestExecutor.SetWorkingDirectory: Failed to set the working directory to '{0}'. {1}", Path.GetDirectoryName(source), setWorkingDirectoryException);
+            }
         }
-    }
 
-    /// <summary>
-    /// Resets the context as it was before calling SetContext()
-    /// </summary>
-    private void ResetContext()
-    {
-        if (!string.IsNullOrEmpty(this.currentDirectory))
+        /// <summary>
+        /// Resets the context as it was before calling SetContext()
+        /// </summary>
+        private void ResetContext()
         {
-            Environment.CurrentDirectory = this.currentDirectory;
+            if (!string.IsNullOrEmpty(this.currentDirectory))
+            {
+                Environment.CurrentDirectory = this.currentDirectory;
+            }
         }
-    }
 
-    private void AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(AssemblyResolver assemblyResolver, string baseDirectory)
-    {
+        private void AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(AssemblyResolver assemblyResolver, string baseDirectory)
+        {
             // Check if user specified any adapter settings
             MSTestAdapterSettings adapterSettings = MSTestSettingsProvider.Settings;
 
@@ -364,7 +364,7 @@ private void AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(Assemb
                 }
             }
         }
-}
+    }
 
 #pragma warning restore SA1649 // SA1649FileNameMustMatchTypeName
 }
diff --git a/src/Adapter/PlatformServices.Desktop/Data/CsvDataConnection.cs b/src/Adapter/PlatformServices.Desktop/Data/CsvDataConnection.cs
@@ -70,7 +70,7 @@ public override List<string> GetColumns(string tableName)
             }
             catch (Exception exception)
             {
-               EqtTrace.ErrorIf(EqtTrace.IsErrorEnabled, exception.Message + " for CSV data source " + this.fileName);
+                EqtTrace.ErrorIf(EqtTrace.IsErrorEnabled, exception.Message + " for CSV data source " + this.fileName);
             }
 
             return null;

diff --git a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSourceHost.cs b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSourceHost.cs
@@ -283,65 +283,65 @@ internal virtual List<string> GetResolutionPaths(string sourceFileName, bool isP
             return resolutionPaths;
         }
 
-    internal virtual string GetTargetFrameworkVersionString(string sourceFileName)
-    {
-        return AppDomainUtilities.GetTargetFrameworkVersionString(sourceFileName);
-    }
-
-    private string GetConfigFileForTestSource(string sourceFileName)
-    {
-        return new DeploymentUtility().GetConfigFile(sourceFileName);
-    }
-
-    /// <summary>
-    /// Sets context required for running tests.
-    /// </summary>
-    /// <param name="source">
-    /// source parameter used for setting context
-    /// </param>
-    private void SetContext(string source)
-    {
-        if (string.IsNullOrEmpty(source))
+        internal virtual string GetTargetFrameworkVersionString(string sourceFileName)
         {
-            return;
+            return AppDomainUtilities.GetTargetFrameworkVersionString(sourceFileName);
         }
 
-        Exception setWorkingDirectoryException = null;
-        this.currentDirectory = Environment.CurrentDirectory;
-
-        try
-        {
-            Environment.CurrentDirectory = Path.GetDirectoryName(source);
-            EqtTrace.Info("MSTestExecutor: Changed the working directory to {0}", Environment.CurrentDirectory);
-        }
-        catch (IOException ex)
-        {
-            setWorkingDirectoryException = ex;
-        }
-        catch (System.Security.SecurityException ex)
+        private string GetConfigFileForTestSource(string sourceFileName)
         {
-            setWorkingDirectoryException = ex;
+            return new DeploymentUtility().GetConfigFile(sourceFileName);
         }
 
-        if (setWorkingDirectoryException != null)
+        /// <summary>
+        /// Sets context required for running tests.
+        /// </summary>
+        /// <param name="source">
+        /// source parameter used for setting context
+        /// </param>
+        private void SetContext(string source)
         {
-            EqtTrace.Error("MSTestExecutor.SetWorkingDirectory: Failed to set the working directory to '{0}'. {1}", Path.GetDirectoryName(source), setWorkingDirectoryException);
+            if (string.IsNullOrEmpty(source))
+            {
+                return;
+            }
+
+            Exception setWorkingDirectoryException = null;
+            this.currentDirectory = Environment.CurrentDirectory;
+
+            try
+            {
+                Environment.CurrentDirectory = Path.GetDirectoryName(source);
+                EqtTrace.Info("MSTestExecutor: Changed the working directory to {0}", Environment.CurrentDirectory);
+            }
+            catch (IOException ex)
+            {
+                setWorkingDirectoryException = ex;
+            }
+            catch (System.Security.SecurityException ex)
+            {
+                setWorkingDirectoryException = ex;
+            }
+
+            if (setWorkingDirectoryException != null)
+            {
+                EqtTrace.Error("MSTestExecutor.SetWorkingDirectory: Failed to set the working directory to '{0}'. {1}", Path.GetDirectoryName(source), setWorkingDirectoryException);
+            }
         }
-    }
 
-    /// <summary>
-    /// Resets the context as it was before calling SetContext()
-    /// </summary>
-    private void ResetContext()
-    {
-        if (!string.IsNullOrEmpty(this.currentDirectory))
+        /// <summary>
+        /// Resets the context as it was before calling SetContext()
+        /// </summary>
+        private void ResetContext()
         {
-            Environment.CurrentDirectory = this.currentDirectory;
+            if (!string.IsNullOrEmpty(this.currentDirectory))
+            {
+                Environment.CurrentDirectory = this.currentDirectory;
+            }
         }
-    }
 
-    private void AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(AssemblyResolver assemblyResolver, string baseDirectory)
-    {
+        private void AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(AssemblyResolver assemblyResolver, string baseDirectory)
+        {
             // Check if user specified any adapter settings
             MSTestAdapterSettings adapterSettings = MSTestSettingsProvider.Settings;
 
@@ -364,7 +364,7 @@ private void AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(Assemb
                 }
             }
         }
-}
+    }
 
 #pragma warning restore SA1649 // SA1649FileNameMustMatchTypeName
 }
diff --git a/src/Adapter/PlatformServices.NetCore/Services/NetCoreTestContextImplementation.cs b/src/Adapter/PlatformServices.NetCore/Services/NetCoreTestContextImplementation.cs
@@ -402,7 +402,7 @@ public string GetDiagnosticMessages()
         /// </summary>
         public void ClearDiagnosticMessages()
         {
-           this.threadSafeStringWriter.ToStringAndClear();
+            this.threadSafeStringWriter.ToStringAndClear();
         }
 
         public void SetDataRow(object dataRow)

diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10FileOperations.cs b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10FileOperations.cs
@@ -36,11 +36,11 @@ public Assembly LoadAssembly(string assemblyName, bool isReflectionOnly)
         /// <returns>Path to the .DLL of the assembly.</returns>
         public string GetAssemblyPath(Assembly assembly)
         {
-            #if NETSTANDARD1_5
-                return assembly.Location;
-            #else
+#if NETSTANDARD1_5
+            return assembly.Location;
+#else
                 return null;
-            #endif
+#endif
         }
 
         /// <summary>

diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.0/Utilities/ns10Validate.cs b/src/Adapter/PlatformServices.Shared/netstandard1.0/Utilities/ns10Validate.cs
@@ -5,7 +5,7 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.Uti
 {
     using System;
 
-    #pragma warning disable SA1649 // File name must match first type name
+#pragma warning disable SA1649 // File name must match first type name
 
     internal class Validate
     {

diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13ThreadSafeStringWriter.cs b/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13ThreadSafeStringWriter.cs
@@ -120,7 +120,7 @@ protected override void Dispose(bool disposing)
                 ThreadSafeStringWriter.State?.Value?.Remove(this.outputType);
                 try
                 {
-                   base.Dispose(disposing);
+                    base.Dispose(disposing);
                 }
                 catch (ObjectDisposedException)
                 {

diff --git a/src/TestFramework/Extension.Desktop/ConfigurationSettings/DataSourceElementCollection.cs b/src/TestFramework/Extension.Desktop/ConfigurationSettings/DataSourceElementCollection.cs
@@ -10,7 +10,7 @@ namespace Microsoft.VisualStudio.TestTools.UnitTesting
     /// <summary>
     /// The Data source element collection.
     /// </summary>
-    [SuppressMessage("Microsoft.Design", "CA1010:CollectionsShouldImplementGenericInterface", Justification ="Compat")]
+    [SuppressMessage("Microsoft.Design", "CA1010:CollectionsShouldImplementGenericInterface", Justification = "Compat")]
     public sealed class DataSourceElementCollection : ConfigurationElementCollection
     {
         /// <summary>

diff --git a/test/ComponentTests/TestAssets/TestProjectForAssemblyResolution/UnitTest1.cs b/test/ComponentTests/TestAssets/TestProjectForAssemblyResolution/UnitTest1.cs
@@ -6,7 +6,7 @@ namespace PlatformServices.Desktop.ComponentTests
     [TestClass]
     [Serializable]
     public class TestProjectForAssemblyResolution : MarshalByRefObject
-    {       
+    {
         [TestMethod]
         public void TestMethod1()
         {

diff --git a/test/E2ETests/DiscoveryAndExecutionTests/OutputTests.cs b/test/E2ETests/DiscoveryAndExecutionTests/OutputTests.cs
@@ -83,7 +83,7 @@ private static void ValidateOutputIsNotMixed(ReadOnlyCollection<OM.TestResult> t
             var testMethod = testResults.Single(t => t.DisplayName == methodName);
             Assert.IsNotNull(testMethod, $"Test method {methodName} was not found.");
             var message = testMethod.Messages.SingleOrDefault(messageFilter);
-            Assert.IsNotNull(message, $"Message for {testMethod.DisplayName} was not found. All messages: { string.Join(Environment.NewLine, testMethod.Messages.Select(m=> $"{m.Category} - {m.Text}")) }");
+            Assert.IsNotNull(message, $"Message for {testMethod.DisplayName} was not found. All messages: { string.Join(Environment.NewLine, testMethod.Messages.Select(m => $"{m.Category} - {m.Text}")) }");
             StringAssert.Matches(message.Text, new Regex(methodName), testMethod.DisplayName);
             StringAssert.Matches(message.Text, new Regex("TestInitialize"), testMethod.DisplayName);
             StringAssert.Matches(message.Text, new Regex("TestCleanup"), testMethod.DisplayName);
@@ -93,7 +93,7 @@ private static void ValidateOutputIsNotMixed(ReadOnlyCollection<OM.TestResult> t
         private static void ValidateInitializeAndCleanup(ReadOnlyCollection<OM.TestResult> testResults, Func<TestResultMessage, bool> messageFilter)
         {
             // It is not deterministic where the class initialize and class cleanup will run, so we look at all tests, to make sure it is includes somewhere.
-            var output = string.Join(Environment.NewLine, testResults.SelectMany(r=>r.Messages).Where(messageFilter).Select(m=>m.Text));
+            var output = string.Join(Environment.NewLine, testResults.SelectMany(r => r.Messages).Where(messageFilter).Select(m => m.Text));
             Assert.IsNotNull(output);
             StringAssert.Matches(output, new Regex("ClassInitialize"));
             StringAssert.Matches(output, new Regex("ClassCleanup"));

diff --git a/test/E2ETests/DiscoveryAndExecutionTests/TestDataSourceExtensibilityTests.cs b/test/E2ETests/DiscoveryAndExecutionTests/TestDataSourceExtensibilityTests.cs
@@ -106,7 +106,7 @@ public void ExecuteCustomTestExtensibilityWithTestDataTests()
         }
 
         [TestMethod]
-        public void BailOutWhenDuplicateTestDisplayName() 
+        public void BailOutWhenDuplicateTestDisplayName()
         {
             // Arrange
             var assemblyPath = Path.IsPathRooted(TestAssembly) ? TestAssembly : this.GetAssetFullPath(TestAssembly);

diff --git a/test/E2ETests/TestAssets/DiscoverInternalsProject/UnitTest1.cs b/test/E2ETests/TestAssets/DiscoverInternalsProject/UnitTest1.cs
@@ -68,7 +68,7 @@ internal void DynamicDataTestMethod(SerializableInternalType serializableInterna
 
         }
 
-        public static IEnumerable<object[]> DynamicData => new []
+        public static IEnumerable<object[]> DynamicData => new[]
         {
             new object[] { new SerializableInternalType() }
         };

diff --git a/test/E2ETests/TestAssets/OutputTestProject/Assembly.cs b/test/E2ETests/TestAssets/OutputTestProject/Assembly.cs
@@ -1,4 +1,4 @@
 using Microsoft.VisualStudio.TestTools.UnitTesting;
 
 [assembly: Parallelize(Scope = ExecutionScope.MethodLevel, Workers = 0)]
-[assembly: ClassCleanupExecution(ClassCleanupBehavior.EndOfClass)] 
+[assembly: ClassCleanupExecution(ClassCleanupBehavior.EndOfClass)]
diff --git a/test/E2ETests/TestAssets/ParallelTestClass/UnitTest1.cs b/test/E2ETests/TestAssets/ParallelTestClass/UnitTest1.cs
@@ -5,7 +5,7 @@ namespace ParallelClassesTestProject
 {
     using System.Threading;
     using Microsoft.VisualStudio.TestTools.UnitTesting;
-    
+
     [TestClass]
     public class UnitTest1
     {

diff --git a/test/E2ETests/TestAssets/ParallelTestClass/UnitTest3.cs b/test/E2ETests/TestAssets/ParallelTestClass/UnitTest3.cs
@@ -5,7 +5,7 @@ namespace ParallelClassesTestProject
 {
     using System.Threading;
     using Microsoft.VisualStudio.TestTools.UnitTesting;
-    
+
     [TestClass]
     public class UnitTest3
     {

diff --git a/test/E2ETests/TestAssets/ParallelTestMethods/UnitTest1.cs b/test/E2ETests/TestAssets/ParallelTestMethods/UnitTest1.cs
@@ -5,7 +5,7 @@ namespace ParallelMethodsTestProject
 {
     using System.Threading;
     using Microsoft.VisualStudio.TestTools.UnitTesting;
-    
+
     [TestClass]
     public class UnitTest1
     {