Replace lodash.invert with native code (#2830)

.changeset/flat-phones-reply.md

@@ -0,0 +1,5 @@
+---
+"victory-core": patch
+---
+
+Replace lodash.invert with native code

packages/victory-core/src/exports.test.ts

@@ -309,6 +309,7 @@ describe("victory-core", () => {
             "getRadius": [Function],
             "getRange": [Function],
             "getStyles": [Function],
+            "invert": [Function],
             "isFunction": [Function],
             "isHorizontal": [Function],
             "isNil": [Function],

packages/victory-core/src/victory-util/axis.tsx

@@ -1,5 +1,5 @@
 import React from "react";
-import { defaults, identity, isObject, invert, uniq, orderBy } from "lodash";
+import { defaults, identity, isObject, uniq, orderBy } from "lodash";
 import * as Collection from "./collection";
 import * as Domain from "./domain";
 import * as Helpers from "./helpers";
@@ -146,7 +146,7 @@ function getDefaultTickFormat(props) {
       ? (x, index) => tickValues[index]
       : fallbackFormat;
   }
-  const invertedStringMap = stringMap && invert(stringMap);
+  const invertedStringMap = stringMap && Helpers.invert(stringMap);
   const tickValueArray = orderBy(Object.values(stringMap), (n) => n);
   const dataNames = tickValueArray.map((tick: any) => invertedStringMap[tick]);
   // string ticks should have one tick of padding at the beginning
@@ -252,7 +252,7 @@ export function getTickFormat(props, scale) {
     return (x, index) => filteredTickFormat[index];
   } else if (tickFormat && Helpers.isFunction(tickFormat)) {
     const applyStringTicks = (tick, index, ticks) => {
-      const invertedStringMap = invert(stringMap);
+      const invertedStringMap = Helpers.invert(stringMap);
       const stringTickArray = ticks.map((t) => invertedStringMap[t]);
       return props.tickFormat(invertedStringMap[tick], index, stringTickArray);
     };

packages/victory-core/src/victory-util/helpers.test.ts

@@ -1,6 +1,20 @@
 import * as Helpers from "./helpers";
 
 describe("victory-util/helpers", () => {
+  describe("invert", () => {
+    it("inverts a given object", () => {
+      const data = { x: 3, y: "2", z: 1 };
+      const invertedData = Helpers.invert(data);
+      expect(invertedData).toEqual({ 3: "x", 2: "y", 1: "z" });
+    });
+
+    it("handles duplicate values by taking the last key", () => {
+      const data = { x: 3, y: 2, z: 1, a: 2 };
+      const invertedData = Helpers.invert(data);
+      expect(invertedData).toEqual({ 3: "x", 2: "a", 1: "z" });
+    });
+  });
+
   describe("omit", () => {
     const data = { x: 3, y: 2, z: 1 };
 

packages/victory-core/src/victory-util/helpers.ts

@@ -27,6 +27,17 @@ function getPolarRange(props, axis) {
 
 // Exported Functions
 
+/**
+ * Creates an object composed of the inverted keys and values of object.
+ * If object contains duplicate values, subsequent values overwrite property assignments of previous values.
+ */
+export function invert(original: Record<string, string | number>) {
+  return Object.entries(original).reduce((acc, current) => {
+    acc[current[1]] = current[0];
+    return acc;
+  }, {});
+}
+
 /**
  * creates an object with some keys excluded
  * replacement for lodash.omit for performance. does not mimic the entire lodash.omit api