fix test and remove dynamic code

Author: TylerLeonhardt

src/PowerShellEditorServices/Services/PowerShellContext/Handlers/GetCommandHandler.cs

@@ -63,12 +63,12 @@ public async Task<List<PSCommandMessage>> Handle(GetCommandParams request, Cance
                 .AddCommand("Microsoft.PowerShell.Utility\\Sort-Object")
                     .AddParameter("Property", "Name");
 
-            IEnumerable<PSObject> result = await _powerShellContextService.ExecuteCommandAsync<PSObject>(psCommand).ConfigureAwait(false);
+            IEnumerable<CommandInfo> result = await _powerShellContextService.ExecuteCommandAsync<CommandInfo>(psCommand).ConfigureAwait(false);
 
             var commandList = new List<PSCommandMessage>();
             if (result != null)
             {
-                foreach (dynamic command in result)
+                foreach (CommandInfo command in result)
                 {
                     if (!_skipPackageManagementCheck && command.ModuleName == "PackageManagement" && command.Version < new Version(1, 4, 6))
                     {
@@ -129,13 +129,24 @@ public async Task<List<PSCommandMessage>> Handle(GetCommandParams request, Cance
                         }
                     }
 
+                    // Get the default ParameterSet
+                    string defaultParameterSet = null;
+                    foreach (CommandParameterSetInfo parameterSetInfo in command.ParameterSets)
+                    {
+                        if (parameterSetInfo.IsDefault)
+                        {
+                            defaultParameterSet = parameterSetInfo.Name;
+                            break;
+                        }
+                    }
+
                     commandList.Add(new PSCommandMessage
                     {
                         Name = command.Name,
                         ModuleName = command.ModuleName,
                         Parameters = command.Parameters,
                         ParameterSets = command.ParameterSets,
-                        DefaultParameterSet = command.DefaultParameterSet
+                        DefaultParameterSet = defaultParameterSet
                     });
                 }
             }

src/PowerShellEditorServices/Services/PowerShellContext/Utilities/CommandHelpers.cs

@@ -45,7 +45,7 @@ public static async Task<CommandInfo> GetCommandInfoAsync(
             Validate.IsNotNull(nameof(commandName), commandName);
             Validate.IsNotNull(nameof(powerShellContext), powerShellContext);
 
-            // Make sure the command's noun isn't blacklisted.  This is
+            // Make sure the command's noun isn't exclusion-listed.  This is
             // currently necessary to make sure that Get-Command doesn't
             // load PackageManagement or PowerShellGet because they cause
             // a major slowdown in IntelliSense.

test/PowerShellEditorServices.Test.E2E/LanguageServerProtocolMessageTests.cs

@@ -970,8 +970,8 @@ await LanguageClient.SendRequest<EvaluateResponseBody>(
         [Fact]
         public async Task CanSendGetCommandRequest()
         {
-            List<PSCommandMessage> pSCommandMessages =
-                await LanguageClient.SendRequest<List<PSCommandMessage>>("powerShell/getCommand", new GetCommandParams());
+            List<object> pSCommandMessages =
+                await LanguageClient.SendRequest<List<object>>("powerShell/getCommand", new GetCommandParams());
 
             Assert.NotEmpty(pSCommandMessages);
             // There should be at least 20 commands or so.