Fix ViGotoBrace tests

These tests need to be cleared with `ddi` and then
asserted to be empty. It's the same fix for the
other tests whose input contains unmatched braces.
Copilot's plausible explanation is that the
unmatched braces cause the input to be flagged by
PowerShell's parser as incomplete, so
`AcceptLineImpl` instead waits for more input,
causing the exception.

Author: andyleejordan

test/MovementTest.VI.cs

@@ -424,6 +424,11 @@ public void ViCursorMovement()
         [SkippableFact]
         public void ViGotoBrace()
         {
+            // NOTE: When the input has unmatched braces, in order to avoid an
+            // exception caused by AcceptLineImpl waiting for incomplete input,
+            // the test needs to end with the Vi command "ddi" and assert that
+            // the result is an empty string.
+
             TestSetup(KeyMode.Vi);
 
             Test("0[2(4{6]8)a}c", Keys(
@@ -451,25 +456,26 @@ public void ViGotoBrace()
                     CheckThat(() => AssertCursorLeftIs(4)),
                     _.Percent,
                     CheckThat(() => AssertCursorLeftIs(4)),
-                    "ddi"
+                    "ddi" // Unmatched brace
                     ));
             }
 
-            // tests when cursor not on any paren
+            // Tests when the cursor is not on any paren
             foreach (var (opening, closing) in new[] { ('(', ')'), ('{', '}'), ('[', ']') })
             {
-                // closing paren with backward match
+                // Closing paren with backward match
                 string input1 = $"0{opening}2{opening}4foo{closing}";
-                Test(input1, Keys(
+                Test("", Keys(
                     input1,
                     CheckThat(() => AssertCursorLeftIs(9)),
                     _.Escape, CheckThat(() => AssertCursorLeftIs(8)),
                     "0ff", CheckThat(() => AssertCursorLeftIs(5)),
                     _.Percent, CheckThat(() => AssertCursorLeftIs(3)),
-                    _.Percent, CheckThat(() => AssertCursorLeftIs(8))
+                    _.Percent, CheckThat(() => AssertCursorLeftIs(8)),
+                    "ddi" // Unmatched closing brace
                 ));
 
-                // closing paren without backward match
+                // Closing paren without backward match
                 string input2 = $"0]2)4foo{closing}";
                 Test(input2, Keys(
                     input2,
@@ -480,7 +486,7 @@ public void ViGotoBrace()
                     _.Percent, CheckThat(() => AssertCursorLeftIs(5))
                 ));
 
-                // opening paren with forward match
+                // Opening paren with forward match
                 string input3 = $"0{opening}2foo6{closing}";
                 Test(input3, Keys(
                     input3,
@@ -490,15 +496,17 @@ public void ViGotoBrace()
                     _.Percent, CheckThat(() => AssertCursorLeftIs(1)),
                     _.Percent, CheckThat(() => AssertCursorLeftIs(7))
                 ));
-                // opening paren without forward match
+
+                // Opening paren without forward match
                 string input4 = $"0)2]4foo{opening}(";
-                Test(input4, Keys(
+                TestMustDing("", Keys(
                     input4,
                     CheckThat(() => AssertCursorLeftIs(10)),
                     _.Escape, CheckThat(() => AssertCursorLeftIs(9)),
                     "0ff", CheckThat(() => AssertCursorLeftIs(5)),
                     _.Percent, CheckThat(() => AssertCursorLeftIs(5)), // stay still
-                    _.Percent, CheckThat(() => AssertCursorLeftIs(5))
+                    _.Percent, CheckThat(() => AssertCursorLeftIs(5)),
+                    "ddi" // Unmatched brace
                 ));
             }