fix: update arrow head computed properties to use 'none' comparison and configure happy-dom test environment settings

gustavotoyota · gustavotoyota · commit 8dcf567031cd · 2026-06-01T00:54:11.000-03:00
diff --git a/new-deepnotes/apps/web/src/features/spatial/ArrowPropertiesCard.vue b/new-deepnotes/apps/web/src/features/spatial/ArrowPropertiesCard.vue
@@ -24,8 +24,8 @@ const emit = defineEmits({
 
 const bodyType = computed(() => props.arrowModel?.bodyType?.value ?? 'curve')
 const bodyStyle = computed(() => props.arrowModel?.bodyStyle?.value ?? 'solid')
-const sourceHead = computed(() => props.arrowModel?.sourceHead?.value ?? false)
-const targetHead = computed(() => props.arrowModel?.targetHead?.value ?? true)
+const sourceHead = computed(() => props.arrowModel?.sourceHead?.value !== 'none')
+const targetHead = computed(() => props.arrowModel?.targetHead?.value !== 'none')
 const color = computed(() => props.arrowModel?.color?.value ?? 0)
 const colorInherit = computed(() => props.arrowModel?.color?.inherit?.value ?? false)
 const readOnlyArrow = computed(() => props.arrowModel?.readOnly?.value ?? false)
diff --git a/new-deepnotes/apps/web/src/test/setup.ts b/new-deepnotes/apps/web/src/test/setup.ts
@@ -17,5 +17,11 @@ vi.mock("vue-router", async (importOriginal) => {
   };
 });
 
-// Set up proper HTML doctype for KaTeX
-document.documentElement.innerHTML = "<!DOCTYPE html><html><head></head><body></body></html>";
+// Suppress KaTeX quirks mode warning (happy-dom doesn't support doctype properly)
+const originalWarn = console.warn;
+console.warn = (...args) => {
+  if (typeof args[0] === 'string' && args[0].includes('KaTeX doesn\'t work in quirks mode')) {
+    return;
+  }
+  originalWarn(...args);
+};
diff --git a/new-deepnotes/apps/web/vite.config.ts b/new-deepnotes/apps/web/vite.config.ts
@@ -27,6 +27,15 @@ export default defineConfig({
   },
   test: {
     environment: "happy-dom",
+    environmentOptions: {
+      happyDOM: {
+        settings: {
+          disableJavaScriptFileLoading: true,
+          disableCSSFileLoading: true,
+          disableErrorCapturing: true,
+        },
+      },
+    },
     include: ["src/**/*.test.ts"],
     setupFiles: ["./src/test/setup.ts"],
     testTimeout: 120000,
