Fixes a bug that prevented early step-out of a container when positio…

…ned on a large scalar value in certain cases.

src/com/amazon/ion/impl/IonCursorBinary.java

@@ -1470,7 +1470,7 @@ private Event slowStepOutOfContainer() {
             if (slowSeek(parent.endIndex - offset)) {
                 return event;
             }
-            peekIndex = parent.endIndex;
+            peekIndex = offset;
         }
         setCheckpointBeforeUnannotatedTypeId();
         if (--containerIndex >= 0) {

test/com/amazon/ion/impl/IonReaderContinuableTopLevelBinaryTest.java

@@ -34,6 +34,7 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.CsvSource;
 import org.junit.jupiter.params.provider.ValueSource;
 
 import java.io.ByteArrayInputStream;
@@ -1723,6 +1724,33 @@ public void reallyLargeString(boolean constructFromBytes) throws Exception {
         closeAndCount();
     }
 
+    @ParameterizedTest
+    @CsvSource({
+        "true, true",
+        "true, false",
+        "false, true",
+        "false, false"
+    })
+    public void skipReallyLargeStringInContainer(boolean constructFromBytes, boolean incremental) throws Exception {
+        StringBuilder sb = new StringBuilder();
+        // 70000 is greater than twice the default buffer size of 32768. This ensures that the string, when skipped,
+        // will not be fully contained in the buffer.
+        for (int i = 0; i < 70000; i++) {
+            sb.append('a');
+        }
+        String string = sb.toString();
+        readerBuilder = readerBuilder.withIncrementalReadingEnabled(incremental);
+        reader = readerFor("{foo: \"" + string + "\"}", constructFromBytes);
+        assertSequence(
+            container(IonType.STRUCT,
+                next(IonType.STRING)
+                // This is an early-step out that causes the string value to be skipped.
+            ),
+            next(null)
+        );
+        closeAndCount();
+    }
+
     @ParameterizedTest(name = "constructFromBytes={0}")
     @ValueSource(booleans = {true, false})
     public void nopPadInAnnotationWrapperFails(boolean constructFromBytes) throws Exception {