more tests

Author: forman

dashi/src/lib/utils/mapObject.test.ts

@@ -0,0 +1,11 @@
+import { describe, it, expect } from "vitest";
+
+import { getValue, setValue, toObjPath } from "./objPath";
+import { mapObject } from "@/lib/utils/mapObject";
+
+describe("Test that mapObject()", () => {
+  it("works", () => {
+    const obj = { a: 1, b: "4", c: true };
+    expect(mapObject(obj, (x) => Number(x))).toEqual({ a: 1, b: 4, c: 1 });
+  });
+});

dashi/src/lib/utils/objPath.test.ts

@@ -1,6 +1,6 @@
 import { describe, it, expect } from "vitest";
 
-import { getValue, setValue } from "./objPath";
+import { getValue, setValue, toObjPath } from "./objPath";
 
 describe("Test that getValue()", () => {
   it("works on 0th level", () => {
@@ -71,3 +71,30 @@ describe("Test that setValue()", () => {
     expect(obj2).toEqual({ a: [1, 2, 3], b: "x" });
   });
 });
+
+describe("Test that toObjPath()", () => {
+  it("does not convert arrays", () => {
+    const value = [1, 2, 3];
+    expect(toObjPath(value)).toBe(value);
+    expect(toObjPath(value)).toEqual([1, 2, 3]);
+  });
+
+  it("converts undefined", () => {
+    expect(toObjPath(undefined)).toEqual([]);
+  });
+
+  it("converts null", () => {
+    expect(toObjPath(null)).toEqual([]);
+  });
+
+  it("converts numbers", () => {
+    expect(toObjPath(3)).toEqual([3]);
+  });
+
+  it("converts strings", () => {
+    expect(toObjPath("")).toEqual([]);
+    expect(toObjPath("colors")).toEqual(["colors"]);
+    expect(toObjPath("colors.6")).toEqual(["colors", 6]);
+    expect(toObjPath("colors.6.red")).toEqual(["colors", 6, "red"]);
+  });
+});

dashi/src/lib/utils/objPath.ts

@@ -66,19 +66,18 @@ export function toObjPath(
 ): ObjPath {
   if (Array.isArray(property)) {
     return property as ObjPath;
-  }
-  if (property === "" || !property) {
+  } else if (property === "" || !property) {
     return [];
-  }
-  if (typeof property === "number") {
+  } else if (typeof property === "number") {
     return [property];
-  }
-  const objPath: ObjPath = property.split(".");
-  for (let i = 0; i < objPath.length; i++) {
-    const index = Number(objPath[i]);
-    if (Number.isInteger(index)) {
-      objPath[i] = index;
+  } else {
+    const objPath: ObjPath = property.split(".");
+    for (let i = 0; i < objPath.length; i++) {
+      const index = Number(objPath[i]);
+      if (Number.isInteger(index)) {
+        objPath[i] = index;
+      }
     }
+    return objPath;
   }
-  return objPath;
 }