From 4cbd03c099341e938134864a29504c1ae8fcf402 Mon Sep 17 00:00:00 2001 From: chaokunyang Date: Sun, 7 May 2023 17:27:52 +0800 Subject: [PATCH 1/2] add missing uint serializers --- .../io/fury/serializer/SerializersTest.java | 29 +++++++++++++++++++ 1 file changed, 29 insertions(+) diff --git a/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java b/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java index 6e13393457..f776220e8a 100644 --- a/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java +++ b/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java @@ -23,10 +23,39 @@ import io.fury.Fury; import io.fury.FuryTestBase; import io.fury.Language; +import io.fury.memory.MemoryBuffer; import org.testng.annotations.Test; public class SerializersTest extends FuryTestBase { + @Test + public void testUint8Serializer() { + Fury fury = Fury.builder().withLanguage(Language.XLANG).disableSecureMode().build(); + Serializers.Uint8Serializer serializer = new Serializers.Uint8Serializer(fury); + MemoryBuffer buffer = MemoryBuffer.newHeapBuffer(8); + serializer.crossLanguageWrite(buffer, 0); + assertEquals(serializer.crossLanguageRead(buffer), Integer.valueOf(0)); + serializer.crossLanguageWrite(buffer, 255); + assertEquals(serializer.crossLanguageRead(buffer), Integer.valueOf(255)); + assertThrows(IllegalArgumentException.class, () -> serializer.crossLanguageWrite(buffer, -1)); + assertThrows(IllegalArgumentException.class, () -> serializer.crossLanguageWrite(buffer, 256)); + } + + @Test + public void testUint16Serializer() { + Fury fury = Fury.builder().withLanguage(Language.XLANG).disableSecureMode().build(); + Serializers.Uint16Serializer serializer = new Serializers.Uint16Serializer(fury); + MemoryBuffer buffer = MemoryBuffer.newHeapBuffer(16); + serializer.crossLanguageWrite(buffer, 0); + assertEquals(serializer.crossLanguageRead(buffer), Integer.valueOf(0)); + serializer.crossLanguageWrite(buffer, 65535); + assertEquals(serializer.crossLanguageRead(buffer), Integer.valueOf(65535)); + assertThrows(IllegalArgumentException.class, () -> serializer.crossLanguageWrite(buffer, -1)); + assertThrows( + IllegalArgumentException.class, () -> serializer.crossLanguageWrite(buffer, 65536)); + } + + @Test(dataProvider = "crossLanguageReferenceTrackingConfig") public void testStringBuilder(boolean referenceTracking, Language language) { Fury fury1 = From dd8223bd9f10b104fc0bcefe54c1f8a276c412b0 Mon Sep 17 00:00:00 2001 From: chaokunyang Date: Sun, 7 May 2023 17:29:09 +0800 Subject: [PATCH 2/2] lint code --- .../src/test/java/io/fury/serializer/SerializersTest.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java b/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java index f776220e8a..ffe7d25885 100644 --- a/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java +++ b/java/fury-core/src/test/java/io/fury/serializer/SerializersTest.java @@ -52,10 +52,9 @@ public void testUint16Serializer() { assertEquals(serializer.crossLanguageRead(buffer), Integer.valueOf(65535)); assertThrows(IllegalArgumentException.class, () -> serializer.crossLanguageWrite(buffer, -1)); assertThrows( - IllegalArgumentException.class, () -> serializer.crossLanguageWrite(buffer, 65536)); + IllegalArgumentException.class, () -> serializer.crossLanguageWrite(buffer, 65536)); } - @Test(dataProvider = "crossLanguageReferenceTrackingConfig") public void testStringBuilder(boolean referenceTracking, Language language) { Fury fury1 =