altoo-ag / scala-kryo-serialization

kryo-based serializers for Scala
Apache License 2.0
11 stars 1 forks source link

Follow-on from work on ScalaObjectSerializer supporting copying. #20

Open sageserpent-open opened 1 month ago

sageserpent-open commented 1 month ago

This follows on from #16 - I noticed a minor omission in one of the tests.

For the sake of expediency, I've enclosed a patch with this ticket. Hope that suffices, the changes are minor.

sageserpent-open commented 1 month ago

Subject: [PATCH] Return the copied object from the helper AbstractKryoTest.testCopyingOf, and use a more revealing name for the corresponding result in AbstractKryoTest.testSerialization.of.

Index: core/src/test/scala/io/altoo/serialization/kryo/scala/testkit/AbstractKryoTest.scala IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP <+>UTF-8

diff --git a/core/src/test/scala/io/altoo/serialization/kryo/scala/testkit/AbstractKryoTest.scala b/core/src/test/scala/io/altoo/serialization/kryo/scala/testkit/AbstractKryoTest.scala --- a/core/src/test/scala/io/altoo/serialization/kryo/scala/testkit/AbstractKryoTest.scala (revision 09e14850342128a4544010c786d9140ef75d716c) +++ b/core/src/test/scala/io/altoo/serialization/kryo/scala/testkit/AbstractKryoTest.scala (date 1728636466202) @@ -46,12 +46,12 @@ output.close()

 val input = new Input(new ByteArrayInputStream(serialized), 4096)
sageserpent-open commented 1 month ago

Move conveniently: issue-20.patch