@marimo-team/frontend 0.22.1-dev25 → 0.22.1-dev27

package/src/utils/__tests__/path.test.ts

@@ -170,36 +170,39 @@ describe("PathBuilder", () => {
   });
 
   describe("edge case filenames", () => {
-    it.each(
-      EDGE_CASE_FILENAMES,
-    )("should handle unicode and spaces in basename: %s", (filename) => {
-      const basename = Paths.basename(filename);
-      expect(basename).toBe(filename);
-      expect(typeof basename).toBe("string");
-      expect(basename).not.toBe("");
-    });
-
-    it.each(
-      EDGE_CASE_FILENAMES,
-    )("should handle unicode and spaces in dirname: %s", (filename) => {
-      const fullPath = `/path/to/${filename}`;
-      const dirname = Paths.dirname(fullPath);
-      expect(dirname).toBe("/path/to");
-    });
-
-    it.each(
-      EDGE_CASE_FILENAMES,
-    )("should handle unicode and spaces in path operations: %s", (filename) => {
-      const baseName = Paths.basename(filename);
-      const extension = Paths.extension(filename);
-
-      // Should preserve unicode characters in basename
-      expect(baseName).toContain(filename.split(".")[0]);
-
-      // Should correctly extract extension
-      if (filename.includes(".")) {
-        expect(extension).toBe(filename.split(".").pop());
-      }
-    });
+    it.each(EDGE_CASE_FILENAMES)(
+      "should handle unicode and spaces in basename: %s",
+      (filename) => {
+        const basename = Paths.basename(filename);
+        expect(basename).toBe(filename);
+        expect(typeof basename).toBe("string");
+        expect(basename).not.toBe("");
+      },
+    );
+
+    it.each(EDGE_CASE_FILENAMES)(
+      "should handle unicode and spaces in dirname: %s",
+      (filename) => {
+        const fullPath = `/path/to/${filename}`;
+        const dirname = Paths.dirname(fullPath);
+        expect(dirname).toBe("/path/to");
+      },
+    );
+
+    it.each(EDGE_CASE_FILENAMES)(
+      "should handle unicode and spaces in path operations: %s",
+      (filename) => {
+        const baseName = Paths.basename(filename);
+        const extension = Paths.extension(filename);
+
+        // Should preserve unicode characters in basename
+        expect(baseName).toContain(filename.split(".")[0]);
+
+        // Should correctly extract extension
+        if (filename.includes(".")) {
+          expect(extension).toBe(filename.split(".").pop());
+        }
+      },
+    );
   });
 });

package/src/utils/__tests__/urls.test.ts

@@ -14,24 +14,25 @@ describe("isUrl", () => {
 });
 
 describe("URL parameter handling with edge case filenames", () => {
-  it.each(
-    EDGE_CASE_FILENAMES,
-  )("should handle unicode filenames in URL parameters: %s", (filename) => {
-    // Test that updateQueryParams can handle unicode filenames
-    updateQueryParams((params) => {
-      params.set("file", filename);
-    });
-
-    // Verify URL encoding/decoding works with unicode
-    const encoded = encodeURIComponent(filename);
-    const decoded = decodeURIComponent(encoded);
-    expect(decoded).toBe(filename);
-
-    // Verify filename can be safely added to URL parameters
-    const url = new URL("https://example.com");
-    url.searchParams.set("file", filename);
-    expect(url.searchParams.get("file")).toBe(filename);
-  });
+  it.each(EDGE_CASE_FILENAMES)(
+    "should handle unicode filenames in URL parameters: %s",
+    (filename) => {
+      // Test that updateQueryParams can handle unicode filenames
+      updateQueryParams((params) => {
+        params.set("file", filename);
+      });
+
+      // Verify URL encoding/decoding works with unicode
+      const encoded = encodeURIComponent(filename);
+      const decoded = decodeURIComponent(encoded);
+      expect(decoded).toBe(filename);
+
+      // Verify filename can be safely added to URL parameters
+      const url = new URL("https://example.com");
+      url.searchParams.set("file", filename);
+      expect(url.searchParams.get("file")).toBe(filename);
+    },
+  );
 
   it("should preserve unicode in query string round-trip", () => {
     EDGE_CASE_FILENAMES.forEach((filename) => {