[incubator-tuweni] branch main updated: ssz: Fix reader.readFixedBytesList

[toulmean]

This is an automated email from the ASF dual-hosted git repository.

toulmean pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-tuweni.git


The following commit(s) were added to refs/heads/main by this push:
     new f62f82d1 ssz: Fix reader.readFixedBytesList
     new eb6d9d6d Merge pull request #424 from zilm13/fix/fixed-bytes-list
f62f82d1 is described below

commit f62f82d192fe989c560d690612843e84a27bb2cd
Author: Dmitrii Shmatko <leode...@gmail.com>
AuthorDate: Wed Jul 13 23:16:00 2022 +0400

    ssz: Fix reader.readFixedBytesList
---
 ssz/src/main/java/org/apache/tuweni/ssz/BytesSSZReader.java     | 2 +-
 ssz/src/test/java/org/apache/tuweni/ssz/BytesSSZReaderTest.java | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/ssz/src/main/java/org/apache/tuweni/ssz/BytesSSZReader.java 
b/ssz/src/main/java/org/apache/tuweni/ssz/BytesSSZReader.java
index de2455cc..1f0fbab1 100644
--- a/ssz/src/main/java/org/apache/tuweni/ssz/BytesSSZReader.java
+++ b/ssz/src/main/java/org/apache/tuweni/ssz/BytesSSZReader.java
@@ -147,7 +147,7 @@ final class BytesSSZReader implements SSZReader {
 
   @Override
   public List<Bytes> readFixedBytesList(int byteLength, int limit) {
-    return readList(remaining -> readFixedBytes(byteLength, limit));
+    return readList(byteLength, () -> readFixedBytes(byteLength, limit));
   }
 
   @Override
diff --git a/ssz/src/test/java/org/apache/tuweni/ssz/BytesSSZReaderTest.java 
b/ssz/src/test/java/org/apache/tuweni/ssz/BytesSSZReaderTest.java
index 46533c8e..2a5656e7 100644
--- a/ssz/src/test/java/org/apache/tuweni/ssz/BytesSSZReaderTest.java
+++ b/ssz/src/test/java/org/apache/tuweni/ssz/BytesSSZReaderTest.java
@@ -18,6 +18,7 @@ import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import org.apache.tuweni.bytes.Bytes;
+import org.apache.tuweni.bytes.Bytes32;
 import org.apache.tuweni.bytes.Bytes48;
 
 import java.math.BigInteger;
@@ -225,4 +226,11 @@ class BytesSSZReaderTest {
     Bytes encoded = SSZ.encode(writer -> 
writer.writeFixedBytesVector(toWrite));
     assertEquals(toWrite, SSZ.decode(encoded, reader -> 
reader.readFixedBytesVector(3, 48)));
   }
+
+  @Test
+  void shouldRoundtripHomogenousBytesList() {
+    List<Bytes32> toWrite = Arrays.asList(Bytes32.random(), Bytes32.random(), 
Bytes32.random());
+    Bytes encoded = SSZ.encode(writer -> writer.writeFixedBytesList(toWrite));
+    assertEquals(toWrite, SSZ.decode(encoded, reader -> 
reader.readFixedBytesList(Bytes32.SIZE)));
+  }
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@tuweni.apache.org
For additional commands, e-mail: commits-h...@tuweni.apache.org