Skip to content

Commit

Permalink
Disable PreEnumerateTheories in default TestAssemblyConfiguration for…
Browse files Browse the repository at this point in the history
… iOS/Android (#676)

It's already disabled for Browser and according to xunit/xunit#1746 (comment) the theory pre-enumeration only makes sense for UI scenarios like VS test explorer, which doesn't apply for xharness.
  • Loading branch information
akoeplinger authored Jul 14, 2021
1 parent b6c2a25 commit e551148
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1081,7 +1081,7 @@ private async Task<XElement> Run(Assembly assembly, string assemblyPath)
{
using (var discoverySink = new TestDiscoverySink())
{
var configuration = GetConfiguration(assembly) ?? new TestAssemblyConfiguration();
var configuration = GetConfiguration(assembly) ?? new TestAssemblyConfiguration() { PreEnumerateTheories = false };
ITestFrameworkDiscoveryOptions discoveryOptions = GetFrameworkOptionsForDiscovery(configuration);
discoveryOptions.SetSynchronousMessageReporting(true);
Logger.OnDebug($"Starting test discovery in the '{assembly}' assembly");
Expand Down

0 comments on commit e551148

Please sign in to comment.