stop using GetValue<T>(name) as it throws when given symbol was not parsed

Author: adamsitnik

src/Cli/dotnet/Commands/Test/ValidationUtility.cs

@@ -47,9 +47,8 @@ static void ValidateOptionsIrrelevantToModulesFilter(ParseResult parseResult)
                 parseResult.HasOption(TestingPlatformOptions.ConfigurationOption) ||
                 parseResult.HasOption(TestingPlatformOptions.FrameworkOption) ||
                 parseResult.HasOption(CommonOptions.OperatingSystemOption) ||
-                ((parseResult.GetValue<string>(CommonOptions.RuntimeOptionLongName) ??
-                 parseResult.GetValue<string>(CommonOptions.RuntimeOptionShortName)) is not null)
-            )
+                parseResult.HasOption(CommonOptions.RuntimeOptionLongName) ||
+                parseResult.HasOption(CommonOptions.RuntimeOptionShortName))
             {
                 throw new GracefulException(CliCommandStrings.CmdOptionCannotBeUsedWithTestModulesDescription);
             }

src/Cli/dotnet/Commands/Tool/Common/ToolAppliedOption.cs

@@ -67,7 +67,7 @@ internal static void EnsureNoConflictGlobalLocalToolPathOption(
             options.Add(LocalOptionName);
         }
 
-        if (!string.IsNullOrWhiteSpace(parseResult.GetValue<string>(ToolPathName)))
+        if (parseResult.RootCommandResult.GetResult(ToolPathName) is not null)
         {
             options.Add(ToolPathName);
         }
@@ -108,7 +108,7 @@ internal static void EnsureNoConflictUpdateAllVersionOption(
     internal static void EnsureToolManifestAndOnlyLocalFlagCombination(ParseResult parseResult)
     {
         if (GlobalOrToolPath(parseResult) &&
-            !string.IsNullOrWhiteSpace(parseResult.GetValue<string>(ToolManifestName)))
+            parseResult.RootCommandResult.GetResult(ToolManifestName) is not null)
         {
             throw new GracefulException(
                 string.Format(
@@ -117,10 +117,7 @@ internal static void EnsureToolManifestAndOnlyLocalFlagCombination(ParseResult p
     }
 
     private static bool GlobalOrToolPath(ParseResult parseResult)
-    {
-        return GlobalOptionWasProvided(parseResult) ||
-               !string.IsNullOrWhiteSpace(parseResult.GetValue<string>(ToolPathName));
-    }
+        => GlobalOptionWasProvided(parseResult) || parseResult.RootCommandResult.GetResult(ToolPathName) is not null;
 
     private static bool GlobalOptionWasProvided(ParseResult parseResult)
         => (parseResult.RootCommandResult.GetResult(GlobalOptionName) ?? parseResult.RootCommandResult.GetResult(GlobalOptionAlias)) is not null;

src/Cli/dotnet/Extensions/ParseResultExtensions.cs

@@ -273,4 +273,11 @@ public static T SafelyGetValueForOption<T>(this ParseResult parseResult, Option<
     /// This is useful for checking if the user has explicitly set an option, as opposed to it being set by default.
     /// </summary>
     public static bool HasOption(this ParseResult parseResult, Option option) => parseResult.GetResult(option) is OptionResult or && !or.Implicit;
+
+    /// <summary>
+    /// Checks if the option with given name/alias is present and not implicit (i.e. not set by default).
+    /// This is useful for checking if the user has explicitly set an option, as opposed to it being set by default.
+    /// </summary>
+    public static bool HasOption(this ParseResult parseResult, string nameOrAlias)
+        => parseResult.RootCommandResult.GetResult(nameOrAlias) is OptionResult or && !or.Implicit;
 }