From 46beb20cb32434d8fe8dd3f8b4107c11094efb85 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Amaury=20Lev=C3=A9?= Date: Fri, 21 Jan 2022 14:58:07 +0100 Subject: [PATCH] Fix some typos in codebase (#3262) --- .../ManagedNameConstants.cs | 2 +- .../Resources/Resources.resx | 4 ++-- .../Resources/xlf/Resources.xlf | 4 ++-- src/Microsoft.TestPlatform.AdapterUtilities/TestIdProvider.cs | 2 +- .../DataCollection/AfterTestRunEndResult.cs | 2 +- .../Filtering/FilterExpression.cs | 2 +- .../Logging/InternalTestLoggerEvents.cs | 2 +- src/Microsoft.TestPlatform.Common/Utilities/FakesUtilities.cs | 2 +- .../Execution/ExecutionManager.cs | 2 +- .../Client/TestSessionInfo.cs | 2 +- .../Attributes/DataCollectorAttachmentProcessor.cs | 2 +- .../BlameDataCollectorTests.cs | 4 ++-- .../RunsettingsTests.cs | 2 +- 13 files changed, 16 insertions(+), 16 deletions(-) diff --git a/src/Microsoft.TestPlatform.AdapterUtilities/ManagedNameConstants.cs b/src/Microsoft.TestPlatform.AdapterUtilities/ManagedNameConstants.cs index caca513122..9415968b99 100644 --- a/src/Microsoft.TestPlatform.AdapterUtilities/ManagedNameConstants.cs +++ b/src/Microsoft.TestPlatform.AdapterUtilities/ManagedNameConstants.cs @@ -4,7 +4,7 @@ namespace Microsoft.TestPlatform.AdapterUtilities { /// - /// Contants to help declare ManagedType and ManagedMethod test properties. + /// Constants to help declare ManagedType and ManagedMethod test properties. /// public static class ManagedNameConstants { diff --git a/src/Microsoft.TestPlatform.AdapterUtilities/Resources/Resources.resx b/src/Microsoft.TestPlatform.AdapterUtilities/Resources/Resources.resx index 52fe4e0b37..31fb65caab 100644 --- a/src/Microsoft.TestPlatform.AdapterUtilities/Resources/Resources.resx +++ b/src/Microsoft.TestPlatform.AdapterUtilities/Resources/Resources.resx @@ -122,7 +122,7 @@ Invalid escape sequence! (segment: {0}, pos: {1}) - An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting postion of invalid sequence. + An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting position of invalid sequence. Method arity must be numeric @@ -133,7 +133,7 @@ A closing single quote was expected at the end of the segment! (segment: {0}) - An error thrown when the type name ended but opened single qoute was not matched. + An error thrown when the type name ended but opened single quote was not matched. Type '{0}' not found diff --git a/src/Microsoft.TestPlatform.AdapterUtilities/Resources/xlf/Resources.xlf b/src/Microsoft.TestPlatform.AdapterUtilities/Resources/xlf/Resources.xlf index cb8d1c6ab6..11e2303f6c 100644 --- a/src/Microsoft.TestPlatform.AdapterUtilities/Resources/xlf/Resources.xlf +++ b/src/Microsoft.TestPlatform.AdapterUtilities/Resources/xlf/Resources.xlf @@ -36,12 +36,12 @@ Example: 'System.Reflection.MethodBase' is not implemented on this platform! Invalid escape sequence! (segment: {0}, pos: {1}) Invalid escape sequence! (segment: {0}, pos: {1}) - An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting postion of invalid sequence. + An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting position of invalid sequence. A closing single quote was expected at the end of the segment! (segment: {0}) A closing single quote was expected at the end of the segment! (segment: {0}) - An error thrown when the type name ended but opened single qoute was not matched. + An error thrown when the type name ended but opened single quote was not matched. diff --git a/src/Microsoft.TestPlatform.AdapterUtilities/TestIdProvider.cs b/src/Microsoft.TestPlatform.AdapterUtilities/TestIdProvider.cs index 5decc3eaf0..7df8ce8cc2 100644 --- a/src/Microsoft.TestPlatform.AdapterUtilities/TestIdProvider.cs +++ b/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; diff --git a/src/Microsoft.TestPlatform.Common/DataCollection/AfterTestRunEndResult.cs b/src/Microsoft.TestPlatform.Common/DataCollection/AfterTestRunEndResult.cs index 53cdd74e1b..a81642ea30 100644 --- a/src/Microsoft.TestPlatform.Common/DataCollection/AfterTestRunEndResult.cs +++ b/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() { diff --git a/src/Microsoft.TestPlatform.Common/Filtering/FilterExpression.cs b/src/Microsoft.TestPlatform.Common/Filtering/FilterExpression.cs index 52b0f74bd4..cc0202fbfb 100644 --- a/src/Microsoft.TestPlatform.Common/Filtering/FilterExpression.cs +++ b/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) { diff --git a/src/Microsoft.TestPlatform.Common/Logging/InternalTestLoggerEvents.cs b/src/Microsoft.TestPlatform.Common/Logging/InternalTestLoggerEvents.cs index d240c8744a..3cf909db96 100644 --- a/src/Microsoft.TestPlatform.Common/Logging/InternalTestLoggerEvents.cs +++ b/src/Microsoft.TestPlatform.Common/Logging/InternalTestLoggerEvents.cs @@ -194,7 +194,7 @@ internal void WaitForEventCompletion() /// /// Raises a test result event to the enabled loggers. /// - /// Arguments to to be raised. + /// Arguments to be raised. internal void RaiseTestResult(TestResultEventArgs args) { ValidateArg.NotNull(args, nameof(args)); diff --git a/src/Microsoft.TestPlatform.Common/Utilities/FakesUtilities.cs b/src/Microsoft.TestPlatform.Common/Utilities/FakesUtilities.cs index e909811f5f..5f368905fe 100644 --- a/src/Microsoft.TestPlatform.Common/Utilities/FakesUtilities.cs +++ b/src/Microsoft.TestPlatform.Common/Utilities/FakesUtilities.cs @@ -106,7 +106,7 @@ private static bool TryAddFakesDataCollectorSettings( IEnumerable 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. diff --git a/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/ExecutionManager.cs b/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/ExecutionManager.cs index fb814ae0c7..332da35a3e 100644 --- a/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/ExecutionManager.cs +++ b/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/ExecutionManager.cs @@ -73,7 +73,7 @@ public void Initialize(IEnumerable pathToAdditionalExtensions, ITestMess this.LoadExtensions(); - //unsubscrive session logger + //unsubscribe session logger this.sessionMessageLogger.TestRunMessage -= this.TestSessionMessageHandler; this.testPlatformEventSource.AdapterSearchStop(); diff --git a/src/Microsoft.TestPlatform.ObjectModel/Client/TestSessionInfo.cs b/src/Microsoft.TestPlatform.ObjectModel/Client/TestSessionInfo.cs index 21ce34307b..ab502d0645 100644 --- a/src/Microsoft.TestPlatform.ObjectModel/Client/TestSessionInfo.cs +++ b/src/Microsoft.TestPlatform.ObjectModel/Client/TestSessionInfo.cs @@ -8,7 +8,7 @@ namespace Microsoft.VisualStudio.TestPlatform.ObjectModel.Client /// /// 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. /// [DataContract] public class TestSessionInfo : IEquatable diff --git a/src/Microsoft.TestPlatform.ObjectModel/DataCollector/Attributes/DataCollectorAttachmentProcessor.cs b/src/Microsoft.TestPlatform.ObjectModel/DataCollector/Attributes/DataCollectorAttachmentProcessor.cs index 08b32fd5d4..3688d0c2e0 100644 --- a/src/Microsoft.TestPlatform.ObjectModel/DataCollector/Attributes/DataCollectorAttachmentProcessor.cs +++ b/src/Microsoft.TestPlatform.ObjectModel/DataCollector/Attributes/DataCollectorAttachmentProcessor.cs @@ -14,7 +14,7 @@ public class DataCollectorAttachmentProcessorAttribute : Attribute /// Initializes a new instance of the class. /// /// - /// The type of the attachement data processor. + /// The type of the attachment data processor. /// public DataCollectorAttachmentProcessorAttribute(Type type) { diff --git a/test/Microsoft.TestPlatform.AcceptanceTests/BlameDataCollectorTests.cs b/test/Microsoft.TestPlatform.AcceptanceTests/BlameDataCollectorTests.cs index 979acae2b2..19bac37859 100644 --- a/test/Microsoft.TestPlatform.AcceptanceTests/BlameDataCollectorTests.cs +++ b/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")] diff --git a/test/Microsoft.TestPlatform.AcceptanceTests/RunsettingsTests.cs b/test/Microsoft.TestPlatform.AcceptanceTests/RunsettingsTests.cs index 93ce21ae97..a00ecb4875 100644 --- a/test/Microsoft.TestPlatform.AcceptanceTests/RunsettingsTests.cs +++ b/test/Microsoft.TestPlatform.AcceptanceTests/RunsettingsTests.cs @@ -67,7 +67,7 @@ public void CommandLineRunSettingsShouldWinAmongAllOptions(RunnerInfo runnerInfo } /// - /// 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. /// [TestMethod] [NetFullTargetFrameworkDataSource]