diff --git a/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumerator.cs b/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumerator.cs index 9ad61717c0..bbb1e42701 100644 --- a/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumerator.cs +++ b/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumerator.cs @@ -71,7 +71,7 @@ internal ICollection EnumerateAssembly(string assemblyFileName, { // We only want to load the source assembly in reflection only context in UWP scenarios where it is always an exe. // For normal test assemblies continue loading it in the default context since: - // 1. There isnt much benefit in terms of Performance loading the assembly in a Reflection Only context during discovery. + // 1. There isn't much benefit in terms of Performance loading the assembly in a Reflection Only context during discovery. // 2. Loading it in Reflection only context entails a bunch of custom logic to identify custom attributes which is over-kill for normal desktop users. assembly = PlatformServiceProvider.Instance.FileOperations.LoadAssembly( assemblyFileName, @@ -112,7 +112,7 @@ internal ICollection EnumerateAssembly(string assemblyFileName, } catch (Exception exception) { - // If we fail to discover type from a class, then dont abort the discovery + // If we fail to discover type from a class, then don't abort the discovery // Move to the next type. string message = string.Format( CultureInfo.CurrentCulture, @@ -123,7 +123,7 @@ internal ICollection EnumerateAssembly(string assemblyFileName, warningMessages.Add(message); PlatformServiceProvider.Instance.AdapterTraceLogger.LogInfo( - "AssemblyEnumerator: Exception occured while enumerating type {0}. {1}", + "AssemblyEnumerator: Exception occurred while enumerating type {0}. {1}", typeFullName, exception); } @@ -179,10 +179,10 @@ internal Type[] GetTypes(Assembly assembly, string assemblyFileName, ICollection } /// - /// Formats load exception as multiline string, each line contains load error message. + /// Formats load exception as multi-line string, each line contains load error message. /// /// The exception. - /// Returns loader exceptions as a multiline string. + /// Returns loader exceptions as a multi-line string. internal string GetLoadExceptionDetails(ReflectionTypeLoadException ex) { Debug.Assert(ex != null, "exception should not be null."); diff --git a/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumeratorWrapper.cs b/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumeratorWrapper.cs index b99e7dd789..fb5683726b 100644 --- a/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumeratorWrapper.cs +++ b/src/Adapter/MSTest.CoreAdapter/Discovery/AssemblyEnumeratorWrapper.cs @@ -28,9 +28,9 @@ internal class AssemblyEnumeratorWrapper /// /// Gets test elements from an assembly. /// - /// The assembly file name. + /// The assembly file name. /// The run Settings. - /// Contains warnings if any, that need to be passed back to the caller. + /// Contains warnings if any, that need to be passed back to the caller. /// A collection of test elements. [SuppressMessage("Microsoft.Design", "CA1031:DoNotCatchGeneralExceptionTypes", Justification = "Catching a generic exception since it is a requirement to not abort discovery in case of any errors.")] [SuppressMessage("Microsoft.Design", "CA1021:AvoidOutParameters", MessageId = "3#", Justification = "This is only for internal use.")] @@ -66,7 +66,7 @@ internal ICollection GetTests( return null; } - // Load the assemly in isolation if required. + // Load the assembly in isolation if required. return this.GetTestsInIsolation(fullFilePath, runSettings, out warnings); } catch (FileNotFoundException ex) diff --git a/src/Adapter/MSTest.CoreAdapter/Discovery/TestMethodValidator.cs b/src/Adapter/MSTest.CoreAdapter/Discovery/TestMethodValidator.cs index 9921926dd5..258595af3c 100644 --- a/src/Adapter/MSTest.CoreAdapter/Discovery/TestMethodValidator.cs +++ b/src/Adapter/MSTest.CoreAdapter/Discovery/TestMethodValidator.cs @@ -51,7 +51,7 @@ internal virtual bool IsValidTestMethod(MethodInfo testMethodInfo, Type type, IC return false; } - // Todo: Decide wheter parameter count matters. + // Todo: Decide whether parameter count matters. // The isGenericMethod check below id to verify that there are no closed generic methods slipping through. // Closed generic methods being GenericMethod and open being GenericMethod. var isValidTestMethod = testMethodInfo.IsPublic && !testMethodInfo.IsAbstract && !testMethodInfo.IsStatic diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/StackTraceHelper.cs b/src/Adapter/MSTest.CoreAdapter/Execution/StackTraceHelper.cs index e08f35a6cd..6b7fded5e5 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/StackTraceHelper.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/StackTraceHelper.cs @@ -64,8 +64,8 @@ internal static StackTraceInformation GetStackTraceInformation(Exception ex) // TODO:Fix the shadow stack-trace used in Private Object // (Look-in Assertion.cs in the UnitTestFramework assembly) - // Sometimes the stacktrace can be null, but the inner stacktrace - // contains information. We are not interested in null stacktraces + // Sometimes the stack trace can be null, but the inner stack trace + // contains information. We are not interested in null stack traces // so we simply ignore this case try { @@ -79,7 +79,7 @@ internal static StackTraceInformation GetStackTraceInformation(Exception ex) // curException.StackTrace can throw exception, Although MSDN doc doesn't say that. try { - // try to get stacktrace + // try to get stack trace if (e.StackTrace != null) { stackTraces.Push(e.StackTrace); @@ -88,7 +88,7 @@ internal static StackTraceInformation GetStackTraceInformation(Exception ex) catch (Exception) { PlatformServiceProvider.Instance.AdapterTraceLogger.LogError( - "StackTraceHelper.GetStackTraceInformation: Failed to get stacktrace info."); + "StackTraceHelper.GetStackTraceInformation: Failed to get stack trace info."); } } } diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TestAssemblyInfo.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TestAssemblyInfo.cs index ca5b9558b9..2d0a23e5b9 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/TestAssemblyInfo.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/TestAssemblyInfo.cs @@ -129,7 +129,7 @@ public void RunAssemblyInitialize(TestContext testContext) // If assembly initialization is not done, then do it. if (!this.IsAssemblyInitializeExecuted) { - // Aquiring a lock is usually a costly operation which does not need to be + // Acquiring a lock is usually a costly operation which does not need to be // performed every time if the assembly init is already executed. lock (this.assemblyInfoExecuteSyncObject) { @@ -152,7 +152,7 @@ public void RunAssemblyInitialize(TestContext testContext) } } - // If assemblyInitialization was successful, then dont do anything + // If assemblyInitialization was successful, then don't do anything if (this.AssemblyInitializationException == null) { return; diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TestClassInfo.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TestClassInfo.cs index 143f6f2f06..95da9337de 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/TestClassInfo.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/TestClassInfo.cs @@ -258,7 +258,7 @@ public void RunClassInitialize(TestContext testContext) return; } - // Aquiring a lock is usually a costly operation which does not need to be + // Acquiring a lock is usually a costly operation which does not need to be // performed every time if the class init is already executed. lock (this.testClassExecuteSyncObject) { @@ -298,7 +298,7 @@ public void RunClassInitialize(TestContext testContext) // and class initialization exception is null, then do it. if (!this.IsClassInitializeExecuted && this.classInitializeMethod != null && this.ClassInitializationException == null) { - // Aquiring a lock is usually a costly operation which does not need to be + // Acquiring a lock is usually a costly operation which does not need to be // performed every time if the class init is already executed. lock (this.testClassExecuteSyncObject) { @@ -322,7 +322,7 @@ public void RunClassInitialize(TestContext testContext) } } - // If classInitialization was successful, then dont do anything + // If classInitialization was successful, then don't do anything if (this.ClassInitializationException == null) { return; diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TestExecutionManager.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TestExecutionManager.cs index d7f27bc823..c8a8620d04 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/TestExecutionManager.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/TestExecutionManager.cs @@ -59,7 +59,7 @@ public TestExecutionManager() /// Tests to be run. /// Context to use when executing the tests. /// Handle to the framework to record results and to do framework operations. - /// Test run cancellation tokenn + /// Test run cancellation token public void RunTests(IEnumerable tests, IRunContext runContext, IFrameworkHandle frameworkHandle, TestRunCancellationToken runCancellationToken) { Debug.Assert(tests != null, "tests"); @@ -235,7 +235,7 @@ private void ExecuteTestsInSource(IEnumerable tests, IRunContext runCo testsToRun = tests.Where(t => MatchTestFilter(filterExpression, t, this.TestMethodFilter)); - // this is done so that appropriate values of testcontext properties are set at source level + // this is done so that appropriate values of test context properties are set at source level // and are merged with session level parameters var sourceLevelParameters = PlatformServiceProvider.Instance.SettingsProvider.GetProperties(source); diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs index 937f0ecd55..fd918bd17d 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodInfo.cs @@ -54,7 +54,7 @@ internal TestMethodInfo( public string NotRunnableReason { get; internal set; } /// - /// Gets a value indicating whether test is runnnable + /// Gets a value indicating whether test is runnable /// public bool IsRunnable => string.IsNullOrEmpty(this.NotRunnableReason); @@ -350,9 +350,9 @@ private TestResult ExecuteInternal(object[] arguments) // Set the current tests outcome before cleanup so it can be used in the cleanup logic. this.TestMethodOptions.TestContext.SetOutcome(result.Outcome); - // TestCleanup can potentially be a long running operation which should'nt ideally be in a finally block. + // TestCleanup can potentially be a long running operation which shouldn't ideally be in a finally block. // Pulling it out so extension writers can abort custom cleanups if need be. Having this in a finally block - // does not allow a threadabort exception to be raised within the block but throws one after finally is executed + // does not allow a thread abort exception to be raised within the block but throws one after finally is executed // crashing the process. This was blocking writing an extension for Dynamic Timeout in VSO. if (classInstance != null && testContextSetup) { @@ -788,7 +788,7 @@ void executeAsyncAction() } else { - // Cancel the token source as test has timedout + // Cancel the token source as test has timed out this.TestMethodOptions.TestContext.Context.CancellationTokenSource.Cancel(); } diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodRunner.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodRunner.cs index cad0203f89..67fb4b3823 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodRunner.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/TestMethodRunner.cs @@ -405,7 +405,7 @@ private UTF.UnitTestOutcome GetAggregateOutcome(List results) } /// - /// Updates given resutls with parent info if results are greater than 1. + /// Updates given results with parent info if results are greater than 1. /// Add parent results as first result in updated result. /// /// Results. diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TestRunCancellationToken.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TestRunCancellationToken.cs index 932d8aa123..0423848c89 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/TestRunCancellationToken.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/TestRunCancellationToken.cs @@ -15,7 +15,7 @@ public class TestRunCancellationToken /// /// Stores whether the test run is canceled or not. /// - private bool cancelled; + private bool canceled; /// /// Callback to be invoked when canceled. @@ -29,13 +29,13 @@ public bool Canceled { get { - return this.cancelled; + return this.canceled; } private set { - this.cancelled = value; - if (this.cancelled) + this.canceled = value; + if (this.canceled) { this.registeredCallback?.Invoke(); } diff --git a/src/Adapter/MSTest.CoreAdapter/Execution/TypeCache.cs b/src/Adapter/MSTest.CoreAdapter/Execution/TypeCache.cs index 6862fe546c..69607ec196 100644 --- a/src/Adapter/MSTest.CoreAdapter/Execution/TypeCache.cs +++ b/src/Adapter/MSTest.CoreAdapter/Execution/TypeCache.cs @@ -176,7 +176,7 @@ private TestClassInfo GetClassInfo(TestMethod testMethod) if (!this.classInfoCache.TryGetValue(typeName, out TestClassInfo classInfo)) { - // Aquiring a lock is usually a costly operation which does not need to be + // Acquiring a lock is usually a costly operation which does not need to be // performed every time if the type is found in the cache. lock (this.classInfoSyncObject) { @@ -402,7 +402,7 @@ private TestAssemblyInfo GetAssemblyInfo(Type type) { // If we fail to discover type from an assembly, then do not abort. Pick the next type. PlatformServiceProvider.Instance.AdapterTraceLogger.LogWarning( - "TypeCache: Exception occured while checking whether type {0} is a test class or not. {1}", + "TypeCache: Exception occurred while checking whether type {0} is a test class or not. {1}", t.FullName, ex); diff --git a/src/Adapter/MSTest.CoreAdapter/Extensions/ExceptionExtensions.cs b/src/Adapter/MSTest.CoreAdapter/Extensions/ExceptionExtensions.cs index 9824400a83..f6e0b98b56 100644 --- a/src/Adapter/MSTest.CoreAdapter/Extensions/ExceptionExtensions.cs +++ b/src/Adapter/MSTest.CoreAdapter/Extensions/ExceptionExtensions.cs @@ -119,7 +119,7 @@ internal static bool TryGetUnitTestAssertException(this Exception exception, out /// /// An instance. /// Appends TestFailedException message to this message. - /// Appends TestFailedExeption stacktrace to this stackTrace + /// Appends TestFailedExeption stack trace to this stackTrace internal static void TryGetTestFailureExceptionMessageAndStackTrace(this TestFailedException testFailureException, StringBuilder message, StringBuilder stackTrace) { if (testFailureException != null) diff --git a/src/Adapter/MSTest.CoreAdapter/Extensions/MethodInfoExtensions.cs b/src/Adapter/MSTest.CoreAdapter/Extensions/MethodInfoExtensions.cs index ca8bc17e51..b221df7923 100644 --- a/src/Adapter/MSTest.CoreAdapter/Extensions/MethodInfoExtensions.cs +++ b/src/Adapter/MSTest.CoreAdapter/Extensions/MethodInfoExtensions.cs @@ -70,7 +70,7 @@ internal static bool HasCorrectTestInitializeOrCleanupSignature(this MethodInfo /// Verifies that the test method has the correct signature /// /// The method to verify. - /// Indicates whether parameter lenght is to be ignored. + /// Indicates whether parameter length is to be ignored. /// True if the method has the right test method signature. internal static bool HasCorrectTestMethodSignature(this MethodInfo method, bool ignoreParameterLength) { diff --git a/src/Adapter/MSTest.CoreAdapter/Helpers/ReflectHelper.cs b/src/Adapter/MSTest.CoreAdapter/Helpers/ReflectHelper.cs index feeeddd7b5..efee7052f8 100644 --- a/src/Adapter/MSTest.CoreAdapter/Helpers/ReflectHelper.cs +++ b/src/Adapter/MSTest.CoreAdapter/Helpers/ReflectHelper.cs @@ -27,7 +27,7 @@ internal class ReflectHelper : MarshalByRefObject /// /// Member/Type to test /// Attribute to search for - /// Look throug inheritence or not + /// Look through inheritance or not /// True if the attribute of the specified type is defined. public virtual bool IsAttributeDefined(MemberInfo memberInfo, Type attributeType, bool inherit) { @@ -68,7 +68,7 @@ public virtual bool IsAttributeDefined(MemberInfo memberInfo, Type attributeType /// /// Member/Type to test /// Attribute to search for - /// Look throug inheritence or not + /// Look through inheritance or not /// True if the specified attribute is defined on the type. public virtual bool IsAttributeDefined(Type type, Type attributeType, bool inherit) { @@ -223,9 +223,9 @@ internal static bool MatchReturnType(MethodInfo method, Type returnType) /// /// Get custom attributes on a member for both normal and reflection only load. /// - /// Memeber for which attributes needs to be retrieved. + /// Member for which attributes needs to be retrieved. /// Type of attribute to retrieve. - /// If inheritied type of attribute. + /// If inherited type of attribute. /// All attributes of give type on member. internal static Attribute[] GetCustomAttributes(MemberInfo memberInfo, Type type, bool inherit) { @@ -245,8 +245,8 @@ internal static Attribute[] GetCustomAttributes(MemberInfo memberInfo, Type type /// /// Get custom attributes on a member for both normal and reflection only load. /// - /// Memeber for which attributes needs to be retrieved. - /// If inheritied type of attribute. + /// Member for which attributes needs to be retrieved. + /// If inherited type of attribute. /// All attributes of give type on member. internal static object[] GetCustomAttributes(MemberInfo memberInfo, bool inherit) { @@ -301,7 +301,7 @@ internal Attribute GetAttribute(Type attributeType, MethodInfo method) } /// - /// Returns true when the method is delcared in the assembly where the type is declared. + /// Returns true when the method is declared in the assembly where the type is declared. /// /// The method to check for. /// The type declared in the assembly to check. @@ -336,7 +336,7 @@ internal virtual string[] GetCategories(MemberInfo categoryAttributeProvider, Ty /// /// Gets the parallelization level set on an assembly. /// - /// The test asembly. + /// The test assembly. /// The parallelization level if set. -1 otherwise. internal ParallelizeAttribute GetParallelizeAttribute(Assembly assembly) { diff --git a/src/Adapter/MSTest.CoreAdapter/IPlatformServiceProvider.cs b/src/Adapter/MSTest.CoreAdapter/IPlatformServiceProvider.cs index f3195fc506..a8311deb2e 100644 --- a/src/Adapter/MSTest.CoreAdapter/IPlatformServiceProvider.cs +++ b/src/Adapter/MSTest.CoreAdapter/IPlatformServiceProvider.cs @@ -64,7 +64,7 @@ internal interface IPlatformServiceProvider /// The run Settings for the session. /// /// - /// The handle to the the test platform. + /// The handle to the test platform. /// /// /// Returns the host for the source provided. diff --git a/src/Adapter/MSTest.CoreAdapter/MSTestExecutor.cs b/src/Adapter/MSTest.CoreAdapter/MSTestExecutor.cs index 31ea7de870..361373b979 100644 --- a/src/Adapter/MSTest.CoreAdapter/MSTestExecutor.cs +++ b/src/Adapter/MSTest.CoreAdapter/MSTestExecutor.cs @@ -19,7 +19,7 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter public class MSTestExecutor : ITestExecutor { /// - /// Token for cancelling the test run. + /// Token for canceling the test run. /// private TestRunCancellationToken cancellationToken = null; diff --git a/src/Adapter/MSTest.CoreAdapter/ObjectModel/StackTraceInformation.cs b/src/Adapter/MSTest.CoreAdapter/ObjectModel/StackTraceInformation.cs index 62c381f479..5b88eb14fa 100644 --- a/src/Adapter/MSTest.CoreAdapter/ObjectModel/StackTraceInformation.cs +++ b/src/Adapter/MSTest.CoreAdapter/ObjectModel/StackTraceInformation.cs @@ -32,17 +32,17 @@ public StackTraceInformation(string stackTrace, string filePath, int lineNumber, public string ErrorStackTrace { get; private set; } /// - /// Gets source code FilePath where the error occured + /// Gets source code FilePath where the error occurred /// public string ErrorFilePath { get; private set; } /// - /// Gets line number in the source code file where the error occured. + /// Gets line number in the source code file where the error occurred. /// public int ErrorLineNumber { get; private set; } /// - /// Gets column number in the source code file where the error occured. + /// Gets column number in the source code file where the error occurred. /// public int ErrorColumnNumber { get; private set; } } diff --git a/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethod.cs b/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethod.cs index 708e9b621f..2af0ef2b98 100644 --- a/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethod.cs +++ b/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethod.cs @@ -57,7 +57,7 @@ public TestMethod(string name, string fullClassName, string assemblyName, bool /// /// Gets or sets the declaring class full name. This will be used while getting navigation data. /// This will be null if AssemblyName is same as DeclaringAssemblyName. - /// Reason to set to null in the above case is to minimise the transfer of data across appdomains and not have a perf hit. + /// Reason to set to null in the above case is to minimize the transfer of data across appdomains and not have a performance hit. /// public string DeclaringAssemblyName { diff --git a/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethodOptions.cs b/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethodOptions.cs index 86ce391d24..59f7bc5d06 100644 --- a/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethodOptions.cs +++ b/src/Adapter/MSTest.CoreAdapter/ObjectModel/TestMethodOptions.cs @@ -7,7 +7,7 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.ObjectModel using Microsoft.VisualStudio.TestTools.UnitTesting; /// - /// A fascade service for options passed to a test method. + /// A facade service for options passed to a test method. /// internal class TestMethodOptions { diff --git a/src/Adapter/MSTest.CoreAdapter/ObjectModel/UnitTestResult.cs b/src/Adapter/MSTest.CoreAdapter/ObjectModel/UnitTestResult.cs index 43bee38e66..24482a8b5a 100644 --- a/src/Adapter/MSTest.CoreAdapter/ObjectModel/UnitTestResult.cs +++ b/src/Adapter/MSTest.CoreAdapter/ObjectModel/UnitTestResult.cs @@ -144,9 +144,9 @@ internal UnitTestResult(UnitTestOutcome outcome, string errorMessage) /// /// Convert parameter unitTestResult to testResult /// - /// The test Case. - /// The start Time. - /// The end Time. + /// The test Case. + /// The start Time. + /// The end Time. /// Current MSTest settings. /// The . internal TestResult ToTestResult(TestCase testCase, DateTimeOffset startTime, DateTimeOffset endTime, MSTestSettings currentSettings) diff --git a/src/Adapter/PlatformServices.Desktop/AssemblyResolver.cs b/src/Adapter/PlatformServices.Desktop/AssemblyResolver.cs index a3d990563b..26b489c009 100644 --- a/src/Adapter/PlatformServices.Desktop/AssemblyResolver.cs +++ b/src/Adapter/PlatformServices.Desktop/AssemblyResolver.cs @@ -64,7 +64,7 @@ public class AssemblyResolver : MarshalByRefObject, IDisposable /// A list of directories for resolution path /// /// - /// If there are additonal paths where a recursive search is required + /// If there are additional paths where a recursive search is required /// call AddSearchDirectoryFromRunSetting method with that list. /// public AssemblyResolver(IList directories) @@ -197,7 +197,7 @@ protected virtual void Dispose(bool disposing) { if (disposing) { - // cleanup Managed resourceslike calling dispose on other managed object created. + // cleanup Managed resources like calling dispose on other managed object created. AppDomain.CurrentDomain.AssemblyResolve -= new ResolveEventHandler(this.OnResolve); AppDomain.CurrentDomain.ReflectionOnlyAssemblyResolve -= new ResolveEventHandler(this.ReflectionOnlyOnResolve); @@ -434,7 +434,7 @@ private Assembly OnResolveInternal(object senderAppDomain, ResolveEventArgs args // see, if we can find it in user specified search directories. while (assembly == null && this.directoryList.Count > 0) { - // instead of loading whole saerch directory in one time, we are adding directory on the basis of need + // instead of loading whole search directory in one time, we are adding directory on the basis of need var currentNode = this.directoryList.Dequeue(); List increamentalSearchDirectory = new List(); @@ -457,7 +457,7 @@ private Assembly OnResolveInternal(object senderAppDomain, ResolveEventArgs args } else { - // generate warning that path doesnot exist. + // generate warning that path does not exist. this.SafeLog( assemblyNameToLoad, () => @@ -484,7 +484,7 @@ private Assembly OnResolveInternal(object senderAppDomain, ResolveEventArgs args if (isReflectionOnly) { // Put it in the resolved assembly cache so that if the Load call below - // triggers another assembly resolution, then we dont end up in stack overflow. + // triggers another assembly resolution, then we don't end up in stack overflow. this.reflectionOnlyResolvedAssemblies[assemblyNameToLoad] = null; assembly = Assembly.ReflectionOnlyLoad(assemblyNameToLoad); @@ -497,7 +497,7 @@ private Assembly OnResolveInternal(object senderAppDomain, ResolveEventArgs args else { // Put it in the resolved assembly cache so that if the Load call below - // triggers another assembly resolution, then we dont end up in stack overflow. + // triggers another assembly resolution, then we don't end up in stack overflow. this.resolvedAssemblies[assemblyNameToLoad] = null; assembly = Assembly.Load(assemblyNameToLoad); @@ -573,7 +573,7 @@ private bool TryLoadFromCache(string assemblyName, bool isReflectionOnly, out As /// CLR does not trigger a load when the EqtTrace messages are in a lamda expression. Leaving it that way /// to preserve readability instead of creating wrapper functions. /// - /// The assembly being rsolved. + /// The assembly being resolved. /// The logger function. private void SafeLog(string assemblyName, Action loggerAction) { @@ -591,7 +591,7 @@ private void SafeLog(string assemblyName, Action loggerAction) /// The requested Name. /// Indicates whether this is called under a Reflection Only Load context. /// The . - [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Reliability", "CA2001:AvoidCallingProblematicMethods", MessageId = "System.Reflection.Assembly.LoadFrom", Justification = "The assembly location is fugred out from the configuration that the user passes in.")] + [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Reliability", "CA2001:AvoidCallingProblematicMethods", MessageId = "System.Reflection.Assembly.LoadFrom", Justification = "The assembly location is figured out from the configuration that the user passes in.")] [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Design", "CA1031:DoNotCatchGeneralExceptionTypes", Justification = "Requirement is to handle all kinds of user exceptions and message appropriately.")] private Assembly SearchAndLoadAssembly(string assemblyPath, string assemblyName, AssemblyName requestedName, bool isReflectionOnly) { @@ -645,7 +645,7 @@ private Assembly SearchAndLoadAssembly(string assemblyPath, string assemblyName, } }); - // Rethrow FileLoadException, because this exception means that the assembly + // Re-throw FileLoadException, because this exception means that the assembly // was found, but could not be loaded. This will allow us to report a more // specific error message to the user for things like access denied. throw; diff --git a/src/Adapter/PlatformServices.Desktop/Data/OdbcDataConnection.cs b/src/Adapter/PlatformServices.Desktop/Data/OdbcDataConnection.cs index 34a2340e93..6280c485bb 100644 --- a/src/Adapter/PlatformServices.Desktop/Data/OdbcDataConnection.cs +++ b/src/Adapter/PlatformServices.Desktop/Data/OdbcDataConnection.cs @@ -107,7 +107,7 @@ private static string FixConnectionString(string connectionString, List } else { - // Fixup magic file paths + // Fix-up magic file paths string fixedFilePath = FixPath(fileName, dataFolders); if (fixedFilePath != null) { diff --git a/src/Adapter/PlatformServices.Desktop/Data/OleDataConnection.cs b/src/Adapter/PlatformServices.Desktop/Data/OleDataConnection.cs index f720018879..e4019c8777 100644 --- a/src/Adapter/PlatformServices.Desktop/Data/OleDataConnection.cs +++ b/src/Adapter/PlatformServices.Desktop/Data/OleDataConnection.cs @@ -95,7 +95,7 @@ private static string FixConnectionString(string connectionString, List } else { - // Fixup magic file paths + // Fix-up magic file paths string fixedFilePath = FixPath(fileName, dataFolders); if (fixedFilePath != null) { diff --git a/src/Adapter/PlatformServices.Desktop/Data/SqlDataConnection.cs b/src/Adapter/PlatformServices.Desktop/Data/SqlDataConnection.cs index c05d3d3487..ebe0019703 100644 --- a/src/Adapter/PlatformServices.Desktop/Data/SqlDataConnection.cs +++ b/src/Adapter/PlatformServices.Desktop/Data/SqlDataConnection.cs @@ -18,7 +18,7 @@ public SqlDataConnection(string invariantProviderName, string connectionString, /// /// Returns default database schema. - /// this.Connection must be alredy opened. + /// this.Connection must be already opened. /// /// The default database schema. public override string GetDefaultSchema() @@ -58,7 +58,7 @@ private static string FixConnectionString(string connectionString, List // for a long time, preventing us from moving files around sqlBuilder.Pooling = false; - // Fixup magic file paths + // Fix-up magic file paths string fixedFilePath = FixPath(attachedFile, dataFolders); if (fixedFilePath != null) { diff --git a/src/Adapter/PlatformServices.Desktop/Data/TestDataConnectionSql.cs b/src/Adapter/PlatformServices.Desktop/Data/TestDataConnectionSql.cs index f6323cdf19..0593e22fcf 100644 --- a/src/Adapter/PlatformServices.Desktop/Data/TestDataConnectionSql.cs +++ b/src/Adapter/PlatformServices.Desktop/Data/TestDataConnectionSql.cs @@ -223,7 +223,7 @@ public string PrepareNameForSql(string tableName) /// /// Take a possibly qualified name and break it down into an - /// array of identifers unquoting any quoted names + /// array of identifiers unquoting any quoted names /// /// A string. /// An array of unquoted parts, or null if the name fails to conform @@ -391,7 +391,7 @@ private string MaybeQuote(string identifier, bool force) } /// - /// Find the first seperator in a string + /// Find the first separator in a string /// /// The string. /// Index. @@ -414,7 +414,7 @@ private int FindIdentifierEnd(string text, int start) // These routine assumes prefixes and suffixes // are single characters string prefix = this.QuotePrefix; - Debug.Assert(prefix.Length == 1, "prefix lenght should be 1."); + Debug.Assert(prefix.Length == 1, "prefix length should be 1."); char prefixChar = prefix[0]; int end = text.Length; @@ -429,7 +429,7 @@ private int FindIdentifierEnd(string text, int start) int here = start + 1; string suffix = this.QuoteSuffix; - Debug.Assert(suffix.Length == 1, "suffix lenght should be 1."); + Debug.Assert(suffix.Length == 1, "suffix length should be 1."); char suffixChar = suffix[0]; while (here < end) @@ -776,7 +776,7 @@ this.Connection is SqlConnection || (odbcConnection != null && IsMSSql(odbcConnection.Driver)), "GetDefaultSchemaMSSql should be called only for MS SQL (either native or Ole Db or Odbc)."); - Debug.Assert(this.IsOpen(), "The connection must aready be open!"); + Debug.Assert(this.IsOpen(), "The connection must already be open!"); Debug.Assert(!string.IsNullOrEmpty(this.Connection.ServerVersion), "GetDefaultSchema: the ServerVersion is null or empty!"); int index = this.Connection.ServerVersion.IndexOf(".", StringComparison.Ordinal); diff --git a/src/Adapter/PlatformServices.Desktop/Deployment/AssemblyLoadWorker.cs b/src/Adapter/PlatformServices.Desktop/Deployment/AssemblyLoadWorker.cs index 7be9285bc2..41db44b578 100644 --- a/src/Adapter/PlatformServices.Desktop/Deployment/AssemblyLoadWorker.cs +++ b/src/Adapter/PlatformServices.Desktop/Deployment/AssemblyLoadWorker.cs @@ -81,7 +81,7 @@ public override object InitializeLifetimeService() /// Get the target dotNet framework string for the assembly /// /// Path of the assembly file - /// String representation of the the target dotNet framework e.g. .NETFramework,Version=v4.0 + /// String representation of the target dotNet framework e.g. .NETFramework,Version=v4.0 internal string GetTargetFrameworkVersionStringFromPath(string path) { if (File.Exists(path)) @@ -114,7 +114,7 @@ internal string GetTargetFrameworkVersionStringFromPath(string path) /// Get the target dot net framework string for the assembly /// /// Assembly from which target framework has to find - /// String representation of the the target dot net framework e.g. .NETFramework,Version=v4.0 + /// String representation of the target dot net framework e.g. .NETFramework,Version=v4.0 private string GetTargetFrameworkStringFromAssembly(Assembly assembly) { string dotNetVersion = string.Empty; diff --git a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestContextImplementation.cs b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestContextImplementation.cs index dda8ac309d..365dd15478 100644 --- a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestContextImplementation.cs +++ b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestContextImplementation.cs @@ -21,7 +21,7 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices /// /// Internal implementation of TestContext exposed to the user. - /// The virtual string properties of the TestContext are retreived from the property dictionary + /// The virtual string properties of the TestContext are retrieved from the property dictionary /// like GetProperty<string>("TestName") or GetProperty<string>("FullyQualifiedTestClassName"); /// public class TestContextImplementation : UTF.TestContext, ITestContext diff --git a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestDataSource.cs b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestDataSource.cs index 0019d0b5ca..20a4d589b5 100644 --- a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestDataSource.cs +++ b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestDataSource.cs @@ -31,7 +31,7 @@ public class TestDataSource : ITestDataSource public IEnumerable GetData(UTF.ITestMethod testMethodInfo, ITestContext testContext) { // Figure out where (as well as the current directory) we could look for data files - // for unit tests this means looking at the the location of the test itself + // for unit tests this means looking at the location of the test itself List dataFolders = new List(); dataFolders.Add(Path.GetDirectoryName(new Uri(testMethodInfo.MethodInfo.Module.Assembly.CodeBase).LocalPath)); @@ -62,7 +62,7 @@ public IEnumerable GetData(UTF.ITestMethod testMethodInfo, ITestContext DataRow[] rows = table.Select(); Debug.Assert(rows != null, "rows should not be null."); - // check for rowlength is 0 + // check for row length is 0 if (rows.Length == 0) { return null; diff --git a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSource.cs b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSource.cs index f1cccd4dbb..f00b3095e3 100644 --- a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSource.cs +++ b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSource.cs @@ -24,7 +24,7 @@ public IEnumerable ValidSourceExtensions { get { - // Since desktop Platform service would also discover other platform tests on dekstop, + // Since desktop Platform service would also discover other platform tests on desktop, // this extension list needs to be updated with all platforms supported file extensions. return new List { @@ -43,7 +43,7 @@ public IEnumerable ValidSourceExtensions /// True if the assembly is referenced. public bool IsAssemblyReferenced(AssemblyName assemblyName, string source) { - // This loads the dll in a different app domain. We can optimize this to load in the current domain since this code ould be run in a new app domain anyway. + // This loads the dll in a different app domain. We can optimize this to load in the current domain since this code could be run in a new app domain anyway. bool? utfReference = AssemblyHelper.DoesReferencesAssembly(source, assemblyName); // If no reference to UTF don't run discovery. Take conservative approach. If not able to find proceed with discovery. diff --git a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSourceHost.cs b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSourceHost.cs index 8bb64e98c1..227d338194 100644 --- a/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSourceHost.cs +++ b/src/Adapter/PlatformServices.Desktop/Services/DesktopTestSourceHost.cs @@ -117,7 +117,7 @@ public void SetupHost() string domainName = string.Format("TestSourceHost: Enumerating source ({0})", this.sourceFileName); this.domain = this.appDomain.CreateDomain(domainName, null, appDomainSetup); - // Load objectModel before creating assembly resolver otherwise in 3.5 process, we run into a recurive assembly resolution + // Load objectModel before creating assembly resolver otherwise in 3.5 process, we run into a recursive assembly resolution // which is trigged by AppContainerUtilities.AttachEventToResolveWinmd method. EqtTrace.SetupRemoteEqtTraceListeners(this.domain); @@ -154,7 +154,7 @@ public void SetupHost() /// If a type is to be created in isolation then it needs to be a MarshalByRefObject. public object CreateInstanceForType(Type type, object[] args) { - // Honour DisableAppDomain setting if it is present in runsettings + // Honor DisableAppDomain setting if it is present in runsettings if (this.isAppDomainCreationDisabled) { return Activator.CreateInstance(type, args); @@ -194,7 +194,7 @@ public void Dispose() if (this.frameworkHandle != null) { // Let the test platform know that it should tear down the test host process - // since we we have issues in unloading appdomain. We do so to avoid any assembly locking issues. + // since we have issues in unloading appdomain. We do so to avoid any assembly locking issues. this.frameworkHandle.EnableShutdownAfterTestRun = true; EqtTrace.Verbose("DesktopTestSourceHost.Dispose(): Notifying the test platform that the test host process should be shut down because the app domain running tests could not be unloaded successfully."); @@ -248,7 +248,7 @@ internal virtual List GetResolutionPaths(string sourceFileName, bool isP { List resolutionPaths = new List(); - // Add path of test assembly in resolution path. Mostly will be used for resovling winmd. + // Add path of test assembly in resolution path. Mostly will be used for resolving winmd. resolutionPaths.Add(Path.GetDirectoryName(sourceFileName)); if (!isPortableMode) @@ -358,7 +358,7 @@ private void AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(Assemb catch (Exception exception) { EqtTrace.Error( - "DesktopTestSourceHost.AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(): Exception hit while trying to set assemly resolver for domain. Exception : {0} \n Message : {1}", + "DesktopTestSourceHost.AddSearchDirectoriesSpecifiedInRunSettingsToAssemblyResolver(): Exception hit while trying to set assembly resolver for domain. Exception : {0} \n Message : {1}", exception, exception.Message); } diff --git a/src/Adapter/PlatformServices.Desktop/Utilities/AppDomainUtilities.cs b/src/Adapter/PlatformServices.Desktop/Utilities/AppDomainUtilities.cs index bd86955670..7be4ff734c 100644 --- a/src/Adapter/PlatformServices.Desktop/Utilities/AppDomainUtilities.cs +++ b/src/Adapter/PlatformServices.Desktop/Utilities/AppDomainUtilities.cs @@ -73,7 +73,7 @@ internal static void SetAppDomainFrameworkVersionBasedOnTestSource(AppDomainSetu /// /// /// Framework string - /// Todo: Need to add components/E2E tests to cover these scenarios. + /// TODO: Need to add components/E2E tests to cover these scenarios. /// [SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1650:ElementDocumentationMustBeSpelledCorrectly", Justification = "Reviewed. Suppression is OK here.")] internal static string GetTargetFrameworkVersionString(string testSourcePath) diff --git a/src/Adapter/PlatformServices.Desktop/Utilities/DesktopDeploymentUtility.cs b/src/Adapter/PlatformServices.Desktop/Utilities/DesktopDeploymentUtility.cs index 593974700b..85d4df61d8 100644 --- a/src/Adapter/PlatformServices.Desktop/Utilities/DesktopDeploymentUtility.cs +++ b/src/Adapter/PlatformServices.Desktop/Utilities/DesktopDeploymentUtility.cs @@ -35,7 +35,7 @@ public override void AddDeploymentItemsBasedOnMsTestSetting(string testSource, I { if (MSTestSettingsProvider.Settings.DeployTestSourceDependencies) { - EqtTrace.Info("Adding the references and satellite assemblies to the deploymentitems list"); + EqtTrace.Info("Adding the references and satellite assemblies to the deployment items list"); // Get the referenced assemblies. this.ProcessNewStorage(testSource, deploymentItems, warnings); @@ -49,7 +49,7 @@ public override void AddDeploymentItemsBasedOnMsTestSetting(string testSource, I } else { - EqtTrace.Info("Adding the test source directory to the deploymentitems list"); + EqtTrace.Info("Adding the test source directory to the deployment items list"); this.DeploymentItemUtility.AddDeploymentItem(deploymentItems, new DeploymentItem(Path.GetDirectoryName(testSource))); } } @@ -81,7 +81,7 @@ protected void ProcessNewStorage(string testSource, IList deploy this.DeploymentItemUtility.AddDeploymentItem(deploymentItems, new DeploymentItem(testSource, string.Empty, DeploymentItemOriginType.TestStorage)); - // Deploy .config file if exists, only for assemlbies, i.e. DLL and EXE. + // Deploy .config file if exists, only for assemblies, i.e. DLL and EXE. // First check .config, then if not found check for App.Config // and deploy AppConfig to .config. if (this.AssemblyUtility.IsAssemblyExtension(Path.GetExtension(testSource))) @@ -173,7 +173,7 @@ protected IEnumerable GetSatellites(IEnumerable new char[] { Path.DirectorySeparatorChar, Path.AltDirectorySeparatorChar }); Debug.Assert(!string.IsNullOrEmpty(satelliteDir), "DeploymentManager.DoDeployment: got empty satellite dir!"); - Debug.Assert(satelliteDir.Length > itemDir.Length + 1, "DeploymentManager.DoDeployment: wrong satellite dir lenght!"); + Debug.Assert(satelliteDir.Length > itemDir.Length + 1, "DeploymentManager.DoDeployment: wrong satellite dir length!"); string localeDir = satelliteDir.Substring(itemDir.Length + 1); Debug.Assert(!string.IsNullOrEmpty(localeDir), "DeploymentManager.DoDeployment: got empty dir name for satellite dir!"); @@ -210,12 +210,12 @@ protected IEnumerable GetSatellites(IEnumerable } /// - /// Process test storage and add dependant assemblies to dependencyDeploymentItems. + /// Process test storage and add dependent assemblies to dependencyDeploymentItems. /// /// The test source. /// The config file. /// Deployment items. - /// Warnigns. + /// Warnings. private void AddDependencies(string testSource, string configFile, IList deploymentItems, IList warnings) { Debug.Assert(!string.IsNullOrEmpty(testSource), "testSource should not be null or empty."); diff --git a/src/Adapter/PlatformServices.Desktop/Utilities/DesktopReflectionUtility.cs b/src/Adapter/PlatformServices.Desktop/Utilities/DesktopReflectionUtility.cs index 4b3127f89f..5cce3d9a3f 100644 --- a/src/Adapter/PlatformServices.Desktop/Utilities/DesktopReflectionUtility.cs +++ b/src/Adapter/PlatformServices.Desktop/Utilities/DesktopReflectionUtility.cs @@ -125,7 +125,7 @@ internal object[] GetCustomAttributes(MemberInfo memberInfo, Type type, bool inh } else { - // Ideally we should not be reaaching here. We only query for attributes on types/methods currently. + // Ideally we should not be reaching here. We only query for attributes on types/methods currently. // Return the attributes that CustomAttributeData returns in this cases not considering inheritance. var firstLevelAttributes = CustomAttributeData.GetCustomAttributes(memberInfo); diff --git a/src/Adapter/PlatformServices.Interface/ITraceListenerManager.cs b/src/Adapter/PlatformServices.Interface/ITraceListenerManager.cs index 8b3d037adc..5cf874af62 100644 --- a/src/Adapter/PlatformServices.Interface/ITraceListenerManager.cs +++ b/src/Adapter/PlatformServices.Interface/ITraceListenerManager.cs @@ -12,19 +12,19 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.Int public interface ITraceListenerManager { /// - /// Adds the arguement traceListener object to TraceListenerCollection. + /// Adds the argument traceListener object to TraceListenerCollection. /// /// The trace listener instance. void Add(ITraceListener traceListener); /// - /// Removes the arguement traceListener object from TraceListenerCollection. + /// Removes the argument traceListener object from TraceListenerCollection. /// /// The trace listener instance. void Remove(ITraceListener traceListener); /// - /// Disposes the traceListener object passed as arguement. + /// Disposes the traceListener object passed as argument. /// /// The trace listener instance. void Dispose(ITraceListener traceListener); diff --git a/src/Adapter/PlatformServices.NetCore/Services/NetCoreTestContextImplementation.cs b/src/Adapter/PlatformServices.NetCore/Services/NetCoreTestContextImplementation.cs index c7802c2cde..b1f11a2dc9 100644 --- a/src/Adapter/PlatformServices.NetCore/Services/NetCoreTestContextImplementation.cs +++ b/src/Adapter/PlatformServices.NetCore/Services/NetCoreTestContextImplementation.cs @@ -21,7 +21,7 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices /// Internal implementation of TestContext exposed to the user. /// /// - /// The virtual string properties of the TestContext are retreived from the property dictionary + /// The virtual string properties of the TestContext are retrieved from the property dictionary /// like GetProperty<string>("TestName") or GetProperty<string>("FullyQualifiedTestClassName"); /// public class TestContextImplementation : UTF.TestContext, ITestContext diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10DiaSessionOperations.cs b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10DiaSessionOperations.cs index b0b26d8df7..6f11b1e746 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10DiaSessionOperations.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10DiaSessionOperations.cs @@ -111,7 +111,7 @@ private static object SafeInvoke(Func action, string messageFormatOnExcept } catch (Exception) { - // todo : Add EqtTrace + // TODO : Add EqtTrace } return null; diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10FileOperations.cs b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10FileOperations.cs index d72dfee51c..cff6d9620c 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10FileOperations.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10FileOperations.cs @@ -52,7 +52,7 @@ public string GetAssemblyPath(Assembly assembly) public bool DoesFileExist(string assemblyFileName) { // For projectK these assemblies can be created on the fly which means the file might not exist on disk. - // Depend on Assembly Load failures intead of this validation. + // Depend on Assembly Load failures instead of this validation. return true; } diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10TestContextImplementation.cs b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10TestContextImplementation.cs index 7442e7301d..7018ad4550 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10TestContextImplementation.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10TestContextImplementation.cs @@ -21,7 +21,7 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices /// Internal implementation of TestContext exposed to the user. /// /// - /// The virtual string properties of the TestContext are retreived from the property dictionary + /// The virtual string properties of the TestContext are retrieved from the property dictionary /// like GetProperty<string>("TestName") or GetProperty<string>("FullyQualifiedTestClassName"); /// public class TestContextImplementation : UTF.TestContext, ITestContext diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10ThreadOperations.cs b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10ThreadOperations.cs index ad06d1ac51..17fbd32a29 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10ThreadOperations.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.0/Services/ns10ThreadOperations.cs @@ -40,7 +40,7 @@ public bool Execute(Action action, int timeout, CancellationToken cancelToken) } catch (OperationCanceledException) { - // Task execution cancelled. + // Task execution canceled. return false; } } @@ -51,7 +51,7 @@ public bool Execute(Action action, int timeout, CancellationToken cancelToken) /// The action to execute. public void ExecuteWithAbortSafety(Action action) { - // There is no Thread abort scenraios yet in .Net Core. Once we move the core platform service to support Thread abort related API's + // There is no Thread abort scenarios yet in .Net Core. Once we move the core platform service to support Thread abort related API's // then this logic would be similar to the desktop platform service. UWP would then be the only diverging platform service since it does not have Thread APIs exposed. action.Invoke(); } diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.0/ns10RecursiveDirectoryPath.cs b/src/Adapter/PlatformServices.Shared/netstandard1.0/ns10RecursiveDirectoryPath.cs index 1671dcec8f..9064d432c1 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.0/ns10RecursiveDirectoryPath.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.0/ns10RecursiveDirectoryPath.cs @@ -20,7 +20,7 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices /// For each directory we need to have two info 1) path 2) includeSubDirectories /// [Serializable] - [SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1603:DocumentationMustContainValidXml", Justification = "Reviewed. Suppression is OK here.")] + [SuppressMessage("StyleCop.CSharp.DocumentationRules", "SA1603:DocumentationMustContainValidXml", Justification = "Reviewed. Suppression is ok here.")] #pragma warning disable SA1649 // File name must match first type name public class RecursiveDirectoryPath : MarshalByRefObject { diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13MSTestAdapterSettings.cs b/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13MSTestAdapterSettings.cs index a622a4dcb7..00b2fb7945 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13MSTestAdapterSettings.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13MSTestAdapterSettings.cs @@ -209,8 +209,8 @@ internal string ResolveEnvironmentVariableAndReturnFullPathIfExist(string path, try { // Get the full path. - // This will cleanup the path converting any "..\" to the appropariate value - // and convert any alternative directory seperators to "\" + // This will cleanup the path converting any "..\" to the appropriate value + // and convert any alternative directory separators to "\" path = Path.GetFullPath(path); } catch (Exception e) @@ -235,7 +235,7 @@ internal string ResolveEnvironmentVariableAndReturnFullPathIfExist(string path, return path; } - // generate warning that path doesnot exist. + // generate warning that path does not exist. EqtTrace.WarningIf(EqtTrace.IsWarningEnabled, string.Format("The Directory: {0}, does not exist.", path)); } diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TestDeployment.cs b/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TestDeployment.cs index d3dcbb3c3d..6a66d03ddc 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TestDeployment.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TestDeployment.cs @@ -58,8 +58,8 @@ internal TestDeployment(DeploymentItemUtility deploymentItemUtility, DeploymentU /// Gets the current run directories for this session. /// /// - /// This is intialized at the beginning of a run session when Deploy is called. - /// Leaving this as a static varaible since the testContext needs to be filled in with this information. + /// This is initialized at the beginning of a run session when Deploy is called. + /// Leaving this as a static variable since the testContext needs to be filled in with this information. /// internal static TestRunDirectories RunDirectories { @@ -132,7 +132,7 @@ public bool Deploy(IEnumerable tests, IRunContext runContext, IFramewo RunDirectories = this.deploymentUtility.CreateDeploymentDirectories(runContext); // Deployment directories are created but deployment will not happen. - // This is added just to keep consistency with MSTestv1 behaviour. + // This is added just to keep consistency with MSTestv1 behavior. if (!hasDeploymentItems) { return false; @@ -204,7 +204,7 @@ internal static IDictionary GetDeploymentInformation(string sour } /// - /// Reset the static varaible to default values. Used only for testing purposes. + /// Reset the static variable to default values. Used only for testing purposes. /// internal static void Reset() { diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TraceListenerManager.cs b/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TraceListenerManager.cs index f99b00d99d..ab3c25ecb7 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TraceListenerManager.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.3/Services/ns13TraceListenerManager.cs @@ -43,7 +43,7 @@ public TraceListenerManager(TextWriter outputWriter, TextWriter errorWriter) } /// - /// Adds the arguement traceListener object to System.Diagnostics.TraceListenerCollection. + /// Adds the argument traceListener object to System.Diagnostics.TraceListenerCollection. /// /// The trace listener instance. public void Add(ITraceListener traceListener) @@ -54,7 +54,7 @@ public void Add(ITraceListener traceListener) } /// - /// Removes the arguement traceListener object from System.Diagnostics.TraceListenerCollection. + /// Removes the argument traceListener object from System.Diagnostics.TraceListenerCollection. /// /// The trace listener instance. public void Remove(ITraceListener traceListener) diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13DeploymentUtilityBase.cs b/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13DeploymentUtilityBase.cs index 006d44bfe4..19abfde94b 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13DeploymentUtilityBase.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13DeploymentUtilityBase.cs @@ -261,7 +261,7 @@ protected IEnumerable Deploy(IList deploymentItems, stri { EqtTrace.WarningIf( EqtTrace.IsWarningEnabled, - "Conflict during copiyng file: '{0}' and '{1}' are from different origins although they might be the same.", + "Conflict during copying file: '{0}' and '{1}' are from different origins although they might be the same.", fileToDeploy, destToSource[relativeDestination]); } @@ -288,7 +288,7 @@ protected IEnumerable Deploy(IList deploymentItems, stri protected string[] GetFullPathToFilesCorrespondingToDeploymentItem(DeploymentItem deploymentItem, string testSource, string resultsDirectory, IList warnings, out bool isDirectory) { Debug.Assert(deploymentItem != null, "deploymentItem should not be null."); - Debug.Assert(!string.IsNullOrEmpty(testSource), "testsource should not be null or empty."); + Debug.Assert(!string.IsNullOrEmpty(testSource), "testSource should not be null or empty."); try { diff --git a/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13FileUtility.cs b/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13FileUtility.cs index 9a4826293e..ef829322de 100644 --- a/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13FileUtility.cs +++ b/src/Adapter/PlatformServices.Shared/netstandard1.3/Utilities/ns13FileUtility.cs @@ -128,7 +128,7 @@ public virtual string CopyFileOverwrite(string source, string destination, out s /// For given file checks if it is a binary, then finds and deploys PDB for line number info in call stack. /// /// - /// Returns deployed destination pdb file path if everything is Ok, otherwise null. + /// Returns deployed destination pdb file path if everything is ok, otherwise null. /// /// The file we need to find PDBs for (we care only about binaries). /// Destination relative to the root of deployment dir. @@ -184,7 +184,7 @@ public string FindAndDeployPdb(string destinationFile, string relativeDestinatio { EqtTrace.WarningIf( EqtTrace.IsWarningEnabled, - "Conflict during copiyng PDBs for line number info: '{0}' and '{1}' are from different origins although they might be the same.", + "Conflict during copying PDBs for line number info: '{0}' and '{1}' are from different origins although they might be the same.", pdbSource, destToSource[relativePdbDestination]); } diff --git a/src/TestFramework/Extension.Core/NetCoreTestContext.cs b/src/TestFramework/Extension.Core/NetCoreTestContext.cs index ed503ae80d..ebecd7fb76 100644 --- a/src/TestFramework/Extension.Core/NetCoreTestContext.cs +++ b/src/TestFramework/Extension.Core/NetCoreTestContext.cs @@ -22,7 +22,7 @@ public abstract class TestContext #region Test run deployment directories /// - /// Gets or sets the cancellation token source. This token source is cancelled when test timesout. Also when explicitly cancelled the test will be aborted + /// Gets or sets the cancellation token source. This token source is canceled when test times out. Also when explicitly canceled the test will be aborted /// public virtual CancellationTokenSource CancellationTokenSource { get; protected set; } diff --git a/src/TestFramework/Extension.Desktop/ConfigurationSettings/ConfigurationNames.cs b/src/TestFramework/Extension.Desktop/ConfigurationSettings/ConfigurationNames.cs index dc9212f095..048f5a7194 100644 --- a/src/TestFramework/Extension.Desktop/ConfigurationSettings/ConfigurationNames.cs +++ b/src/TestFramework/Extension.Desktop/ConfigurationSettings/ConfigurationNames.cs @@ -34,7 +34,7 @@ internal static class ConfigurationNames internal const string ConnectionStringAttributeName = "connectionString"; /// - /// Attrbiute name for 'DataAccessMethod' + /// Attribute name for 'DataAccessMethod' /// internal const string DataAccessMethodAttributeName = "dataAccessMethod"; diff --git a/src/TestFramework/Extension.Desktop/DesktopTestContext.cs b/src/TestFramework/Extension.Desktop/DesktopTestContext.cs index 8ef0694658..1b95acd060 100644 --- a/src/TestFramework/Extension.Desktop/DesktopTestContext.cs +++ b/src/TestFramework/Extension.Desktop/DesktopTestContext.cs @@ -24,7 +24,7 @@ public abstract class TestContext public abstract IDictionary Properties { get; } /// - /// Gets or sets the cancellation token source. This token source is cancelled when test timesout. Also when explicitly cancelled the test will be aborted + /// Gets or sets the cancellation token source. This token source is canceled when test times out. Also when explicitly canceled the test will be aborted /// public virtual CancellationTokenSource CancellationTokenSource { get; protected set; } diff --git a/src/TestFramework/Extension.Desktop/PrivateObject.cs b/src/TestFramework/Extension.Desktop/PrivateObject.cs index 6220b88e1c..a6c656affc 100644 --- a/src/TestFramework/Extension.Desktop/PrivateObject.cs +++ b/src/TestFramework/Extension.Desktop/PrivateObject.cs @@ -36,7 +36,7 @@ public class PrivateObject /// the already existing object of the private class /// /// object that serves as starting point to reach the private members - /// the derefrencing string using . that points to the object to be retrived as in m_X.m_Y.m_Z + /// the de-referencing string using . that points to the object to be retrieved as in m_X.m_Y.m_Z [SuppressMessage("Microsoft.Naming", "CA1720:IdentifiersShouldNotContainTypeNames", MessageId = "obj", Justification = "We don't know anything about the object other than that it's an object, so 'obj' seems reasonable")] public PrivateObject(object obj, string memberToAccess) { @@ -68,7 +68,7 @@ public PrivateObject(object obj, string memberToAccess) /// /// Name of the assembly /// fully qualified name - /// Argmenets to pass to the constructor + /// Arguments to pass to the constructor public PrivateObject(string assemblyName, string typeName, params object[] args) : this(assemblyName, typeName, null, args) { @@ -81,7 +81,7 @@ public PrivateObject(string assemblyName, string typeName, params object[] args) /// Name of the assembly /// fully qualified name /// An array of objects representing the number, order, and type of the parameters for the constructor to get - /// Argmenets to pass to the constructor + /// Arguments to pass to the constructor public PrivateObject(string assemblyName, string typeName, Type[] parameterTypes, object[] args) : this(Type.GetType(string.Format(CultureInfo.InvariantCulture, "{0}, {1}", typeName, assemblyName), false), parameterTypes, args) { @@ -94,7 +94,7 @@ public PrivateObject(string assemblyName, string typeName, Type[] parameterTypes /// specified type. /// /// type of the object to create - /// Argmenets to pass to the constructor + /// Arguments to pass to the constructor public PrivateObject(Type type, params object[] args) : this(type, null, args) { @@ -107,7 +107,7 @@ public PrivateObject(Type type, params object[] args) /// /// type of the object to create /// An array of objects representing the number, order, and type of the parameters for the constructor to get - /// Argmenets to pass to the constructor + /// Arguments to pass to the constructor public PrivateObject(Type type, Type[] parameterTypes, object[] args) { Helper.CheckParameterNotNull(type, "type", string.Empty); @@ -161,7 +161,7 @@ public PrivateObject(object obj) /// /// object to wrap /// PrivateType object - [SuppressMessage("Microsoft.Naming", "CA1720:IdentifiersShouldNotContainTypeNames", MessageId = "obj", Justification = "We don't know anything about the object other than that it's an an object, so 'obj' seems reasonable")] + [SuppressMessage("Microsoft.Naming", "CA1720:IdentifiersShouldNotContainTypeNames", MessageId = "obj", Justification = "We don't know anything about the object other than that it's an object, so 'obj' seems reasonable")] public PrivateObject(object obj, PrivateType type) { Helper.CheckParameterNotNull(type, "type", string.Empty); @@ -382,7 +382,7 @@ public object Invoke(string name, BindingFlags bindingFlags, Type[] parameterTyp // Fix up the parameter types MethodInfo member = this.originalType.GetMethod(name, bindingFlags, null, parameterTypes, null); - // If the method was not found and type arguments were provided for generic paramaters, + // If the method was not found and type arguments were provided for generic parameters, // attempt to look up a generic method. if ((member == null) && (typeArguments != null)) { @@ -430,11 +430,11 @@ public object Invoke(string name, BindingFlags bindingFlags, Type[] parameterTyp } /// - /// Gets the array element using array of subsrcipts for each dimension + /// Gets the array element using array of subscripts for each dimension /// /// Name of the member /// the indices of array - /// An arrya of elements. + /// An array of elements. public object GetArrayElement(string name, params int[] indices) { Helper.CheckParameterNotNull(name, "name", string.Empty); @@ -442,7 +442,7 @@ public object GetArrayElement(string name, params int[] indices) } /// - /// Sets the array element using array of subsrcipts for each dimension + /// Sets the array element using array of subscripts for each dimension /// /// Name of the member /// Value to set @@ -454,12 +454,12 @@ public void SetArrayElement(string name, object value, params int[] indices) } /// - /// Gets the array element using array of subsrcipts for each dimension + /// Gets the array element using array of subscripts for each dimension /// /// Name of the member /// A bitmask comprised of one or more that specify how the search is conducted. /// the indices of array - /// An arrya of elements. + /// An array of elements. public object GetArrayElement(string name, BindingFlags bindingFlags, params int[] indices) { Helper.CheckParameterNotNull(name, "name", string.Empty); @@ -468,7 +468,7 @@ public object GetArrayElement(string name, BindingFlags bindingFlags, params int } /// - /// Sets the array element using array of subsrcipts for each dimension + /// Sets the array element using array of subscripts for each dimension /// /// Name of the member /// A bitmask comprised of one or more that specify how the search is conducted. @@ -728,7 +728,7 @@ private static void ValidateAccessString(string access) } /// - /// Invokes the memeber + /// Invokes the member /// /// Name of the member /// Additional attributes @@ -796,7 +796,7 @@ private void BuildGenericMethodCacheForType(Type t) /// An array of types corresponding to the types of the generic arguments. /// to further filter the method signatures. /// Modifiers for parameters. - /// A methodinfo instance. + /// A method info instance. private MethodInfo GetGenericMethodFromCache(string methodName, Type[] parameterTypes, Type[] typeArguments, BindingFlags bindingFlags, ParameterModifier[] modifiers) { Debug.Assert(!string.IsNullOrEmpty(methodName), "Invalid method name."); diff --git a/src/TestFramework/Extension.Desktop/PrivateType.cs b/src/TestFramework/Extension.Desktop/PrivateType.cs index 65133bdbc8..413ff1cba3 100644 --- a/src/TestFramework/Extension.Desktop/PrivateType.cs +++ b/src/TestFramework/Extension.Desktop/PrivateType.cs @@ -9,7 +9,7 @@ namespace Microsoft.VisualStudio.TestTools.UnitTesting using System.Reflection; /// - /// This class represents a private class for the Private Accessor functionality. + /// This class represents a private class for the Private Accessors functionality. /// public class PrivateType { @@ -63,7 +63,7 @@ public PrivateType(Type type) /// Invokes static member /// /// Name of the member to InvokeHelper - /// Arguements to the invoction + /// Arguments to the invocation /// Result of invocation public object InvokeStatic(string name, params object[] args) { @@ -75,7 +75,7 @@ public object InvokeStatic(string name, params object[] args) /// /// Name of the member to InvokeHelper /// An array of objects representing the number, order, and type of the parameters for the method to invoke - /// Arguements to the invoction + /// Arguments to the invocation /// Result of invocation public object InvokeStatic(string name, Type[] parameterTypes, object[] args) { @@ -87,7 +87,7 @@ public object InvokeStatic(string name, Type[] parameterTypes, object[] args) /// /// Name of the member to InvokeHelper /// An array of objects representing the number, order, and type of the parameters for the method to invoke - /// Arguements to the invoction + /// Arguments to the invocation /// An array of types corresponding to the types of the generic arguments. /// Result of invocation public object InvokeStatic(string name, Type[] parameterTypes, object[] args, Type[] typeArguments) @@ -99,7 +99,7 @@ public object InvokeStatic(string name, Type[] parameterTypes, object[] args, Ty /// Invokes the static method /// /// Name of the member - /// Arguements to the invocation + /// Arguments to the invocation /// Culture /// Result of invocation public object InvokeStatic(string name, object[] args, CultureInfo culture) @@ -112,7 +112,7 @@ public object InvokeStatic(string name, object[] args, CultureInfo culture) /// /// Name of the member /// An array of objects representing the number, order, and type of the parameters for the method to invoke - /// Arguements to the invocation + /// Arguments to the invocation /// Culture info /// Result of invocation public object InvokeStatic(string name, Type[] parameterTypes, object[] args, CultureInfo culture) @@ -125,7 +125,7 @@ public object InvokeStatic(string name, Type[] parameterTypes, object[] args, Cu /// /// Name of the member /// Additional invocation attributes - /// Arguements to the invocation + /// Arguments to the invocation /// Result of invocation public object InvokeStatic(string name, BindingFlags bindingFlags, params object[] args) { @@ -138,7 +138,7 @@ public object InvokeStatic(string name, BindingFlags bindingFlags, params object /// Name of the member /// Additional invocation attributes /// An array of objects representing the number, order, and type of the parameters for the method to invoke - /// Arguements to the invocation + /// Arguments to the invocation /// Result of invocation public object InvokeStatic(string name, BindingFlags bindingFlags, Type[] parameterTypes, object[] args) { @@ -150,7 +150,7 @@ public object InvokeStatic(string name, BindingFlags bindingFlags, Type[] parame /// /// Name of the member /// Additional invocation attributes - /// Arguements to the invocation + /// Arguments to the invocation /// Culture /// Result of invocation public object InvokeStatic(string name, BindingFlags bindingFlags, object[] args, CultureInfo culture) @@ -164,7 +164,7 @@ public object InvokeStatic(string name, BindingFlags bindingFlags, object[] args /// Name of the member /// Additional invocation attributes /// /// An array of objects representing the number, order, and type of the parameters for the method to invoke - /// Arguements to the invocation + /// Arguments to the invocation /// Culture /// Result of invocation public object InvokeStatic(string name, BindingFlags bindingFlags, Type[] parameterTypes, object[] args, CultureInfo culture) @@ -178,7 +178,7 @@ public object InvokeStatic(string name, BindingFlags bindingFlags, Type[] parame /// Name of the member /// Additional invocation attributes /// /// An array of objects representing the number, order, and type of the parameters for the method to invoke - /// Arguements to the invocation + /// Arguments to the invocation /// Culture /// An array of types corresponding to the types of the generic arguments. /// Result of invocation @@ -238,7 +238,7 @@ public object GetStaticArrayElement(string name, params int[] indices) } /// - /// Sets the memeber of the static array + /// Sets the member of the static array /// /// Name of the array /// value to set @@ -253,7 +253,7 @@ public void SetStaticArrayElement(string name, object value, params int[] indice } /// - /// Gets the element in satatic array + /// Gets the element in static array /// /// Name of the array /// Additional InvokeHelper attributes @@ -261,7 +261,7 @@ public void SetStaticArrayElement(string name, object value, params int[] indice /// A one-dimensional array of 32-bit integers that represent the indexes specifying /// the position of the element to get. For instance, to access a[10][11] the array would be {10,11} /// - /// element at the spcified location + /// element at the specified location public object GetStaticArrayElement(string name, BindingFlags bindingFlags, params int[] indices) { Helper.CheckParameterNotNull(name, "name", string.Empty); @@ -270,7 +270,7 @@ public object GetStaticArrayElement(string name, BindingFlags bindingFlags, para } /// - /// Sets the memeber of the static array + /// Sets the member of the static array /// /// Name of the array /// Additional InvokeHelper attributes @@ -301,7 +301,7 @@ public object GetStaticField(string name) /// Sets the static field /// /// Name of the field - /// Arguement to the invocation + /// Argument to the invocation public void SetStaticField(string name, object value) { Helper.CheckParameterNotNull(name, "name", string.Empty); @@ -325,7 +325,7 @@ public object GetStaticField(string name, BindingFlags bindingFlags) /// /// Name of the field /// Additional InvokeHelper attributes - /// Arguement to the invocation + /// Argument to the invocation public void SetStaticField(string name, BindingFlags bindingFlags, object value) { Helper.CheckParameterNotNull(name, "name", string.Empty); @@ -382,7 +382,7 @@ public void SetStaticFieldOrProperty(string name, BindingFlags bindingFlags, obj /// Gets the static property /// /// Name of the field or property - /// Arguements to the invocation + /// Arguments to the invocation /// The static property. public object GetStaticProperty(string name, params object[] args) { @@ -500,7 +500,7 @@ public void SetStaticProperty(string name, BindingFlags bindingFlags, object val /// /// Name of the member /// Additional invocation attributes - /// Arguements to the invocation + /// Arguments to the invocation /// Culture /// Result of invocation private object InvokeHelperStatic(string name, BindingFlags bindingFlags, object[] args, CultureInfo culture) diff --git a/src/TestFramework/Extension.Desktop/RuntimeTypeHelper.cs b/src/TestFramework/Extension.Desktop/RuntimeTypeHelper.cs index e087b76c5c..987a27cf94 100644 --- a/src/TestFramework/Extension.Desktop/RuntimeTypeHelper.cs +++ b/src/TestFramework/Extension.Desktop/RuntimeTypeHelper.cs @@ -18,7 +18,7 @@ internal class RuntimeTypeHelper /// /// Method1 /// Method2 - /// True if they are similiar. + /// True if they are similar. internal static bool CompareMethodSigAndName(MethodBase m1, MethodBase m2) { ParameterInfo[] params1 = m1.GetParameters(); @@ -62,7 +62,7 @@ internal static int GetHierarchyDepth(Type t) } /// - /// Finds most dervied type with the provided information. + /// Finds most derived type with the provided information. /// /// Candidate matches. /// Number of matches. @@ -240,10 +240,10 @@ internal static MethodBase SelectMethod(BindingFlags bindingAttr, MethodBase[] m /// /// Method 1 /// Parameter order for Method 1 - /// Paramter array type. + /// Parameter array type. /// Method 2 /// Parameter order for Method 2 - /// >Paramter array type. + /// >Parameter array type. /// Types to search in. /// Args. /// An int representing the match. @@ -305,10 +305,10 @@ internal static int FindMostSpecificMethod( /// /// Method 1 /// Parameter order for Method 1 - /// Paramter array type. + /// Parameter array type. /// Method 2 /// Parameter order for Method 2 - /// >Paramter array type. + /// >Parameter array type. /// Types to search in. /// Args. /// An int representing the match. diff --git a/src/TestFramework/Extension.UWP/DeploymentItemAttribute.cs b/src/TestFramework/Extension.UWP/DeploymentItemAttribute.cs index 82b5f07962..13dbd70a9d 100644 --- a/src/TestFramework/Extension.UWP/DeploymentItemAttribute.cs +++ b/src/TestFramework/Extension.UWP/DeploymentItemAttribute.cs @@ -18,7 +18,7 @@ namespace Microsoft.VisualStudio.TestTools.UnitTesting /// /// /// Putting this in here so that UWP discovery works. We still do not want users to be using DeploymentItem in the UWP world - Hence making it internal. - /// We should separate out DeploymentItem logic in the adapter via a Framework extensiblity point. + /// We should separate out DeploymentItem logic in the adapter via a Framework extensibility point. /// Filed https://github.com/Microsoft/testfx/issues/100 to track this. /// [AttributeUsage(AttributeTargets.Method | AttributeTargets.Class, AllowMultiple = true)] diff --git a/src/TestFramework/MSTest.Core/Assertions/StringAssert.cs b/src/TestFramework/MSTest.Core/Assertions/StringAssert.cs index 514429e431..004e9bd585 100644 --- a/src/TestFramework/MSTest.Core/Assertions/StringAssert.cs +++ b/src/TestFramework/MSTest.Core/Assertions/StringAssert.cs @@ -285,7 +285,7 @@ public static void EndsWith(string value, string substring, string message, para #endregion Substrings - #region Regular Expresssions + #region Regular Expressions /// /// Tests whether the specified string matches a regular expression and @@ -444,6 +444,6 @@ public static void DoesNotMatch(string value, Regex pattern, string message, par } } - #endregion Regular Expresssions + #endregion Regular Expressions } } diff --git a/src/TestFramework/MSTest.Core/Attributes/DataRowAttribute.cs b/src/TestFramework/MSTest.Core/Attributes/DataRowAttribute.cs index 05b3e57e1e..c8c641e461 100644 --- a/src/TestFramework/MSTest.Core/Attributes/DataRowAttribute.cs +++ b/src/TestFramework/MSTest.Core/Attributes/DataRowAttribute.cs @@ -10,7 +10,7 @@ namespace Microsoft.VisualStudio.TestTools.UnitTesting using System.Reflection; /// - /// Attribute to define inline data for a test method. + /// Attribute to define in-line data for a test method. /// [AttributeUsage(AttributeTargets.Method, AllowMultiple = true)] public class DataRowAttribute : Attribute, ITestDataSource diff --git a/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionAttribute.cs b/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionAttribute.cs index a8a20a27d0..274d322c39 100644 --- a/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionAttribute.cs +++ b/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionAttribute.cs @@ -106,7 +106,7 @@ protected internal override void Verify(Exception exception) { if (!this.ExceptionType.GetTypeInfo().IsAssignableFrom(thrownExceptionType.GetTypeInfo())) { - // If the exception is an AssertFailedException or an AssertInconclusiveException, then rethrow it to + // If the exception is an AssertFailedException or an AssertInconclusiveException, then re-throw it to // preserve the test outcome and error message this.RethrowIfAssertException(exception); @@ -123,7 +123,7 @@ protected internal override void Verify(Exception exception) { if (thrownExceptionType != this.ExceptionType) { - // If the exception is an AssertFailedException or an AssertInconclusiveException, then rethrow it to + // If the exception is an AssertFailedException or an AssertInconclusiveException, then re-throw it to // preserve the test outcome and error message this.RethrowIfAssertException(exception); diff --git a/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionBaseAttribute.cs b/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionBaseAttribute.cs index 1c999575d4..c9ef6d6358 100644 --- a/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionBaseAttribute.cs +++ b/src/TestFramework/MSTest.Core/Attributes/ExpectedExceptionBaseAttribute.cs @@ -43,7 +43,7 @@ protected ExpectedExceptionBaseAttribute(string noExceptionMessage) #region Properties - // Todo: Test Context needs to be put in here for source compat. + // TODO: Test Context needs to be put in here for source compat. /// /// Gets the message to include in the test result if the test fails due to not throwing an exception diff --git a/src/TestFramework/MSTest.Core/Attributes/TestCategoryAttribute.cs b/src/TestFramework/MSTest.Core/Attributes/TestCategoryAttribute.cs index 9c5b75f237..6a801f7f1b 100644 --- a/src/TestFramework/MSTest.Core/Attributes/TestCategoryAttribute.cs +++ b/src/TestFramework/MSTest.Core/Attributes/TestCategoryAttribute.cs @@ -11,7 +11,7 @@ namespace Microsoft.VisualStudio.TestTools.UnitTesting /// TestCategory attribute; used to specify the category of a unit test. /// [AttributeUsage(AttributeTargets.Method | AttributeTargets.Class | AttributeTargets.Assembly, AllowMultiple = true)] - [SuppressMessage("Microsoft.Design", "CA1019:DefineAccessorsForAttributeArguments", Justification = "The TestCategories accessor propety exposes the testCategory argument.")] + [SuppressMessage("Microsoft.Design", "CA1019:DefineAccessorsForAttributeArguments", Justification = "The TestCategories accessor property exposes the testCategory argument.")] public sealed class TestCategoryAttribute : TestCategoryBaseAttribute { private IList testCategories; diff --git a/src/TestFramework/MSTest.Core/Attributes/VSTestAttributes.cs b/src/TestFramework/MSTest.Core/Attributes/VSTestAttributes.cs index fb2b9629fd..a87c0bd61a 100644 --- a/src/TestFramework/MSTest.Core/Attributes/VSTestAttributes.cs +++ b/src/TestFramework/MSTest.Core/Attributes/VSTestAttributes.cs @@ -104,7 +104,7 @@ public virtual TestResult[] Execute(ITestMethod testMethod) } /// - /// Attribute for data driven test where data can be specified inline. + /// Attribute for data driven test where data can be specified in-line. /// [AttributeUsage(AttributeTargets.Method, AllowMultiple = false)] public class DataTestMethodAttribute : TestMethodAttribute @@ -398,7 +398,7 @@ public WorkItemAttribute(int id) } /// - /// Gets the Id to a workitem associated. + /// Gets the Id to a work item associated. /// public int Id { get; private set; } } @@ -601,7 +601,7 @@ public DataSourceAttribute(string dataSourceSettingName) this.DataSourceSettingName = dataSourceSettingName; } - // Different providers use dfferent connection strings and provider itself is a part of connection string. + // Different providers use different connection strings and provider itself is a part of connection string. /// /// Gets a value representing the data provider of the data source. diff --git a/src/TestFramework/MSTest.Core/DataAccessMethod.cs b/src/TestFramework/MSTest.Core/DataAccessMethod.cs index 8b516a0786..5039ee634f 100644 --- a/src/TestFramework/MSTest.Core/DataAccessMethod.cs +++ b/src/TestFramework/MSTest.Core/DataAccessMethod.cs @@ -4,7 +4,7 @@ namespace Microsoft.VisualStudio.TestTools.UnitTesting { /// - /// Enumeration for how how we access data rows in data driven testing. + /// Enumeration for how we access data rows in data driven testing. /// public enum DataAccessMethod { diff --git a/src/TestFramework/MSTest.Core/GenericParameterHelper.cs b/src/TestFramework/MSTest.Core/GenericParameterHelper.cs index 096c574c3e..1d4c40459b 100644 --- a/src/TestFramework/MSTest.Core/GenericParameterHelper.cs +++ b/src/TestFramework/MSTest.Core/GenericParameterHelper.cs @@ -20,7 +20,7 @@ namespace Microsoft.VisualStudio.TestTools.UnitTesting // This next suppression could mask a problem, since Equals and CompareTo may not agree! [SuppressMessage("Microsoft.Design", "CA1036:OverrideMethodsOnComparableTypes", Justification = "Compat reasons.")] - // GenericParameterHelper in full CLR version also implements ICloneable, but we dont have ICloneable in core CLR + // GenericParameterHelper in full CLR version also implements ICloneable, but we don't have ICloneable in core CLR public class GenericParameterHelper : IComparable, IEnumerable { #region Private Fields diff --git a/src/TestFramework/MSTest.Core/Logger.cs b/src/TestFramework/MSTest.Core/Logger.cs index 47834d4291..c0c3739931 100644 --- a/src/TestFramework/MSTest.Core/Logger.cs +++ b/src/TestFramework/MSTest.Core/Logger.cs @@ -42,7 +42,7 @@ public static void LogMessage(string format, params object[] args) string message = string.Format(CultureInfo.InvariantCulture, format, args); - // Making sure all event handlers are called in sunc on same thread. + // Making sure all event handlers are called in sync on same thread. foreach (var invoker in OnLogMessage.GetInvocationList()) { try diff --git a/test/ComponentTests/PlatformServices.Desktop.Component.Tests/DesktopTestSourceHostTests.cs b/test/ComponentTests/PlatformServices.Desktop.Component.Tests/DesktopTestSourceHostTests.cs index 4ce931e013..74291efdf1 100644 --- a/test/ComponentTests/PlatformServices.Desktop.Component.Tests/DesktopTestSourceHostTests.cs +++ b/test/ComponentTests/PlatformServices.Desktop.Component.Tests/DesktopTestSourceHostTests.cs @@ -94,7 +94,7 @@ public void DisposeShouldUnloadChildAppDomain() this.testSourceHost = new TestSourceHost(this.testSource, null, null); this.testSourceHost.SetupHost(); - // Check that child appdmon was indeed created + // Check that child appdomain was indeed created Assert.IsNotNull(this.testSourceHost.AppDomain); this.testSourceHost.Dispose(); diff --git a/test/ComponentTests/PlatformServices.Desktop.Component.Tests/ReflectionUtilityTests.cs b/test/ComponentTests/PlatformServices.Desktop.Component.Tests/ReflectionUtilityTests.cs index c751d1569e..17fc6813be 100644 --- a/test/ComponentTests/PlatformServices.Desktop.Component.Tests/ReflectionUtilityTests.cs +++ b/test/ComponentTests/PlatformServices.Desktop.Component.Tests/ReflectionUtilityTests.cs @@ -46,7 +46,7 @@ public ReflectionUtilityTests() "TestAssets"); this.testAsset = Assembly.ReflectionOnlyLoadFrom(Path.Combine(testAssetPath, "TestProjectForDiscovery.dll")); - // Also preload the Test Framework assembly. + // Also pre-load the Test Framework assembly. Assembly.ReflectionOnlyLoadFrom( Path.Combine(testAssetPath, "Microsoft.VisualStudio.TestPlatform.TestFramework.dll")); diff --git a/test/E2ETests/Automation.CLI/RunConfiguration.cs b/test/E2ETests/Automation.CLI/RunConfiguration.cs index 6964e404a1..828d30eaf9 100644 --- a/test/E2ETests/Automation.CLI/RunConfiguration.cs +++ b/test/E2ETests/Automation.CLI/RunConfiguration.cs @@ -16,7 +16,7 @@ public class RunConfiguration public string SettingsName { get; set; } /// - /// Gets or sets paths at which enigine should look for test adapters + /// Gets or sets paths at which engine should look for test adapters /// public string TestAdaptersPaths { get; set; } diff --git a/test/E2ETests/Automation.CLI/XmlRunSettingsUtilities.cs b/test/E2ETests/Automation.CLI/XmlRunSettingsUtilities.cs index b86b25bbd5..537968091d 100644 --- a/test/E2ETests/Automation.CLI/XmlRunSettingsUtilities.cs +++ b/test/E2ETests/Automation.CLI/XmlRunSettingsUtilities.cs @@ -43,7 +43,7 @@ public static string CreateDefaultRunSettings() public static class Constants { /// - /// Name of data collection settigns node in RunSettings. + /// Name of data collection settings node in RunSettings. /// public const string DataCollectionRunSettingsName = "DataCollectionRunSettings"; diff --git a/test/E2ETests/Smoke.E2E.Tests/DataRowTests.cs b/test/E2ETests/Smoke.E2E.Tests/DataRowTests.cs index a1658cca3e..280c4aeecd 100644 --- a/test/E2ETests/Smoke.E2E.Tests/DataRowTests.cs +++ b/test/E2ETests/Smoke.E2E.Tests/DataRowTests.cs @@ -23,8 +23,8 @@ public void ExecuteOnlyDerivedClassDataRowsWhenBothBaseAndDerviedClassHasDataRow "DataRowTestMethod (DerivedString1)", "DataRowTestMethod (DerivedString2)"); - // 4 tests of BaseClass.DataRowTestMethod - 3 datarow result and 1 parent result - // 3 tests of DerivedClass.DataRowTestMethod - 2 datarow result and 1 parent result + // 4 tests of BaseClass.DataRowTestMethod - 3 data row results and 1 parent result + // 3 tests of DerivedClass.DataRowTestMethod - 2 data row results and 1 parent result // Total 7 tests - Making sure that DerivedClass doesn't run BaseClass tests this.ValidatePassedTestsCount(7); } diff --git a/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionAttributeTests.cs b/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionAttributeTests.cs index 21377c609c..d332691f6a 100644 --- a/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionAttributeTests.cs +++ b/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionAttributeTests.cs @@ -19,7 +19,7 @@ namespace UnitTestFramework.Tests public class ExpectedExceptionAttributeTests { /// - /// ExpectedExceptionAttribute constructer should throw ArgumentNullException when parameter exceptionType = null + /// ExpectedExceptionAttribute constructor should throw ArgumentNullException when parameter exceptionType = null /// [TestFrameworkV1.TestMethod] public void ExpectedExceptionAttributeConstructerShouldThrowArgumentNullExceptionWhenExceptionTypeIsNull() @@ -30,7 +30,7 @@ public void ExpectedExceptionAttributeConstructerShouldThrowArgumentNullExceptio } /// - /// ExpectedExceptionAttribute constructer should throw ArgumentNullException when parameter exceptionType = typeof(AnyClassNotDerivedFromExceptionClass) + /// ExpectedExceptionAttribute constructor should throw ArgumentNullException when parameter exceptionType = typeof(AnyClassNotDerivedFromExceptionClass) /// [TestFrameworkV1.TestMethod] public void ExpectedExceptionAttributeConstructerShouldThrowArgumentException() @@ -41,7 +41,7 @@ public void ExpectedExceptionAttributeConstructerShouldThrowArgumentException() } /// - /// ExpectedExceptionAttribute constructer should not throw exception when parameter exceptionType = typeof(AnyClassDerivedFromExceptionClass) + /// ExpectedExceptionAttribute constructor should not throw exception when parameter exceptionType = typeof(AnyClassDerivedFromExceptionClass) /// [TestFrameworkV1.TestMethod] public void ExpectedExceptionAttributeConstructerShouldNotThrowAnyException() diff --git a/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionBaseAttributeTests.cs b/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionBaseAttributeTests.cs index fffcf9af51..e74692a706 100644 --- a/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionBaseAttributeTests.cs +++ b/test/UnitTests/MSTest.Core.Unit.Tests/Attributes/ExpectedExceptionBaseAttributeTests.cs @@ -94,9 +94,9 @@ public string GetNoExceptionMessage() } /// - /// Rethrow the exception if it is an AssertFailedException or an AssertInconclusiveException + /// Re-throw the exception if it is an AssertFailedException or an AssertInconclusiveException /// - /// The exception to rethrow if it is an assertion exception + /// The exception to re-throw if it is an assertion exception public new void RethrowIfAssertException(Exception exception) { base.RethrowIfAssertException(exception); diff --git a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestExecutionManagerTests.cs b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestExecutionManagerTests.cs index 4ec01b1046..9442a30d06 100644 --- a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestExecutionManagerTests.cs +++ b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestExecutionManagerTests.cs @@ -177,7 +177,7 @@ public void RunTestsForMultipleTestShouldSendMultipleResults() } [TestMethodV1] - public void RunTestsForCancellationTokenCancelledSetToTrueShouldSendZeroResults() + public void RunTestsForCancellationTokencanceledSetToTrueShouldSendZeroResults() { var testCase = this.GetTestCase(typeof(DummyTestClass), "PassingTest"); @@ -364,7 +364,7 @@ public void RunTestsShouldClearSessionParametersAcrossRuns() #region Run Tests on Sources - // Todo: This tests needs to be mocked. + // TODO: This tests needs to be mocked. [Ignore] [TestMethodV1] public void RunTestsForSourceShouldRunTestsInASource() @@ -378,7 +378,7 @@ public void RunTestsForSourceShouldRunTestsInASource() CollectionAssert.Contains(this.frameworkHandle.ResultsList, "PassingTest Passed"); } - // Todo: This tests needs to be mocked. + // TODO: This tests needs to be mocked. [Ignore] [TestMethodV1] public void RunTestsForSourceShouldPassInTestRunParametersInformationAsPropertiesToTheTest() @@ -937,7 +937,7 @@ public static void Cleanup() [UTF.TestMethod] public void TestMethod1() { - // Ensures stability.. for the thread to be not used for another testmethod + // Ensures stability.. for the thread to be not used for another test method System.Threading.Thread.Sleep(2000); threadIds.Add(Thread.CurrentThread.ManagedThreadId); } @@ -945,7 +945,7 @@ public void TestMethod1() [UTF.TestMethod] public void TestMethod2() { - // Ensures stability.. for the thread to be not used for another testmethod + // Ensures stability.. for the thread to be not used for another test method System.Threading.Thread.Sleep(2000); threadIds.Add(Thread.CurrentThread.ManagedThreadId); } @@ -972,7 +972,7 @@ public static void Cleanup() [UTF.TestMethod] public void TestMethod1() { - // Ensures stability.. for the thread to be not used for another testmethod + // Ensures stability.. for the thread to be not used for another test method System.Threading.Thread.Sleep(2000); threadIds.Add(Thread.CurrentThread.ManagedThreadId); } @@ -980,7 +980,7 @@ public void TestMethod1() [UTF.TestMethod] public void TestMethod2() { - // Ensures stability.. for the thread to be not used for another testmethod + // Ensures stability.. for the thread to be not used for another test method System.Threading.Thread.Sleep(2000); threadIds.Add(Thread.CurrentThread.ManagedThreadId); } @@ -1007,7 +1007,7 @@ public static void Cleanup() [UTF.TestMethod] public void TestMethod1() { - // Ensures stability.. for the thread to be not used for another testmethod + // Ensures stability.. for the thread to be not used for another test method Thread.Sleep(2000); threadIds.Add(Thread.CurrentThread.ManagedThreadId); } @@ -1060,7 +1060,7 @@ public static void Cleanup() [UTF.TestMethod] public void TestMethod1() { - // Ensures stability.. for the thread to be not used for another testmethod + // Ensures stability.. for the thread to be not used for another test method Thread.Sleep(2000); parallelizableTestsThreadIds.Add(Thread.CurrentThread.ManagedThreadId); threadApartmentStates.Add(Thread.CurrentThread.GetApartmentState()); @@ -1071,7 +1071,7 @@ public void TestMethod1() [UTF.TestMethod] public void TestMethod2() { - // Ensures stability.. for the thread to be not used for another testmethod + // Ensures stability.. for the thread to be not used for another test method Thread.Sleep(2000); parallelizableTestsThreadIds.Add(Thread.CurrentThread.ManagedThreadId); threadApartmentStates.Add(Thread.CurrentThread.GetApartmentState()); diff --git a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodFilterTests.cs b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodFilterTests.cs index 31dac97d28..d097e8e064 100644 --- a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodFilterTests.cs +++ b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodFilterTests.cs @@ -149,7 +149,7 @@ public void GetFilterExpressionForDiscoveryContextWithGetTestCaseFilterReturnsVa } /// - /// GetFilterExpression shoould return null test case filter expression in case DiscoveryContext doesnt have GetTestCaseFilter. + /// GetFilterExpression should return null test case filter expression in case DiscoveryContext doesn't have GetTestCaseFilter. /// [TestMethod] public void GetFilterExpressionForDiscoveryContextWithoutGetTestCaseFilterReturnsNullTestCaseFilterExpression() diff --git a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodInfoTests.cs b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodInfoTests.cs index 6ef9d380cf..032dd1965d 100644 --- a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodInfoTests.cs +++ b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/TestMethodInfoTests.cs @@ -1263,7 +1263,7 @@ public void TestMethodInfoInvokeShouldCancelTokenSourceOnTimeout() Assert.AreEqual(UTF.UnitTestOutcome.Timeout, result.Outcome); StringAssert.Contains(result.TestFailureException.Message, "exceeded execution timeout period"); - Assert.IsTrue(this.testContextImplementation.CancellationTokenSource.IsCancellationRequested, "Not cancelled.."); + Assert.IsTrue(this.testContextImplementation.CancellationTokenSource.IsCancellationRequested, "Not canceled.."); }); } @@ -1295,7 +1295,7 @@ public void TestMethodInfoInvokeShouldFailOnTokenSourceCancellation() Assert.AreEqual(UTF.UnitTestOutcome.Timeout, result.Outcome); StringAssert.Contains(result.TestFailureException.Message, "execution has been aborted"); - Assert.IsTrue(this.testContextImplementation.CancellationTokenSource.IsCancellationRequested, "Not cancelled.."); + Assert.IsTrue(this.testContextImplementation.CancellationTokenSource.IsCancellationRequested, "Not canceled.."); }); } @@ -1564,7 +1564,7 @@ public void DummyTestCleanupMethod() #region Dummy implementation /// - /// Custom Expected exception attribute which doverrides the Verify method. + /// Custom Expected exception attribute which overrides the Verify method. /// public class CustomExpectedExceptionAttribute : UTF.ExpectedExceptionBaseAttribute { @@ -1593,7 +1593,7 @@ protected override void Verify(Exception exception) } /// - /// Custom Expected exception attribute which doverrides the Verify method. + /// Custom Expected exception attribute which overrides the Verify method. /// public class DerivedCustomExpectedExceptionAttribute : CustomExpectedExceptionAttribute { diff --git a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/UnitTestRunnerTests.cs b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/UnitTestRunnerTests.cs index 1a4b430b08..e7795818a5 100644 --- a/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/UnitTestRunnerTests.cs +++ b/test/UnitTests/MSTest.CoreAdapter.Unit.Tests/Execution/UnitTestRunnerTests.cs @@ -344,7 +344,7 @@ private MSTestSettings GetSettingsWithDebugTrace(bool captureDebugTraceValue) #endregion - #region Dummmy implementations + #region Dummy implementations [DummyTestClass] private class DummyTestClass diff --git a/test/UnitTests/PlatformServices.Desktop.Unit.Tests/Services/DesktopThreadOperationsTests.cs b/test/UnitTests/PlatformServices.Desktop.Unit.Tests/Services/DesktopThreadOperationsTests.cs index a65702abc2..d7d690f1b6 100644 --- a/test/UnitTests/PlatformServices.Desktop.Unit.Tests/Services/DesktopThreadOperationsTests.cs +++ b/test/UnitTests/PlatformServices.Desktop.Unit.Tests/Services/DesktopThreadOperationsTests.cs @@ -127,7 +127,7 @@ public void TokenCancelShouldAbortExecutingAction() } [TestMethod] - public void TokenCancelShouldAbortIfAlreadyCancelled() + public void TokenCancelShouldAbortIfAlreadycanceled() { // setup var cancellationTokenSource = new CancellationTokenSource(); diff --git a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13MSTestSettingsProviderTests.cs b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13MSTestSettingsProviderTests.cs index bf7401202f..b13bfd4056 100644 --- a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13MSTestSettingsProviderTests.cs +++ b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13MSTestSettingsProviderTests.cs @@ -41,7 +41,7 @@ public void TestInit() [TestMethod] public void GetPropertiesShouldReturnDeploymentInformation() { - // this is a base case and we just validating that properties does not remain uninitialised, + // this is a base case and we just validating that properties does not remain un-initialized, // so passing 'null' source will also suffice. var properties = this.settingsProvider.GetProperties(null); diff --git a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13TestDeploymentTests.cs b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13TestDeploymentTests.cs index 092bebcfd8..1e589d701b 100644 --- a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13TestDeploymentTests.cs +++ b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/Services/ns13TestDeploymentTests.cs @@ -228,7 +228,7 @@ public void DeployShouldReturnFalseWhenDeploymentEnabledSetToFalseButHasDeployme // Deployment should not happen Assert.IsFalse(testDeployment.Deploy(new List { testCase }, null, null)); - // Deplyment directories should not be created + // Deployment directories should not be created Assert.IsNull(testDeployment.GetDeploymentDirectory()); } @@ -280,7 +280,7 @@ public void DeployShouldReturnFalseWhenDeploymentEnabledSetToTrueButHasNoDeploym Assert.IsNotNull(testDeployment.GetDeploymentDirectory()); } - // [Todo] This test has to have mocks. It actually deploys stuff and we cannot assume that all the dependencies get copied over to bin\debug. + // TODO: This test has to have mocks. It actually deploys stuff and we cannot assume that all the dependencies get copied over to bin\debug. [TestMethod] [Ignore] public void DeployShouldReturnTrueWhenDeploymentEnabledSetToTrueAndHasDeploymentItems() diff --git a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerManagerTests.cs b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerManagerTests.cs index 323498ea47..3a69ab0e52 100644 --- a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerManagerTests.cs +++ b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerManagerTests.cs @@ -70,7 +70,7 @@ public void DisposeShouldCallDisposeOnCorrespondingTraceListener() traceListenerManager.Add(traceListener); traceListenerManager.Dispose(traceListener); - // Tring to write after closing textWriter should throw exception + // Trying to write after closing textWriter should throw exception Action shouldThrowException = () => writer.WriteLine("Try to write something"); ActionUtility.ActionShouldThrowExceptionOfType(shouldThrowException, typeof(ObjectDisposedException)); } diff --git a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerTests.cs b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerTests.cs index 86382dcc62..85dc54b86f 100644 --- a/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerTests.cs +++ b/test/UnitTests/PlatformServices.Shared.Unit.Tests/netstandard1.3/ns13TraceListenerTests.cs @@ -40,7 +40,7 @@ public void DisposeShouldDisposeCorrespondingTextWriter() var traceListener = new TraceListenerWrapper(writer); traceListener.Dispose(); - // Tring to write after disposing textWriter should throw exception + // Trying to write after disposing textWriter should throw exception Action shouldThrowException = () => writer.WriteLine("Try to write something"); ActionUtility.ActionShouldThrowExceptionOfType(shouldThrowException, typeof(ObjectDisposedException)); }