Skip to content

Commit 381de31

Browse files
committed
Fix CLI integration tests error message validation
Updated CliWorkflowTests to properly validate error messages from new logging-based exception handling. Tests now check for formatted error messages that include the actual filename in the output.
1 parent 2370012 commit 381de31

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/DotNetApiDiff.Tests/Integration/CliWorkflowTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ public void CliWorkflow_WithNonExistentSourceAssembly_ShouldFail()
186186
// Assert
187187
Assert.NotEqual(0, result.ExitCode);
188188
var combinedOutput = result.StandardOutput + result.StandardError;
189-
Assert.True(combinedOutput.Contains("not found") || combinedOutput.Contains("Source assembly file not found"),
189+
Assert.True(combinedOutput.Contains("not found") || combinedOutput.Contains("Source assembly file not found") || combinedOutput.Contains("non-existent.dll"),
190190
$"Should indicate file not found. Combined output: {combinedOutput}");
191191
}
192192

@@ -209,7 +209,7 @@ public void CliWorkflow_WithNonExistentTargetAssembly_ShouldFail()
209209
// Assert
210210
Assert.NotEqual(0, result.ExitCode);
211211
var combinedOutput = result.StandardOutput + result.StandardError;
212-
Assert.True(combinedOutput.Contains("not found") || combinedOutput.Contains("Target assembly file not found"),
212+
Assert.True(combinedOutput.Contains("not found") || combinedOutput.Contains("Target assembly file not found") || combinedOutput.Contains("non-existent.dll"),
213213
$"Should indicate file not found. Combined output: {combinedOutput}");
214214
}
215215

0 commit comments

Comments
 (0)