commit eb6d9d6da7bd1c0ff2d6ae90da18ef5909799847
author Antoine Toulme <antoine@lunar-ocean.com> Thu Jul 14 01:39:50 2022 +0200
committer GitHub <noreply@github.com> Thu Jul 14 01:39:50 2022 +0200
tree 4449206c5d74ca2b5bb13d530585e52a564440ac
parent 43cfa02659b49d1ca6954bbde141a94cc15c2921
parent f62f82d192fe989c560d690612843e84a27bb2cd

Merge pull request #424 from zilm13/fix/fixed-bytes-list

ssz: Fix reader.readFixedBytesList

ssz/src/main/java/org/apache/tuweni/ssz/BytesSSZReader.java

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 de2455c..1f0fbab 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 @@
 
   @Override
   public List<Bytes> readFixedBytesList(int byteLength, int limit) {
-    return readList(remaining -> readFixedBytes(byteLength, limit));
+    return readList(byteLength, () -> readFixedBytes(byteLength, limit));
   }
 
   @Override

ssz/src/test/java/org/apache/tuweni/ssz/BytesSSZReaderTest.java

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 46533c8..2a5656e 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.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 @@
     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)));
+  }
 }