Skip to content

Commit

Permalink
[benc/split-number-line-widget] Create getNumberLinePublicWidgetOptions
Browse files Browse the repository at this point in the history
  • Loading branch information
benchristel committed Jan 31, 2025
1 parent 67e5e6c commit 8489c6b
Show file tree
Hide file tree
Showing 5 changed files with 71 additions and 1 deletion.
1 change: 1 addition & 0 deletions packages/perseus-core/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -117,3 +117,4 @@ export {default as getCategorizerPublicWidgetOptions} from "./widgets/categorize
export {default as getCSProgramPublicWidgetOptions} from "./widgets/cs-program/cs-program-util";
export {default as getExpressionPublicWidgetOptions} from "./widgets/expression/expression-util";
export {default as getLabelImagePublicWidgetOptions} from "./widgets/label-image/label-image-util";
export {default as getNumberLinePublicWidgetOptions} from "./widgets/number-line/number-line-util";
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
import getNumberLinePublicWidgetOptions from "./number-line-util";

import type {PerseusNumberLineWidgetOptions} from "../../data-schema"

describe("getNumberLinePublicWidgetOptions", () => {
it("removes correctX and correctRel", () => {
const options: PerseusNumberLineWidgetOptions = {
correctRel: "lt",
correctX: 42,
range: [0, 100],
numDivisions: 5,
tickStep: 6,
divisionRange: [],
initialX: 3,
labelRange: [],
labelStyle: "",
labelTicks: false,
snapDivisions: 0,
isTickCtrl: false,
showTooltips: false,
static: false,
};

const publicOptions = getNumberLinePublicWidgetOptions(options);

expect(publicOptions).toEqual({
range: [0, 100],
numDivisions: 5,
tickStep: 6,
divisionRange: [],
initialX: 3,
labelRange: [],
labelStyle: "",
labelTicks: false,
snapDivisions: 0,
isTickCtrl: false,
showTooltips: false,
static: false,
})
})
})
24 changes: 24 additions & 0 deletions packages/perseus-core/src/widgets/number-line/number-line-util.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
import type {PerseusNumberLineWidgetOptions} from "../../data-schema";

type NumberLinePublicWidgetOptions = Pick<
PerseusNumberLineWidgetOptions,
| "range"
| "labelRange"
| "labelStyle"
| "labelTicks"
| "isTickCtrl"
| "divisionRange"
| "numDivisions"
| "snapDivisions"
| "tickStep"
| "initialX"
| "showTooltips"
| "static"
>;

export default function getNumberLinePublicWidgetOptions(
options: PerseusNumberLineWidgetOptions,
): NumberLinePublicWidgetOptions {
const {correctX: _, correctRel: __, ...publicOptions} = options;
return publicOptions;
}
4 changes: 3 additions & 1 deletion packages/perseus/src/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import type {
getCategorizerPublicWidgetOptions,
getCSProgramPublicWidgetOptions,
getExpressionPublicWidgetOptions,
getNumberLinePublicWidgetOptions,
} from "@khanacademy/perseus-core";
import type {LinterContextProps} from "@khanacademy/perseus-linter";
import type {
Expand Down Expand Up @@ -547,7 +548,8 @@ export type PublicWidgetOptionsFunction =
| typeof getOrdererPublicWidgetOptions
| typeof getExpressionPublicWidgetOptions
| typeof getLabelImagePublicWidgetOptions
| typeof getCSProgramPublicWidgetOptions;
| typeof getCSProgramPublicWidgetOptions
| typeof getNumberLinePublicWidgetOptions;

export type WidgetExports<
T extends React.ComponentType<any> & Widget = React.ComponentType<any>,
Expand Down
2 changes: 2 additions & 0 deletions packages/perseus/src/widgets/number-line/number-line.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import {number as knumber, KhanMath} from "@khanacademy/kmath";
import {getNumberLinePublicWidgetOptions} from "@khanacademy/perseus-core";
import {scoreNumberLine, validateNumberLine} from "@khanacademy/perseus-score";
import * as React from "react";
import ReactDOM from "react-dom";
Expand Down Expand Up @@ -808,4 +809,5 @@ export default {
// TODO(LEMS-2656): remove TS suppression
// @ts-expect-error: Type 'UserInput' is not assignable to type 'PerseusNumberLineUserInput'.
validator: validateNumberLine,
getPublicWidgetOptions: getNumberLinePublicWidgetOptions,
} satisfies WidgetExports<typeof NumberLine>;

0 comments on commit 8489c6b

Please sign in to comment.