Skip to content

Commit fb140ba

Browse files
authored
[ci] de-flake app rewards test (#3643)
Signed-off-by: Itai Segall <itai.segall@digitalasset.com>
1 parent ec57d35 commit fb140ba

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

apps/app/src/test/scala/org/lfdecentralizedtrust/splice/integration/tests/WalletManualRoundsIntegrationTest.scala

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ import com.digitalasset.canton.logging.SuppressionRule
4141
import org.slf4j.event.Level
4242

4343
import java.time.Duration
44-
import scala.concurrent.duration.DurationInt
4544
import scala.jdk.CollectionConverters.*
4645

4746
class WalletManualRoundsIntegrationTest
@@ -315,9 +314,8 @@ class WalletManualRoundsIntegrationTest
315314

316315
aliceWalletClient.tap(20.0)
317316

318-
eventually(40.seconds) {
319-
aliceValidatorWalletClient.listAppRewardCoupons() should be(empty)
320-
}
317+
// We might have app reward coupons still from previous tests, we'll just ignore those in later stages of this test
318+
val initialAppRewardCoupons = aliceValidatorWalletClient.listAppRewardCoupons()
321319

322320
clue("Check that no payment requests exist") {
323321
aliceWalletClient.listAppPaymentRequests() shouldBe empty
@@ -344,7 +342,11 @@ class WalletManualRoundsIntegrationTest
344342
"Request no longer exists and validator has one unfeatured app reward",
345343
_ => {
346344
aliceWalletClient.listAppPaymentRequests() should have length 0
347-
inside(aliceValidatorWalletClient.listAppRewardCoupons()) { case Seq(c) =>
345+
inside(
346+
aliceValidatorWalletClient
347+
.listAppRewardCoupons()
348+
.filter(!initialAppRewardCoupons.contains(_))
349+
) { case Seq(c) =>
348350
// Award for the first (locking) leg goes to the sender's validator
349351
// The wallet is not a featured app, so no featured app reward even if the validator party is featured!
350352
c.payload.featured shouldBe false

0 commit comments

Comments
 (0)