From 593fedcdd46249788516607947a6989a0181762d Mon Sep 17 00:00:00 2001
From: Satoshi Qazi Muhammed <info@pezkuwichain.io>
Date: Thu, 9 Jul 2026 12:34:02 -0700
Subject: [PATCH] fix: avoid suspend call inside non-inline
 removeAll(predicate)

removeAll { assetDao.getAsset(...) } failed to compile ("Suspension
functions can only be called within coroutine body"). filter{} is
unambiguously inline and safe for a suspend call inside a suspend
function; pair it with the plain Collection-based removeAll(elements)
overload instead, which takes no lambda at all.
---
 .../nova/balances/PezkuwiFullArchitectureBalancesTest.kt       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/src/androidTest/java/io/novafoundation/nova/balances/PezkuwiFullArchitectureBalancesTest.kt b/app/src/androidTest/java/io/novafoundation/nova/balances/PezkuwiFullArchitectureBalancesTest.kt
index 7ecf6dd6..d0d6fe34 100644
--- a/app/src/androidTest/java/io/novafoundation/nova/balances/PezkuwiFullArchitectureBalancesTest.kt
+++ b/app/src/androidTest/java/io/novafoundation/nova/balances/PezkuwiFullArchitectureBalancesTest.kt
@@ -68,9 +68,10 @@ class PezkuwiFullArchitectureBalancesTest {
             val stillMissing = fixture.assets.toMutableList()
             withTimeoutOrNull(120.seconds) {
                 while (stillMissing.isNotEmpty()) {
-                    stillMissing.removeAll { asset ->
+                    val found = stillMissing.filter { asset ->
                         assetDao.getAsset(metaId, asset.chainId, asset.assetId) != null
                     }
+                    stillMissing.removeAll(found)
                     if (stillMissing.isNotEmpty()) delay(2.seconds)
                 }
             }