itest: preparatory fee estimation changes

This commit is contained in:
Slyghtning
2024-02-20 14:30:28 +01:00
parent 7d9589ecbf
commit ef069b658d
5 changed files with 46 additions and 41 deletions

View File

@@ -89,7 +89,14 @@ func testSendPaymentAMPInvoiceCase(ht *lntest.HarnessTest,
// Increase Dave's fee to make the test deterministic. Otherwise it
// would be unpredictable whether pathfinding would go through Charlie
// or Dave for the first shard.
expectedPolicy := mts.updateDaveGlobalPolicy()
expectedPolicy := &lnrpc.RoutingPolicy{
FeeBaseMsat: 500_000,
FeeRateMilliMsat: int64(0.001 * 1_000_000),
TimeLockDelta: 40,
MinHtlc: 1000, // default value
MaxHtlcMsat: 133_650_000,
}
mts.dave.UpdateGlobalPolicy(expectedPolicy)
// Make sure Alice has heard it for both Dave's channels.
ht.AssertChannelPolicyUpdate(
@@ -382,10 +389,17 @@ func testSendPaymentAMP(ht *lntest.HarnessTest) {
mts.openChannels(mppReq)
chanPointAliceDave := mts.channelPoints[1]
// Increase Dave's fee to make the test deterministic. Otherwise it
// Increase Dave's fee to make the test deterministic. Otherwise, it
// would be unpredictable whether pathfinding would go through Charlie
// or Dave for the first shard.
expectedPolicy := mts.updateDaveGlobalPolicy()
expectedPolicy := &lnrpc.RoutingPolicy{
FeeBaseMsat: 500_000,
FeeRateMilliMsat: int64(0.001 * 1_000_000),
TimeLockDelta: 40,
MinHtlc: 1000, // default value
MaxHtlcMsat: 133_650_000,
}
mts.dave.UpdateGlobalPolicy(expectedPolicy)
// Make sure Alice has heard it.
ht.AssertChannelPolicyUpdate(
@@ -493,7 +507,7 @@ func testSendToRouteAMP(ht *lntest.HarnessTest) {
// Alice -- Carol ---- Bob
// \ /
// \__ Dave ____/
///
//
mppReq := &mppOpenChannelRequest{
// Since the channel Alice-> Carol will have to carry two
// shards, we make it larger.

View File

@@ -347,33 +347,3 @@ func (m *mppTestScenario) buildRoute(amt btcutil.Amount,
return routeResp.Route
}
// updatePolicy updates a Dave's global channel policy and returns the expected
// policy for further check. It changes Dave's `FeeBaseMsat` from 1000 msat to
// 500,000 msat, and `FeeProportionalMillonths` from 1 msat to 1000 msat.
func (m *mppTestScenario) updateDaveGlobalPolicy() *lnrpc.RoutingPolicy {
const (
baseFeeMsat = 500_000
feeRate = 0.001
maxHtlcMsat = 133_650_000
)
expectedPolicy := &lnrpc.RoutingPolicy{
FeeBaseMsat: baseFeeMsat,
FeeRateMilliMsat: feeRate * testFeeBase,
TimeLockDelta: 40,
MinHtlc: 1000, // default value
MaxHtlcMsat: maxHtlcMsat,
}
updateFeeReq := &lnrpc.PolicyUpdateRequest{
BaseFeeMsat: baseFeeMsat,
FeeRate: feeRate,
TimeLockDelta: 40,
Scope: &lnrpc.PolicyUpdateRequest_Global{Global: true},
MaxHtlcMsat: maxHtlcMsat,
}
m.dave.RPC.UpdateChannelPolicy(updateFeeReq)
return expectedPolicy
}

View File

@@ -38,10 +38,17 @@ func testSendMultiPathPayment(ht *lntest.HarnessTest) {
mts.openChannels(req)
chanPointAliceDave := mts.channelPoints[1]
// Increase Dave's fee to make the test deterministic. Otherwise it
// Increase Dave's fee to make the test deterministic. Otherwise, it
// would be unpredictable whether pathfinding would go through Charlie
// or Dave for the first shard.
expectedPolicy := mts.updateDaveGlobalPolicy()
expectedPolicy := &lnrpc.RoutingPolicy{
FeeBaseMsat: 500_000,
FeeRateMilliMsat: int64(0.001 * 1_000_000),
TimeLockDelta: 40,
MinHtlc: 1000, // default value
MaxHtlcMsat: 133_650_000,
}
mts.dave.UpdateGlobalPolicy(expectedPolicy)
// Make sure Alice has heard it.
ht.AssertChannelPolicyUpdate(