todo removed, tests improved

ergoplatform · Jun 3, 2024 · bc71b2f · bc71b2f
1 parent b78ff7a
commit bc71b2f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/core/shared/src/main/scala/sigma/serialization/CoreDataSerializer.scala b/core/shared/src/main/scala/sigma/serialization/CoreDataSerializer.scala
@@ -74,7 +74,6 @@ class CoreDataSerializer {
         serialize(v, elemType, w)
       }
 
-    // TODO v6.0 (3h): support Option[T] (see https://github.com/ScorexFoundation/sigmastate-interpreter/issues/659)
     case _ =>
       CheckSerializableTypeCode(tpe.typeCode)
       throw new SerializerException(s"Don't know how to serialize ($v, $tpe)")

diff --git a/interpreter/shared/src/test/scala/sigma/serialization/DataSerializerSpecification.scala b/interpreter/shared/src/test/scala/sigma/serialization/DataSerializerSpecification.scala
@@ -111,6 +111,7 @@ class DataSerializerSpecification extends SerializationSpecification {
         roundtrip[SOption[T]](Some(in), SOption(tpe))
         roundtrip[SOption[SCollection[T]]](Some(Colls.fromItems(in)(tT)), SOption(SCollectionType(tpe)))
         roundtrip[SCollection[SOption[T]]](Colls.fromItems(Option(in), None.asInstanceOf[Option[T#WrappedType]])(OptionType(tT)), SCollectionType(SOption(tpe)))
+        roundtrip[SOption[SOption[T]]](None, SOption(SOption(tpe)))
         roundtrip[SOption[SOption[T]]](Some(Some(in)), SOption(SOption(tpe)))
       }
     }