Ran cargo fmt and cargo clippy

Author: AmosOO7

tests/test_rellocktime_issue.rs

@@ -4,18 +4,21 @@
 /// which can panic if given an invalid value (> 16,777,215).
 ///
 /// This file shows the problem and how to catch it.
-
 use bdk_wallet::descriptor;
 
 #[test]
 fn test_older_with_invalid_rellocktime_too_large() {
     // Value with high bit set causes RelLockTime::from_consensus() to fail
     let invalid_value = 0x80000000; // 2147483648
-    
+
     // This should now return an error instead of panicking
     let result = descriptor!(wsh(older(invalid_value)));
-    assert!(result.is_err(), "Invalid RelLockTime {} should return an error", invalid_value);
-    
+    assert!(
+        result.is_err(),
+        "Invalid RelLockTime {} should return an error",
+        invalid_value
+    );
+
     // Check that it's the right kind of error
     if let Err(descriptor::DescriptorError::RelLockTime(_)) = result {
         // Good, it's the expected error type
@@ -36,7 +39,7 @@ fn test_older_with_valid_rellocktime_max() {
 fn test_older_with_valid_rellocktime_min() {
     // Min valid value is 1 (0 is not valid for RelLockTime)
     let min_value = 1;
-    
+
     let result = descriptor!(wsh(older(min_value)));
     assert!(result.is_ok(), "Min valid RelLockTime should work");
 }
@@ -45,20 +48,16 @@ fn test_older_with_valid_rellocktime_min() {
 fn test_older_with_valid_common_values() {
     // Common usage: blocks or seconds
     let test_cases = vec![
-        1,           // 1 block/second
-        144,         // ~1 day in blocks
-        1000,        // Common value
-        65535,       // Max 16-bit value (common for CSV)
-        4_209_713,   // Valid but large
+        1,         // 1 block/second
+        144,       // ~1 day in blocks
+        1000,      // Common value
+        65535,     // Max 16-bit value (common for CSV)
+        4_209_713, // Valid but large
     ];
-    
+
     for value in test_cases {
         let result = descriptor!(wsh(older(value)));
-        assert!(
-            result.is_ok(),
-            "Valid RelLockTime {} should work",
-            value
-        );
+        assert!(result.is_ok(), "Valid RelLockTime {} should work", value);
     }
 }
 
@@ -67,20 +66,19 @@ fn test_older_with_valid_common_values() {
 fn test_demonstrate_proper_error_handling() {
     // This is what the fix should look like - proper Result handling
     // instead of .expect() which panics
-    
+
     use miniscript::RelLockTime;
-    
+
     // Valid case
     match RelLockTime::from_consensus(144) {
         Ok(lock_time) => println!("Valid: {:?}", lock_time),
         Err(e) => println!("Invalid: {}", e),
     }
-    
+
     // Invalid case - properly handled
     match RelLockTime::from_consensus(16_777_216) {
         Ok(lock_time) => println!("Valid: {:?}", lock_time),
         Err(e) => {
-            // Instead of panicking, we could return a proper DescriptorError
             println!("Invalid RelLockTime value: {}", e);
         }
     }