Fix some typos in codebase (#3262)

src/Microsoft.TestPlatform.AdapterUtilities/ManagedNameConstants.cs

@@ -4,7 +4,7 @@
 namespace Microsoft.TestPlatform.AdapterUtilities
 {
     /// <summary>
-    /// Contants to help declare ManagedType and ManagedMethod test properties.
+    /// Constants to help declare ManagedType and ManagedMethod test properties.
     /// </summary>
     public static class ManagedNameConstants
     {

src/Microsoft.TestPlatform.AdapterUtilities/Resources/Resources.resx

@@ -122,7 +122,7 @@
   </data>
   <data name="ErrorInvalidSequenceAt" xml:space="preserve">
     <value>Invalid escape sequence! (segment: {0}, pos: {1})</value>
-    <comment>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting postion of invalid sequence.</comment>
+    <comment>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting position of invalid sequence.</comment>
   </data>
   <data name="ErrorMethodArityMustBeNumeric" xml:space="preserve">
     <value>Method arity must be numeric</value>
@@ -133,7 +133,7 @@
   </data>
   <data name="ErrorNoClosingQuote" xml:space="preserve">
     <value>A closing single quote was expected at the end of the segment! (segment: {0})</value>
-    <comment>An error thrown when the type name ended but opened single qoute was not matched.</comment>
+    <comment>An error thrown when the type name ended but opened single quote was not matched.</comment>
   </data>
   <data name="ErrorTypeNotFound" xml:space="preserve">
     <value>Type '{0}' not found</value>

src/Microsoft.TestPlatform.AdapterUtilities/Resources/xlf/Resources.xlf

@@ -36,12 +36,12 @@ Example: 'System.Reflection.MethodBase' is not implemented on this platform!</no
       <trans-unit id="ErrorInvalidSequenceAt">
         <source>Invalid escape sequence! (segment: {0}, pos: {1})</source>
         <target state="new">Invalid escape sequence! (segment: {0}, pos: {1})</target>
-        <note>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting postion of invalid sequence.</note>
+        <note>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting position of invalid sequence.</note>
       </trans-unit>
       <trans-unit id="ErrorNoClosingQuote">
         <source>A closing single quote was expected at the end of the segment! (segment: {0})</source>
         <target state="new">A closing single quote was expected at the end of the segment! (segment: {0})</target>
-        <note>An error thrown when the type name ended but opened single qoute was not matched.</note>
+        <note>An error thrown when the type name ended but opened single quote was not matched.</note>
       </trans-unit>
     </body>
   </file>

src/Microsoft.TestPlatform.AdapterUtilities/TestIdProvider.cs

@@ -47,7 +47,7 @@ public void AppendString(string str)
             hasher.ProcessBlock(lastBlock, 0, lastBlock.Length);
             position = 0;
 
-            // We proccessed the entire string already
+            // We processed the entire string already
             if (end == bytes.Length)
             {
                 return;

src/Microsoft.TestPlatform.Common/DataCollection/AfterTestRunEndResult.cs

@@ -14,7 +14,7 @@ namespace Microsoft.VisualStudio.TestPlatform.Common.DataCollection
     [DataContract]
     public class AfterTestRunEndResult
     {
-        // We have more than one ctor for backward-compatibility reason but we don't want to add dependency on Newtosoft([JsonConstructor])
+        // We have more than one ctor for backward-compatibility reason but we don't want to add dependency on Newtonsoft([JsonConstructor])
         // We want to fallback to the non-public default constructor https://www.newtonsoft.com/json/help/html/T_Newtonsoft_Json_ConstructorHandling.htm during deserialization
         private AfterTestRunEndResult()
         {

src/Microsoft.TestPlatform.Common/Filtering/FilterExpression.cs

@@ -166,7 +166,7 @@ internal static FilterExpression Parse(string filterString, out FastFilter fastF
         {
             ValidateArg.NotNull(filterString, nameof(filterString));
 
-            // Below parsing doesn't error out on pattern (), so explicitly search for that (empty parethesis).
+            // Below parsing doesn't error out on pattern (), so explicitly search for that (empty parenthesis).
             var invalidInput = Regex.Match(filterString, @"\(\s*\)");
             if (invalidInput.Success)
             {

src/Microsoft.TestPlatform.Common/Logging/InternalTestLoggerEvents.cs

@@ -194,7 +194,7 @@ internal void WaitForEventCompletion()
         /// <summary>
         /// Raises a test result event to the enabled loggers.
         /// </summary>
-        /// <param name="args">Arguments to to be raised.</param>
+        /// <param name="args">Arguments to be raised.</param>
         internal void RaiseTestResult(TestResultEventArgs args)
         {
             ValidateArg.NotNull(args, nameof(args));

src/Microsoft.TestPlatform.Common/Utilities/FakesUtilities.cs

@@ -106,7 +106,7 @@ private static bool TryAddFakesDataCollectorSettings(
             IEnumerable<string> sources,
             FrameworkVersion framework)
         {
-            // A new Fakes Congigurator API makes the decision to add the right datacollector uri to the configuration
+            // A new Fakes Configurator API makes the decision to add the right datacollector uri to the configuration
             // There now exist two data collector URIs to support two different scenarios. The new scenario involves 
             // using the CLRIE profiler, and the old involves using the Intellitrace profiler (which isn't supported in 
             // .NET Core scenarios). The old API still exists for fallback measures. 

src/Microsoft.TestPlatform.CrossPlatEngine/Execution/ExecutionManager.cs

@@ -73,7 +73,7 @@ public void Initialize(IEnumerable<string> pathToAdditionalExtensions, ITestMess
 
             this.LoadExtensions();
 
-            //unsubscrive session logger
+            //unsubscribe session logger
             this.sessionMessageLogger.TestRunMessage -= this.TestSessionMessageHandler;
 
             this.testPlatformEventSource.AdapterSearchStop();

src/Microsoft.TestPlatform.ObjectModel/Client/TestSessionInfo.cs

@@ -8,7 +8,7 @@ namespace Microsoft.VisualStudio.TestPlatform.ObjectModel.Client
 
     /// <summary>
     /// Defines the test session info object to be passed around between vstest.console and
-    /// vstest.console wrapper in order to indentify the current session.
+    /// vstest.console wrapper in order to identify the current session.
     /// </summary>
     [DataContract]
     public class TestSessionInfo : IEquatable<TestSessionInfo>

src/Microsoft.TestPlatform.ObjectModel/DataCollector/Attributes/DataCollectorAttachmentProcessor.cs

@@ -14,7 +14,7 @@ public class DataCollectorAttachmentProcessorAttribute : Attribute
         /// Initializes a new instance of the <see cref="DataCollectorAttachmentProcessorAttribute"/> class.
         /// </summary>
         /// <param name="type">
-        /// The type of the attachement data processor.
+        /// The type of the attachment data processor.
         /// </param>
         public DataCollectorAttachmentProcessorAttribute(Type type)
         {

test/Microsoft.TestPlatform.AcceptanceTests/BlameDataCollectorTests.cs

@@ -131,7 +131,7 @@ public void HangDumpOnTimeout(RunnerInfo runnerInfo)
         }
 
         [TestMethod]
-        // net5.0 does not suppord dump on exit
+        // net5.0 does not support dump on exit
         [NetCoreRunner("net452;net472;netcoreapp3.1")]
         // should make no difference, keeping for easy debug
         // [NetFrameworkRunner("net452;net472;netcoreapp3.1")]
@@ -150,7 +150,7 @@ public void CrashDumpWhenThereIsNoTimeout(RunnerInfo runnerInfo)
         }
 
         [TestMethod]
-        // net5.0 does not suppord dump on exit
+        // net5.0 does not support dump on exit
         [NetCoreRunner("net452;net472;netcoreapp3.1")]
         // should make no difference, keeping for easy debug
         // [NetFrameworkRunner("net452;net472;netcoreapp3.1")]

test/Microsoft.TestPlatform.AcceptanceTests/RunsettingsTests.cs

@@ -67,7 +67,7 @@ public void CommandLineRunSettingsShouldWinAmongAllOptions(RunnerInfo runnerInfo
         }
 
         /// <summary>
-        /// Command line run settings should have high precedence btween cli runsettings and cli switches.
+        /// Command line run settings should have high precedence between cli runsettings and cli switches.
         /// </summary>
         [TestMethod]
         [NetFullTargetFrameworkDataSource]