test(roam): run vitest in ci

Author: mdroidian

.github/workflows/ci.yaml

@@ -34,6 +34,9 @@ jobs:
       - name: Check TypeScript Types
         run: npx turbo check-types
 
+      - name: Run Tests
+        run: npx turbo run test --filter=roam --ui stream
+
   lint-changed-files:
     if: github.event_name == 'pull_request'
     runs-on: ubuntu-latest

apps/roam/package.json

@@ -10,8 +10,8 @@
     "lint:fix": "eslint . --fix",
     "publish": "tsx scripts/publish.ts",
     "check-types": "tsc --noEmit --skipLibCheck",
-    "test": "vitest run src/utils/__tests__",
-    "test:watch": "vitest src/utils/__tests__"
+    "test": "vitest run --config vitest.config.mts",
+    "test:watch": "vitest --config vitest.config.mts"
   },
   "license": "Apache-2.0",
   "devDependencies": {
@@ -21,6 +21,7 @@
     "@repo/typescript-config": "workspace:*",
     "@types/file-saver": "2.0.5",
     "@types/nanoid": "2.0.0",
+    "@types/node": "^20",
     "@types/react": "catalog:roam",
     "@types/react-dom": "catalog:roam",
     "@types/react-vertical-timeline-component": "^3.3.3",

apps/roam/src/utils/__tests__/datalogUtils.test.ts

@@ -72,7 +72,7 @@ describe("replaceDatalogVariables", () => {
       [
         {
           type: "fn-expr",
-          fn: "identity",
+          fn: "get",
           arguments: [{ type: "variable", value: "node" }],
           binding: {
             type: "bind-scalar",

apps/roam/src/utils/__tests__/fireQuery.test.ts

@@ -1,7 +1,12 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
 
+type ConditionToDatalogArgs = {
+  source: string;
+  target: string;
+};
+
 vi.mock("~/utils/conditionToDatalog", () => ({
-  default: vi.fn(({ source, target }) => [
+  default: vi.fn(({ source, target }: ConditionToDatalogArgs) => [
     {
       type: "data-pattern",
       arguments: [
@@ -73,15 +78,21 @@ describe("fireQuery", () => {
               q: vi
                 .fn()
                 .mockResolvedValue([
-                  [{ ":node/title": "Local", ":block/uid": "l1" }],
+                  [
+                    { ":node/title": "Local", ":block/uid": "l1" },
+                    { ":block/uid": "l1" },
+                  ],
                 ]),
             },
           },
           backend: {
             q: vi
               .fn()
               .mockResolvedValue([
-                [{ ":node/title": "Remote", ":block/uid": "r1" }],
+                [
+                  { ":node/title": "Remote", ":block/uid": "r1" },
+                  { ":block/uid": "r1" },
+                ],
               ]),
           },
           fast: {

apps/roam/src/utils/__tests__/queryParsing.test.ts

@@ -1,11 +1,21 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
 
+type SettingsTreeNode = {
+  text: string;
+  children?: SettingsTreeNode[];
+};
+
+type GetSettingValueArgs = {
+  tree?: SettingsTreeNode[];
+  key: string;
+};
+
 vi.mock("roamjs-components/util/getSubTree", () => ({
   default: vi.fn(),
 }));
 vi.mock("roamjs-components/util/getSettingValueFromTree", () => ({
   default: vi.fn(
-    ({ tree, key }) =>
+    ({ tree = [], key }: GetSettingValueArgs): string =>
       tree.find((t: { text: string }) => t.text === key)?.children?.[0]?.text ||
       "",
   ),
@@ -42,23 +52,36 @@ describe("parseQuery", () => {
       ],
     };
     mockedGetSubTree.mockImplementation(({ key }) => {
-      if (key === "conditions") return { uid: "conditions-uid", children: [] };
+      if (key === "conditions") {
+        return { uid: "conditions-uid", text: "conditions", children: [] };
+      }
       if (key === "selections") {
         return {
           uid: "selections-uid",
+          text: "selections",
           children: [
-            { uid: "sel-node", text: "node", children: [{ text: "Title" }] },
+            {
+              uid: "sel-node",
+              text: "node",
+              children: [{ uid: "title-label", text: "Title", children: [] }],
+            },
             {
               uid: "sel-created",
               text: "created",
-              children: [{ text: "Created" }],
+              children: [
+                { uid: "created-label", text: "Created", children: [] },
+              ],
             },
           ],
         };
       }
       return {
         uid: "custom-uid",
-        children: [{ text: "[:find ?x]" }, { text: "enabled" }],
+        text: "custom",
+        children: [
+          { uid: "custom-query", text: "[:find ?x]", children: [] },
+          { uid: "custom-enabled", text: "enabled", children: [] },
+        ],
       };
     });
 

apps/roam/vitest.config.mts

@@ -0,0 +1,17 @@
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import { defineConfig } from "vitest/config";
+
+const dirname = path.dirname(fileURLToPath(import.meta.url));
+
+export default defineConfig({
+  test: {
+    environment: "node",
+    include: ["src/utils/__tests__/**/*.test.ts"],
+  },
+  resolve: {
+    alias: {
+      "~": path.resolve(dirname, "src"),
+    },
+  },
+});