@khanacademy/perseus 10.0.0 → 10.0.1

package/package.json

@@ -3,7 +3,7 @@
     "description": "Core Perseus API (includes renderers and widgets)",
     "author": "Khan Academy",
     "license": "MIT",
-    "version": "10.0.0",
+    "version": "10.0.1",
     "publishConfig": {
         "access": "public"
     },
@@ -23,7 +23,7 @@
     "dependencies": {
         "@khanacademy/kas": "^0.3.1",
         "@khanacademy/kmath": "^0.1.2",
-        "@khanacademy/math-input": "^10.0.1",
+        "@khanacademy/math-input": "^10.1.0",
         "@khanacademy/perseus-core": "0.2.0",
         "@khanacademy/perseus-linter": "^0.3.2",
         "@khanacademy/pure-markdown": "^0.2.4",

package/src/__tests__/mock-asset-loading-widget.tsx

@@ -2,9 +2,11 @@ import * as React from "react";
 
 import AssetContext from "../asset-context";
 
+import type {PerseusItem} from "../perseus-types";
 import type {WidgetExports} from "../types";
 
-export const mockedAssetItem = {
+// @ts-expect-error - TS2322 - Type '"mocked-asset-widget"' is not assignable to type '"categorizer" | "cs-program" | "definition" | "dropdown" | "example-graphie-widget" | "example-widget" | "explanation" | "expression" | "graded-group-set" | "graded-group" | ... 29 more ... | "video"'.
+export const mockedAssetItem: PerseusItem = {
     question: {
         content: "[[\u2603 mocked-asset-widget 1]]",
         images: Object.freeze({}),
@@ -27,7 +29,6 @@ export const mockedAssetItem = {
         zTable: false,
     },
     itemDataVersion: {major: 0, minor: 1},
-    // $FlowIgnore[signature-verification-failure]
     hints: [],
     _multi: null,
     answer: null,

package/src/__tests__/widgets.test.ts

@@ -0,0 +1,73 @@
+import {registerAllWidgetsForTesting} from "../util/register-all-widgets-for-testing";
+import * as Widgets from "../widgets";
+
+describe("Widget API support", () => {
+    beforeAll(() => {
+        registerAllWidgetsForTesting();
+    });
+
+    // This verifies a known list of widgets to ensure they provide the static
+    // validate function. Not all widgets support this function so even though
+    // this list looks exhaustive, it's not!
+    it.each([
+        "radio",
+        "input-number",
+        "numeric-input",
+        "expression",
+        "categorizer",
+        "cs-program",
+        "dropdown",
+        "explanation",
+        "definition",
+        "grapher",
+        "iframe",
+        "image",
+        "interaction",
+        "interactive-graph",
+        "label-image",
+        "matrix",
+        "matcher",
+        "measurer",
+        "number-line",
+        "orderer",
+        "passage",
+        "passage-ref",
+        "passage-ref-target",
+        "plotter",
+        "simulator",
+        "sorter",
+        "table",
+        "transformer",
+        "unit-input",
+        "video",
+    ])("%s widget should provide static validate function", (widgetType) => {
+        expect(Widgets.getWidget(widgetType)).toHaveProperty("validate");
+    });
+
+    // This list is mirrored in Khan Academy's webapp for the coach reports.
+    // If you change this, be sure to double-check that consuming applications
+    // agree on this list of supported widgets.
+    it.each([
+        "categorizer",
+        "expression",
+        "grapher",
+        "input-number",
+        "interactive-graph",
+        "numeric-input",
+        "radio",
+    ])(
+        "%s widget should provide static getUserInputFromProps function",
+        (widgetType) => {
+            const Widget = Widgets.getWidget(widgetType);
+            expect(Widget).toHaveProperty("getUserInputFromProps");
+        },
+    );
+
+    it.each(["expression", "input-number", "numeric-input"])(
+        "%s widget should provide static getOneCorrectAnswerFromRubric function",
+        (widgetType) => {
+            const Widget = Widgets.getWidget(widgetType);
+            expect(Widget).toHaveProperty("getOneCorrectAnswerFromRubric");
+        },
+    );
+});

package/src/widgets/expression.tsx

@@ -680,6 +680,19 @@ const ExpressionWithDependencies = React.forwardRef<
     return <Expression ref={ref} analytics={deps.analytics} {...props} />;
 });
 
+// HACK: Propogate "static" methods onto our wrapper component.
+// In the future we should adjust client apps to not depend on these static
+// methods and instead adjust Peresus to provide these facilities through
+// instance methods on our Renderers.
+// @ts-expect-error - TS2339 - Property 'validate' does not exist on type
+ExpressionWithDependencies.validate = Expression.validate;
+// @ts-expect-error - TS2339 - Property 'validate' does not exist on type
+ExpressionWithDependencies.getUserInputFromProps =
+    Expression.getUserInputFromProps;
+// @ts-expect-error - TS2339 - Property 'validate' does not exist on type
+ExpressionWithDependencies.getOneCorrectAnswerFromRubric =
+    Expression.getOneCorrectAnswerFromRubric;
+
 export default {
     name: "expression",
     displayName: "Expression / Equation",