fix(🐛): Make getYForX() robust to x value outside the graph

wcandillon · Oct 9, 2020 · b959959 · b959959
1 parent 3bd75eb
commit b959959
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Paths.ts b/src/Paths.ts
@@ -235,7 +235,6 @@ export const interpolatePath = (
 /**
  * @summary Interpolate two paths with an animation value that goes from 0 to 1
  */
-
 export const mixPath = (
   value: number,
   p1: Path,
@@ -298,7 +297,7 @@ export const getYForX = (path: Path, x: number) => {
     }
     return false;
   });
-  if (isCurve(p[0])) {
+  if (p.length > 0 && isCurve(p[0])) {
     return cubicBezierYForX(x, p[0].from, p[0].c1, p[0].c2, p[0].to);
   }
   return 0;

diff --git a/src/__tests__/Paths.test.ts b/src/__tests__/Paths.test.ts
@@ -16,6 +16,7 @@ test("getYForX()", () => {
   );
   expect(getYForX(p1, 200)).toBe(75);
   expect(getYForX(p1, 50)).toBe(151.1683950839424);
+  expect(getYForX(p1, 750)).toBe(0);
 });
 
 test("getYForX2()", () => {