@khanacademy/perseus-editor 2.10.0 → 2.11.0

package/package.json

@@ -3,7 +3,7 @@
     "description": "Perseus editors",
     "author": "Khan Academy",
     "license": "MIT",
-    "version": "2.10.0",
+    "version": "2.11.0",
     "publishConfig": {
         "access": "public"
     },
@@ -24,7 +24,7 @@
     "dependencies": {
         "@khanacademy/kas": "^0.3.5",
         "@khanacademy/kmath": "^0.1.6",
-        "@khanacademy/perseus": "^13.2.0",
+        "@khanacademy/perseus": "^14.0.0",
         "@khanacademy/perseus-core": "1.3.0"
     },
     "devDependencies": {

package/src/widgets/__tests__/expression-editor.test.tsx

@@ -0,0 +1,56 @@
+import {Dependencies} from "@khanacademy/perseus";
+import {render, screen} from "@testing-library/react";
+import * as React from "react";
+
+import "@testing-library/jest-dom";
+
+import {testDependencies} from "../../../../../testing/test-dependencies";
+import ExpressionEditor from "../expression-editor";
+
+import type {PropsFor} from "@khanacademy/wonder-blocks-core";
+
+describe("expression-editor", () => {
+    beforeEach(() => {
+        jest.spyOn(Dependencies, "getDependencies").mockReturnValue(
+            testDependencies,
+        );
+    });
+
+    it("should render", async () => {
+        render(<ExpressionEditor onChange={() => undefined} />);
+
+        expect(await screen.findByText(/Add new answer/)).toBeInTheDocument();
+    });
+
+    it("should render answerForms missing a key", async () => {
+        const answerForms: PropsFor<typeof ExpressionEditor>["answerForms"] = [
+            {
+                value: "x\\cdot3=y",
+                form: true,
+                simplify: true,
+                considered: "correct",
+            },
+            {
+                value: "x^2=y",
+                form: true,
+                simplify: true,
+                considered: "wrong",
+            },
+            {
+                value: "x=y\\cdotπ",
+                form: true,
+                simplify: true,
+                considered: "wrong",
+            },
+        ];
+
+        render(
+            <ExpressionEditor
+                onChange={() => undefined}
+                answerForms={answerForms}
+            />,
+        );
+
+        expect(await screen.findByText(/π/)).toBeInTheDocument();
+    });
+});

package/src/widgets/expression-editor.tsx

@@ -18,7 +18,6 @@ import SortableArea from "../components/sortable";
 import type {PerseusExpressionWidgetOptions} from "@khanacademy/perseus";
 
 const {InfoTip, PropCheckBox, TexButtons} = components;
-const {getDependencies} = Dependencies;
 
 type Props = {
     widgetId?: any;
@@ -38,11 +37,12 @@ type DefaultProps = {
 };
 
 const parseAnswerKey = ({key}: AnswerForm): number => {
-    const parsedKey = Number.parseInt(key ?? "");
-    if (Number.isNaN(parsedKey)) {
-        throw new Error(`Invalid answer key: ${key}`);
-    }
-    return parsedKey;
+    // We don't throw here because there is data stored in some
+    // exercises/articles where the answer forms don't have a key. If we throw,
+    // it blocks content editors from loading the page at all.
+    // TODO(Jeremy): find a way to handle these answer forms that are missing
+    // keys more gracefully.
+    return Number.parseInt(key ?? "");
 };
 
 // Pick a key that isn't currently used by an answer in answerForms
@@ -182,7 +182,7 @@ class ExpressionEditor extends React.Component<Props, State> {
             },
         );
 
-        const {TeX} = getDependencies(); // OldExpression only
+        const {TeX} = Dependencies.getDependencies(); // OldExpression only
 
         buttonSetChoices.splice(
             1,