Merge pull request #1981 from paulvanbrenk/preUnitTest

cleanup namespaces

Author: paulvanbrenk

Common/Product/SharedProject/ProcessOutput.cs

@@ -640,10 +640,10 @@ public ProcessPriorityClass PriorityClass
         /// <param name="line"></param>
         public void WriteInputLine(string line)
         {
-            if (IsStarted && redirector != null && !redirector.CloseStandardInput())
+            if (this.IsStarted && this.redirector != null && !this.redirector.CloseStandardInput())
             {
-                Process.StandardInput.WriteLine(line);
-                Process.StandardInput.Flush();
+                this.Process.StandardInput.WriteLine(line);
+                this.Process.StandardInput.Flush();
             }
         }
 
@@ -824,11 +824,7 @@ private void OnExited(object sender, EventArgs e)
             }
             this.haveRaisedExitedEvent = true;
             FlushAndCloseOutput();
-            var evt = Exited;
-            if (evt != null)
-            {
-                evt(this, e);
-            }
+            Exited?.Invoke(this, e);
         }
 
         /// <summary>

Nodejs/Product/TestAdapterImpl/ServiceProviderExtension.cs

@@ -4,7 +4,7 @@
 using System.Globalization;
 using Microsoft.VisualStudio.TestPlatform.ObjectModel;
 
-namespace Microsoft.VisualStudioTools.TestAdapter
+namespace Microsoft.NodejsTools.TestAdapter
 {
     internal static class ServiceProviderExtensions
     {

Nodejs/Product/TestAdapterImpl/TestFileAddRemoveListener.cs

@@ -6,7 +6,7 @@
 using Microsoft.VisualStudio.Shell.Interop;
 using Microsoft.VisualStudio.TestPlatform.ObjectModel;
 
-namespace Microsoft.VisualStudioTools.TestAdapter
+namespace Microsoft.NodejsTools.TestAdapter
 {
     internal sealed class TestFileChangedEventArgs : EventArgs
     {
@@ -24,7 +24,7 @@ public TestFileChangedEventArgs(string file, WatcherChangeTypes reason, IVsProje
 
     internal sealed class TestFileAddRemoveListener : IVsTrackProjectDocumentsEvents2, IDisposable
     {
-        private IVsTrackProjectDocuments2 _projectDocTracker;
+        private readonly IVsTrackProjectDocuments2 _projectDocTracker;
         private uint _cookie = VSConstants.VSCOOKIE_NIL;
         private Guid _testProjectGuid;
 

Nodejs/Product/TestAdapterImpl/TestFilesUpdateWatcher.cs

@@ -8,7 +8,7 @@
 using Microsoft.VisualStudio.Shell.Interop;
 using Microsoft.VisualStudio.TestPlatform.ObjectModel;
 
-namespace Microsoft.VisualStudioTools.TestAdapter
+namespace Microsoft.NodejsTools.TestAdapter
 {
     // TODO: consider replacing this with Microsoft.VisualStudioTools.Project.FileChangeManager. (When we have an assembly we can use to share code)
 

Nodejs/Product/TestAdapterImpl/TestMethodResolver.cs

@@ -7,7 +7,6 @@
 using Microsoft.VisualStudio.Shell;
 using Microsoft.VisualStudio.Shell.Interop;
 using Microsoft.VisualStudio.TestWindow.Extensibility;
-using Microsoft.VisualStudioTools.TestAdapter;
 using MSBuild = Microsoft.Build.Evaluation;
 
 namespace Microsoft.NodejsTools.TestAdapter

Nodejs/Product/TestAdapterImpl/VsProjectExtensions.cs

@@ -3,14 +3,12 @@
 using System;
 using System.Collections.Generic;
 using System.Diagnostics;
-using Microsoft.NodejsTools;
 using Microsoft.VisualStudio;
-using Microsoft.VisualStudio.Shell;
 using Microsoft.VisualStudio.Shell.Flavor;
 using Microsoft.VisualStudio.Shell.Interop;
 using Microsoft.VisualStudio.TestPlatform.ObjectModel;
 
-namespace Microsoft.VisualStudioTools.TestAdapter
+namespace Microsoft.NodejsTools.TestAdapter
 {
     internal static class VSProjectExtensions
     {
@@ -100,12 +98,7 @@ out var extObject
             try
             {
                 var projHome = props.Item("ProjectHome");
-                if (projHome == null)
-                {
-                    return null;
-                }
-
-                return projHome.Value as string;
+                return projHome == null ? null : projHome.Value as string;
             }
             catch (ArgumentException)
             {