Handle redundant 'configurationDone' requests in the DebugAdapter

Author: daviwil

src/PowerShellEditorServices.Protocol/Server/DebugAdapter.cs

@@ -122,10 +122,19 @@ protected async Task HandleConfigurationDoneRequest(
         {
             if (!string.IsNullOrEmpty(this.scriptPathToLaunch))
             {
-                // Configuration is done, launch the script
-                var nonAwaitedTask =
-                    this.LaunchScript(requestContext)
-                        .ConfigureAwait(false);
+                if (this.editorSession.PowerShellContext.SessionState == PowerShellContextState.Ready)
+                {
+                    // Configuration is done, launch the script
+                    var nonAwaitedTask =
+                        this.LaunchScript(requestContext)
+                            .ConfigureAwait(false);
+                }
+                else
+                {
+                    Logger.Write(
+                        LogLevel.Verbose,
+                        "configurationDone request called after script was already launched, skipping it.");
+                }
             }
 
             await requestContext.SendResult(null);

test/PowerShellEditorServices.Test.Host/DebugAdapterTests.cs

@@ -122,7 +122,6 @@ public async Task DebugAdapterReceivesOutputEvents()
         private async Task LaunchScript(string scriptPath)
         {
             await this.debugAdapterClient.LaunchScript(scriptPath);
-            await this.SendRequest(ConfigurationDoneRequest.Type, null);
         }
     }
 }