chore: Fix test irregularities

src/ActionsTests/Misc/ExtensionMethodsTests.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft. All rights reserved.
+// Copyright (c) Microsoft. All rights reserved.
 // Licensed under the MIT license. See LICENSE file in the project root for full license information.
 
 using Axe.Windows.Actions.Misc;
@@ -12,14 +12,14 @@
 
 namespace Axe.Windows.ActionsTests.Misc
 {
-    [TestClass()]
+    [TestClass]
     public class ExtensionMethodsTests
     {
         /// <summary>
         /// Create 10 elements with bounding rectangle (i, i, i, i) from [0, 9]
         /// See which one is identified as smallest enclosing area of (5, 5)
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void GetSmallestElementFromPointTest()
         {
             var boundingRects = new List<System.Drawing.Rectangle>();
@@ -39,7 +39,7 @@ public void GetSmallestElementFromPointTest()
         /// See which one is identified as smallest enclosing area of (5, 5)
         /// Normal answer (3, 3, 3, 3) is off-screen
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void GetSmallestElementFromPointTest_Offscreen()
         {
             var boundingRects = new List<System.Drawing.Rectangle>();
@@ -59,7 +59,7 @@ public void GetSmallestElementFromPointTest_Offscreen()
         /// Create 10 elements with bounding rectangle (i, i, i, i) from [0, 9]
         /// See which one is identified as smallest enclosing area of (200, 200)
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void GetSmallestElementFromPointTest_NoneOverlaps()
         {
             var boundingRects = new List<System.Drawing.Rectangle>();
@@ -74,13 +74,13 @@ public void GetSmallestElementFromPointTest_NoneOverlaps()
             Assert.AreEqual(null, answer);
         }
 
-        [TestMethod()]
+        [TestMethod]
         public void GetSmallestElementFromPointTest_NullArgument()
         {
             Assert.ThrowsException<ArgumentNullException>(() => ExtensionMethods.GetSmallestElementFromPoint(null, System.Drawing.Point.Empty));
         }
 
-        [TestMethod()]
+        [TestMethod]
         public void GetSmallestElementFromPointTest_NoElementsProvided()
         {
             Assert.ThrowsException<ArgumentException>(() => ExtensionMethods.GetSmallestElementFromPoint(new Dictionary<int, ICoreA11yElement>(), System.Drawing.Point.Empty));

src/AutomationTests/ScanResultsAssemblerTests.cs

@@ -13,7 +13,7 @@
 
 namespace Axe.Windows.AutomationTests
 {
-    [TestClass()]
+    [TestClass]
     public class ScanResultsAssemblerTests
     {
         readonly ScanResultsAssembler _assembler = new ScanResultsAssembler();

src/CoreTests/Bases/A11yElementTests.cs

@@ -12,10 +12,10 @@ namespace Axe.Windows.CoreTests.Bases
     /// <summary>
     /// Tests A11yElement class
     /// </summary>
-    [TestClass()]
+    [TestClass]
     public class A11yElementTests
     {
-        [TestMethod()]
+        [TestMethod]
         [Timeout(2000)]
         public void FindDescendent_SimpleCondition_ReturnsChild()
         {
@@ -31,7 +31,7 @@ public void FindDescendent_SimpleCondition_ReturnsChild()
             Assert.AreEqual(result, child);
         }
 
-        [TestMethod()]
+        [TestMethod]
         [Timeout(2000)]
         public void FindDescendent_SimpleCondition_ReturnsNull()
         {
@@ -47,7 +47,7 @@ public void FindDescendent_SimpleCondition_ReturnsNull()
             Assert.IsNull(result);
         }
 
-        [TestMethod()]
+        [TestMethod]
         [Timeout(2000)]
         public void FindDescendent_SimpleCondition_ReturnsGrandchild()
         {
@@ -73,7 +73,7 @@ public void FindDescendent_SimpleCondition_ReturnsGrandchild()
             Assert.AreEqual(result, grandChild);
         }
 
-        [TestMethod()]
+        [TestMethod]
         [Timeout(2000)]
         public void FindDescendent_SimpleConditionNonePass_ReturnsNull()
         {
@@ -99,7 +99,7 @@ public void FindDescendent_SimpleConditionNonePass_ReturnsNull()
         /// <summary>
         /// Test various getters which use GetPropertySafely
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void GetPropertySafelyTest()
         {
             A11yElement element = Utility.LoadA11yElementsFromJSON("Resources/A11yElementTest.hier");

src/CoreTests/Bases/A11yPatternPropertyTests.cs

@@ -10,13 +10,13 @@ namespace Axe.Windows.CoreTests.Bases
     /// <summary>
     /// Tests A11yPatternProperty class
     /// </summary>
-    [TestClass()]
+    [TestClass]
     public class A11yPatternPropertyTests
     {
         /// <summary>
         /// Checks NodeValue string output
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void NodeValueTest()
         {
             A11yElement element = Utility.LoadA11yElementsFromJSON("Resources/A11yPatternTest.hier");

src/CoreTests/Bases/A11yPatternTests.cs

@@ -10,13 +10,13 @@ namespace Axe.Windows.CoreTests.Bases
     /// <summary>
     /// Tests A11yPattern class
     /// </summary>
-    [TestClass()]
+    [TestClass]
     public class A11yPatternTests
     {
         /// <summary>
         /// Test ToString and constructor for A11yPattern
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void ToStringTest()
         {
             A11yElement element = Utility.LoadA11yElementsFromJSON("Resources/A11yPatternTest.hier");

src/CoreTests/Bases/A11yPropertyTests.cs

@@ -12,14 +12,14 @@ namespace Axe.Windows.CoreTests.Bases
     /// <summary>
     /// Tests A11yProperty class
     /// </summary>
-    [TestClass()]
+    [TestClass]
     public class A11yPropertyTests
     {
         /// <summary>
         /// Tests A11yProperty.ToString() through A11yProperty.TextValue.
         /// Also tests constructor.
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void ToStringTest()
         {
             A11yElement element = Utility.LoadA11yElementsFromJSON("Resources/A11yPropertyTest.hier");
@@ -44,7 +44,7 @@ public void ToStringTest()
             Assert.AreEqual("False", kpVal);
         }
 
-        [TestMethod()]
+        [TestMethod]
         public void SimpleCustomPropertyTest()
         {
             const int testId = 42;
@@ -55,7 +55,7 @@ public void SimpleCustomPropertyTest()
             Assert.AreEqual(expectedRendering, kp.ToString());
         }
 
-        [TestMethod()]
+        [TestMethod]
         public void RegisterTwiceCustomPropertyTest()
         {
             const int testId = 42;

src/CoreTests/CustomObjects/ConfigTests.cs

@@ -8,7 +8,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects
 {
-    [TestClass()]
+    [TestClass]
     public class ConfigTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/Converters/BoolConverterTests.cs

@@ -7,7 +7,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects.Converters
 {
-    [TestClass()]
+    [TestClass]
     public class BoolConverterTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/Converters/DoubleConverterTests.cs

@@ -7,7 +7,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects.Converters
 {
-    [TestClass()]
+    [TestClass]
     public class DoubleConverterTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/Converters/ElementConverterTests.cs

@@ -8,7 +8,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects.Converters
 {
-    [TestClass()]
+    [TestClass]
     public class ElementConverterTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/Converters/EnumConverterTests.cs

@@ -8,7 +8,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects.Converters
 {
-    [TestClass()]
+    [TestClass]
     public class EnumConverterTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/Converters/IntConverterTests.cs

@@ -7,7 +7,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects.Converters
 {
-    [TestClass()]
+    [TestClass]
     public class IntConverterTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/Converters/PointConverterTests.cs

@@ -7,7 +7,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects.Converters
 {
-    [TestClass()]
+    [TestClass]
     public class PointConverterTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/Converters/StringConverterTests.cs

@@ -7,7 +7,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects.Converters
 {
-    [TestClass()]
+    [TestClass]
     public class StringConverterTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/CustomObjects/CustomPropertyTests.cs

@@ -10,7 +10,7 @@
 
 namespace Axe.Windows.CoreTests.CustomObjects
 {
-    [TestClass()]
+    [TestClass]
     public class CustomPropertyTests
     {
         [TestMethod, Timeout(1000)]

src/CoreTests/Misc/MiscTests.cs

@@ -13,14 +13,14 @@ namespace Axe.Windows.CoreTests.Misc
     /// <summary>
     /// Tests for the misc folder in Axe.Windows.Core
     /// </summary>
-    [TestClass()]
+    [TestClass]
     public class MiscTests
     {
         /// <summary>
         /// Tests whether the aggregates status counts of a monster
         /// snapshot are accurate
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void GetStatusCounts()
         {
             A11yElement element = UnitTestSharedLibrary.Utility.LoadA11yElementsFromJSON("Snapshots/Taskbar.snapshot");

src/CoreTests/Results/ScanMetaInfoTests.cs

@@ -9,13 +9,13 @@
 
 namespace Axe.Windows.CoreTests.Results
 {
-    [TestClass()]
+    [TestClass]
     public class ScanMetaInfoTests
     {
         /// <summary>
         /// Test with PropertyId Set
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void ScanMetaInfoTest()
         {
             A11yElement e = new A11yElement
@@ -37,7 +37,7 @@ public void ScanMetaInfoTest()
         /// <summary>
         /// Test without PropertyId
         /// </summary>
-        [TestMethod()]
+        [TestMethod]
         public void ScanMetaInfoTest2()
         {
             A11yElement e = new A11yElement

src/CoreTests/Types/ControlTypesTests.cs

@@ -7,23 +7,23 @@
 
 namespace Axe.Windows.CoreTests.Types
 {
-    [TestClass()]
+    [TestClass]
     public class ControlTypesTest
     {
-        [TestMethod()]
+        [TestMethod]
         public void CheckExists()
         {
             Assert.AreEqual(true, ControlType.GetInstance().Exists(ControlType.UIA_AppBarControlTypeId));
             Assert.AreEqual(false, ControlType.GetInstance().Exists(0));
         }
 
-        [TestMethod()]
+        [TestMethod]
         public void CheckGetNameById()
         {
             Assert.AreEqual("AppBar(50040)", ControlType.GetInstance().GetNameById(ControlType.UIA_AppBarControlTypeId));
         }
 
-        [TestMethod()]
+        [TestMethod]
         [Timeout(1000)]
         public void CheckHasExpectedValues()
         {