Add UT for #1.

Author: CXuesong

Luaon.NET/LuaTableTextReader.cs

@@ -40,8 +40,8 @@ public class LuaTableTextReader : IDisposable
         private readonly List<LuaContainerContext> contextStack;
         private LuaContainerContext currentContext;
         private readonly char[] buffer;
-        private int bufferLength;
-        private int bufferPos;
+        private int bufferLength;   // Loaded buffer end position
+        private int bufferPos;      // Start of next buffer character
         private bool readerEof;
 
         private static readonly object boxedTrue = true;
@@ -98,6 +98,7 @@ private enum Token
         private void GotoNextState(Token token)
         {
             var next = NextStateTable[(int)currentState][(int)token];
+            Debug.WriteLine("GotoNextState: {0}[{1}] -> {2}", currentState, token, next);
             if (next == State.Error)
             {
                 if (currentState == State.End)
@@ -230,6 +231,7 @@ private bool EnsureBuffer(int length = 0)
                 bufferLength = bufferLength - bufferPos;
                 bufferPos = 0;
             }
+            // Read from where the current loaded buffer ends.
             var charsRead = Reader.ReadBlock(buffer, bufferLength, charsNeeded);
             bufferLength += charsRead;
             if (charsRead < charsNeeded)
@@ -438,6 +440,7 @@ public LuaTableReaderToken Read()
             if ((o = ReadLiteral()) != null)
             {
                 GotoNextState(Token.Literal);
+                Debug.WriteLine("Literal: {0}", o);
                 if (currentState == State.Key)
                 {
                     currentContext.BoxedKey = o;
@@ -456,6 +459,7 @@ public LuaTableReaderToken Read()
                 GotoNextState(Token.KeyStart);
                 GotoNextState(Token.Literal);
                 GotoNextState(Token.KeyEnd);
+                Debug.WriteLine("Identifier: {0}", o);
                 CurrentValue = o;
                 // We keep using BoxedKey instead of Key in Reader.
                 currentContext.BoxedKey = (string)o;

XUnitTestProject1/Tests/RegressionTests.cs

@@ -14,22 +14,25 @@ public class RegressionTests
         /// <summary>
         /// UnexpectedCharacterException when parsing long Lua table (> LuaTableTextReader buffer size, i.e., 1024).
         /// </summary>
-        [Fact]
-        public void Issue1()
+        [Theory]
+		[InlineData("0123456789ABCDEF", 3)]
+		[InlineData("0123456789ABCDEF", 10)]
+        [InlineData("0123456789ABCDEF", 64)]
+        [InlineData("0123456789ABCDEF", 100)]
+        public void Issue1(string seed, int repeat)
         {
-            var longString1 = string.Join("", Enumerable.Repeat("0123456789ABCDEF", 64)); // 16 x 64 = 1024
-            var longString2 = longString1 + "||" + longString1;
             var bigTable = new LTable
             {
-                ["Field1"] = longString1,
-                ["Field2"] = 12345,
-                ["Field2"] = longString2,
+                ["Field1"] = 12345,
+                ["Field2"] = "Test",
             };
+            for (var i = 0; i < repeat; i++)
+                bigTable[i] = seed + i;
             var serialized = bigTable.ToString(Formatting.Prettified);
-            Assert.Contains(longString1, serialized);
-            Assert.Contains(longString2, serialized);
+            Assert.Contains("12345", serialized);
+            Assert.Contains("Test", serialized);
             var bigTable2 = LToken.Parse(serialized);
-            Assert.Equal(bigTable, bigTable2);
+            Assert.Equal(bigTable.ToString(), bigTable2.ToString());
         }
 
     }