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

[kakserpom]

Description

Implements #493

[coveralls]

Pull Request Test Coverage Report for Build 20173570354

Details

• 15 of 42 (35.71%) changed or added relevant lines in 3 files are covered.
• 2 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.04%) to 35.19%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/types/array/array_key.rs	5	10	50.0%
src/types/array/conversions/btree_set.rs	5	16	31.25%
src/types/array/conversions/hash_set.rs	5	16	31.25%

Files with Coverage Reduction	New Missed Lines	%
src/types/array/array_key.rs	2	53.06%

Totals
Change from base Build 20110536661:	-0.04%
Covered Lines:	1611
Relevant Lines:	4578

---

💛 - Coveralls

[Xenira]

Thanks. Can you add unit tests?

[kakserpom]

I am not sure what should be tested here.

[ptondereau]

Thanks for the implementation! Just some comments.

[ptondereau]

I am not sure what should be tested here.

for example:

#[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());
      });
}