how does InvalidOperationException result in "Skipping test adapter"

[SimonCropp]

in VSTestDiscoverTestExecutionRequestFactory.CreateRequestAsync
https://github.com/microsoft/testfx/blob/main/src/Platform/Microsoft.Testing.Extensions.VSTestBridge/Requests/VSTestDiscoverTestExecutionRequestFactory.cs#L55

there is this code

throw new InvalidOperationException($"Skipping test adapter {_testFrameworkAdapter.DisplayName} because it is not {nameof(IVSTestFlattenedTestNodesReportCapability)} capable.")

and i cant seem to understand how a InvalidOperationException is skipped?

Removing that throw passes all the test #4288

[Evangelink]

There are some issues there, thanks @SimonCropp!

1. Message is incorrect, it's not skipping the test framework, it's causing the test app to fail. This is a safeguard because the new platform supports tree based test structure but sadly we are limited by Test Explorer and so when using the VSTest bridge we do want the framework to provide a flattened list of tests (what VSTest and Test Explorer expect).

2. Message of exceptions should be localized

3. We don't have a test for this scenario