Merging the two test files for the same class and removing the redund…

…ant test.

test/vstest.console.UnitTests/TestPlatformHelpers/TestRequestManagerTests.cs

@@ -21,6 +21,7 @@ namespace vstest.console.UnitTests.TestPlatformHelpers
     using System.Threading;
     using System.Diagnostics;
     using Microsoft.VisualStudio.TestPlatform.ObjectModel;
+    using vstest.console.UnitTests.TestDoubles;
 
     [TestClass]
     public class TestRequestManagerTests
@@ -43,6 +44,42 @@ public void TestInit()
                 this.mockTestPlatformEventSource.Object);
         }
 
+        [TestCleanup]
+        public void Cleanup()
+        {
+            CommandLineOptions.Instance.Reset();
+        }
+
+        [TestMethod]
+        public void TestRequestManagerShouldInitializeConsoleLogger()
+        {
+            var mockLoggerEvents = new DummyLoggerEvents(TestSessionMessageLogger.Instance);
+            var mockLoggerManager = new DummyTestLoggerManager(mockLoggerEvents);
+            CommandLineOptions.Instance.IsDesignMode = false;
+            var requestManager = new TestRequestManager(CommandLineOptions.Instance,
+                new Mock<ITestPlatform>().Object,
+                mockLoggerManager,
+                TestRunResultAggregator.Instance,
+                new Mock<ITestPlatformEventSource>().Object);
+
+            Assert.IsTrue(mockLoggerEvents.EventsSubscribed());
+        }
+
+        [TestMethod]
+        public void TestRequestManagerShouldNotInitializeConsoleLoggerIfDesignModeIsSet()
+        {
+            var mockLoggerEvents = new DummyLoggerEvents(TestSessionMessageLogger.Instance);
+            var mockLoggerManager = new DummyTestLoggerManager(mockLoggerEvents);
+            CommandLineOptions.Instance.IsDesignMode = true;
+            var requestManager = new TestRequestManager(CommandLineOptions.Instance,
+                new Mock<ITestPlatform>().Object,
+                mockLoggerManager,
+                TestRunResultAggregator.Instance,
+                new Mock<ITestPlatformEventSource>().Object);
+
+            Assert.IsFalse(mockLoggerEvents.EventsSubscribed());
+        }
+
         [TestMethod]
         public void InitializeExtensionsShouldCallTestPlatformToInitialize()
         {