Remove no longer relevant tests

Author: Youssef1313

test/UnitTests/MSTestAdapter.PlatformServices.UnitTests/Execution/TypeCacheTests.cs

@@ -998,79 +998,6 @@ public void GetTestMethodInfoShouldReturnTestMethodInfoForMethodsAdornedWithADer
         testMethodInfo.Executor.Should().BeOfType<DerivedTestMethodAttribute>();
     }
 
-    public void GetTestMethodInfoShouldSetTestContextWithCustomProperty()
-    {
-        // Not using _typeCache here which uses a mocked ReflectHelper which doesn't work well with this test.
-        // Setting up the mock feels unnecessary when the original production implementation can work just fine.
-        var typeCache = new TypeCache(new ReflectHelper());
-        Type type = typeof(DummyTestClassWithTestMethods);
-        MethodInfo methodInfo = type.GetMethod("TestMethodWithCustomProperty")!;
-        TestMethod testMethod = CreateTestMethod(methodInfo.Name, type.FullName!, "A", displayName: null);
-
-        TestContextImplementation testContext = CreateTestContextImplementationForMethod(testMethod);
-
-        typeCache.GetTestMethodInfo(testMethod, testContext);
-        KeyValuePair<string, object?> customProperty = testContext.Properties.FirstOrDefault(p => p.Key.Equals("WhoAmI", StringComparison.Ordinal));
-
-        customProperty.Should().NotBeNull();
-        (customProperty.Value as string).Should().Be("Me");
-    }
-
-    public void GetTestMethodInfoShouldAllowActualOwnerAttribute()
-    {
-        // Test that the actual OwnerAttribute is allowed
-        var typeCache = new TypeCache(new ReflectHelper());
-        Type type = typeof(DummyTestClassWithTestMethods);
-        MethodInfo methodInfo = type.GetMethod("TestMethodWithActualOwnerAttribute")!;
-        TestMethod testMethod = CreateTestMethod(methodInfo.Name, type.FullName!, "A", displayName: null);
-        TestContextImplementation testContext = CreateTestContextImplementationForMethod(testMethod);
-
-        TestMethodInfo? testMethodInfo = typeCache.GetTestMethodInfo(testMethod, testContext);
-
-        testMethodInfo.Should().NotBeNull();
-
-        // The Owner property should be added to the test context
-        testContext.TryGetPropertyValue("Owner", out object? ownerValue).Should().BeTrue();
-        ownerValue?.ToString().Should().Be("TestOwner");
-    }
-
-    public void GetTestMethodInfoShouldAllowActualPriorityAttribute()
-    {
-        // Test that the actual PriorityAttribute is allowed
-        var typeCache = new TypeCache(new ReflectHelper());
-        Type type = typeof(DummyTestClassWithTestMethods);
-        MethodInfo methodInfo = type.GetMethod("TestMethodWithActualPriorityAttribute")!;
-        TestMethod testMethod = CreateTestMethod(methodInfo.Name, type.FullName!, "A", displayName: null);
-        TestContextImplementation testContext = CreateTestContextImplementationForMethod(testMethod);
-
-        TestMethodInfo? testMethodInfo = typeCache.GetTestMethodInfo(testMethod, testContext);
-
-        testMethodInfo.Should().NotBeNull();
-
-        // The Priority property should be added to the test context
-        testContext.TryGetPropertyValue("Priority", out object? priorityValue).Should().BeTrue();
-        priorityValue?.ToString().Should().Be("1");
-    }
-
-    public void GetTestMethodInfoShouldNotAddDuplicateTestPropertiesToTestContext()
-    {
-        // Not using _typeCache here which uses a mocked ReflectHelper which doesn't work well with this test.
-        // Setting up the mock feels unnecessary when the original production implementation can work just fine.
-        var typeCache = new TypeCache(new ReflectHelper());
-        Type type = typeof(DummyTestClassWithTestMethods);
-        MethodInfo methodInfo = type.GetMethod("TestMethodWithDuplicateCustomPropertyNames")!;
-        TestMethod testMethod = CreateTestMethod(methodInfo.Name, type.FullName!, "A", displayName: null);
-        TestContextImplementation testContext = CreateTestContextImplementationForMethod(testMethod);
-
-        TestMethodInfo? testMethodInfo = typeCache.GetTestMethodInfo(testMethod, testContext);
-
-        testMethodInfo.Should().NotBeNull();
-
-        // Verify that the first value gets set.
-        testContext.Properties.TryGetValue("WhoAmI", out object? value).Should().BeTrue();
-        value.Should().Be("Me");
-    }
-
     public void GetTestMethodInfoShouldReturnTestMethodInfoForDerivedTestClasses()
     {
         Type type = typeof(DerivedTestClass);
@@ -1275,12 +1202,6 @@ public void TestMethodWithTimeoutOfZero()
         {
         }
 
-        [TestMethod]
-        [TestProperty("WhoAmI", "Me")]
-        public void TestMethodWithCustomProperty()
-        {
-        }
-
         [TestMethod]
         [TestProperty("Owner", "You")]
         public void TestMethodWithOwnerAsCustomProperty()
@@ -1293,18 +1214,6 @@ public void TestMethodWithTestCategoryAsCustomProperty()
         {
         }
 
-        [TestMethod]
-        [Owner("TestOwner")]
-        public void TestMethodWithActualOwnerAttribute()
-        {
-        }
-
-        [TestMethod]
-        [Priority(1)]
-        public void TestMethodWithActualPriorityAttribute()
-        {
-        }
-
         [TestMethod]
         [TestProperty("Priority", "2")]
         public void TestMethodWithPriorityAsCustomProperty()

test/UnitTests/MSTestAdapter.PlatformServices.UnitTests/Execution/UnitTestRunnerTests.cs

@@ -109,30 +109,6 @@ public async Task RunSingleTestShouldReturnTestResultIndicateATestNotFoundIfTest
         results[0].IgnoreReason.Should().Be("Test method M was not found.");
     }
 
-    public async Task RunSingleTestShouldReturnTestResultIndicatingNotRunnableTestIfTestMethodCannotBeRun()
-    {
-        Type type = typeof(TypeCacheTests.DummyTestClassWithTestMethods);
-        MethodInfo methodInfo = type.GetMethod("TestMethodWithNullCustomPropertyName")!;
-        TestMethod testMethod = CreateTestMethod(methodInfo.Name, type.FullName!, "A", displayName: null);
-
-        _testablePlatformServiceProvider.MockFileOperations.Setup(fo => fo.LoadAssembly("A"))
-            .Returns(Assembly.GetExecutingAssembly());
-
-        UnitTestRunner unitTestRunner = CreateUnitTestRunner([new UnitTestElement(testMethod)]);
-        TestResult[] results = await unitTestRunner.RunSingleTestAsync(testMethod, _testRunParameters, null!);
-
-        string expectedMessage = string.Format(
-            CultureInfo.InvariantCulture,
-            "UTA021: {0}: Null or empty custom property defined on method {1}. The custom property must have a valid name.",
-            methodInfo.DeclaringType!.FullName,
-            methodInfo.Name);
-
-        results.Should().NotBeNull();
-        results.Length.Should().Be(1);
-        results[0].Outcome.Should().Be(UTF.UnitTestOutcome.NotRunnable);
-        results[0].IgnoreReason.Should().Be(expectedMessage);
-    }
-
     public async Task ExecuteShouldSkipTestAndFillInClassIgnoreMessageIfIgnoreAttributeIsPresentOnTestClassAndHasMessage()
     {
         Type type = typeof(TypeCacheTests.DummyTestClassWithIgnoreClassWithMessage);