Resolve ViGotoBrace Reviews

Author: sharpchen

PSReadLine/Movement.vi.cs

@@ -261,6 +261,10 @@ private int ViFindBrace(int i)
                         for (int idx = 0; idx < parenthese.Length; idx++)
                             if (parenthese[idx] == _buffer[nextParen]) goto Outer;
 
+                    // if not found, nextParen could exceed the range
+                    if (nextParen >= _buffer.Length)
+                        return i;
+
                 Outer:
                     int match = _buffer[nextParen] switch
                     {

test/MovementTest.VI.cs

@@ -1,5 +1,4 @@
-using System.Collections.Generic;
-using Xunit;
+using Xunit;
 
 namespace Test
 {
@@ -477,7 +476,7 @@ public void ViGotoBrace()
 
                 // Closing paren without backward match
                 string input2 = $"0]2)4foo{closing}";
-                Test(input2, Keys(
+                TestMustDing(input2, Keys(
                     input2,
                     CheckThat(() => AssertCursorLeftIs(9)),
                     _.Escape, CheckThat(() => AssertCursorLeftIs(8)),