Lambda-Associates/sql-utilities
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test(pgsql): wrap CommandVisitor asserts in Assert.Multiple

Browse Source 
Resolves SonarQube NUnit2045. The three independent parameter-index
assertions now report together instead of short-circuiting on the first
failure. The Is.Not.Null guard in the reflection helper stays outside the
multiple block because it gates the subsequent Invoke.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Thom Lamb
2026-05-20 13:02:28 -05:00
co-authored by Claude Opus 4.7
parent 5cb32d2311
commit 2fae7738e3
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/Strata.SqlTools.PostgreSql.Tests/PostgreSql/CommandVisitorTests.cs
+6 -3
View File
@@ -15,9 +15,12 @@ public class CommandVisitorTests
var param1b = InvokeFormatParameterName(visitor1, "p"); var param1b = InvokeFormatParameterName(visitor1, "p");
var param2a = InvokeFormatParameterName(visitor2, "p"); var param2a = InvokeFormatParameterName(visitor2, "p");
Assert.That(param1a, Is.EqualTo("$1"), "first visitor's first parameter should be $1"); Assert.Multiple(() =>
Assert.That(param1b, Is.EqualTo("$2"), "first visitor's second parameter should be $2"); {
Assert.That(param2a, Is.EqualTo("$1"), "second visitor must start at $1, not inherit visitor1's counter"); Assert.That(param1a, Is.EqualTo("$1"), "first visitor's first parameter should be $1");
Assert.That(param1b, Is.EqualTo("$2"), "first visitor's second parameter should be $2");
Assert.That(param2a, Is.EqualTo("$1"), "second visitor must start at $1, not inherit visitor1's counter");
});
} }
private static string InvokeFormatParameterName(CommandVisitor visitor, string name) private static string InvokeFormatParameterName(CommandVisitor visitor, string name)