tx-signing-js: fixed test for isoUnsignedTransaction

ergoplatform · Aug 5, 2023 · 4c5b84c · 4c5b84c
1 parent a9fe1c5
commit 4c5b84c
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/interpreter/shared/src/test/scala/sigmastate/serialization/generators/ObjectGenerators.scala b/interpreter/shared/src/test/scala/sigmastate/serialization/generators/ObjectGenerators.scala
@@ -27,6 +27,7 @@ import special.sigma._
 
 import java.math.BigInteger
 import scala.collection.compat.immutable.ArraySeq
+import scala.collection.immutable.VectorMap
 import scala.collection.mutable.ListBuffer
 import scala.reflect.ClassTag
 
@@ -238,7 +239,7 @@ trait ObjectGenerators extends TypeGenerators
 
   lazy val contextExtensionGen: Gen[ContextExtension] = for {
     values <- Gen.sequence(contextExtensionValuesGen(0, 5))(Buildable.buildableSeq)
-  } yield ContextExtension(values.toMap)
+  } yield ContextExtension(VectorMap.from(values.sortBy(_._1)))
 
   lazy val serializedProverResultGen: Gen[ProverResult] = for {
     bytes <- arrayOfRange(1, 100, arbByte.arbitrary)

diff --git a/sdk/js/src/main/scala/org/ergoplatform/sdk/js/Isos.scala b/sdk/js/src/main/scala/org/ergoplatform/sdk/js/Isos.scala
@@ -200,7 +200,7 @@ object Isos {
   implicit val isoContextExtension: Iso[contextExtensionMod.ContextExtension, ContextExtension] = new Iso[contextExtensionMod.ContextExtension, ContextExtension] {
     override def to(x: contextExtensionMod.ContextExtension): ContextExtension = {
       var map = new ListMap[Byte, Constant[SType]]()
-      val keys = js.Object.keys(x)
+      val keys = js.Object.keys(x).sorted
       for ( k <- keys ) {
         val id = k.toInt.toByte
         val c = isoHexStringToConstant.to(x.apply(id).get.get)

diff --git a/sdk/js/src/test/scala/org/ergoplatform/sdk/js/IsosSpec.scala b/sdk/js/src/test/scala/org/ergoplatform/sdk/js/IsosSpec.scala
@@ -37,8 +37,7 @@ class IsosSpec  extends AnyPropSpec with Matchers with ObjectGenerators with Sca
     )
 
   def roundtrip[A,B](iso: Iso[A,B])(b: B): Unit = {
-    val invIso = iso.inverse
-    invIso.from(invIso.to(b)) shouldBe b
+    iso.to(iso.from(b)) shouldBe b
   }
 
   override implicit val generatorDrivenConfig: PropertyCheckConfiguration = PropertyCheckConfiguration(minSuccessful = 30)
@@ -164,7 +163,7 @@ class IsosSpec  extends AnyPropSpec with Matchers with ObjectGenerators with Sca
     }
   }
 
-  ignore("Iso.isoUnsignedTransaction") {
+  property("Iso.isoUnsignedTransaction") {
     forAll { (tx: UnsignedErgoLikeTransaction) =>
       roundtrip(Isos.isoUnsignedTransaction)(tx)
     }

diff --git a/sdk/shared/src/test/scala/org/ergoplatform/sdk/ExtensionsSpec.scala b/sdk/shared/src/test/scala/org/ergoplatform/sdk/ExtensionsSpec.scala
@@ -91,7 +91,7 @@ class ExtensionsSpec extends AnyPropSpec with ScalaCheckPropertyChecks with Matc
       val left = builder.pairColl(leftKeys, leftValues)
       val right = builder.pairColl(rightKeys, rightValues)
       val res = builder.outerJoin(left, right)(l => l._2 - 2, r => r._2 - 3, i => i._2._1 + 5)
-      val (ks, vs) = builder.unzip(res)
+      val (_, vs) = builder.unzip(res)
       vs.sum shouldBe (col.sum * 2 + col.map(_ + 5).sum)
     }
     //    test(builder.fromItems(0))