Skip to content

Commit f61b53e

Browse files
committed
test: drop a redundant ToString in the weak-validator test
CodeQL cs/redundant-tostring-call (alert #309): string concatenation already calls ToString, so the explicit call and its null-forgiving operator were both noise.
1 parent 70b6f88 commit f61b53e

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tests/NeoReports.AspNetCore.IntegrationTests/ReportEditEndpointsTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -472,7 +472,7 @@ public async Task A_weak_validator_never_satisfies_if_match()
472472
HttpClient client = await CreateSalesAsync(host);
473473

474474
HttpResponseMessage config = await client.GetAsync("/api/reports/sales/config");
475-
string weak = "W/" + config.Headers.ETag!.ToString();
475+
string weak = $"W/{config.Headers.ETag}";
476476

477477
string edited = Original.Replace("\"pageSize\": 100", "\"pageSize\": 250", StringComparison.Ordinal);
478478
(await SendJsonAsync(client, HttpMethod.Put, "/api/reports/sales", edited, weak)).StatusCode

0 commit comments

Comments
 (0)