20173570354

Committed 12 Dec 2025 04:44PM UTC coverage: 35.19% (-0.04%) from 35.229%

Build # 20173570354

Build Type

Pull #598

github

Committed by

web-flow

Commit Message

Merge d36bd8d87 into ea4577563

Pull Request Pull Request #598: feat(types): Argument coercion into HashSet/BTreeSet #493

Coverage Stats

15 of 42 new or added lines in 3 files covered. (35.71%)

2 existing lines in 1 file now uncovered.

1611 of 4578 relevant lines covered (35.19%)

8.51 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

31.25

/src/types/array/conversions/hash_set.rs

use super::super::ZendHashTable;
use crate::{
    boxed::ZBox,
    convert::{FromZval, IntoZval},
    error::{Error, Result},
    flags::DataType,
    types::Zval,
};
use std::collections::HashSet;
use std::convert::TryFrom;
use std::hash::{BuildHasher, Hash};

impl<'a, V, H> TryFrom<&'a ZendHashTable> for HashSet<V, H>
where
    V: FromZval<'a> + Eq + Hash,
    H: BuildHasher + Default,
{
    type Error = Error;

    fn try_from(value: &'a ZendHashTable) -> Result<Self> {
        let mut set = Self::with_capacity_and_hasher(value.len(), H::default());

        for (_key, val) in value {
            set.insert(V::from_zval(val).ok_or_else(|| Error::ZvalConversion(val.get_type()))?);
        }

        Ok(set)
    }
}

impl<V, H> TryFrom<HashSet<V, H>> for ZBox<ZendHashTable>
where
    V: IntoZval,
    H: BuildHasher,
{
    type Error = Error;

    fn try_from(value: HashSet<V, H>) -> Result<Self> {
        let mut ht = ZendHashTable::with_capacity(
            value.len().try_into().map_err(|_| Error::IntegerOverflow)?,
        );

        for (k, v) in value.into_iter().enumerate() {
            ht.insert(k, v)?;
        }

        Ok(ht)
    }
}

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,
    H: BuildHasher,
{
    const TYPE: DataType = DataType::Array;
    const NULLABLE: bool = false;

    fn set_zval(self, zv: &mut Zval, _: bool) -> Result<()> {
        let arr = self.try_into()?;
        zv.set_hashtable(arr);
        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());
        });
    }
}

1	use super::super::ZendHashTable;
2	use crate::{
3	boxed::ZBox,
4	convert::{FromZval, IntoZval},
5	error::{Error, Result},
6	flags::DataType,
7	types::Zval,
8	};
9	use std::collections::HashSet;
10	use std::convert::TryFrom;
11	use std::hash::{BuildHasher, Hash};
12
13	impl<'a, V, H> TryFrom<&'a ZendHashTable> for HashSet<V, H>
14	where
15	V: FromZval<'a> + Eq + Hash,
16	H: BuildHasher + Default,
17	{
18	type Error = Error;
19
NEW 20	fn try_from(value: &'a ZendHashTable) -> Result<Self> {	×
NEW 21	let mut set = Self::with_capacity_and_hasher(value.len(), H::default());	×
22
NEW 23	for (_key, val) in value {	×
NEW 24	set.insert(V::from_zval(val).ok_or_else(\|\| Error::ZvalConversion(val.get_type()))?);	×
25	}
26
NEW 27	Ok(set)	×
28	}
29	}
30
31	impl<V, H> TryFrom<HashSet<V, H>> for ZBox<ZendHashTable>
32	where
33	V: IntoZval,
34	H: BuildHasher,
35	{
36	type Error = Error;
37
38	fn try_from(value: HashSet<V, H>) -> Result<Self> {	1✔
39	let mut ht = ZendHashTable::with_capacity(
40	value.len().try_into().map_err(\|_\| Error::IntegerOverflow)?,	4✔
41	);
42
43	for (k, v) in value.into_iter().enumerate() {	5✔
44	ht.insert(k, v)?;	3✔
45	}
46
47	Ok(ht)	1✔
48	}
49	}
50
51	impl<'a, V, H> FromZval<'a> for HashSet<V, H>
52	where
53	V: FromZval<'a> + Eq + Hash,
54	H: BuildHasher + Default,
55	{
56	const TYPE: DataType = DataType::Array;
57
NEW 58	fn from_zval(zval: &'a Zval) -> Option<Self> {	×
NEW 59	zval.array().and_then(\|arr\| arr.try_into().ok())	×
60	}
61	}
62
63	impl<V, H> IntoZval for HashSet<V, H>
64	where
65	V: IntoZval,
66	H: BuildHasher,
67	{
68	const TYPE: DataType = DataType::Array;
69	const NULLABLE: bool = false;
70
NEW 71	fn set_zval(self, zv: &mut Zval, _: bool) -> Result<()> {	×
NEW 72	let arr = self.try_into()?;	×
NEW 73	zv.set_hashtable(arr);	×
NEW 74	Ok(())	×
75	}
76	}
77
78	#[cfg(test)]
79	#[cfg(feature = "embed")]
80	#[allow(clippy::unwrap_used)]
81	mod tests {
82	use std::collections::HashSet;
83
84	use crate::boxed::ZBox;
85	use crate::embed::Embed;
86	use crate::types::ZendHashTable;
87
88	#[test]
89	fn test_hash_table_try_from_hash_set() {
90	Embed::run(\|\| {
91	let mut set = HashSet::new();
92	set.insert("one");
93	let ht: ZBox<ZendHashTable> = set.try_into().unwrap();
94	assert_eq!(ht.len(), 1);
95	assert!(ht.get(0).is_some());
96	});
97	}
98	}

extphprs / ext-php-rs / 20173570354

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous