Remove getState() from RelpParser, add toString() that prints only current state, add tests for it (#56)

Author: StrongestNumber9

src/main/java/com/teragrep/rlp_01/RelpParser.java

@@ -73,8 +73,11 @@ public ByteBuffer getData() {
         return this.frameData;
     }
 
-    public RelpParserState getState() {
-        return state;
+    @Override
+    public String toString() {
+        return "RelpParser{" +
+                "state=" + state +
+                '}';
     }
 
     private enum RelpParserState {
@@ -95,6 +98,9 @@ private enum RelpParserState {
      Byte to be parsed.
      */
     public void parse(byte b) {
+        if(this.isComplete) {
+            throw new IllegalStateException("parser was not reset after completing");
+        }
         switch (this.state) {
             case TXN:
                 if (b == ' '){

src/test/java/com/teragrep/rlp_01/RelpParserTest.java

@@ -161,6 +161,54 @@ public void testMultipleMessagesInRow() {
         Assertions.assertEquals("five!", results.get(2), "parser getData() differs");
     }
 
+    @Test
+    public void testToStringStateZeroLengthNewline() {
+        RelpParser parser = createParser("0 syslog 0\n");
+        Assertions.assertEquals("RelpParser{state=NL}", parser.toString(), "Parser toString() differs");
+    }
+
+    @Test
+    public void testToStringStateZeroLengthSpace() {
+        RelpParser parser = createParser("0 syslog 0 ");
+        Assertions.assertEquals("RelpParser{state=NL}", parser.toString(), "Parser toString() differs");
+    }
+
+    @Test
+    public void testToStringStateTxn() {
+        RelpParser parser = createParser("0");
+        Assertions.assertEquals("RelpParser{state=TXN}", parser.toString(), "Parser toString() differs");
+    }
+
+    @Test
+    public void testToStringStateCommand() {
+        RelpParser parser = createParser("0 syslog");
+        Assertions.assertEquals("RelpParser{state=COMMAND}", parser.toString(), "Parser toString() differs");
+    }
+
+    @Test
+    public void testToStringStateLength() {
+        RelpParser parser = createParser("0 syslog 3");
+        Assertions.assertEquals("RelpParser{state=LENGTH}", parser.toString(), "Parser toString() differs");
+    }
+
+    @Test
+    public void testToStringStateDataIncomplete() {
+        RelpParser parser = createParser("0 syslog 3 he");
+        Assertions.assertEquals("RelpParser{state=DATA}", parser.toString(), "Parser toString() differs");
+    }
+
+    @Test
+    public void testToStringStateDataComplete() {
+        RelpParser parser = createParser("0 syslog 3 hey");
+        Assertions.assertEquals("RelpParser{state=NL}", parser.toString(), "Parser toString() differs");
+    }
+
+    @Test
+    public void testToStringStateNLAfterData() {
+        RelpParser parser = createParser("0 syslog 3 hey\n");
+        Assertions.assertEquals("RelpParser{state=NL}", parser.toString(), "Parser toString() differs");
+    }
+
     @Disabled(value="Triggers BufferOverflow and is not gracefully handled")
     @Test
     public void testVeryLongTxnId() {