Skip to content

Commit

Permalink
fixed tests
Browse files Browse the repository at this point in the history
  • Loading branch information
phearnot committed Nov 27, 2023
1 parent 8eddc72 commit 5a822a7
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 10 deletions.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package com.wavesplatform.api.grpc.test

import scala.concurrent.{Await, Future}
import scala.concurrent.duration.Duration
import scala.concurrent.duration.*
import com.wavesplatform.account.Address
import com.wavesplatform.common.state.ByteStr
import com.wavesplatform.test.{FlatSpec, TestTime}
Expand Down Expand Up @@ -91,7 +91,7 @@ class GRPCBroadcastSpec extends FlatSpec with BeforeAndAfterAll with PathMockFac

@throws[StatusException]("on failed broadcast")
def assertBroadcast(tx: Transaction): Unit = {
Await.result(grpcTxApi.broadcast(PBTransactions.protobuf(tx)), Duration.Inf)
Await.result(grpcTxApi.broadcast(PBTransactions.protobuf(tx)), 10.seconds)
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -420,7 +420,7 @@ class BlockchainUpdatesSpec extends FreeSpec with WithBUDomain with ScalaFutures
(1 to blocksCount + 1).foreach(_ => d.appendBlock())

val result = Await
.result(r.getBlockUpdatesRange(GetBlockUpdatesRangeRequest(1, blocksCount)), Duration.Inf)
.result(r.getBlockUpdatesRange(GetBlockUpdatesRangeRequest(1, blocksCount)), 1.minute)
.updates
.map(_.update.append.map(_.getBlock.vrf.toByteStr).filterNot(_.isEmpty))

Expand Down Expand Up @@ -1215,7 +1215,7 @@ class BlockchainUpdatesSpec extends FreeSpec with WithBUDomain with ScalaFutures
Await
.result(
repo.getBlockUpdate(GetBlockUpdateRequest(height)),
Duration.Inf
1.minute
)
.getUpdate
.update
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ class BlockWithMaxBaseTargetTest extends FreeSpec with WithNewDBForEachTest with
case _: SecurityException =>
Task.unit
}
Await.result(blockAppendTask.runToFuture(scheduler), Duration.Inf)
Await.result(blockAppendTask.runToFuture(scheduler), 1.minute)

signal.tryAcquire(10, TimeUnit.SECONDS)

Expand Down
12 changes: 10 additions & 2 deletions node/src/test/scala/com/wavesplatform/state/LightNodeTest.scala
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,15 @@ class LightNodeTest extends PropSpec with WithDomain {
)

val appender =
ExtensionAppender(d.blockchain, d.utxPool, d.posSelector, TestTime(), InvalidBlockStorage.NoOp, PeerDatabase.NoOp, Scheduler.global)(
ExtensionAppender(
d.blockchain,
d.utxPool,
d.posSelector,
TestTime(extensionBlocks.blocks.last.header.timestamp),
InvalidBlockStorage.NoOp,
PeerDatabase.NoOp,
Scheduler.global
)(
null,
_
)
Expand All @@ -162,7 +170,7 @@ class LightNodeTest extends PropSpec with WithDomain {
val challengingBlock = d.createChallengingBlock(challengingMiner, invalidBlock, strictTime = true)
val txSnapshots = getTxSnapshots(d, challengingBlock)

val appender = BlockAppender(d.blockchainUpdater, TestTime(), d.utxPool, d.posSelector, Scheduler.global) _
val appender = BlockAppender(d.blockchainUpdater, TestTime(challengingBlock.header.timestamp), d.utxPool, d.posSelector, Scheduler.global) _

appender(challengingBlock, Some(BlockSnapshot(challengingBlock.id(), txSnapshots))).runSyncUnsafe() shouldBe Right(
Applied(Seq.empty, d.blockchain.score)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ import java.util.concurrent.locks.ReentrantLock
import scala.concurrent.ExecutionContext.Implicits.global
import scala.concurrent.{Await, Future}
import scala.concurrent.duration.*
import scala.concurrent.duration.Duration.Inf

class TransactionsByAddressSpec extends FreeSpec with BlockGen with WithDomain {
def transfers(sender: KeyPair, rs: AddressOrAlias, amount: Long): Seq[TransferTransaction] =
Expand Down Expand Up @@ -122,7 +121,7 @@ class TransactionsByAddressSpec extends FreeSpec with BlockGen with WithDomain {
val txs = Future { d.addressTransactions(defaultAddress).map(_._2.tpe) }
d.blockchain.bestLiquidSnapshot.synchronized(d.appendKeyBlock())
startRead.unlock()
Await.result(txs, Inf).map(_.tpe) shouldBe List(TransactionType.Issue)
Await.result(txs, 1.minute).map(_.tpe) shouldBe List(TransactionType.Issue)
}
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1994,7 +1994,7 @@ class ExchangeTransactionDiffTest extends PropSpec with Inside with WithDomain w
TxExchangeAmount(1),
TxOrderPrice(1),
System.currentTimeMillis(),
System.currentTimeMillis() + 10000,
System.currentTimeMillis() + 10.hours,
TxMatcherFee.unsafeFrom(0.003.waves)
)
val signature = EthOrders.signOrder(buyOrder, signer)
Expand Down

0 comments on commit 5a822a7

Please sign in to comment.