uppercase (#4852)

app/packages/state/src/utils.test.ts

@@ -0,0 +1,10 @@
+import { describe, expect, it } from "vitest";
+import { convertTargets } from "./utils";
+
+describe("convertTargets", () => {
+  it("upper cases rgb hex targets", () => {
+    expect(
+      convertTargets([{ target: "#ffffff", value: "white" }])
+    ).toStrictEqual({ "#FFFFFF": { label: "white", intTarget: 1 } });
+  });
+});

app/packages/state/src/utils.ts

@@ -113,22 +113,22 @@ export const getStandardizedUrls = (
   return Object.fromEntries(urls.map(({ field, url }) => [field, url]));
 };
 
-const convertTargets = (
+export const convertTargets = (
   targets: {
     target: string;
     value: string;
   }[]
-) => {
+): { [key: string]: { label: string; intTarget: number } | string } => {
   return Object.fromEntries(
     (targets || []).map(({ target, value }, i) => {
-      if (!isNaN(Number(target))) {
+      if (!Number.isNaN(Number(target))) {
         // masks targets is for non-rgb masks
         return [target, value];
       }
 
       // convert into RGB mask representation
       // offset of 1 in intTarget because 0 has a special significance
-      return [target, { label: value, intTarget: i + 1 }];
+      return [target.toUpperCase(), { label: value, intTarget: i + 1 }];
     })
   );
 };