From 349eac1e22584fe34830f6e5ee075ed27ed00311 Mon Sep 17 00:00:00 2001 From: Frank Ray <52075808+FrankRay78@users.noreply.github.com> Date: Fri, 11 Oct 2024 17:41:19 +0100 Subject: [PATCH] Bug fix: Tests with remaining args should not be expected to pass strict parsing --- .../Unit/CommandAppTests.Version.cs | 27 +++++++------------ 1 file changed, 9 insertions(+), 18 deletions(-) diff --git a/src/Tests/Spectre.Console.Cli.Tests/Unit/CommandAppTests.Version.cs b/src/Tests/Spectre.Console.Cli.Tests/Unit/CommandAppTests.Version.cs index 995b532..ec01092 100644 --- a/src/Tests/Spectre.Console.Cli.Tests/Unit/CommandAppTests.Version.cs +++ b/src/Tests/Spectre.Console.Cli.Tests/Unit/CommandAppTests.Version.cs @@ -91,17 +91,14 @@ public sealed partial class CommandAppTests } [Theory] - [InlineData("-v", false)] - [InlineData("-v", true)] - [InlineData("--version", false)] - [InlineData("--version", true)] - public void Should_Execute_Command_Not_Output_Application_Version_To_The_Console(string versionOption, bool strictParsing) + [InlineData("-v")] + [InlineData("--version")] + public void Should_Execute_Command_Not_Output_Application_Version_To_The_Console(string versionOption) { // Given var fixture = new CommandAppTester(); fixture.Configure(configurator => { - configurator.Settings.StrictParsing = strictParsing; configurator.SetApplicationVersion("1.0"); configurator.AddCommand("empty"); }); @@ -141,17 +138,14 @@ public sealed partial class CommandAppTests } [Theory] - [InlineData("-v", false)] - [InlineData("-v", true)] - [InlineData("--version", false)] - [InlineData("--version", true)] - public void Should_Execute_Branch_Default_Command_Not_Output_Application_Version_To_The_Console(string versionOption, bool strictParsing) + [InlineData("-v")] + [InlineData("--version")] + public void Should_Execute_Branch_Default_Command_Not_Output_Application_Version_To_The_Console(string versionOption) { // Given var fixture = new CommandAppTester(); fixture.Configure(configurator => { - configurator.Settings.StrictParsing = strictParsing; configurator.SetApplicationVersion("1.0"); configurator.AddBranch("branch", branch => { @@ -168,17 +162,14 @@ public sealed partial class CommandAppTests } [Theory] - [InlineData("-v", false)] - [InlineData("-v", true)] - [InlineData("--version", false)] - [InlineData("--version", true)] - public void Should_Execute_Branch_Command_Not_Output_Application_Version_To_The_Console(string versionOption, bool strictParsing) + [InlineData("-v")] + [InlineData("--version")] + public void Should_Execute_Branch_Command_Not_Output_Application_Version_To_The_Console(string versionOption) { // Given var fixture = new CommandAppTester(); fixture.Configure(configurator => { - configurator.Settings.StrictParsing = strictParsing; configurator.SetApplicationVersion("1.0"); configurator.AddBranch("branch", branch => {