Update comments and fix mistakes

Nirmal4G · Nirmal4G · commit 8297f88bbd21 · 2022-10-22T08:08:30.000+05:30
- Fix message text.
- Fix Include logic.
- Fix spelling mistakes.
- Fix leading whitespace.
- Fix casing in code and comments.
diff --git a/Microsoft.Toolkit.Uwp.Notifications/Microsoft.Toolkit.Uwp.Notifications.csproj b/Microsoft.Toolkit.Uwp.Notifications/Microsoft.Toolkit.Uwp.Notifications.csproj
@@ -12,7 +12,7 @@
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(TargetFramework)' == 'net461' or '$(TargetFramework)' == 'netcoreapp3.1' or '$(TargetFramework)' == 'net5.0-windows10.0.17763.0'">
-    <!-- Define the WINDOWS_UWP conditional symbol, since the Windows.Data.Xml and the Windows.UI.Notification namespaces are available -->
+    <!-- Define the WINDOWS_UWP conditional symbol, since the 'Windows.Data.Xml' and the 'Windows.UI.Notification' namespaces are available -->
     <DefineConstants>$(DefineConstants);WINDOWS_UWP;WIN32</DefineConstants>
   </PropertyGroup>
 
@@ -42,13 +42,12 @@
     <None Include="Microsoft.Toolkit.Uwp.Notifications.targets" Pack="true" PackagePath="build\native" />
   </ItemGroup>
 
-  <!-- Desktop Win32 apps -->
+  <!-- For Desktop Win32 apps' WinRT APIs support -->
   <ItemGroup Condition="'$(TargetFramework)' == 'net461' or '$(TargetFramework)' == 'netcoreapp3.1'">
-    <!-- Reference Windows SDK NuGet of correct target platform version -->
     <PackageReference Include="Microsoft.Windows.SDK.Contracts" Version="10.0.19041.1" />
   </ItemGroup>
 
-  <!-- The .NET Core desktop apps also need the Registry NuGet package and System.Reflection.Emit for generating COM class dynamically -->
+  <!-- The .NET Core desktop apps also need the Registry NuGet package and 'System.Reflection.Emit' for generating COM class dynamically -->
   <ItemGroup Condition="'$(TargetFramework)' == 'netcoreapp3.1' or '$(TargetFramework)' == 'net5.0-windows10.0.17763.0'">
     <PackageReference Include="Microsoft.Win32.Registry" Version="4.7.0" />
     <PackageReference Include="System.Reflection.Emit" Version="4.7.0" />
@@ -59,12 +58,15 @@
     <PackageReference Include="Microsoft.NETCore.UniversalWindowsPlatform" Version="$(ExtrasUwpMetaPackageVersion)" PrivateAssets="All" IsImplicitlyDefined="true" />
   </ItemGroup>
 
-  <!-- Native (C++) doesn't need 'System.ValueTuple' (plus it's incompatible with this package) -->
+  <!-- Native (C++) does not need 'System.ValueTuple' (plus it is incompatible with this package) -->
   <ItemGroup Condition="'$(TargetFramework)' != 'native'">
     <PackageReference Include="System.ValueTuple" Version="4.5.0" />
   </ItemGroup>
 
-  <!-- Set the nuspec properties. Dependent on version which isn't updated till after 'GetBuildVersion'. Condition ensures we only set once since this runs multiple times for each target. -->
+  <!--
+      Set the nuspec properties. Dependent on version which is not updated till after 'GetBuildVersion'.
+      Condition ensures we only set once since this runs multiple times for each target.
+  -->
   <Target Name="SetNuspecProperties" AfterTargets="GetBuildVersion">
     <PropertyGroup Condition="'$(NuspecProperties)' == ''">
       <NuspecProperties>buildOutput=bin\$(Configuration);version=$(Version)</NuspecProperties>
diff --git a/SmokeTests/SmokeTests.proj b/SmokeTests/SmokeTests.proj
@@ -33,7 +33,7 @@
           Condition="'$(NuGetPackageVersion)' == ''">
       <Output TaskParameter="ConsoleOutput" PropertyName="NuGetPackageVersion" />
     </Exec>
-    <Message Text="Got GitBank Version... $(NuGetPackageVersion)" Importance="High" />
+    <Message Text="Got NuGet Package Version... $(NuGetPackageVersion)" Importance="High" />
   </Target>
 
   <Target Name="Build"
@@ -44,7 +44,7 @@
     <Message Importance="High" Text="Building project %(ProjectsToBuild.Identity): (%(ProjectsToBuild.Configuration)|%(ProjectsToBuild.Platform))" />
 
     <MSBuild Projects="SmokeTest.csproj"
-              Targets="restore;build"
+              Targets="Restore;Build"
               Properties="CurrentProject=%(ProjectsToBuild.Identity);Configuration=%(ProjectsToBuild.Configuration);Platform=%(ProjectsToBuild.Platform);NuGetPackageVersion=$(NuGetPackageVersion)" />
   </Target>
 
@@ -77,10 +77,10 @@
 
   <Target Name="CheckNuGetPackages">
     <PropertyGroup>
-      <NuGetOutputPath>$(MSBuildThisFileDirectory)..\bin\nupkg</NuGetOutputPath>
+      <NuGetOutputPath>$(MSBuildThisFileDirectory)..\bin\nupkg\</NuGetOutputPath>
     </PropertyGroup>
     <ItemGroup>
-      <NuGetPackage Include="$(NuGetOutputPath)\*.nupkg" />
+      <NuGetPackage Include="$(NuGetOutputPath)*.nupkg" />
     </ItemGroup>
     <Error Condition="'@(NuGetPackage)' == ''" Text="Directory '$(NuGetOutputPath)' is empty" />
   </Target>
diff --git a/UITests/UITests.Tests.Shared/Controls/ColorPickerButtonTest.cs b/UITests/UITests.Tests.Shared/Controls/ColorPickerButtonTest.cs
@@ -25,7 +25,7 @@ public class ColorPickerButtonTest : UITestBase
         [TestProperty("Platform", "Any")]
         public static void ClassInitialize(TestContext testContext)
         {
-            TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);
+            TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);
         }
 
         /// <summary>
diff --git a/UITests/UITests.Tests.Shared/Controls/ConstrainedBoxTest.cs b/UITests/UITests.Tests.Shared/Controls/ConstrainedBoxTest.cs
@@ -27,7 +27,7 @@ public class ConstrainedBoxTest : UITestBase
         [TestProperty("Platform", "Any")]
         public static void ClassInitialize(TestContext testContext)
         {
-            TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);
+            TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);
         }
 
         [TestMethod]
diff --git a/UITests/UITests.Tests.Shared/Controls/GridSplitterTest.cs b/UITests/UITests.Tests.Shared/Controls/GridSplitterTest.cs
@@ -33,7 +33,7 @@ public class GridSplitterTest : UITestBase
         [TestProperty("Platform", "Any")]
         public static void ClassInitialize(TestContext testContext)
         {
-            TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);
+            TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);
         }
 
         [TestMethod]
diff --git a/UITests/UITests.Tests.Shared/Controls/RangeSelectorTest.cs b/UITests/UITests.Tests.Shared/Controls/RangeSelectorTest.cs
@@ -26,7 +26,7 @@ public class RangeSelectorTest : UITestBase
         [TestProperty("Platform", "Any")]
         public static void ClassInitialize(TestContext testContext)
         {
-            TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);
+            TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);
         }
 
         [TestMethod]
diff --git a/UITests/UITests.Tests.Shared/Controls/RichSuggestBoxTest.cs b/UITests/UITests.Tests.Shared/Controls/RichSuggestBoxTest.cs
@@ -27,7 +27,7 @@ public class RichSuggestBoxTest : UITestBase
         [TestProperty("Platform", "Any")]
         public static void ClassInitialize(TestContext testContext)
         {
-            TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);
+            TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);
         }
 
         [TestMethod]
diff --git a/UITests/UITests.Tests.Shared/Controls/TextBoxMaskTest.cs b/UITests/UITests.Tests.Shared/Controls/TextBoxMaskTest.cs
@@ -25,7 +25,7 @@ public class TextBoxMaskTest : UITestBase
         [TestProperty("Platform", "Any")]
         public static void ClassInitialize(TestContext testContext)
         {
-            TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);
+            TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);
         }
 
         [TestMethod]
diff --git a/UITests/UITests.Tests.Shared/Examples/SimpleTest.cs b/UITests/UITests.Tests.Shared/Examples/SimpleTest.cs
@@ -25,7 +25,7 @@ public class SimpleTest : UITestBase
         [TestProperty("Platform", "Any")]
         public static void ClassInitialize(TestContext testContext)
         {
-            TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);
+            TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);
         }
 
         [TestMethod]
diff --git a/UITests/UITests.Tests.Shared/TestAssembly.cs b/UITests/UITests.Tests.Shared/TestAssembly.cs
@@ -36,10 +36,10 @@ public static void AssemblyInitialize(TestContext testContext)
         [AssemblyCleanup]
         public static void AssemblyCleanup()
         {
-            TestEnvironment.AssemblyCleanupWorker(UITestBase.WinUICsUWPSampleApp);
+            TestEnvironment.AssemblyCleanupWorker(UITestBase.WinUICSharpUWPSampleApp);
         }
 
-        private static async Task InitalizeComService()
+        private static async Task InitializeComService()
         {
             CommunicationService = new AppServiceConnection();
 
@@ -84,7 +84,7 @@ internal static async Task<AppServiceResponse> SendCustomMessageToApp(ValueSet m
         {
             if (CommunicationService is null)
             {
-                await InitalizeComService();
+                await InitializeComService();
             }
 
             return await CommunicationService.SendMessageAsync(message);
@@ -94,7 +94,7 @@ private static async Task<bool> SendMessageToApp(ValueSet message)
         {
             if (CommunicationService is null)
             {
-                await InitalizeComService();
+                await InitializeComService();
             }
 
             var response = await CommunicationService.SendMessageAsync(message);
diff --git a/UITests/UITests.Tests.Shared/UITestBase.cs b/UITests/UITests.Tests.Shared/UITestBase.cs
@@ -25,7 +25,7 @@ public abstract class UITestBase
     {
         private TestSetupHelper helper;
 
-        internal static TestApplicationInfo WinUICsUWPSampleApp
+        internal static TestApplicationInfo WinUICSharpUWPSampleApp
         {
             get
             {
@@ -132,13 +132,13 @@ private static string GetPageForTest(TestContext testContext)
 
             Log.Comment($"Found {testMethodString}.");
 
-            var testpageAttributeString = $"\"{typeof(TestPageAttribute)}\" on {testMethodString}";
+            var testPageAttributeString = $"\"{typeof(TestPageAttribute)}\" on {testMethodString}";
             if (method.GetCustomAttribute(typeof(TestPageAttribute), true) is not TestPageAttribute attribute)
             {
-                throw new Exception($"Could not find {testpageAttributeString}.");
+                throw new Exception($"Could not find {testPageAttributeString}.");
             }
 
-            Log.Comment($"Found {testpageAttributeString}. {nameof(TestPageAttribute.XamlFile)}: {attribute.XamlFile}.");
+            Log.Comment($"Found {testPageAttributeString}. {nameof(TestPageAttribute.XamlFile)}: {attribute.XamlFile}.");
 
             return attribute.XamlFile;
         }
diff --git a/UnitTests/UnitTests.UWP/Extensions/Test_DispatcherQueueExtensions.cs b/UnitTests/UnitTests.UWP/Extensions/Test_DispatcherQueueExtensions.cs
@@ -16,7 +16,7 @@
 namespace UnitTests.Extensions
 {
     [TestClass]
-    [Ignore("Ignored until issue on .Net Native is fixed. These are working.")]
+    [Ignore("Ignored until issue on .NET Native is fixed. These are working.")]
     public class Test_DispatcherQueueExtensions
     {
         private const int TIME_OUT = 5000;
diff --git a/build/Windows.Toolkit.VisualStudio.Design.props b/build/Windows.Toolkit.VisualStudio.Design.props
@@ -15,7 +15,7 @@
 
   <ItemGroup>
     <AppDesigner Include="Properties\"/>
-      <!-- The 'LogicalName' metadata changes the default manifest naming scheme -->
+    <!-- The 'LogicalName' metadata changes the default manifest naming scheme -->
     <EmbeddedResource Include="Icons\*.icon.png" LogicalName="$(ParentProjectRootNamespace).%(Filename)%(Extension)"/>
     <EmbeddedResource Include="$(ParentProjectOutputPath)$(ParentProjectName).xml" LinkBase="Properties\" LogicalName="%(Filename)%(Extension)"/>
   </ItemGroup>

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ public class ColorPickerButtonTest : UITestBase`
`25`	`25`	`[TestProperty("Platform", "Any")]`
`26`	`26`	`public static void ClassInitialize(TestContext testContext)`
`27`	`27`	`{`
`28`		`- TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);`
	`28`	`+ TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public class ConstrainedBoxTest : UITestBase`
`27`	`27`	`[TestProperty("Platform", "Any")]`
`28`	`28`	`public static void ClassInitialize(TestContext testContext)`
`29`	`29`	`{`
`30`		`- TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);`
	`30`	`+ TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`[TestMethod]`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public class GridSplitterTest : UITestBase`
`33`	`33`	`[TestProperty("Platform", "Any")]`
`34`	`34`	`public static void ClassInitialize(TestContext testContext)`
`35`	`35`	`{`
`36`		`- TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);`
	`36`	`+ TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`[TestMethod]`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ public class RangeSelectorTest : UITestBase`
`26`	`26`	`[TestProperty("Platform", "Any")]`
`27`	`27`	`public static void ClassInitialize(TestContext testContext)`
`28`	`28`	`{`
`29`		`- TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);`
	`29`	`+ TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`[TestMethod]`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public class RichSuggestBoxTest : UITestBase`
`27`	`27`	`[TestProperty("Platform", "Any")]`
`28`	`28`	`public static void ClassInitialize(TestContext testContext)`
`29`	`29`	`{`
`30`		`- TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);`
	`30`	`+ TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`[TestMethod]`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ public class TextBoxMaskTest : UITestBase`
`25`	`25`	`[TestProperty("Platform", "Any")]`
`26`	`26`	`public static void ClassInitialize(TestContext testContext)`
`27`	`27`	`{`
`28`		`- TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);`
	`28`	`+ TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`[TestMethod]`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ public class SimpleTest : UITestBase`
`25`	`25`	`[TestProperty("Platform", "Any")]`
`26`	`26`	`public static void ClassInitialize(TestContext testContext)`
`27`	`27`	`{`
`28`		`- TestEnvironment.Initialize(testContext, WinUICsUWPSampleApp);`
	`28`	`+ TestEnvironment.Initialize(testContext, WinUICSharpUWPSampleApp);`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`[TestMethod]`
Original file line number	Diff line number	Diff line change
`@@ -36,10 +36,10 @@ public static void AssemblyInitialize(TestContext testContext)`
`36`	`36`	`[AssemblyCleanup]`
`37`	`37`	`public static void AssemblyCleanup()`
`38`	`38`	`{`
`39`		`- TestEnvironment.AssemblyCleanupWorker(UITestBase.WinUICsUWPSampleApp);`
	`39`	`+ TestEnvironment.AssemblyCleanupWorker(UITestBase.WinUICSharpUWPSampleApp);`
`40`	`40`	`}`
`41`	`41`
`42`		`- private static async Task InitalizeComService()`
	`42`	`+ private static async Task InitializeComService()`
`43`	`43`	`{`
`44`	`44`	`CommunicationService = new AppServiceConnection();`
`45`	`45`
`@@ -84,7 +84,7 @@ internal static async Task<AppServiceResponse> SendCustomMessageToApp(ValueSet m`
`84`	`84`	`{`
`85`	`85`	`if (CommunicationService is null)`
`86`	`86`	`{`
`87`		`- await InitalizeComService();`
	`87`	`+ await InitializeComService();`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`return await CommunicationService.SendMessageAsync(message);`
`@@ -94,7 +94,7 @@ private static async Task<bool> SendMessageToApp(ValueSet message)`
`94`	`94`	`{`
`95`	`95`	`if (CommunicationService is null)`
`96`	`96`	`{`
`97`		`- await InitalizeComService();`
	`97`	`+ await InitializeComService();`
`98`	`98`	`}`
`99`	`99`
`100`	`100`	`var response = await CommunicationService.SendMessageAsync(message);`