Add HashMap.toValues and HashSet.toValues (#4317)

Effect-TS · Feb 14, 2025 · 7080b67 · 7080b67
1 parent adc60e8
commit 7080b67
Show file tree

Hide file tree

Showing 7 changed files with 38 additions and 2 deletions.
diff --git a/.changeset/three-bees-attack.md b/.changeset/three-bees-attack.md
@@ -0,0 +1,5 @@
+---
+"effect": minor
+---
+
+Add `HashMap.toValues` and `HashSet.toValues` getters
diff --git a/packages/effect/src/HashMap.ts b/packages/effect/src/HashMap.ts
@@ -229,6 +229,14 @@ export const keySet: <K, V>(self: HashMap<K, V>) => HashSet<K> = keySet_.keySet
  */
 export const values: <K, V>(self: HashMap<K, V>) => IterableIterator<V> = HM.values
 
+/**
+ * Returns an `Array` of the values within the `HashMap`.
+ *
+ * @since 3.13.0
+ * @category getters
+ */
+export const toValues = <K, V>(self: HashMap<K, V>): Array<V> => Array.from(values(self))
+
 /**
  * Returns an `IterableIterator` of the entries within the `HashMap`.
  *

diff --git a/packages/effect/src/HashSet.ts b/packages/effect/src/HashSet.ts
@@ -115,6 +115,14 @@ export const isSubset: {
  */
 export const values: <A>(self: HashSet<A>) => IterableIterator<A> = HS.values
 
+/**
+ * Returns an `Array` of the values within the `HashSet`.
+ *
+ * @since 3.13.0
+ * @category getters
+ */
+export const toValues = <A>(self: HashSet<A>): Array<A> => Array.from(values(self))
+
 /**
  * Calculates the number of values in the `HashSet`.
  *

diff --git a/packages/effect/src/internal/keyedPool.ts b/packages/effect/src/internal/keyedPool.ts
@@ -174,7 +174,7 @@ const makeImpl = <K, A, E, R>(
           }
         })
       const activePools: Effect.Effect<Array<Pool.Pool<A, E>>> = core.suspend(() =>
-        core.forEachSequential(Array.from(HashMap.values(MutableRef.get(map))), (value) => {
+        core.forEachSequential(HashMap.toValues(MutableRef.get(map)), (value) => {
           switch (value._tag) {
             case "Complete": {
               return core.succeed(value.pool)

diff --git a/packages/effect/test/HashMap.test.ts b/packages/effect/test/HashMap.test.ts
@@ -394,6 +394,13 @@ describe("HashMap", () => {
     deepStrictEqual(result, [value("a"), value("b")])
   })
 
+  it("toValues", () => {
+    const map = HM.make([key(0), value("a")], [key(1), value("b")])
+    const result = HM.toValues(map)
+
+    deepStrictEqual(result, [value("a"), value("b")])
+  })
+
   it("entries", () => {
     const map = HM.make([key(0), value("a")], [key(1), value("b")])
     const result = Array.from(HM.entries(map))

diff --git a/packages/effect/test/HashSet.test.ts b/packages/effect/test/HashSet.test.ts
@@ -220,6 +220,14 @@ describe("HashSet", () => {
     deepStrictEqual(result, [value(0), value(1), value(2)])
   })
 
+  it("toValues", () => {
+    const hashSet = makeTestHashSet(0, 1, 2)
+
+    const result = HashSet.toValues(hashSet)
+
+    deepStrictEqual(result, [value(0), value(1), value(2)])
+  })
+
   it("pipe()", () => {
     strictEqual(
       HashSet.empty<string>().pipe(HashSet.add("value"), HashSet.size),

diff --git a/packages/effect/test/utils/cache/WatchableLookup.ts b/packages/effect/test/utils/cache/WatchableLookup.ts
@@ -101,7 +101,7 @@ export const makeEffect = <Key, Value, Error = never>(
       const assertAllCleaned = () =>
         Effect.flatMap(createdResources(), (resources) =>
           resourcesCleaned(
-            Chunk.flatten(Chunk.unsafeFromArray(Array.from(HashMap.values(resources))))
+            Chunk.flatten(Chunk.unsafeFromArray(HashMap.toValues(resources)))
           ))
       const assertAllCleanedForKey = (key: Key) =>
         Effect.flatMap(createdResources(), (resources) =>