Skip to content

Cleanup tests #4406

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 19, 2023
Merged

Cleanup tests #4406

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions test/Microsoft.TestPlatform.Acceptance.IntegrationTests/DifferentTestFrameworkSimpleTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ public void CPPRunAllTestExecutionPlatformx64Net(RunnerInfo runnerInfo)
[TestMethod]
[TestCategory("Windows-Review")]
[NetFullTargetFrameworkDataSource]
// After the bump due to the new vesion it doesn't fine anymore a correct Microsoft.VisualStudio.Telemetry version, new version are taking newer versions of it and the test if for the legacy platform.
[Ignore("After the bump of TestPlatformRemoteExternalsVersion to 17.6 it doesn't work anymore, it's a test for the legacy platform and fails for wrong Microsoft.VisualStudio.Telemetry version.")]
public void WebTestRunAllTestsWithRunSettings(RunnerInfo runnerInfo)
{
Expand Down Expand Up @@ -129,7 +128,7 @@ public void WebTestRunAllTestsWithRunSettings(RunnerInfo runnerInfo)
[TestMethod]
[TestCategory("Windows-Review")]
[NetFullTargetFrameworkDataSource]
[Ignore("Test expected .NETFramework,Version=v4.5.2 support but the minimum one is .NETFramework,Version=v4.6.2")]
[Ignore("Test expects .NETFramework,Version=v4.5.2 support but the minimum one is .NETFramework,Version=v4.6.2")]
public void CodedWebTestRunAllTests(RunnerInfo runnerInfo)
{
if (!IsCI)
Expand Down