test: add treeshaking, improve CI (#1750)

Author: atilafassina

.github/workflows/server-function.yml

@@ -0,0 +1,53 @@
+name: 🧪 Tests
+
+on:
+  workflow_dispatch:
+  pull_request:
+    branches: [main]
+  push:
+    branches: [main]
+
+jobs:
+  tests:
+    name: "🧪 Tests"
+    runs-on: ubuntu-22.04
+    steps:
+      - uses: actions/checkout@v3
+      - uses: pnpm/action-setup@v4
+        with:
+          # https://github.com/pnpm/pnpm/issues/8953
+          version: 9.15.3
+
+      - name: Install project dependencies
+        run: pnpm i
+
+      - name: Build SolidStart
+        run: pnpm --filter @solidjs/start build
+
+      - name: Build Test Project
+        run: pnpm --filter tests build
+
+      - name: Run unit tests
+        run: pnpm --filter tests unit:ci
+
+      - name: Install Cypress dependencies
+        uses: cypress-io/github-action@v6
+        with:
+          working-directory: .
+          runTests: false
+
+      - name: E2E Firefox
+        uses: cypress-io/github-action@v6
+        with:
+          install: false
+          working-directory: packages/tests
+          start: pnpm start
+          browser: firefox
+
+      - name: E2E Chromium
+        uses: cypress-io/github-action@v6
+        with:
+          install: false
+          working-directory: packages/tests
+          start: pnpm start
+          browser: chromium

.github/workflows/test.yml

@@ -49,3 +49,6 @@ vite.config.ts.timestamp*
 
 .nitro
 .output
+
+# ignore cypress screenshots
+**/cypress/screenshots

.github/workflows/tests.yml

@@ -65,7 +65,7 @@
   "dependencies": {
     "@vinxi/plugin-directives": "^0.5.0",
     "@vinxi/server-components": "^0.5.0",
-    "@tanstack/server-functions-plugin": "^1.97.2",
+    "@tanstack/server-functions-plugin": "^1.97.19",
     "defu": "^6.1.2",
     "error-stack-parser": "^2.1.4",
     "html-to-image": "^1.11.11",

.gitignore

@@ -7,7 +7,7 @@ interface Route {
   children?: Route[];
   page?: boolean;
   $component?: any;
-	$HEAD?: any;
+  $HEAD?: any;
   $GET?: any;
   $POST?: any;
   $PUT?: any;
@@ -24,9 +24,7 @@ declare module "vinxi/routes" {
   }
 }
 
-export const pageRoutes = defineRoutes(
-  (fileRoutes as unknown as Route[]).filter(o => o.page)
-);
+export const pageRoutes = defineRoutes((fileRoutes as unknown as Route[]).filter(o => o.page));
 
 function defineRoutes(fileRoutes: Route[]) {
   function processRoute(routes: Route[], route: Route, id: string, full: string) {
@@ -35,7 +33,11 @@ function defineRoutes(fileRoutes: Route[]) {
     });
 
     if (!parentRoute) {
-      routes.push({ ...route, id, path: id.replace(/\/\([^)/]+\)/g, "").replace(/\([^)/]+\)/g, "") });
+      routes.push({
+        ...route,
+        id,
+        path: id.replace(/\/\([^)/]+\)/g, "").replace(/\([^)/]+\)/g, "")
+      });
       return routes;
     }
     processRoute(
@@ -58,7 +60,8 @@ function defineRoutes(fileRoutes: Route[]) {
 export function matchAPIRoute(path: string, method: string) {
   const match = router.lookup(path);
   if (match && match.route) {
-    const handler = method === "HEAD" ? match.route["$HEAD"] || match.route["$GET"] : match.route[`$${method}`];
+    const handler =
+      method === "HEAD" ? match.route["$HEAD"] || match.route["$GET"] : match.route[`$${method}`];
     if (handler === undefined) return;
     return {
       handler,
@@ -68,19 +71,34 @@ export function matchAPIRoute(path: string, method: string) {
 }
 
 function containsHTTP(route: Route) {
-  return route["$HEAD"] || route["$GET"] || route["$POST"] || route["$PUT"] || route["$PATCH"] || route["$DELETE"];
+  return (
+    route["$HEAD"] ||
+    route["$GET"] ||
+    route["$POST"] ||
+    route["$PUT"] ||
+    route["$PATCH"] ||
+    route["$DELETE"]
+  );
 }
 
 const router = createRouter({
   routes: (fileRoutes as unknown as Route[]).reduce((memo, route) => {
     if (!containsHTTP(route)) return memo;
-    let path = route.path.replace(/\/\([^)/]+\)/g, "").replace(/\([^)/]+\)/g, "").replace(/\*([^/]*)/g, (_, m) => `**:${m}`).split('/').map(s => (s.startsWith(':') || s.startsWith('*')) ? s : encodeURIComponent(s)).join('/');
+    let path = route.path
+      .replace(/\/\([^)/]+\)/g, "")
+      .replace(/\([^)/]+\)/g, "")
+      .replace(/\*([^/]*)/g, (_, m) => `**:${m}`)
+      .split("/")
+      .map(s => (s.startsWith(":") || s.startsWith("*") ? s : encodeURIComponent(s)))
+      .join("/");
     if (/:[^/]*\?/g.test(path)) {
       throw new Error(`Optional parameters are not supported in API routes: ${path}`);
     }
     if (memo[path]) {
       throw new Error(
-        `Duplicate API routes for "${path}" found at "${memo[path]!.route.path}" and "${route.path}"`
+        `Duplicate API routes for "${path}" found at "${memo[path]!.route.path}" and "${
+          route.path
+        }"`
       );
     }
     memo[path] = { route };