Removed loading of Discoverers while executing tests. (#955)

* Removed loading of Discoverers while executing tests. * Fixed failing tests.
microsoft · Jul 26, 2017 · 2c3a9f5 · 2c3a9f5
1 parent a771cd3
commit 2c3a9f5
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 13 deletions.
diff --git a/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/ExecutionManager.cs b/src/Microsoft.TestPlatform.CrossPlatEngine/Execution/ExecutionManager.cs
@@ -171,10 +171,6 @@ private void LoadExtensions()
                 // Load the extensions on creation so that we dont have to spend time during first execution.
                 EqtTrace.Verbose("TestExecutorService: Loading the extensions");
 
-                TestDiscoveryExtensionManager.LoadAndInitializeAllExtensions(false);
-
-                EqtTrace.Verbose("TestExecutorService: Loaded the discoverers");
-
                 TestExecutorExtensionManager.LoadAndInitializeAllExtensions(false);
 
                 EqtTrace.Verbose("TestExecutorService: Loaded the executors");

diff --git a/test/Microsoft.TestPlatform.CrossPlatEngine.UnitTests/Execution/ExecutionManagerTests.cs b/test/Microsoft.TestPlatform.CrossPlatEngine.UnitTests/Execution/ExecutionManagerTests.cs
@@ -65,15 +65,6 @@ public void InitializeShouldLoadAndInitializeAllExtension()
 
             Assert.IsNotNull(TestPluginCache.Instance.TestExtensions);
 
-            // Discoverers
-            Assert.IsTrue(TestPluginCache.Instance.TestExtensions.TestDiscoverers.Count > 0);
-            var allDiscoverers = TestDiscoveryExtensionManager.Create().Discoverers;
-
-            foreach (var discoverer in allDiscoverers)
-            {
-                Assert.IsTrue(discoverer.IsExtensionCreated);
-            }
-
             // Executors
             Assert.IsTrue(TestPluginCache.Instance.TestExtensions.TestExecutors.Count > 0);
             var allExecutors = TestExecutorExtensionManager.Create().TestExtensions;