fix(managed): fix issue with releasing of resource happens multiple t…

…imes
tusharmath · Oct 20, 2019 · bfddfe5 · bfddfe5
1 parent d5becbb
commit bfddfe5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/main/Managed.ts b/src/main/Managed.ts
@@ -36,12 +36,12 @@ export class Managed<E1, A1, R1> {
     acquire: FIO<E1, A1, R1>,
     release: (a: A1) => FIO<never, void, R2>
   ): Managed<E1, A1, R1 & R2> {
-    return Managed.of(
-      acquire
-        .map(a1 =>
-          Reservation.of(FIO.of(a1).addEnv<R2>(), release(a1).addEnv<R1>())
+    return Managed.of<E1, A1, R1 & R2>(
+      acquire.chain(a1 =>
+        release(a1).once.map(r =>
+          Reservation.of(FIO.of(a1).addEnv<R1>(), r.addEnv<R2>())
         )
-        .addEnv()
+      )
     )
   }
 

diff --git a/test/Managed.test.ts b/test/Managed.test.ts
@@ -68,7 +68,7 @@ describe('Managed', () => {
     assert.ok(r.isReleased)
   })
 
-  it.skip('should release only once', () => {
+  it('should release only once', () => {
     const r = Resource()
     const runtime = testRuntime()