Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some typos in codebase #3262

Merged
merged 1 commit into from
Jan 21, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Fix some typos in codebase
  • Loading branch information
Evangelink committed Jan 18, 2022
commit 4d09e18cc73f837cf51b0ac964e7d3d88a8083bb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
namespace Microsoft.TestPlatform.AdapterUtilities
{
/// <summary>
/// Contants to help declare ManagedType and ManagedMethod test properties.
/// Constants to help declare ManagedType and ManagedMethod test properties.
/// </summary>
public static class ManagedNameConstants
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@
</data>
<data name="ErrorInvalidSequenceAt" xml:space="preserve">
<value>Invalid escape sequence! (segment: {0}, pos: {1})</value>
<comment>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting postion of invalid sequence.</comment>
<comment>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting position of invalid sequence.</comment>
</data>
<data name="ErrorMethodArityMustBeNumeric" xml:space="preserve">
<value>Method arity must be numeric</value>
Expand All @@ -133,7 +133,7 @@
</data>
<data name="ErrorNoClosingQuote" xml:space="preserve">
<value>A closing single quote was expected at the end of the segment! (segment: {0})</value>
<comment>An error thrown when the type name ended but opened single qoute was not matched.</comment>
<comment>An error thrown when the type name ended but opened single quote was not matched.</comment>
</data>
<data name="ErrorTypeNotFound" xml:space="preserve">
<value>Type '{0}' not found</value>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,12 @@ Example: 'System.Reflection.MethodBase' is not implemented on this platform!</no
<trans-unit id="ErrorInvalidSequenceAt">
Haplois marked this conversation as resolved.
Show resolved Hide resolved
<source>Invalid escape sequence! (segment: {0}, pos: {1})</source>
<target state="new">Invalid escape sequence! (segment: {0}, pos: {1})</target>
<note>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting postion of invalid sequence.</note>
<note>An escape sequence started with '\', but it wasn't legal. {0}: Invalid string, {1}: starting position of invalid sequence.</note>
</trans-unit>
<trans-unit id="ErrorNoClosingQuote">
<source>A closing single quote was expected at the end of the segment! (segment: {0})</source>
<target state="new">A closing single quote was expected at the end of the segment! (segment: {0})</target>
<note>An error thrown when the type name ended but opened single qoute was not matched.</note>
<note>An error thrown when the type name ended but opened single quote was not matched.</note>
</trans-unit>
</body>
</file>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ public void AppendString(string str)
hasher.ProcessBlock(lastBlock, 0, lastBlock.Length);
position = 0;

// We proccessed the entire string already
// We processed the entire string already
if (end == bytes.Length)
{
return;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ namespace Microsoft.VisualStudio.TestPlatform.Common.DataCollection
[DataContract]
public class AfterTestRunEndResult
{
// We have more than one ctor for backward-compatibility reason but we don't want to add dependency on Newtosoft([JsonConstructor])
// We have more than one ctor for backward-compatibility reason but we don't want to add dependency on Newtonsoft([JsonConstructor])
// We want to fallback to the non-public default constructor https://www.newtonsoft.com/json/help/html/T_Newtonsoft_Json_ConstructorHandling.htm during deserialization
private AfterTestRunEndResult()
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ internal static FilterExpression Parse(string filterString, out FastFilter fastF
{
ValidateArg.NotNull(filterString, nameof(filterString));

// Below parsing doesn't error out on pattern (), so explicitly search for that (empty parethesis).
// Below parsing doesn't error out on pattern (), so explicitly search for that (empty parenthesis).
var invalidInput = Regex.Match(filterString, @"\(\s*\)");
if (invalidInput.Success)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ internal void WaitForEventCompletion()
/// <summary>
/// Raises a test result event to the enabled loggers.
/// </summary>
/// <param name="args">Arguments to to be raised.</param>
/// <param name="args">Arguments to be raised.</param>
internal void RaiseTestResult(TestResultEventArgs args)
{
ValidateArg.NotNull(args, nameof(args));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ private static bool TryAddFakesDataCollectorSettings(
IEnumerable<string> sources,
FrameworkVersion framework)
{
// A new Fakes Congigurator API makes the decision to add the right datacollector uri to the configuration
// A new Fakes Configurator API makes the decision to add the right datacollector uri to the configuration
// There now exist two data collector URIs to support two different scenarios. The new scenario involves
// using the CLRIE profiler, and the old involves using the Intellitrace profiler (which isn't supported in
// .NET Core scenarios). The old API still exists for fallback measures.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ public void Initialize(IEnumerable<string> pathToAdditionalExtensions, ITestMess

this.LoadExtensions();

//unsubscrive session logger
//unsubscribe session logger
this.sessionMessageLogger.TestRunMessage -= this.TestSessionMessageHandler;

this.testPlatformEventSource.AdapterSearchStop();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ namespace Microsoft.VisualStudio.TestPlatform.ObjectModel.Client

/// <summary>
/// Defines the test session info object to be passed around between vstest.console and
/// vstest.console wrapper in order to indentify the current session.
/// vstest.console wrapper in order to identify the current session.
/// </summary>
[DataContract]
public class TestSessionInfo : IEquatable<TestSessionInfo>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ public class DataCollectorAttachmentProcessorAttribute : Attribute
/// Initializes a new instance of the <see cref="DataCollectorAttachmentProcessorAttribute"/> class.
/// </summary>
/// <param name="type">
/// The type of the attachement data processor.
/// The type of the attachment data processor.
/// </param>
public DataCollectorAttachmentProcessorAttribute(Type type)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ public void HangDumpOnTimeout(RunnerInfo runnerInfo)
}

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

[TestMethod]
// net5.0 does not suppord dump on exit
// net5.0 does not support dump on exit
[NetCoreRunner("net452;net472;netcoreapp3.1")]
// should make no difference, keeping for easy debug
// [NetFrameworkRunner("net452;net472;netcoreapp3.1")]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ public void CommandLineRunSettingsShouldWinAmongAllOptions(RunnerInfo runnerInfo
}

/// <summary>
/// Command line run settings should have high precedence btween cli runsettings and cli switches.
/// Command line run settings should have high precedence between cli runsettings and cli switches.
/// </summary>
[TestMethod]
[NetFullTargetFrameworkDataSource]
Expand Down