unit test

Author: sommeeeer

packages/open-next/src/adapters/config/index.ts

@@ -19,6 +19,7 @@ export const NEXT_DIR = path.join(__dirname, ".next");
 export const OPEN_NEXT_DIR = path.join(__dirname, ".open-next");
 
 debug({ NEXT_DIR, OPEN_NEXT_DIR });
+console.log(`Next.js directory: ${NEXT_DIR}`);
 
 //TODO: inject these values at build time
 export const NextConfig = /* @__PURE__ */ loadConfig(NEXT_DIR);

packages/open-next/src/core/routing/routeMatcher.ts

@@ -12,7 +12,7 @@ const optionalBasepathPrefixRegex = RoutesManifest.basePath
   : "^/";
 
 // Add the basePath prefix to the api routes
-export const apiPrefix = `${RoutesManifest.basePath ?? ""}/api`;
+const apiPrefix = `${RoutesManifest.basePath ?? ""}/api`;
 
 const optionalPrefix = optionalLocalePrefixRegex.replace(
   "^/",
@@ -68,7 +68,7 @@ export const dynamicRouteMatcher = routeMatcher(RoutesManifest.routes.dynamic);
  *
  * Without it handleFallbackFalse will 404 on static API routes if there is a catch-all route on root level.
  */
-export function getStaticAPIRoutes(): RouteDefinition[] {
+function getStaticAPIRoutes(): RouteDefinition[] {
   const createRouteDefinition = (route: string) => ({
     page: route,
     regex: `^${route}(?:/)?$`,

packages/tests-unit/tests/core/routing/matcher.test.ts

@@ -71,12 +71,26 @@ vi.mock("@opennextjs/aws/adapters/config/index.js", () => ({
       regex: "^/api/app(?:/)?$",
     },
   ],
+  NEXT_DIR:
+    "/home/opennextuser/coding/git/mynextproject/.open-next/server-functions/default/.next",
 }));
 vi.mock("@opennextjs/aws/core/routing/i18n/index.js", () => ({
   localizePath: (event: InternalEvent) => event.rawPath,
   handleLocaleRedirect: (_event: InternalEvent) => false,
 }));
 
+vi.mock("@opennextjs/aws/adapters/config/util.js", () => ({
+  loadPagesManifest: () => ({
+    "/_app": "pages/_app.js",
+    "/_document": "pages/_document.js",
+    "/api/hello": "pages/api/hello.js",
+    "/mypage": "pages/mypage.html",
+    "/_error": "pages/_error.js",
+    "/[[...subpage]]": "pages/[[...subpage]].js",
+    "/404": "pages/404.html",
+  }),
+}));
+
 type PartialEvent = Partial<
   Omit<InternalEvent, "body" | "rawPath" | "query">
 > & { body?: string };

packages/tests-unit/tests/core/routing/routeMatcher.test.ts

@@ -66,6 +66,25 @@ vi.mock("@opennextjs/aws/adapters/config/index.js", () => ({
       regex: "^/api/app(?:/)?$",
     },
   ],
+  NEXT_DIR:
+    "/home/opennextuser/coding/git/mynextproject/.open-next/server-functions/default/.next",
+  loadPagesManifest: () => ({
+    "/api/app": "pages/api/app.js",
+    "/page": "pages/page.js",
+    "/app": "app/page.js",
+  }),
+}));
+
+vi.mock("@opennextjs/aws/adapters/config/util.js", () => ({
+  loadPagesManifest: () => ({
+    "/_app": "pages/_app.js",
+    "/_document": "pages/_document.js",
+    "/api/hello": "pages/api/hello.js",
+    "/mypage": "pages/mypage.html",
+    "/_error": "pages/_error.js",
+    "/[[...subpage]]": "pages/[[...subpage]].js",
+    "/404": "pages/404.html",
+  }),
 }));
 
 describe("routeMatcher", () => {
@@ -92,6 +111,14 @@ describe("routeMatcher", () => {
           type: "route",
         },
       ]);
+
+      const helloRoute = staticRouteMatcher("/api/hello");
+      expect(helloRoute).toEqual([
+        {
+          route: "/api/hello",
+          type: "page",
+        },
+      ]);
     });
 
     it("should not match app dynamic route", () => {