Update mstest monorepo to 3.11.0 (#1508)

* Update mstest monorepo to 3.11.0

* Remove DataTestMethod attribute

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Jamie Magee <[email protected]>

Author: renovate[bot]

Directory.Packages.props

@@ -21,9 +21,9 @@
     <PackageVersion Include="MinVer" Version="5.0.0" />
     <PackageVersion Include="Moq" Version="4.18.4" />
     <PackageVersion Include="morelinq" Version="4.4.0" />
-    <PackageVersion Include="MSTest.TestFramework" Version="3.6.1" />
-    <PackageVersion Include="MSTest.Analyzers" Version="3.6.1" />
-    <PackageVersion Include="MSTest.TestAdapter" Version="3.6.1" />
+    <PackageVersion Include="MSTest.TestFramework" Version="3.11.0" />
+    <PackageVersion Include="MSTest.Analyzers" Version="3.11.0" />
+    <PackageVersion Include="MSTest.TestAdapter" Version="3.11.0" />
     <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.14.1" />
     <PackageVersion Include="Newtonsoft.Json" Version="13.0.4" />
     <PackageVersion Include="Newtonsoft.Json.Schema" Version="3.0.16" />

test/Microsoft.ComponentDetection.Detectors.Tests/GoComponentDetectorTests.cs

@@ -875,7 +875,7 @@ public async Task GoModDetector_GoModFileFound_GoModParserIsExecuted()
     /// Verifies that if Go CLI is enabled/available and succeeds, go.sum file is not parsed and vice-versa.
     /// </summary>
     /// <returns>Task.</returns>
-    [DataTestMethod]
+    [TestMethod]
     [DataRow(true)]
     [DataRow(false)]
     public async Task GoDetector_GoSum_GoSumParserExecuted(bool goCliSucceeds)

test/Microsoft.ComponentDetection.Detectors.Tests/VcpkgComponentDetectorTests.cs

@@ -180,7 +180,6 @@ public async Task TestInvalidFileAsync()
     }
 
     [TestMethod]
-    [DataTestMethod]
     [DataRow("vcpkg_installed\\manifest-info.json", "vcpkg.json")]
     [DataRow("vcpkg_installed\\vcpkg\\manifest-info.json", "vcpkg.json")]
     [DataRow("bad_location\\manifest-info.json", "vcpkg_installed\\packageLocation\\vcpkg.spdx.json")]

test/Microsoft.ComponentDetection.Orchestrator.Tests/Commands/ScanSettingsTests.cs

@@ -63,7 +63,7 @@ public void CanSerialize()
         action.Should().NotThrow();
     }
 
-    [DataTestMethod]
+    [TestMethod]
     [DataRow(-1)]
     [DataRow(0)]
     public void Validate_FailInvalidMaxThreads(int? input)
@@ -79,7 +79,7 @@ public void Validate_FailInvalidMaxThreads(int? input)
         result.Successful.Should().BeFalse();
     }
 
-    [DataTestMethod]
+    [TestMethod]
     [DataRow(null)]
     [DataRow(1)]
     [DataRow(99)]

test/Microsoft.ComponentDetection.Orchestrator.Tests/Services/DetectorProcessingServiceTests.cs

@@ -294,7 +294,7 @@ public async Task ProcessDetectorsAsync_ExperimentalDetectorsThrowingDoesntKillD
         experimentalDetectorRecord.DetectedComponentCount.Should().Be(0);
         experimentalDetectorRecord.IsExperimental.Should().BeTrue();
         experimentalDetectorRecord.ReturnCode.Should().Be((int)ProcessingResultCode.InputError);
-        experimentalDetectorRecord.ExperimentalInformation.Contains("Simulated experimental failure");
+        experimentalDetectorRecord.ExperimentalInformation.Should().Contain("Simulated experimental failure");
 
         // We should have all components except the ones that came from our experimental detector
         this.GetDiscoveredComponentsFromDetectorProcessingResult(results).Should().HaveCount(records.Sum(x => x.DetectedComponentCount ?? 0));