Drop fake time advancing in scoring tests

Now that we use explicit times passed to decay methods, there's no
reason to make calls to `SinceEpoch::advance` in scoring tests.

Author: TheBlueMatt

lightning/src/routing/scoring.rs

@@ -2145,7 +2145,6 @@ mod tests {
 	use super::{ChannelLiquidity, HistoricalBucketRangeTracker, ProbabilisticScoringFeeParameters, ProbabilisticScoringDecayParameters, ProbabilisticScorer};
 	use crate::blinded_path::{BlindedHop, BlindedPath};
 	use crate::util::config::UserConfig;
-	use crate::util::time::tests::SinceEpoch;
 
 	use crate::ln::channelmanager;
 	use crate::ln::msgs::{ChannelAnnouncement, ChannelUpdate, UnsignedChannelAnnouncement, UnsignedChannelUpdate};
@@ -2819,7 +2818,6 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), u64::max_value());
 
 		// Half decay (i.e., three-quarter life)
-		SinceEpoch::advance(Duration::from_secs(5));
 		scorer.decay_liquidity_certainty(Duration::from_secs(5));
 		let usage = ChannelUsage { amount_msat: 128, ..usage };
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 22);
@@ -2831,7 +2829,6 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), u64::max_value());
 
 		// One decay (i.e., half life)
-		SinceEpoch::advance(Duration::from_secs(5));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10));
 		let usage = ChannelUsage { amount_msat: 64, ..usage };
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 0);
@@ -2843,7 +2840,6 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), u64::max_value());
 
 		// Fully decay liquidity lower bound.
-		SinceEpoch::advance(Duration::from_secs(10 * 7));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10 * 8));
 		let usage = ChannelUsage { amount_msat: 0, ..usage };
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 0);
@@ -2855,14 +2851,12 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), u64::max_value());
 
 		// Fully decay liquidity upper bound.
-		SinceEpoch::advance(Duration::from_secs(10));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10 * 9));
 		let usage = ChannelUsage { amount_msat: 0, ..usage };
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 0);
 		let usage = ChannelUsage { amount_msat: 1_024, ..usage };
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), u64::max_value());
 
-		SinceEpoch::advance(Duration::from_secs(10));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10 * 10));
 		let usage = ChannelUsage { amount_msat: 0, ..usage };
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 0);
@@ -2899,7 +2893,6 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 281);
 
 		// Decaying knowledge gives less confidence (128, 896), meaning a higher penalty.
-		SinceEpoch::advance(Duration::from_secs(10));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10));
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 291);
 
@@ -2914,7 +2907,6 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 245);
 
 		// Further decaying affects the lower bound more than the upper bound (128, 928).
-		SinceEpoch::advance(Duration::from_secs(10));
 		scorer.decay_liquidity_certainty(Duration::from_secs(20));
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 280);
 	}
@@ -2944,7 +2936,6 @@ mod tests {
 		scorer.payment_path_failed(&payment_path_for_amount(500), 42, Duration::ZERO);
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), u64::max_value());
 
-		SinceEpoch::advance(Duration::from_secs(10));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10));
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 473);
 
@@ -2985,7 +2976,6 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), u64::max_value());
 
 		if decay_before_reload {
-			SinceEpoch::advance(Duration::from_secs(10));
 			scorer.decay_liquidity_certainty(Duration::from_secs(10));
 		}
 
@@ -2996,7 +2986,6 @@ mod tests {
 		let mut deserialized_scorer =
 			<ProbabilisticScorer<_, _>>::read(&mut serialized_scorer, (decay_params, &network_graph, &logger)).unwrap();
 		if !decay_before_reload {
-			SinceEpoch::advance(Duration::from_secs(10));
 			scorer.decay_liquidity_certainty(Duration::from_secs(10));
 			deserialized_scorer.decay_liquidity_certainty(Duration::from_secs(10));
 		}
@@ -3005,7 +2994,6 @@ mod tests {
 		scorer.payment_path_failed(&payment_path_for_amount(250), 43, Duration::from_secs(10));
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 300);
 
-		SinceEpoch::advance(Duration::from_secs(10));
 		deserialized_scorer.decay_liquidity_certainty(Duration::from_secs(20));
 		assert_eq!(deserialized_scorer.channel_penalty_msat(42, &source, &target, usage, &params), 370);
 	}
@@ -3278,7 +3266,6 @@ mod tests {
 
 		// Advance the time forward 16 half-lives (which the docs claim will ensure all data is
 		// gone), and check that we're back to where we started.
-		SinceEpoch::advance(Duration::from_secs(10 * 16));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10 * 16));
 		assert_eq!(scorer.channel_penalty_msat(42, &source, &target, usage, &params), 168);
 		// Once fully decayed we still have data, so we get 0s for the buckets.
@@ -3304,7 +3291,6 @@ mod tests {
 		assert_eq!(scorer.channel_penalty_msat(42, &target, &source, usage, &params), 2048);
 
 		// Advance to decay all liquidity offsets to zero.
-		SinceEpoch::advance(Duration::from_secs(60 * 60 * 10));
 		scorer.decay_liquidity_certainty(Duration::from_secs(10 * (16 + 60 * 60)));
 
 		// Once even the bounds have decayed information about the channel should be removed