Skip to content

Commit

Permalink
nits
Browse files Browse the repository at this point in the history
  • Loading branch information
pm47 committed Jun 19, 2023
1 parent 309540c commit dbf8952
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ sealed trait ConfirmationPriority {
case ConfirmationPriority.Medium => feerates.medium
case ConfirmationPriority.Fast => feerates.fast
}
override def toString: String = super.toString.toLowerCase
override def toString: String = super.toString.toLowerCase
}
object ConfirmationPriority {
case object Slow extends ConfirmationPriority
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -245,9 +245,6 @@ private class ReplaceableTxPublisher(nodeParams: NodeParams,
None
}
}



// We avoid a herd effect whenever we fee bump transactions.
targetFeerate_opt.foreach(targetFeerate => timers.startSingleTimer(BumpFeeKey, BumpFee(targetFeerate), (1 + Random.nextLong(nodeParams.channelConf.maxTxPublishRetryDelay.toMillis)).millis))
Behaviors.same
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ import fr.acinq.eclair.Features.StaticRemoteKey
import fr.acinq.eclair.TestConstants.{Alice, Bob}
import fr.acinq.eclair._
import fr.acinq.eclair.blockchain.bitcoind.ZmqWatcher._
import fr.acinq.eclair.blockchain.fee.{ConfirmationTarget, FeeratePerByte, FeeratePerKw, FeeratesPerKw}
import fr.acinq.eclair.blockchain.fee.{ConfirmationPriority, ConfirmationTarget, FeeratePerByte, FeeratePerKw, FeeratesPerKw}
import fr.acinq.eclair.blockchain.{CurrentBlockHeight, CurrentFeerates}
import fr.acinq.eclair.channel.RealScidStatus.Final
import fr.acinq.eclair.channel._
Expand Down Expand Up @@ -3439,7 +3439,7 @@ class NormalStateSpec extends TestKitBaseClass with FixtureAnyFunSuiteLike with
} else {
val localAnchor = alice2blockchain.expectMsgType[PublishReplaceableTx]
// When there are no pending HTLCs, there is no absolute deadline to get the commit tx confirmed, we use priority
assert(localAnchor.txInfo.confirmationTarget.isInstanceOf[ConfirmationTarget.Priority])
assert(localAnchor.txInfo.confirmationTarget == ConfirmationTarget.Priority(ConfirmationPriority.Medium))
val claimMain = alice2blockchain.expectMsgType[PublishFinalTx]
assert(alice2blockchain.expectMsgType[WatchTxConfirmed].txId === aliceCommitTx.txid)
assert(alice2blockchain.expectMsgType[WatchTxConfirmed].txId === claimMain.tx.txid)
Expand Down

0 comments on commit dbf8952

Please sign in to comment.