Conversions for package & revision types

src/lib.rs

@@ -259,6 +259,12 @@ impl From<&str> for Package {
     }
 }
 
+impl From<&Package> for Package {
+    fn from(value: &Package) -> Self {
+        value.clone()
+    }
+}
+
 impl std::fmt::Display for Package {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         write!(f, "{}", self.0)
@@ -318,6 +324,12 @@ impl From<&str> for Revision {
     }
 }
 
+impl From<&Revision> for Revision {
+    fn from(value: &Revision) -> Self {
+        value.clone()
+    }
+}
+
 impl std::fmt::Display for Revision {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {

src/locator.rs

@@ -409,6 +409,18 @@ mod tests {
 
     use super::*;
 
+    #[test]
+    fn from_existing() {
+        let first = locator!(Git, "github.com/foo/bar");
+        let second = Locator::builder()
+            .fetcher(first.fetcher())
+            .maybe_org_id(first.org_id())
+            .package(first.package())
+            .maybe_revision(first.revision().as_ref())
+            .build();
+        assert_eq!(first, second);
+    }
+
     #[test]
     fn optional_fields() {
         let with_options = Locator::builder()

src/locator_package.rs

@@ -266,6 +266,17 @@ mod tests {
 
     use super::*;
 
+    #[test]
+    fn from_existing() {
+        let first = package!(Git, "github.com/foo/bar");
+        let second = PackageLocator::builder()
+            .fetcher(first.fetcher())
+            .maybe_org_id(first.org_id())
+            .package(first.package())
+            .build();
+        assert_eq!(first, second);
+    }
+
     #[test]
     fn optional_fields() {
         let with_options = PackageLocator::builder()

src/locator_strict.rs

@@ -239,6 +239,18 @@ mod tests {
 
     use super::*;
 
+    #[test]
+    fn from_existing() {
+        let first = strict!(Git, "github.com/foo/bar", "abcd");
+        let second = StrictLocator::builder()
+            .fetcher(first.fetcher())
+            .maybe_org_id(first.org_id())
+            .package(first.package())
+            .revision(first.revision())
+            .build();
+        assert_eq!(first, second);
+    }
+
     #[test]
     fn optional_fields() {
         let with_options = StrictLocator::builder()