From 5f2aa22ecb80c5eabd82951f7dce2d017738fa5c Mon Sep 17 00:00:00 2001
From: NSoiffer <NSoiffer@users.noreply.github.com>
Date: Wed, 13 Dec 2023 01:21:41 +0000
Subject: [PATCH] Modified `example_11_5_3` to use output from MathJax. The
 previous MathML was strange and I couldn't figure out where it came from.

Added `overscript_grouping_aph_5_4_8` to test issue #220.
---
 tests/braille/Nemeth/rules.rs |  1 -
 tests/braille/UEB/iceb.rs     | 19 ++++++++++++++++---
 tests/braille/UEB/other.rs    |  7 +++++++
 3 files changed, 23 insertions(+), 4 deletions(-)

diff --git a/tests/braille/Nemeth/rules.rs b/tests/braille/Nemeth/rules.rs
index 5929d28f..484c4bd2 100644
--- a/tests/braille/Nemeth/rules.rs
+++ b/tests/braille/Nemeth/rules.rs
@@ -272,7 +272,6 @@ fn greek_24_b_1() {
 
 #[test]
 fn eli_nemeth_UEB_rule_book_4_11_1() {
-    init_logger();
     let expr = "<math>
             <mtext>p</mtext><mtext>&#xA0;</mtext>
             <mtext>D</mtext><mtext>&#xA0;</mtext>
diff --git a/tests/braille/UEB/iceb.rs b/tests/braille/UEB/iceb.rs
index 302d296d..f9f8f7b5 100644
--- a/tests/braille/UEB/iceb.rs
+++ b/tests/braille/UEB/iceb.rs
@@ -949,10 +949,23 @@ fn example_11_5_2() {
 
 #[test]
 fn example_11_5_3() {
+    // from MathJaX
+    let expr = "<math>
+        <msup><mrow/><mi>n</mi></msup>
+        <msub><mi>C</mi><mi>r</mi></msub>
+        <mo>=</mo>
+        <mrow>
+            <mo minsize='2.047em' maxsize='2.047em'>(</mo>
+            <mfrac linethickness='0'><mi>n</mi><mi>r</mi></mfrac>
+            <mo minsize='2.047em' maxsize='2.047em'>)</mo>
+        </mrow>
+        <mo>=</mo>
+        <mfrac>
+            <mrow><mi>n</mi><mo>!</mo></mrow>
+            <mrow><mi>r</mi><mo>!</mo><mo stretchy='false'>(</mo><mi>n</mi><mo>&#x2212;</mo><mi>r</mi><mo stretchy='false'>)</mo><mo>!</mo></mrow>
+        </mfrac>
+    </math>";
     // modified to use "shape" as recommended in a comment on this example
-    let expr = "<math><msub><mmultiscripts><mi>C</mi><mprescripts/><none/><mi>n</mi></mmultiscripts><mi>r</mi></msub><mo>=</mo>
-            <mo>(</mo><mfrac linethickness='0'><mi>n</mi><mi>r</mi></mfrac><mo>)</mo><mo>=</mo>
-            <mfrac><mrow><mi>n</mi><mo>!</mo></mrow><mrow><mi>r</mi><mo>!</mo><mo>(</mo><mi>n</mi><mo>-</mo><mi>r</mi><mo>)</mo><mo>!</mo></mrow></mfrac></math>";
     test_braille("UEB", expr, "⠰⠰⠰⠔⠝⠠⠉⠢⠗⠀⠐⠶⠀⠐⠣⠝⠰⠻⠗⠐⠜⠀⠐⠶⠀⠷⠝⠖⠨⠌⠗⠖⠐⠣⠝⠐⠤⠗⠐⠜⠖⠾⠰⠄");
 }
 
diff --git a/tests/braille/UEB/other.rs b/tests/braille/UEB/other.rs
index e0c0e611..d811a840 100644
--- a/tests/braille/UEB/other.rs
+++ b/tests/braille/UEB/other.rs
@@ -21,6 +21,13 @@ fn blank_aph_7_1_ex5() {
     test_braille("UEB", expr, "⠼⠉⠒⠼⠁⠑⠀⠒⠒⠀⠼⠋⠚⠒⠨⠤");
 }
 
+#[test]
+fn overscript_grouping_aph_5_4_8() {
+    // this test was added because #220 (failed to add grouping around overscript)
+    let expr = "<math> <mover> <mi>MN</mi> <mo>&#x2194;</mo> </mover> </math>";
+    test_braille("UEB", expr, "⠰⠰⠣⠠⠠⠍⠝⠜⠨⠔⠳⠺⠗⠕");
+}
+
 // Extra tests targeted at special cases in MathCAT
 #[test]
 fn number_space_before() {