Skip to content

Commit 8e47a55

Browse files
bartlomiejuclaude
andcommitted
fix: update basePath validation test for URL round-trip check
The @ character is valid in URL paths, so it should no longer be rejected. Updated test to match the new URL-based validation. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 2382dc4 commit 8e47a55

1 file changed

Lines changed: 1 addition & 5 deletions

File tree

packages/fresh/tests/basepath_test.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,8 @@ Deno.test("basePath validation - rejects invalid paths", () => {
1111
'Invalid basePath: "/ui/". Must not end with "/" except for root path',
1212
);
1313

14-
expect(() => new App({ basePath: "/ui@admin" })).toThrow(
15-
'Invalid basePath: "/ui@admin". Contains invalid characters',
16-
);
17-
1814
expect(() => new App({ basePath: "/ui admin" })).toThrow(
19-
'Invalid basePath: "/ui admin". Contains invalid characters',
15+
'Invalid basePath: "/ui admin"',
2016
);
2117
});
2218

0 commit comments

Comments
 (0)