Cleanup of redundant class (#1963)

## Why make this change?

- Cleanup CliBoolean class as it is not adding any value and creating
confusion with CliBool.cs

## How was this tested?
- Existing tests

Author: abhishekkumams

src/Cli.Tests/TestHelper.cs

@@ -1082,7 +1082,7 @@ public static InitOptions CreateBasicInitOptionsForMsSqlWithConfig(string? confi
                 hostMode: HostMode.Development,
                 corsOrigin: new List<string>(),
                 authenticationProvider: EasyAuthType.StaticWebApps.ToString(),
-                restRequestBodyStrict: CliBoolean.True,
+                restRequestBodyStrict: CliBool.True,
                 config: config);
         }
     }

src/Cli/Commands/InitOptions.cs

@@ -36,7 +36,7 @@ public InitOptions(
             bool graphqlDisabled = false,
             CliBool restEnabled = CliBool.None,
             CliBool graphqlEnabled = CliBool.None,
-            CliBoolean restRequestBodyStrict = CliBoolean.None,
+            CliBool restRequestBodyStrict = CliBool.None,
             string? config = null)
             : base(config)
         {
@@ -118,7 +118,7 @@ public InitOptions(
         // Since the rest.request-body-strict option does not have a default value, it is required to specify a value for this option if it is
         // included in the init command.
         [Option("rest.request-body-strict", Required = false, HelpText = "(Default: true) Allow extraneous fields in the request body for REST.")]
-        public CliBoolean RestRequestBodyStrict { get; }
+        public CliBool RestRequestBodyStrict { get; }
 
         public void Handler(ILogger logger, FileSystemRuntimeConfigLoader loader, IFileSystem fileSystem)
         {

src/Cli/ConfigGenerator.cs

@@ -142,7 +142,7 @@ public static bool TryCreateRuntimeConfig(InitOptions options, FileSystemRuntime
                         _logger.LogWarning("Configuration option --rest.path is not honored for cosmosdb_nosql since CosmosDB does not support REST.");
                     }
 
-                    if (options.RestRequestBodyStrict is not CliBoolean.None)
+                    if (options.RestRequestBodyStrict is not CliBool.None)
                     {
                         _logger.LogWarning("Configuration option --rest.request-body-strict is not honored for cosmosdb_nosql since CosmosDB does not support REST.");
                     }
@@ -232,7 +232,7 @@ public static bool TryCreateRuntimeConfig(InitOptions options, FileSystemRuntime
                 Schema: dabSchemaLink,
                 DataSource: dataSource,
                 Runtime: new(
-                    Rest: new(restEnabled, restPath ?? RestRuntimeOptions.DEFAULT_PATH, options.RestRequestBodyStrict is CliBoolean.False ? false : true),
+                    Rest: new(restEnabled, restPath ?? RestRuntimeOptions.DEFAULT_PATH, options.RestRequestBodyStrict is CliBool.False ? false : true),
                     GraphQL: new(graphQLEnabled, graphQLPath),
                     Host: new(
                         Cors: new(options.CorsOrigin?.ToArray() ?? Array.Empty<string>()),

src/Config/ObjectModel/CliBool.cs

@@ -3,6 +3,11 @@
 
 namespace Azure.DataApiBuilder.Config.ObjectModel;
 
+/// <summary>
+/// Class to represent boolean values in CLI. This is required over primitive boolean types because of the limitation of the CommandLineParser library
+/// where if a boolean option is included in the CLI command, it is set as true.
+/// Doesn't matter what value we specify for the option as that value is ignored.
+/// </summary>
 public enum CliBool
 {
     // The enum value None is required to determine whether a value was provided for a CLI option. In case the option is not included in the init command,