From dc2cbc21f973251e22901dbf3b4fe80e2fa25b58 Mon Sep 17 00:00:00 2001 From: "Tobin C. Harding" Date: Thu, 22 May 2025 11:26:59 +1000 Subject: [PATCH] Make fee_rate test identifiers uniform We have a bunch of unit tests, some use `f` and some use `fee_rate`. Uniform would be better. Elect to use the longer form just because there are only 4 instances of the shorter one (although I personally prefer the shorter form). --- units/src/fee_rate/mod.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/units/src/fee_rate/mod.rs b/units/src/fee_rate/mod.rs index f132c01c7..b9aa229ca 100644 --- a/units/src/fee_rate/mod.rs +++ b/units/src/fee_rate/mod.rs @@ -257,8 +257,8 @@ mod tests { assert_eq!(one.checked_add(two).unwrap(), three); - let f = FeeRate::from_sat_per_kwu(u64::MAX).checked_add(one); - assert!(f.is_none()); + let fee_rate = FeeRate::from_sat_per_kwu(u64::MAX).checked_add(one); + assert!(fee_rate.is_none()); } #[test] @@ -268,8 +268,8 @@ mod tests { let three = FeeRate::from_sat_per_kwu(3); assert_eq!(three.checked_sub(two).unwrap(), one); - let f = FeeRate::ZERO.checked_sub(one); - assert!(f.is_none()); + let fee_rate = FeeRate::ZERO.checked_sub(one); + assert!(fee_rate.is_none()); } #[test]