Removed the RelLockTime enum Error in error.rs

Author: AmosOO7

src/descriptor/error.rs

@@ -44,8 +44,6 @@ pub enum Error {
     Hex(bitcoin::hex::HexToBytesError),
     /// The provided wallet descriptors are identical
     ExternalAndInternalAreTheSame,
-    /// RelLockTime validation error
-    RelLockTime(miniscript::RelLockTimeError),
 }
 
 impl From<crate::keys::KeyError> for Error {
@@ -86,7 +84,6 @@ impl fmt::Display for Error {
             Self::ExternalAndInternalAreTheSame => {
                 write!(f, "External and internal descriptors are the same")
             }
-            Self::RelLockTime(err) => write!(f, "RelLockTime error: {err}"),
         }
     }
 }

tests/descriptor_macro.rs

@@ -20,7 +20,9 @@ fn test_older_with_invalid_rellocktime_too_large() {
     );
 
     // Check that it's the right kind of error
-    if let Err(descriptor::DescriptorError::Miniscript(miniscript::Error::RelativeLockTime(_))) = result {
+    if let Err(descriptor::DescriptorError::Miniscript(miniscript::Error::RelativeLockTime(_))) =
+        result
+    {
     } else {
         panic!("Expected RelLockTime error, got {:?}", result);
     }
@@ -91,30 +93,30 @@ fn test_rel_lock_time_error() {
     let invalid_value = 0x80000000u32;
     let rel_lock_time_result = miniscript::RelLockTime::from_consensus(invalid_value);
     assert!(rel_lock_time_result.is_err());
-    
+
     let rel_lock_time_error = rel_lock_time_result.unwrap_err();
-    
+
     // 2. Wrap it in the Miniscript error variant (matching your new macro logic)
     let minisc_err = miniscript::Error::RelativeLockTime(rel_lock_time_error);
-    
+
     // 3. Test the From impl (Miniscript -> DescriptorError)
     let error: descriptor::DescriptorError = minisc_err.into();
-    
+
     // Check that it matches the nested structure
     assert!(matches!(
-        error, 
+        error,
         descriptor::DescriptorError::Miniscript(miniscript::Error::RelativeLockTime(_))
     ));
 
     // 4. Test the Display impl
     let display_string = format!("{}", error);
-    
-    // FIX: Since the error is now wrapped in DescriptorError::Miniscript, 
+
+    // FIX: Since the error is now wrapped in DescriptorError::Miniscript,
     // the string likely starts with "Miniscript error:" or similar.
     // Using .contains() is more robust for nested errors.
     assert!(
         display_string.contains("relative locktime"),
-        "Display string was: '{}'", display_string
+        "Display string was: '{}'",
+        display_string
     );
 }
-