Remove constructor-based DML tool tests per reviewer feedback

Keep only E2E tests that verify CLI option name parsing via Program.Execute().

Author: souvikghosh04

src/Cli.Tests/ConfigureOptionsTests.cs

@@ -1081,96 +1081,6 @@ public void TestConfigureDescriptionForMcpSettings(string descriptionValue)
             Assert.AreEqual(descriptionValue, runtimeConfig.Runtime.Mcp.Description);
         }
 
-        /// <summary>
-        /// Tests that running "dab configure --runtime.mcp.dml-tools.{tool} {value}" updates
-        /// the individual DML tool boolean in the runtime config. Most tools are configured as
-        /// direct booleans in the schema (e.g., "describe-entities": true) rather than nested
-        /// objects with .enabled; "aggregate-records" additionally supports an object form
-        /// when specifying a query-timeout.
-        /// </summary>
-        [DataTestMethod]
-        [DataRow(true, DisplayName = "Enable individual DML tool: describe-entities")]
-        [DataRow(false, DisplayName = "Disable individual DML tool: describe-entities")]
-        public void TestConfigureIndividualDmlToolForMcpSettings(bool updatedValue)
-        {
-            // Arrange
-            SetupFileSystemWithInitialConfig(INITIAL_CONFIG);
-
-            // Act: Set describe-entities via the corrected option name (no .enabled suffix)
-            ConfigureOptions options = new(
-                runtimeMcpDmlToolsDescribeEntitiesEnabled: updatedValue,
-                config: TEST_RUNTIME_CONFIG_FILE
-            );
-            bool isSuccess = TryConfigureSettings(options, _runtimeConfigLoader!, _fileSystem!);
-
-            // Assert
-            Assert.IsTrue(isSuccess);
-            string updatedConfig = _fileSystem!.File.ReadAllText(TEST_RUNTIME_CONFIG_FILE);
-            Assert.IsTrue(RuntimeConfigLoader.TryParseConfig(updatedConfig, out RuntimeConfig? runtimeConfig));
-            Assert.IsNotNull(runtimeConfig.Runtime?.Mcp?.DmlTools);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.DescribeEntities);
-        }
-
-        /// <summary>
-        /// Tests that running "dab configure --runtime.mcp.dml-tools.enabled {value}" sets all
-        /// DML tools at once via the bulk toggle.
-        /// </summary>
-        [DataTestMethod]
-        [DataRow(true, DisplayName = "Enable all DML tools at once")]
-        [DataRow(false, DisplayName = "Disable all DML tools at once")]
-        public void TestConfigureAllDmlToolsForMcpSettings(bool updatedValue)
-        {
-            // Arrange
-            SetupFileSystemWithInitialConfig(INITIAL_CONFIG);
-
-            // Act: Set all tools via the bulk enabled toggle
-            ConfigureOptions options = new(
-                runtimeMcpDmlToolsEnabled: updatedValue,
-                config: TEST_RUNTIME_CONFIG_FILE
-            );
-            bool isSuccess = TryConfigureSettings(options, _runtimeConfigLoader!, _fileSystem!);
-
-            // Assert
-            Assert.IsTrue(isSuccess);
-            string updatedConfig = _fileSystem!.File.ReadAllText(TEST_RUNTIME_CONFIG_FILE);
-            Assert.IsTrue(RuntimeConfigLoader.TryParseConfig(updatedConfig, out RuntimeConfig? runtimeConfig));
-            Assert.IsNotNull(runtimeConfig.Runtime?.Mcp?.DmlTools);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.DescribeEntities);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.CreateRecord);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.ReadRecords);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.UpdateRecord);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.DeleteRecord);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.ExecuteEntity);
-            Assert.AreEqual(updatedValue, runtimeConfig.Runtime.Mcp.DmlTools.AggregateRecords);
-        }
-
-        /// <summary>
-        /// Tests that running "dab configure" with multiple individual DML tool options
-        /// correctly updates each tool independently.
-        /// </summary>
-        [TestMethod]
-        public void TestConfigureMultipleIndividualDmlToolsForMcpSettings()
-        {
-            // Arrange
-            SetupFileSystemWithInitialConfig(INITIAL_CONFIG);
-
-            // Act: Enable describe-entities, disable create-record
-            ConfigureOptions options = new(
-                runtimeMcpDmlToolsDescribeEntitiesEnabled: true,
-                runtimeMcpDmlToolsCreateRecordEnabled: false,
-                config: TEST_RUNTIME_CONFIG_FILE
-            );
-            bool isSuccess = TryConfigureSettings(options, _runtimeConfigLoader!, _fileSystem!);
-
-            // Assert
-            Assert.IsTrue(isSuccess);
-            string updatedConfig = _fileSystem!.File.ReadAllText(TEST_RUNTIME_CONFIG_FILE);
-            Assert.IsTrue(RuntimeConfigLoader.TryParseConfig(updatedConfig, out RuntimeConfig? runtimeConfig));
-            Assert.IsNotNull(runtimeConfig.Runtime?.Mcp?.DmlTools);
-            Assert.AreEqual(true, runtimeConfig.Runtime.Mcp.DmlTools.DescribeEntities);
-            Assert.AreEqual(false, runtimeConfig.Runtime.Mcp.DmlTools.CreateRecord);
-        }
-
         /// <summary>
         /// Validates that `dab configure --show-effective-permissions` correctly displays
         /// effective permissions without modifying the config file.