feat(types): array: impl FromZval for HashSet/BTreeSet

Author: kakserpom

src/types/array/conversions/btree_set.rs

@@ -45,6 +45,17 @@ where
     }
 }
 
+impl<'a, V> FromZval<'a> for BTreeSet<V>
+where
+    V: FromZval<'a> + Ord,
+{
+    const TYPE: DataType = DataType::Array;
+
+    fn from_zval(zval: &'a Zval) -> Option<Self> {
+        zval.array().and_then(|arr| arr.try_into().ok())
+    }
+}
+
 impl<V> IntoZval for BTreeSet<V>
 where
     V: IntoZval,
@@ -58,3 +69,25 @@ where
         Ok(())
     }
 }
+
+#[cfg(test)]
+#[cfg(feature = "embed")]
+#[allow(clippy::unwrap_used)]
+mod tests {
+    use std::collections::BTreeSet;
+
+    use crate::boxed::ZBox;
+    use crate::embed::Embed;
+    use crate::types::ZendHashTable;
+
+    #[test]
+    fn test_hash_table_try_from_btree_set() {
+        Embed::run(|| {
+            let mut set = BTreeSet::new();
+            set.insert("one");
+            let ht: ZBox<ZendHashTable> = set.try_into().unwrap();
+            assert_eq!(ht.len(), 1);
+            assert!(ht.get(0).is_some());
+        });
+    }
+}

src/types/array/conversions/hash_set.rs

@@ -48,6 +48,18 @@ where
     }
 }
 
+impl<'a, V, H> FromZval<'a> for HashSet<V, H>
+where
+    V: FromZval<'a> + Eq + Hash,
+    H: BuildHasher + Default,
+{
+    const TYPE: DataType = DataType::Array;
+
+    fn from_zval(zval: &'a Zval) -> Option<Self> {
+        zval.array().and_then(|arr| arr.try_into().ok())
+    }
+}
+
 impl<V, H> IntoZval for HashSet<V, H>
 where
     V: IntoZval,
@@ -62,3 +74,25 @@ where
         Ok(())
     }
 }
+
+#[cfg(test)]
+#[cfg(feature = "embed")]
+#[allow(clippy::unwrap_used)]
+mod tests {
+    use std::collections::HashSet;
+
+    use crate::boxed::ZBox;
+    use crate::embed::Embed;
+    use crate::types::ZendHashTable;
+
+    #[test]
+    fn test_hash_table_try_from_hash_set() {
+        Embed::run(|| {
+            let mut set = HashSet::new();
+            set.insert("one");
+            let ht: ZBox<ZendHashTable> = set.try_into().unwrap();
+            assert_eq!(ht.len(), 1);
+            assert!(ht.get(0).is_some());
+        });
+    }
+}

src/types/array/conversions/mod.rs

@@ -7,7 +7,9 @@
 //! ## Supported Collections
 //!
 //! - `BTreeMap<K, V>` ↔ `ZendHashTable` (via `btree_map` module)
+//! - `BTreeSet<V>` ↔ `ZendHashTable` (via `btree_set` module)
 //! - `HashMap<K, V>` ↔ `ZendHashTable` (via `hash_map` module)
+//! - `HashSet<V>` ↔ `ZendHashTable` (via `hash_set` module)
 //! - `Vec<T>` and `Vec<(K, V)>` ↔ `ZendHashTable` (via `vec` module)
 
 mod btree_map;