chore(flat-routes): add support for optional and escape route segments

Signed-off-by: Logan McAnsh <logan@mcan.sh>
remix-run · Dec 20, 2022 · 7e579db · 7e579db
1 parent 909b2d2
commit 7e579db
Show file tree

Hide file tree

Showing 3 changed files with 248 additions and 235 deletions.
diff --git a/packages/remix-dev/__tests__/flat-routes-test.ts b/packages/remix-dev/__tests__/flat-routes-test.ts
@@ -1,4 +1,8 @@
-import { createRoutePath } from "../config/flat-routes";
+import {
+  createRoutePath,
+  getRouteSegments,
+  isIndexRoute,
+} from "../config/flat-routes";
 
 describe("createRoutePath", () => {
   describe("creates proper route paths", () => {
@@ -10,6 +14,7 @@ describe("createRoutePath", () => {
       ["_auth", undefined],
       ["_landing.about", "about"],
       ["_landing._index", undefined],
+      ["_landing.index", "index"],
       ["_landing", undefined],
       ["app.calendar.$day", "app/calendar/:day"],
       ["app.calendar._index", "app/calendar"],
@@ -94,7 +99,11 @@ describe("createRoutePath", () => {
 
     for (let [input, expected] of tests) {
       it(`"${input}" -> "${expected}"`, () => {
-        expect(createRoutePath(input)).toBe(expected);
+        let segments = getRouteSegments(input);
+        let index = isIndexRoute(input);
+        let result = createRoutePath(segments, index);
+        let expectedPath = expected === undefined ? undefined : "/" + expected;
+        expect(result).toBe(expectedPath);
       });
     }
   });