From 61097c29436cbb3944cdffd2da5b07c6d61a1b30 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 13 Apr 2026 22:51:17 +0000 Subject: [PATCH] Add --result alias for --results command-line argument with tests Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/c1f21142-ee4f-4da2-9ab0-6b7bd285be14 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --- src/DemaConsulting.ReviewMark/Cli/Context.cs | 1 + .../Cli/ContextTests.cs | 25 +++++++++++++++++++ 2 files changed, 26 insertions(+) diff --git a/src/DemaConsulting.ReviewMark/Cli/Context.cs b/src/DemaConsulting.ReviewMark/Cli/Context.cs index f1e58f7..51380bb 100644 --- a/src/DemaConsulting.ReviewMark/Cli/Context.cs +++ b/src/DemaConsulting.ReviewMark/Cli/Context.cs @@ -347,6 +347,7 @@ private int ParseArgument(string arg, string[] args, int index) LogFile = GetRequiredStringArgument(arg, args, index, FilenameArgument); return index + 1; + case "--result": case "--results": ResultsFile = GetRequiredStringArgument(arg, args, index, "a results filename argument"); return index + 1; diff --git a/test/DemaConsulting.ReviewMark.Tests/Cli/ContextTests.cs b/test/DemaConsulting.ReviewMark.Tests/Cli/ContextTests.cs index 2c94bd3..7d88511 100644 --- a/test/DemaConsulting.ReviewMark.Tests/Cli/ContextTests.cs +++ b/test/DemaConsulting.ReviewMark.Tests/Cli/ContextTests.cs @@ -226,6 +226,31 @@ public void Context_Create_ResultsFlag_WithoutValue_ThrowsArgumentException() Assert.Contains("--results", exception.Message); } + /// + /// Test creating a context with the --result alias sets the results file. + /// + [TestMethod] + public void Context_Create_ResultAlias_SetsResultsFile() + { + // Act + using var context = Context.Create(["--result", "test.trx"]); + + // Assert — ResultsFile is set to the provided path and exit code is zero + Assert.AreEqual("test.trx", context.ResultsFile); + Assert.AreEqual(0, context.ExitCode); + } + + /// + /// Test creating a context with --result alias but no value throws exception. + /// + [TestMethod] + public void Context_Create_ResultAlias_WithoutValue_ThrowsArgumentException() + { + // Act & Assert + var exception = Assert.ThrowsExactly(() => Context.Create(["--result"])); + Assert.Contains("--result", exception.Message); + } + /// /// Test WriteLine writes to console output when not silent. ///