fix(pollux): add asc ordering to the VerificationPolicy constraints A…

…TL-6040 Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
hyperledger · Oct 24, 2023 · 5a38b6e · 5a38b6e
1 parent 0bd234e
commit 5a38b6e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.../lib/sql-doobie/src/main/scala/io/iohk/atala/pollux/sql/model/db/VerificationPolicy.scala b/.../lib/sql-doobie/src/main/scala/io/iohk/atala/pollux/sql/model/db/VerificationPolicy.scala
@@ -75,7 +75,9 @@ object VerificationPolicySql extends DoobieContext.Postgres(SnakeCase) {
   def getVerificationPolicyConstrains(fk_ids: Seq[UUID]) =
     run(
       quote(
-        query[VerificationPolicyConstraint].filter(vpc => liftQuery(fk_ids).contains(vpc.fk_id))
+        query[VerificationPolicyConstraint]
+          .filter(vpc => liftQuery(fk_ids).contains(vpc.fk_id))
+          .sortBy(_.index)(Ord.asc)
       )
     )
 

diff --git a/...doobie/src/test/scala/io/iohk/atala/pollux/sql/VerificationPolicySqlIntegrationSpec.scala b/...doobie/src/test/scala/io/iohk/atala/pollux/sql/VerificationPolicySqlIntegrationSpec.scala
@@ -286,7 +286,7 @@ object VerificationPolicySqlIntegrationSpec extends ZIOSpecDefault, PostgresTest
       name <- name
       description <- description
       constraints <- Gen
-        .setOfBounded(min = 1, max = 10)(verificationPolicyConstraint)
+        .setOfBounded(min = 2, max = 10)(verificationPolicyConstraint)
         .map(_.toVector)
     } yield VerificationPolicy.make(
       name = name,