Merge pull request #1365 from mathjax/issue3447

Fix handling of argument to \set and others.  (mathjax/MathJax#3447)

Author: dpvc

testsuite/tests/input/tex/Braket.test.ts

@@ -665,6 +665,20 @@ describe('Braket', () => {
 
   /********************************************************************************/
 
+  it('Braket-braces', () => {
+    expectTexError('\\braket{')
+      .toBe('Missing close brace');
+  });
+
+  /********************************************************************************/
+
+  it('Braket-braces', () => {
+    expectTexError('\\braket}')
+      .toBe('Extra close brace or missing open brace');
+  });
+
+  /********************************************************************************/
+
 });
 
 /**********************************************************************************/

ts/input/tex/braket/BraketMethods.ts

@@ -25,15 +25,14 @@ import { ParseMethod, ParseResult } from '../Types.js';
 import BaseMethods from '../base/BaseMethods.js';
 import TexParser from '../TexParser.js';
 import { TEXCLASS } from '../../../core/MmlTree/MmlNode.js';
-import TexError from '../TexError.js';
 import { BraketItem } from './BraketItems.js';
 
 const BraketMethods: { [key: string]: ParseMethod } = {
   /**
    * Generate a bra-ket expression.
    *
    * @param {TexParser} parser The current TeX parser.
-   * @param {string} _name Name of the current control sequence.
+   * @param {string} name Name of the current control sequence.
    * @param {string} open Opening delimiter.
    * @param {string} close Closing delimiter.
    * @param {boolean} stretchy Is it stretchy.
@@ -42,21 +41,17 @@ const BraketMethods: { [key: string]: ParseMethod } = {
    */
   Braket(
     parser: TexParser,
-    _name: string,
+    name: string,
     open: string,
     close: string,
     stretchy: boolean,
     barmax: number,
     space: boolean = false
   ) {
+    const i = parser.i;
+    parser.GetArgument(name); // Error if there isn't a proper argument
+    parser.i = i;
     const next = parser.GetNext();
-    if (next === '') {
-      throw new TexError(
-        'MissingArgFor',
-        'Missing argument for %1',
-        parser.currentCS
-      );
-    }
     let single = true;
     if (next === '{') {
       parser.i++;