From b53d39e16f8ecf8651c44ec4f90e374dec3f1af9 Mon Sep 17 00:00:00 2001 From: flash2048 Date: Thu, 29 Mar 2018 14:27:07 +0300 Subject: [PATCH] Corrects spelling of some comments, method name and remove an excess flag --- .../Rendering/BrowserRendererRegistry.cs | 2 +- .../DirectiveRazorIntegrationTest.cs | 2 +- .../ComponentTagHelperDescriptorProviderTest.cs | 6 +++--- test/Microsoft.AspNetCore.Blazor.Test/RendererTest.cs | 2 +- .../Routing/RouteTableTests.cs | 2 +- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/Microsoft.AspNetCore.Blazor.Browser/Rendering/BrowserRendererRegistry.cs b/src/Microsoft.AspNetCore.Blazor.Browser/Rendering/BrowserRendererRegistry.cs index 9efcb8403d..805c772d44 100644 --- a/src/Microsoft.AspNetCore.Blazor.Browser/Rendering/BrowserRendererRegistry.cs +++ b/src/Microsoft.AspNetCore.Blazor.Browser/Rendering/BrowserRendererRegistry.cs @@ -8,7 +8,7 @@ namespace Microsoft.AspNetCore.Blazor.Browser.Rendering /// /// Provides mechanisms for locating instances /// by ID. This is used when receiving incoming events from the browser. It - /// implictly ensures that the instances and + /// implicitly ensures that the instances and /// their associated component instances aren't GCed when events may still /// be received for them. /// diff --git a/test/Microsoft.AspNetCore.Blazor.Build.Test/DirectiveRazorIntegrationTest.cs b/test/Microsoft.AspNetCore.Blazor.Build.Test/DirectiveRazorIntegrationTest.cs index 6e30602dfa..8cc58b1b61 100644 --- a/test/Microsoft.AspNetCore.Blazor.Build.Test/DirectiveRazorIntegrationTest.cs +++ b/test/Microsoft.AspNetCore.Blazor.Build.Test/DirectiveRazorIntegrationTest.cs @@ -84,7 +84,7 @@ namespace Microsoft.AspNetCore.Blazor.Build.Test $"Hello from @MyService1 and @MyService2").GetType(); // Assert 1: Compiled type has correct properties - var propertyFlags = BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.NonPublic; + var propertyFlags = BindingFlags.Instance | BindingFlags.NonPublic; var injectableProperties = componentType.GetProperties(propertyFlags) .Where(p => p.GetCustomAttribute() != null); Assert.Collection(injectableProperties.OrderBy(p => p.Name), diff --git a/test/Microsoft.AspNetCore.Blazor.Razor.Extensions.Test/ComponentTagHelperDescriptorProviderTest.cs b/test/Microsoft.AspNetCore.Blazor.Razor.Extensions.Test/ComponentTagHelperDescriptorProviderTest.cs index a2f4f27872..d10c331c53 100644 --- a/test/Microsoft.AspNetCore.Blazor.Razor.Extensions.Test/ComponentTagHelperDescriptorProviderTest.cs +++ b/test/Microsoft.AspNetCore.Blazor.Razor.Extensions.Test/ComponentTagHelperDescriptorProviderTest.cs @@ -85,13 +85,13 @@ namespace Test // No documentation in this test Assert.Null(component.Documentation); - // These are all trivally derived from the assembly/namespace/type name + // These are all trivially derived from the assembly/namespace/type name Assert.Equal("TestAssembly", component.AssemblyName); Assert.Equal("Test.MyComponent", component.Name); Assert.Equal("Test.MyComponent", component.DisplayName); Assert.Equal("Test.MyComponent", component.GetTypeName()); - // Our use of matching rules is also very simple, and derived from the name. Veriying + // Our use of matching rules is also very simple, and derived from the name. Verifying // it once in detail here and then ignoring it. var rule = Assert.Single(component.TagMatchingRules); Assert.Empty(rule.Attributes); @@ -329,7 +329,7 @@ namespace Test Assert.True(attribute.IsDelegateProperty()); } - // For simplicity in testing, exlude the built-in components. We'll add more and we + // For simplicity in testing, exclude the built-in components. We'll add more and we // don't want to update the tests when that happens. private TagHelperDescriptor[] ExcludeBuiltInComponents(TagHelperDescriptorProviderContext context) { diff --git a/test/Microsoft.AspNetCore.Blazor.Test/RendererTest.cs b/test/Microsoft.AspNetCore.Blazor.Test/RendererTest.cs index ee9fc6088d..8db5dd9eec 100644 --- a/test/Microsoft.AspNetCore.Blazor.Test/RendererTest.cs +++ b/test/Microsoft.AspNetCore.Blazor.Test/RendererTest.cs @@ -134,7 +134,7 @@ namespace Microsoft.AspNetCore.Blazor.Test var nestedComponent = (MessageComponent)nestedComponentFrame.Component; var nestedComponentId = nestedComponentFrame.ComponentId; - // Assert: inital render + // Assert: initial render nestedComponent.Message = "Render 1"; nestedComponent.TriggerRender(); var batch = renderer.Batches[1]; diff --git a/test/Microsoft.AspNetCore.Blazor.Test/Routing/RouteTableTests.cs b/test/Microsoft.AspNetCore.Blazor.Test/Routing/RouteTableTests.cs index 61790aa561..5851e24221 100644 --- a/test/Microsoft.AspNetCore.Blazor.Test/Routing/RouteTableTests.cs +++ b/test/Microsoft.AspNetCore.Blazor.Test/Routing/RouteTableTests.cs @@ -279,7 +279,7 @@ namespace Microsoft.AspNetCore.Blazor.Test.Routing [InlineData("/{parameter}", "/{parameter}/")] [InlineData("/literal/{parameter}", "/Literal/{something}")] [InlineData("/{parameter}/literal/{something}", "{param}/Literal/{else}")] - public void DetectsAmbigousRoutes(string left, string right) + public void DetectsAmbiguousRoutes(string left, string right) { // Arrange var expectedMessage = $@"The following routes are ambiguous: