@pie-lib/math-toolbar 3.1.0-beta.3 → 3.1.0-beta.5

package/lib/__tests__/editor-and-pad-mathlive.test.js

@@ -0,0 +1,272 @@
+"use strict";
+
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
+var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
+var _editorAndPad = require("../editor-and-pad");
+function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
+function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2["default"])(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
+/**
+ * Covers the call sites that changed when this package moved from
+ * @pie-lib/math-input (MathQuill) to @pie-lib/math-input-mathlive.
+ *
+ * All three were MathQuill-shaped and silently changed meaning:
+ *  - shouldComponentUpdate read `this.input.mathField.latex()`, which throws now
+ *    (and the field loads asynchronously, so `this.input` can be null)
+ *  - the mixed-number workaround poked `__controller.cursor`, a MathQuill
+ *    internal with no MathLive equivalent
+ *  - onClick routes keypad presses through the imperative field API
+ */
+describe('EditorAndPad after the MathLive migration', function () {
+  var props = function props() {
+    var over = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+    return _objectSpread({
+      classes: {},
+      classNames: {},
+      onChange: jest.fn()
+    }, over);
+  };
+  describe('shouldComponentUpdate', function () {
+    it('treats a missing input ref as different, so the first render is not skipped', function () {
+      var c = new _editorAndPad.EditorAndPad(props({
+        latex: 'x'
+      }));
+      c.input = null;
+      expect(c.shouldComponentUpdate({
+        latex: 'x'
+      }, {})).toBe(true);
+    });
+    it('does not throw when the math engine has not loaded yet', function () {
+      var c = new _editorAndPad.EditorAndPad(props({
+        latex: 'x'
+      }));
+
+      // latex() returns '' until MathLive is ready
+      c.input = {
+        latex: function latex() {
+          return '';
+        }
+      };
+      expect(function () {
+        return c.shouldComponentUpdate({
+          latex: 'x'
+        }, {});
+      }).not.toThrow();
+      expect(c.shouldComponentUpdate({
+        latex: 'x'
+      }, {})).toBe(true);
+    });
+    it('skips the update when the field already holds the incoming latex', function () {
+      var c = new _editorAndPad.EditorAndPad(props({
+        latex: 'x',
+        keypadMode: 'a',
+        noDecimal: false
+      }));
+      c.input = {
+        latex: function latex() {
+          return 'x';
+        }
+      };
+      c.state = {
+        equationEditor: 'geometry'
+      };
+      expect(c.shouldComponentUpdate({
+        latex: 'x',
+        keypadMode: 'a',
+        noDecimal: false
+      }, c.state)).toBe(false);
+    });
+    it('re-renders when the field differs from the incoming latex', function () {
+      var c = new _editorAndPad.EditorAndPad(props({
+        latex: 'y'
+      }));
+      c.input = {
+        latex: function latex() {
+          return 'x';
+        }
+      };
+      expect(c.shouldComponentUpdate({
+        latex: 'y'
+      }, {})).toBe(true);
+    });
+    it('uses input.latex(), never input.mathField.latex()', function () {
+      var c = new _editorAndPad.EditorAndPad(props({
+        latex: 'x'
+      }));
+      var latex = jest.fn(function () {
+        return 'x';
+      });
+
+      // deliberately no `mathField` - the old code would have thrown here
+      c.input = {
+        latex: latex
+      };
+      c.state = {
+        equationEditor: 'geometry'
+      };
+      c.shouldComponentUpdate({
+        latex: 'x'
+      }, c.state);
+      expect(latex).toHaveBeenCalled();
+    });
+  });
+  describe('onClick key routing', function () {
+    var withInput = function withInput() {
+      var over = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+      var c = new _editorAndPad.EditorAndPad(props(over));
+      var calls = {
+        write: [],
+        command: [],
+        keystroke: [],
+        clear: 0
+      };
+      c.input = {
+        write: function write(v) {
+          return calls.write.push(v);
+        },
+        command: function command(v) {
+          return calls.command.push(v);
+        },
+        keystroke: function keystroke(v) {
+          return calls.keystroke.push(v);
+        },
+        clear: function clear() {
+          return calls.clear++;
+        }
+      };
+      c.updateDisable = function () {};
+      return {
+        c: c,
+        calls: calls
+      };
+    };
+
+    // toNodeData keys this off `value === 'clear'`, not `type`
+    it('clear', function () {
+      var _withInput = withInput(),
+        c = _withInput.c,
+        calls = _withInput.calls;
+      c.onClick({
+        value: 'clear'
+      });
+      expect(calls.clear).toBe(1);
+    });
+    it('command', function () {
+      var _withInput2 = withInput(),
+        c = _withInput2.c,
+        calls = _withInput2.calls;
+      c.onClick({
+        type: 'command',
+        value: '\\frac'
+      });
+      expect(calls.command).toEqual(['\\frac']);
+    });
+    it('cursor keystroke', function () {
+      var _withInput3 = withInput(),
+        c = _withInput3.c,
+        calls = _withInput3.calls;
+      c.onClick({
+        type: 'cursor',
+        value: 'Left'
+      });
+      expect(calls.keystroke).toEqual(['Left']);
+    });
+    it('answer block writes the response token', function () {
+      var _withInput4 = withInput(),
+        c = _withInput4.c,
+        calls = _withInput4.calls;
+      c.onClick({
+        type: 'answer'
+      });
+      expect(calls.write).toEqual(['%response%']);
+    });
+    it('plain value', function () {
+      var _withInput5 = withInput(),
+        c = _withInput5.c,
+        calls = _withInput5.calls;
+      c.onClick({
+        value: 'x'
+      });
+      expect(calls.write).toEqual(['x']);
+    });
+    it('ignores a nullish key', function () {
+      var _withInput6 = withInput(),
+        c = _withInput6.c,
+        calls = _withInput6.calls;
+      expect(function () {
+        return c.onClick(null);
+      }).not.toThrow();
+      expect(calls.write).toEqual([]);
+    });
+  });
+  describe('onEditorChange', function () {
+    var withInput = function withInput() {
+      var over = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+      var onChange = jest.fn();
+      var c = new _editorAndPad.EditorAndPad(props(_objectSpread({
+        onChange: onChange
+      }, over)));
+      var calls = {
+        keystroke: [],
+        clear: 0,
+        write: []
+      };
+      c.input = {
+        keystroke: function keystroke(v) {
+          return calls.keystroke.push(v);
+        },
+        clear: function clear() {
+          return calls.clear++;
+        },
+        write: function write(v) {
+          return calls.write.push(v);
+        }
+      };
+      c.updateDisable = function () {};
+      return {
+        c: c,
+        onChange: onChange,
+        calls: calls
+      };
+    };
+    it('propagates ordinary latex to onChange', function () {
+      var _withInput7 = withInput(),
+        c = _withInput7.c,
+        onChange = _withInput7.onChange;
+      c.onEditorChange('\\frac{1}{2}');
+      expect(onChange).toHaveBeenCalledWith('\\frac{1}{2}');
+    });
+    it('strips decimals when noDecimal is set', function () {
+      var _withInput8 = withInput({
+          noDecimal: true
+        }),
+        c = _withInput8.c,
+        onChange = _withInput8.onChange,
+        calls = _withInput8.calls;
+      c.onEditorChange('1.5');
+      expect(calls.clear).toBe(1);
+      expect(calls.write).toEqual(['15']);
+      expect(onChange).not.toHaveBeenCalled();
+    });
+    it('undoes the empty fraction after a digit with a public delete, not MathQuill internals', function () {
+      var _withInput9 = withInput(),
+        c = _withInput9.c,
+        onChange = _withInput9.onChange,
+        calls = _withInput9.calls;
+
+      // the shape the old __controller hack targeted
+      c.onEditorChange('2\\ \\frac{1}{ }');
+      expect(calls.keystroke).toEqual(['Backspace']);
+      expect(onChange).not.toHaveBeenCalled();
+    });
+    it('does not treat normal fractions as the mixed-number case', function () {
+      var _withInput0 = withInput(),
+        c = _withInput0.c,
+        onChange = _withInput0.onChange,
+        calls = _withInput0.calls;
+      c.onEditorChange('\\frac{1}{2}');
+      expect(calls.keystroke).toEqual([]);
+      expect(onChange).toHaveBeenCalled();
+    });
+  });
+});
+//# sourceMappingURL=editor-and-pad-mathlive.test.js.map

package/lib/__tests__/editor-and-pad-mathlive.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"editor-and-pad-mathlive.test.js","names":["_editorAndPad","require","ownKeys","e","r","t","Object","keys","getOwnPropertySymbols","o","filter","getOwnPropertyDescriptor","enumerable","push","apply","_objectSpread","arguments","length","forEach","_defineProperty2","getOwnPropertyDescriptors","defineProperties","defineProperty","describe","props","over","undefined","classes","classNames","onChange","jest","fn","it","c","EditorAndPad","latex","input","expect","shouldComponentUpdate","toBe","not","toThrow","keypadMode","noDecimal","state","equationEditor","toHaveBeenCalled","withInput","calls","write","command","keystroke","clear","v","updateDisable","_withInput","onClick","value","_withInput2","type","toEqual","_withInput3","_withInput4","_withInput5","_withInput6","_withInput7","onEditorChange","toHaveBeenCalledWith","_withInput8","_withInput9","_withInput0"],"sources":["../../src/__tests__/editor-and-pad-mathlive.test.js"],"sourcesContent":["import { EditorAndPad } from '../editor-and-pad';\n\n/**\n * Covers the call sites that changed when this package moved from\n * @pie-lib/math-input (MathQuill) to @pie-lib/math-input-mathlive.\n *\n * All three were MathQuill-shaped and silently changed meaning:\n *  - shouldComponentUpdate read `this.input.mathField.latex()`, which throws now\n *    (and the field loads asynchronously, so `this.input` can be null)\n *  - the mixed-number workaround poked `__controller.cursor`, a MathQuill\n *    internal with no MathLive equivalent\n *  - onClick routes keypad presses through the imperative field API\n */\ndescribe('EditorAndPad after the MathLive migration', () => {\n  const props = (over = {}) => ({ classes: {}, classNames: {}, onChange: jest.fn(), ...over });\n\n  describe('shouldComponentUpdate', () => {\n    it('treats a missing input ref as different, so the first render is not skipped', () => {\n      const c = new EditorAndPad(props({ latex: 'x' }));\n\n      c.input = null;\n\n      expect(c.shouldComponentUpdate({ latex: 'x' }, {})).toBe(true);\n    });\n\n    it('does not throw when the math engine has not loaded yet', () => {\n      const c = new EditorAndPad(props({ latex: 'x' }));\n\n      // latex() returns '' until MathLive is ready\n      c.input = { latex: () => '' };\n\n      expect(() => c.shouldComponentUpdate({ latex: 'x' }, {})).not.toThrow();\n      expect(c.shouldComponentUpdate({ latex: 'x' }, {})).toBe(true);\n    });\n\n    it('skips the update when the field already holds the incoming latex', () => {\n      const c = new EditorAndPad(props({ latex: 'x', keypadMode: 'a', noDecimal: false }));\n\n      c.input = { latex: () => 'x' };\n      c.state = { equationEditor: 'geometry' };\n\n      expect(c.shouldComponentUpdate({ latex: 'x', keypadMode: 'a', noDecimal: false }, c.state)).toBe(false);\n    });\n\n    it('re-renders when the field differs from the incoming latex', () => {\n      const c = new EditorAndPad(props({ latex: 'y' }));\n\n      c.input = { latex: () => 'x' };\n\n      expect(c.shouldComponentUpdate({ latex: 'y' }, {})).toBe(true);\n    });\n\n    it('uses input.latex(), never input.mathField.latex()', () => {\n      const c = new EditorAndPad(props({ latex: 'x' }));\n      const latex = jest.fn(() => 'x');\n\n      // deliberately no `mathField` - the old code would have thrown here\n      c.input = { latex };\n      c.state = { equationEditor: 'geometry' };\n      c.shouldComponentUpdate({ latex: 'x' }, c.state);\n\n      expect(latex).toHaveBeenCalled();\n    });\n  });\n\n  describe('onClick key routing', () => {\n    const withInput = (over = {}) => {\n      const c = new EditorAndPad(props(over));\n      const calls = { write: [], command: [], keystroke: [], clear: 0 };\n\n      c.input = {\n        write: (v) => calls.write.push(v),\n        command: (v) => calls.command.push(v),\n        keystroke: (v) => calls.keystroke.push(v),\n        clear: () => calls.clear++,\n      };\n      c.updateDisable = () => {};\n\n      return { c, calls };\n    };\n\n    // toNodeData keys this off `value === 'clear'`, not `type`\n    it('clear', () => {\n      const { c, calls } = withInput();\n\n      c.onClick({ value: 'clear' });\n      expect(calls.clear).toBe(1);\n    });\n\n    it('command', () => {\n      const { c, calls } = withInput();\n\n      c.onClick({ type: 'command', value: '\\\\frac' });\n      expect(calls.command).toEqual(['\\\\frac']);\n    });\n\n    it('cursor keystroke', () => {\n      const { c, calls } = withInput();\n\n      c.onClick({ type: 'cursor', value: 'Left' });\n      expect(calls.keystroke).toEqual(['Left']);\n    });\n\n    it('answer block writes the response token', () => {\n      const { c, calls } = withInput();\n\n      c.onClick({ type: 'answer' });\n      expect(calls.write).toEqual(['%response%']);\n    });\n\n    it('plain value', () => {\n      const { c, calls } = withInput();\n\n      c.onClick({ value: 'x' });\n      expect(calls.write).toEqual(['x']);\n    });\n\n    it('ignores a nullish key', () => {\n      const { c, calls } = withInput();\n\n      expect(() => c.onClick(null)).not.toThrow();\n      expect(calls.write).toEqual([]);\n    });\n  });\n\n  describe('onEditorChange', () => {\n    const withInput = (over = {}) => {\n      const onChange = jest.fn();\n      const c = new EditorAndPad(props({ onChange, ...over }));\n      const calls = { keystroke: [], clear: 0, write: [] };\n\n      c.input = {\n        keystroke: (v) => calls.keystroke.push(v),\n        clear: () => calls.clear++,\n        write: (v) => calls.write.push(v),\n      };\n      c.updateDisable = () => {};\n\n      return { c, onChange, calls };\n    };\n\n    it('propagates ordinary latex to onChange', () => {\n      const { c, onChange } = withInput();\n\n      c.onEditorChange('\\\\frac{1}{2}');\n      expect(onChange).toHaveBeenCalledWith('\\\\frac{1}{2}');\n    });\n\n    it('strips decimals when noDecimal is set', () => {\n      const { c, onChange, calls } = withInput({ noDecimal: true });\n\n      c.onEditorChange('1.5');\n\n      expect(calls.clear).toBe(1);\n      expect(calls.write).toEqual(['15']);\n      expect(onChange).not.toHaveBeenCalled();\n    });\n\n    it('undoes the empty fraction after a digit with a public delete, not MathQuill internals', () => {\n      const { c, onChange, calls } = withInput();\n\n      // the shape the old __controller hack targeted\n      c.onEditorChange('2\\\\ \\\\frac{1}{ }');\n\n      expect(calls.keystroke).toEqual(['Backspace']);\n      expect(onChange).not.toHaveBeenCalled();\n    });\n\n    it('does not treat normal fractions as the mixed-number case', () => {\n      const { c, onChange, calls } = withInput();\n\n      c.onEditorChange('\\\\frac{1}{2}');\n\n      expect(calls.keystroke).toEqual([]);\n      expect(onChange).toHaveBeenCalled();\n    });\n  });\n});\n"],"mappings":";;;;AAAA,IAAAA,aAAA,GAAAC,OAAA;AAAiD,SAAAC,QAAAC,CAAA,EAAAC,CAAA,QAAAC,CAAA,GAAAC,MAAA,CAAAC,IAAA,CAAAJ,CAAA,OAAAG,MAAA,CAAAE,qBAAA,QAAAC,CAAA,GAAAH,MAAA,CAAAE,qBAAA,CAAAL,CAAA,GAAAC,CAAA,KAAAK,CAAA,GAAAA,CAAA,CAAAC,MAAA,WAAAN,CAAA,WAAAE,MAAA,CAAAK,wBAAA,CAAAR,CAAA,EAAAC,CAAA,EAAAQ,UAAA,OAAAP,CAAA,CAAAQ,IAAA,CAAAC,KAAA,CAAAT,CAAA,EAAAI,CAAA,YAAAJ,CAAA;AAAA,SAAAU,cAAAZ,CAAA,aAAAC,CAAA,MAAAA,CAAA,GAAAY,SAAA,CAAAC,MAAA,EAAAb,CAAA,UAAAC,CAAA,WAAAW,SAAA,CAAAZ,CAAA,IAAAY,SAAA,CAAAZ,CAAA,QAAAA,CAAA,OAAAF,OAAA,CAAAI,MAAA,CAAAD,CAAA,OAAAa,OAAA,WAAAd,CAAA,QAAAe,gBAAA,aAAAhB,CAAA,EAAAC,CAAA,EAAAC,CAAA,CAAAD,CAAA,SAAAE,MAAA,CAAAc,yBAAA,GAAAd,MAAA,CAAAe,gBAAA,CAAAlB,CAAA,EAAAG,MAAA,CAAAc,yBAAA,CAAAf,CAAA,KAAAH,OAAA,CAAAI,MAAA,CAAAD,CAAA,GAAAa,OAAA,WAAAd,CAAA,IAAAE,MAAA,CAAAgB,cAAA,CAAAnB,CAAA,EAAAC,CAAA,EAAAE,MAAA,CAAAK,wBAAA,CAAAN,CAAA,EAAAD,CAAA,iBAAAD,CAAA;AAEjD;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACAoB,QAAQ,CAAC,2CAA2C,EAAE,YAAM;EAC1D,IAAMC,KAAK,GAAG,SAARA,KAAKA,CAAA;IAAA,IAAIC,IAAI,GAAAT,SAAA,CAAAC,MAAA,QAAAD,SAAA,QAAAU,SAAA,GAAAV,SAAA,MAAG,CAAC,CAAC;IAAA,OAAAD,aAAA;MAAQY,OAAO,EAAE,CAAC,CAAC;MAAEC,UAAU,EAAE,CAAC,CAAC;MAAEC,QAAQ,EAAEC,IAAI,CAACC,EAAE,CAAC;IAAC,GAAKN,IAAI;EAAA,CAAG;EAE5FF,QAAQ,CAAC,uBAAuB,EAAE,YAAM;IACtCS,EAAE,CAAC,6EAA6E,EAAE,YAAM;MACtF,IAAMC,CAAC,GAAG,IAAIC,0BAAY,CAACV,KAAK,CAAC;QAAEW,KAAK,EAAE;MAAI,CAAC,CAAC,CAAC;MAEjDF,CAAC,CAACG,KAAK,GAAG,IAAI;MAEdC,MAAM,CAACJ,CAAC,CAACK,qBAAqB,CAAC;QAAEH,KAAK,EAAE;MAAI,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAACI,IAAI,CAAC,IAAI,CAAC;IAChE,CAAC,CAAC;IAEFP,EAAE,CAAC,wDAAwD,EAAE,YAAM;MACjE,IAAMC,CAAC,GAAG,IAAIC,0BAAY,CAACV,KAAK,CAAC;QAAEW,KAAK,EAAE;MAAI,CAAC,CAAC,CAAC;;MAEjD;MACAF,CAAC,CAACG,KAAK,GAAG;QAAED,KAAK,EAAE,SAAPA,KAAKA,CAAA;UAAA,OAAQ,EAAE;QAAA;MAAC,CAAC;MAE7BE,MAAM,CAAC;QAAA,OAAMJ,CAAC,CAACK,qBAAqB,CAAC;UAAEH,KAAK,EAAE;QAAI,CAAC,EAAE,CAAC,CAAC,CAAC;MAAA,EAAC,CAACK,GAAG,CAACC,OAAO,CAAC,CAAC;MACvEJ,MAAM,CAACJ,CAAC,CAACK,qBAAqB,CAAC;QAAEH,KAAK,EAAE;MAAI,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAACI,IAAI,CAAC,IAAI,CAAC;IAChE,CAAC,CAAC;IAEFP,EAAE,CAAC,kEAAkE,EAAE,YAAM;MAC3E,IAAMC,CAAC,GAAG,IAAIC,0BAAY,CAACV,KAAK,CAAC;QAAEW,KAAK,EAAE,GAAG;QAAEO,UAAU,EAAE,GAAG;QAAEC,SAAS,EAAE;MAAM,CAAC,CAAC,CAAC;MAEpFV,CAAC,CAACG,KAAK,GAAG;QAAED,KAAK,EAAE,SAAPA,KAAKA,CAAA;UAAA,OAAQ,GAAG;QAAA;MAAC,CAAC;MAC9BF,CAAC,CAACW,KAAK,GAAG;QAAEC,cAAc,EAAE;MAAW,CAAC;MAExCR,MAAM,CAACJ,CAAC,CAACK,qBAAqB,CAAC;QAAEH,KAAK,EAAE,GAAG;QAAEO,UAAU,EAAE,GAAG;QAAEC,SAAS,EAAE;MAAM,CAAC,EAAEV,CAAC,CAACW,KAAK,CAAC,CAAC,CAACL,IAAI,CAAC,KAAK,CAAC;IACzG,CAAC,CAAC;IAEFP,EAAE,CAAC,2DAA2D,EAAE,YAAM;MACpE,IAAMC,CAAC,GAAG,IAAIC,0BAAY,CAACV,KAAK,CAAC;QAAEW,KAAK,EAAE;MAAI,CAAC,CAAC,CAAC;MAEjDF,CAAC,CAACG,KAAK,GAAG;QAAED,KAAK,EAAE,SAAPA,KAAKA,CAAA;UAAA,OAAQ,GAAG;QAAA;MAAC,CAAC;MAE9BE,MAAM,CAACJ,CAAC,CAACK,qBAAqB,CAAC;QAAEH,KAAK,EAAE;MAAI,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAACI,IAAI,CAAC,IAAI,CAAC;IAChE,CAAC,CAAC;IAEFP,EAAE,CAAC,mDAAmD,EAAE,YAAM;MAC5D,IAAMC,CAAC,GAAG,IAAIC,0BAAY,CAACV,KAAK,CAAC;QAAEW,KAAK,EAAE;MAAI,CAAC,CAAC,CAAC;MACjD,IAAMA,KAAK,GAAGL,IAAI,CAACC,EAAE,CAAC;QAAA,OAAM,GAAG;MAAA,EAAC;;MAEhC;MACAE,CAAC,CAACG,KAAK,GAAG;QAAED,KAAK,EAALA;MAAM,CAAC;MACnBF,CAAC,CAACW,KAAK,GAAG;QAAEC,cAAc,EAAE;MAAW,CAAC;MACxCZ,CAAC,CAACK,qBAAqB,CAAC;QAAEH,KAAK,EAAE;MAAI,CAAC,EAAEF,CAAC,CAACW,KAAK,CAAC;MAEhDP,MAAM,CAACF,KAAK,CAAC,CAACW,gBAAgB,CAAC,CAAC;IAClC,CAAC,CAAC;EACJ,CAAC,CAAC;EAEFvB,QAAQ,CAAC,qBAAqB,EAAE,YAAM;IACpC,IAAMwB,SAAS,GAAG,SAAZA,SAASA,CAAA,EAAkB;MAAA,IAAdtB,IAAI,GAAAT,SAAA,CAAAC,MAAA,QAAAD,SAAA,QAAAU,SAAA,GAAAV,SAAA,MAAG,CAAC,CAAC;MAC1B,IAAMiB,CAAC,GAAG,IAAIC,0BAAY,CAACV,KAAK,CAACC,IAAI,CAAC,CAAC;MACvC,IAAMuB,KAAK,GAAG;QAAEC,KAAK,EAAE,EAAE;QAAEC,OAAO,EAAE,EAAE;QAAEC,SAAS,EAAE,EAAE;QAAEC,KAAK,EAAE;MAAE,CAAC;MAEjEnB,CAAC,CAACG,KAAK,GAAG;QACRa,KAAK,EAAE,SAAPA,KAAKA,CAAGI,CAAC;UAAA,OAAKL,KAAK,CAACC,KAAK,CAACpC,IAAI,CAACwC,CAAC,CAAC;QAAA;QACjCH,OAAO,EAAE,SAATA,OAAOA,CAAGG,CAAC;UAAA,OAAKL,KAAK,CAACE,OAAO,CAACrC,IAAI,CAACwC,CAAC,CAAC;QAAA;QACrCF,SAAS,EAAE,SAAXA,SAASA,CAAGE,CAAC;UAAA,OAAKL,KAAK,CAACG,SAAS,CAACtC,IAAI,CAACwC,CAAC,CAAC;QAAA;QACzCD,KAAK,EAAE,SAAPA,KAAKA,CAAA;UAAA,OAAQJ,KAAK,CAACI,KAAK,EAAE;QAAA;MAC5B,CAAC;MACDnB,CAAC,CAACqB,aAAa,GAAG,YAAM,CAAC,CAAC;MAE1B,OAAO;QAAErB,CAAC,EAADA,CAAC;QAAEe,KAAK,EAALA;MAAM,CAAC;IACrB,CAAC;;IAED;IACAhB,EAAE,CAAC,OAAO,EAAE,YAAM;MAChB,IAAAuB,UAAA,GAAqBR,SAAS,CAAC,CAAC;QAAxBd,CAAC,GAAAsB,UAAA,CAADtB,CAAC;QAAEe,KAAK,GAAAO,UAAA,CAALP,KAAK;MAEhBf,CAAC,CAACuB,OAAO,CAAC;QAAEC,KAAK,EAAE;MAAQ,CAAC,CAAC;MAC7BpB,MAAM,CAACW,KAAK,CAACI,KAAK,CAAC,CAACb,IAAI,CAAC,CAAC,CAAC;IAC7B,CAAC,CAAC;IAEFP,EAAE,CAAC,SAAS,EAAE,YAAM;MAClB,IAAA0B,WAAA,GAAqBX,SAAS,CAAC,CAAC;QAAxBd,CAAC,GAAAyB,WAAA,CAADzB,CAAC;QAAEe,KAAK,GAAAU,WAAA,CAALV,KAAK;MAEhBf,CAAC,CAACuB,OAAO,CAAC;QAAEG,IAAI,EAAE,SAAS;QAAEF,KAAK,EAAE;MAAS,CAAC,CAAC;MAC/CpB,MAAM,CAACW,KAAK,CAACE,OAAO,CAAC,CAACU,OAAO,CAAC,CAAC,QAAQ,CAAC,CAAC;IAC3C,CAAC,CAAC;IAEF5B,EAAE,CAAC,kBAAkB,EAAE,YAAM;MAC3B,IAAA6B,WAAA,GAAqBd,SAAS,CAAC,CAAC;QAAxBd,CAAC,GAAA4B,WAAA,CAAD5B,CAAC;QAAEe,KAAK,GAAAa,WAAA,CAALb,KAAK;MAEhBf,CAAC,CAACuB,OAAO,CAAC;QAAEG,IAAI,EAAE,QAAQ;QAAEF,KAAK,EAAE;MAAO,CAAC,CAAC;MAC5CpB,MAAM,CAACW,KAAK,CAACG,SAAS,CAAC,CAACS,OAAO,CAAC,CAAC,MAAM,CAAC,CAAC;IAC3C,CAAC,CAAC;IAEF5B,EAAE,CAAC,wCAAwC,EAAE,YAAM;MACjD,IAAA8B,WAAA,GAAqBf,SAAS,CAAC,CAAC;QAAxBd,CAAC,GAAA6B,WAAA,CAAD7B,CAAC;QAAEe,KAAK,GAAAc,WAAA,CAALd,KAAK;MAEhBf,CAAC,CAACuB,OAAO,CAAC;QAAEG,IAAI,EAAE;MAAS,CAAC,CAAC;MAC7BtB,MAAM,CAACW,KAAK,CAACC,KAAK,CAAC,CAACW,OAAO,CAAC,CAAC,YAAY,CAAC,CAAC;IAC7C,CAAC,CAAC;IAEF5B,EAAE,CAAC,aAAa,EAAE,YAAM;MACtB,IAAA+B,WAAA,GAAqBhB,SAAS,CAAC,CAAC;QAAxBd,CAAC,GAAA8B,WAAA,CAAD9B,CAAC;QAAEe,KAAK,GAAAe,WAAA,CAALf,KAAK;MAEhBf,CAAC,CAACuB,OAAO,CAAC;QAAEC,KAAK,EAAE;MAAI,CAAC,CAAC;MACzBpB,MAAM,CAACW,KAAK,CAACC,KAAK,CAAC,CAACW,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC;IACpC,CAAC,CAAC;IAEF5B,EAAE,CAAC,uBAAuB,EAAE,YAAM;MAChC,IAAAgC,WAAA,GAAqBjB,SAAS,CAAC,CAAC;QAAxBd,CAAC,GAAA+B,WAAA,CAAD/B,CAAC;QAAEe,KAAK,GAAAgB,WAAA,CAALhB,KAAK;MAEhBX,MAAM,CAAC;QAAA,OAAMJ,CAAC,CAACuB,OAAO,CAAC,IAAI,CAAC;MAAA,EAAC,CAAChB,GAAG,CAACC,OAAO,CAAC,CAAC;MAC3CJ,MAAM,CAACW,KAAK,CAACC,KAAK,CAAC,CAACW,OAAO,CAAC,EAAE,CAAC;IACjC,CAAC,CAAC;EACJ,CAAC,CAAC;EAEFrC,QAAQ,CAAC,gBAAgB,EAAE,YAAM;IAC/B,IAAMwB,SAAS,GAAG,SAAZA,SAASA,CAAA,EAAkB;MAAA,IAAdtB,IAAI,GAAAT,SAAA,CAAAC,MAAA,QAAAD,SAAA,QAAAU,SAAA,GAAAV,SAAA,MAAG,CAAC,CAAC;MAC1B,IAAMa,QAAQ,GAAGC,IAAI,CAACC,EAAE,CAAC,CAAC;MAC1B,IAAME,CAAC,GAAG,IAAIC,0BAAY,CAACV,KAAK,CAAAT,aAAA;QAAGc,QAAQ,EAARA;MAAQ,GAAKJ,IAAI,CAAE,CAAC,CAAC;MACxD,IAAMuB,KAAK,GAAG;QAAEG,SAAS,EAAE,EAAE;QAAEC,KAAK,EAAE,CAAC;QAAEH,KAAK,EAAE;MAAG,CAAC;MAEpDhB,CAAC,CAACG,KAAK,GAAG;QACRe,SAAS,EAAE,SAAXA,SAASA,CAAGE,CAAC;UAAA,OAAKL,KAAK,CAACG,SAAS,CAACtC,IAAI,CAACwC,CAAC,CAAC;QAAA;QACzCD,KAAK,EAAE,SAAPA,KAAKA,CAAA;UAAA,OAAQJ,KAAK,CAACI,KAAK,EAAE;QAAA;QAC1BH,KAAK,EAAE,SAAPA,KAAKA,CAAGI,CAAC;UAAA,OAAKL,KAAK,CAACC,KAAK,CAACpC,IAAI,CAACwC,CAAC,CAAC;QAAA;MACnC,CAAC;MACDpB,CAAC,CAACqB,aAAa,GAAG,YAAM,CAAC,CAAC;MAE1B,OAAO;QAAErB,CAAC,EAADA,CAAC;QAAEJ,QAAQ,EAARA,QAAQ;QAAEmB,KAAK,EAALA;MAAM,CAAC;IAC/B,CAAC;IAEDhB,EAAE,CAAC,uCAAuC,EAAE,YAAM;MAChD,IAAAiC,WAAA,GAAwBlB,SAAS,CAAC,CAAC;QAA3Bd,CAAC,GAAAgC,WAAA,CAADhC,CAAC;QAAEJ,QAAQ,GAAAoC,WAAA,CAARpC,QAAQ;MAEnBI,CAAC,CAACiC,cAAc,CAAC,cAAc,CAAC;MAChC7B,MAAM,CAACR,QAAQ,CAAC,CAACsC,oBAAoB,CAAC,cAAc,CAAC;IACvD,CAAC,CAAC;IAEFnC,EAAE,CAAC,uCAAuC,EAAE,YAAM;MAChD,IAAAoC,WAAA,GAA+BrB,SAAS,CAAC;UAAEJ,SAAS,EAAE;QAAK,CAAC,CAAC;QAArDV,CAAC,GAAAmC,WAAA,CAADnC,CAAC;QAAEJ,QAAQ,GAAAuC,WAAA,CAARvC,QAAQ;QAAEmB,KAAK,GAAAoB,WAAA,CAALpB,KAAK;MAE1Bf,CAAC,CAACiC,cAAc,CAAC,KAAK,CAAC;MAEvB7B,MAAM,CAACW,KAAK,CAACI,KAAK,CAAC,CAACb,IAAI,CAAC,CAAC,CAAC;MAC3BF,MAAM,CAACW,KAAK,CAACC,KAAK,CAAC,CAACW,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC;MACnCvB,MAAM,CAACR,QAAQ,CAAC,CAACW,GAAG,CAACM,gBAAgB,CAAC,CAAC;IACzC,CAAC,CAAC;IAEFd,EAAE,CAAC,uFAAuF,EAAE,YAAM;MAChG,IAAAqC,WAAA,GAA+BtB,SAAS,CAAC,CAAC;QAAlCd,CAAC,GAAAoC,WAAA,CAADpC,CAAC;QAAEJ,QAAQ,GAAAwC,WAAA,CAARxC,QAAQ;QAAEmB,KAAK,GAAAqB,WAAA,CAALrB,KAAK;;MAE1B;MACAf,CAAC,CAACiC,cAAc,CAAC,kBAAkB,CAAC;MAEpC7B,MAAM,CAACW,KAAK,CAACG,SAAS,CAAC,CAACS,OAAO,CAAC,CAAC,WAAW,CAAC,CAAC;MAC9CvB,MAAM,CAACR,QAAQ,CAAC,CAACW,GAAG,CAACM,gBAAgB,CAAC,CAAC;IACzC,CAAC,CAAC;IAEFd,EAAE,CAAC,0DAA0D,EAAE,YAAM;MACnE,IAAAsC,WAAA,GAA+BvB,SAAS,CAAC,CAAC;QAAlCd,CAAC,GAAAqC,WAAA,CAADrC,CAAC;QAAEJ,QAAQ,GAAAyC,WAAA,CAARzC,QAAQ;QAAEmB,KAAK,GAAAsB,WAAA,CAALtB,KAAK;MAE1Bf,CAAC,CAACiC,cAAc,CAAC,cAAc,CAAC;MAEhC7B,MAAM,CAACW,KAAK,CAACG,SAAS,CAAC,CAACS,OAAO,CAAC,EAAE,CAAC;MACnCvB,MAAM,CAACR,QAAQ,CAAC,CAACiB,gBAAgB,CAAC,CAAC;IACrC,CAAC,CAAC;EACJ,CAAC,CAAC;AACJ,CAAC,CAAC","ignoreList":[]}

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "3.1.0-beta.3",
+  "version": "3.1.0-beta.5",
   "description": "Math toolbar for editing math equations",
   "keywords": [
     "math",
@@ -21,7 +21,7 @@
     "@emotion/styled": "^11.14.1",
     "@mui/icons-material": "^7.3.4",
     "@mui/material": "^7.3.4",
-    "@pie-lib/math-input-mathlive": "^1.0.7",
+    "@pie-lib/math-input-mathlive": "^1.0.9",
     "@pie-lib/render-ui": "^6.1.0",
     "debug": "^4.1.1",
     "lodash-es": "^4.17.23",

package/src/__tests__/editor-and-pad-mathlive.test.js

@@ -0,0 +1,178 @@
+import { EditorAndPad } from '../editor-and-pad';
+
+/**
+ * Covers the call sites that changed when this package moved from
+ * @pie-lib/math-input (MathQuill) to @pie-lib/math-input-mathlive.
+ *
+ * All three were MathQuill-shaped and silently changed meaning:
+ *  - shouldComponentUpdate read `this.input.mathField.latex()`, which throws now
+ *    (and the field loads asynchronously, so `this.input` can be null)
+ *  - the mixed-number workaround poked `__controller.cursor`, a MathQuill
+ *    internal with no MathLive equivalent
+ *  - onClick routes keypad presses through the imperative field API
+ */
+describe('EditorAndPad after the MathLive migration', () => {
+  const props = (over = {}) => ({ classes: {}, classNames: {}, onChange: jest.fn(), ...over });
+
+  describe('shouldComponentUpdate', () => {
+    it('treats a missing input ref as different, so the first render is not skipped', () => {
+      const c = new EditorAndPad(props({ latex: 'x' }));
+
+      c.input = null;
+
+      expect(c.shouldComponentUpdate({ latex: 'x' }, {})).toBe(true);
+    });
+
+    it('does not throw when the math engine has not loaded yet', () => {
+      const c = new EditorAndPad(props({ latex: 'x' }));
+
+      // latex() returns '' until MathLive is ready
+      c.input = { latex: () => '' };
+
+      expect(() => c.shouldComponentUpdate({ latex: 'x' }, {})).not.toThrow();
+      expect(c.shouldComponentUpdate({ latex: 'x' }, {})).toBe(true);
+    });
+
+    it('skips the update when the field already holds the incoming latex', () => {
+      const c = new EditorAndPad(props({ latex: 'x', keypadMode: 'a', noDecimal: false }));
+
+      c.input = { latex: () => 'x' };
+      c.state = { equationEditor: 'geometry' };
+
+      expect(c.shouldComponentUpdate({ latex: 'x', keypadMode: 'a', noDecimal: false }, c.state)).toBe(false);
+    });
+
+    it('re-renders when the field differs from the incoming latex', () => {
+      const c = new EditorAndPad(props({ latex: 'y' }));
+
+      c.input = { latex: () => 'x' };
+
+      expect(c.shouldComponentUpdate({ latex: 'y' }, {})).toBe(true);
+    });
+
+    it('uses input.latex(), never input.mathField.latex()', () => {
+      const c = new EditorAndPad(props({ latex: 'x' }));
+      const latex = jest.fn(() => 'x');
+
+      // deliberately no `mathField` - the old code would have thrown here
+      c.input = { latex };
+      c.state = { equationEditor: 'geometry' };
+      c.shouldComponentUpdate({ latex: 'x' }, c.state);
+
+      expect(latex).toHaveBeenCalled();
+    });
+  });
+
+  describe('onClick key routing', () => {
+    const withInput = (over = {}) => {
+      const c = new EditorAndPad(props(over));
+      const calls = { write: [], command: [], keystroke: [], clear: 0 };
+
+      c.input = {
+        write: (v) => calls.write.push(v),
+        command: (v) => calls.command.push(v),
+        keystroke: (v) => calls.keystroke.push(v),
+        clear: () => calls.clear++,
+      };
+      c.updateDisable = () => {};
+
+      return { c, calls };
+    };
+
+    // toNodeData keys this off `value === 'clear'`, not `type`
+    it('clear', () => {
+      const { c, calls } = withInput();
+
+      c.onClick({ value: 'clear' });
+      expect(calls.clear).toBe(1);
+    });
+
+    it('command', () => {
+      const { c, calls } = withInput();
+
+      c.onClick({ type: 'command', value: '\\frac' });
+      expect(calls.command).toEqual(['\\frac']);
+    });
+
+    it('cursor keystroke', () => {
+      const { c, calls } = withInput();
+
+      c.onClick({ type: 'cursor', value: 'Left' });
+      expect(calls.keystroke).toEqual(['Left']);
+    });
+
+    it('answer block writes the response token', () => {
+      const { c, calls } = withInput();
+
+      c.onClick({ type: 'answer' });
+      expect(calls.write).toEqual(['%response%']);
+    });
+
+    it('plain value', () => {
+      const { c, calls } = withInput();
+
+      c.onClick({ value: 'x' });
+      expect(calls.write).toEqual(['x']);
+    });
+
+    it('ignores a nullish key', () => {
+      const { c, calls } = withInput();
+
+      expect(() => c.onClick(null)).not.toThrow();
+      expect(calls.write).toEqual([]);
+    });
+  });
+
+  describe('onEditorChange', () => {
+    const withInput = (over = {}) => {
+      const onChange = jest.fn();
+      const c = new EditorAndPad(props({ onChange, ...over }));
+      const calls = { keystroke: [], clear: 0, write: [] };
+
+      c.input = {
+        keystroke: (v) => calls.keystroke.push(v),
+        clear: () => calls.clear++,
+        write: (v) => calls.write.push(v),
+      };
+      c.updateDisable = () => {};
+
+      return { c, onChange, calls };
+    };
+
+    it('propagates ordinary latex to onChange', () => {
+      const { c, onChange } = withInput();
+
+      c.onEditorChange('\\frac{1}{2}');
+      expect(onChange).toHaveBeenCalledWith('\\frac{1}{2}');
+    });
+
+    it('strips decimals when noDecimal is set', () => {
+      const { c, onChange, calls } = withInput({ noDecimal: true });
+
+      c.onEditorChange('1.5');
+
+      expect(calls.clear).toBe(1);
+      expect(calls.write).toEqual(['15']);
+      expect(onChange).not.toHaveBeenCalled();
+    });
+
+    it('undoes the empty fraction after a digit with a public delete, not MathQuill internals', () => {
+      const { c, onChange, calls } = withInput();
+
+      // the shape the old __controller hack targeted
+      c.onEditorChange('2\\ \\frac{1}{ }');
+
+      expect(calls.keystroke).toEqual(['Backspace']);
+      expect(onChange).not.toHaveBeenCalled();
+    });
+
+    it('does not treat normal fractions as the mixed-number case', () => {
+      const { c, onChange, calls } = withInput();
+
+      c.onEditorChange('\\frac{1}{2}');
+
+      expect(calls.keystroke).toEqual([]);
+      expect(onChange).toHaveBeenCalled();
+    });
+  });
+});