refactor: remove PartialEq and Eq from Specifier (#148)

oxc-project · Apr 24, 2024 · dd6f236 · dd6f236
1 parent e6adb88
commit dd6f236
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/src/specifier.rs b/src/specifier.rs
@@ -1,7 +1,7 @@
 use crate::error::SpecifierError;
 use std::borrow::Cow;
 
-#[derive(Debug, Eq, PartialEq)]
+#[derive(Debug)]
 pub struct Specifier<'a> {
     path: Cow<'a, str>,
     pub query: Option<&'a str>,
@@ -82,14 +82,21 @@ impl<'a> Specifier<'a> {
 mod tests {
     use super::{Specifier, SpecifierError};
 
+    #[test]
+    fn debug() {
+        let specifier = Specifier::parse("/").unwrap();
+        assert_eq!(
+            format!("{specifier:?}"),
+            r#"Specifier { path: "/", query: None, fragment: None }"#
+        );
+    }
+
     #[test]
     fn empty() {
         let specifiers = ["", "?"];
         for specifier in specifiers {
-            assert_eq!(
-                Specifier::parse(specifier),
-                Err(SpecifierError::Empty(specifier.to_string()))
-            );
+            let error = Specifier::parse(specifier).unwrap_err();
+            assert_eq!(error, SpecifierError::Empty(specifier.to_string()));
         }
     }