Skip to content

Commit 859502b

Browse files
test: narrow API route assertion to /api/ prefix pattern
Avoid broad substring matches on controller names (chat, listingsourcecode, mcptoken) which could cause false failures if a legitimate content route ever contains those words. The /api/ prefix is the precise invariant.
1 parent 8ad5a85 commit 859502b

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

EssentialCSharp.Web.Tests/SitemapXmlHelpersTests.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,8 @@ public async Task GenerateSitemapXml_DoesNotIncludeApiRoutes()
4949

5050
var allUrls = nodes.Select(n => n.Url).ToList();
5151

52-
// Verify no API routes are included
52+
// Verify no API routes are included (assert on the /api/ pattern, not specific controller names)
5353
await Assert.That(allUrls).DoesNotContain(url => url.Contains("/api/", StringComparison.OrdinalIgnoreCase));
54-
await Assert.That(allUrls).DoesNotContain(url => url.Contains("chat", StringComparison.OrdinalIgnoreCase));
55-
await Assert.That(allUrls).DoesNotContain(url => url.Contains("listingsourcecode", StringComparison.OrdinalIgnoreCase));
56-
await Assert.That(allUrls).DoesNotContain(url => url.Contains("mcptoken", StringComparison.OrdinalIgnoreCase));
5754
}
5855

5956
[Test]

0 commit comments

Comments
 (0)