From f7d9135179ea632367323bc2f6a31eadd8de03cf Mon Sep 17 00:00:00 2001
From: Simon Cropp <simon.cropp@gmail.com>
Date: Thu, 23 May 2024 12:48:49 +1000
Subject: [PATCH] rawArgument in GetArgumentList cant be null

since it is always guarded in the preceding code
---
 .../Utilities/ArgumentProcessorUtilities.cs         | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/src/vstest.console/Processors/Utilities/ArgumentProcessorUtilities.cs b/src/vstest.console/Processors/Utilities/ArgumentProcessorUtilities.cs
index efac4c6614..a5744648cd 100644
--- a/src/vstest.console/Processors/Utilities/ArgumentProcessorUtilities.cs
+++ b/src/vstest.console/Processors/Utilities/ArgumentProcessorUtilities.cs
@@ -18,12 +18,19 @@ internal class ArgumentProcessorUtilities
     /// <param name="argumentSeparator">Argument separator.</param>
     /// <param name="exceptionMessage">Exception Message.</param>
     /// <returns>Argument list.</returns>
-    public static string[] GetArgumentList(string? rawArgument, char[] argumentSeparator, string exceptionMessage)
+    public static string[] GetArgumentList(string rawArgument, char[] argumentSeparator, string exceptionMessage)
     {
-        var argumentList = rawArgument?.Split(argumentSeparator, StringSplitOptions.RemoveEmptyEntries);
+        var argumentList = rawArgument.Split(argumentSeparator, StringSplitOptions.RemoveEmptyEntries);
 
         // Throw error in case of invalid argument.
-        return argumentList == null || argumentList.Length <= 0 ? throw new CommandLineException(exceptionMessage) : argumentList;
+        if (argumentList.Length <= 0)
+        {
+            throw new CommandLineException(exceptionMessage);
+        }
+        else
+        {
+            return argumentList;
+        }
     }
 
     /// <summary>