Skip to content

Commit 8d5a2a6

Browse files
Potential fix for pull request finding
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 09c73d3 commit 8d5a2a6

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

src/Service.Tests/UnitTests/DwSqlQueryBuilderUnitTests.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,11 +91,12 @@ public void DwSqlUpsert_UpdateBranch_AppliesPrimaryKeyAndUpdatePolicyPredicates(
9191

9292
// The SET clause must be followed by a WHERE before the IF/END block closes, i.e.
9393
// "UPDATE ... SET ... WHERE ..." rather than "UPDATE ... SET ... END".
94-
int setIdx = query.IndexOf("SET", StringComparison.Ordinal);
94+
int updateIdx = query.IndexOf("UPDATE", StringComparison.Ordinal);
95+
int setIdx = query.IndexOf("SET", updateIdx, StringComparison.Ordinal);
9596
int whereIdx = query.IndexOf("WHERE", setIdx, StringComparison.Ordinal);
96-
int endIdx = query.IndexOf("END", setIdx, StringComparison.Ordinal);
97-
Assert.IsTrue(whereIdx > setIdx && (endIdx == -1 || whereIdx < endIdx),
98-
$"UPDATE branch must contain a WHERE clause before the IF/END block terminates:{Environment.NewLine}{query}");
97+
int endIdx = query.IndexOf("END", updateIdx, StringComparison.Ordinal);
98+
Assert.IsTrue(setIdx > updateIdx && whereIdx > setIdx && (endIdx == -1 || whereIdx < endIdx),
99+
$"UPDATE branch must contain a WHERE clause after SET and before the IF/END block terminates:{Environment.NewLine}{query}");
99100
}
100101

101102
/// <summary>

0 commit comments

Comments
 (0)