From 77ed935dae337b3ae14659dc85adb430356d1b7c Mon Sep 17 00:00:00 2001
From: Emmanuel Touzery <etouzery@gmail.com>
Date: Sun, 12 Aug 2018 00:36:27 +0200
Subject: [PATCH] fix another brown paper bag Future.mapFailure bug

---
 src/Future.ts   | 2 +-
 tests/Future.ts | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Future.ts b/src/Future.ts
index 3b4af3c..1c61386 100644
--- a/src/Future.ts
+++ b/src/Future.ts
@@ -269,7 +269,7 @@ export class Future<T> {
      * This method is eager, will trigger the underlying Promise.
      */
     mapFailure(fn: (x:any)=>any): Future<T> {
-        const lazy = Lazy.of(()=>this.promise.get().catch(x => {throw [fn(x)]}));
+        const lazy = Lazy.of(()=>this.promise.get().catch(x => {throw fn(x)}));
         lazy.get();
         return new Future<T>(lazy);
     }
diff --git a/tests/Future.ts b/tests/Future.ts
index 6a2d50a..60a8dcf 100644
--- a/tests/Future.ts
+++ b/tests/Future.ts
@@ -9,7 +9,7 @@ async function ensureFailedWithValue<T>(val: any, promise:Promise<T>) {
         v = await promise;
         assert.ok(false);
     } catch (err) {
-        assert.equal(val, err);
+        assert.deepEqual(val, err);
     }
 }
 
@@ -96,7 +96,7 @@ describe("Future.map*", () => {
     });
     it("mapFailure works", async () => {
         return ensureFailedWithValue(
-            "oops, sorry", Future.failed("sorry").mapFailure(err => "oops, " + err).toPromise());
+            "SORRY", Future.failed("sorry").mapFailure(err => err.toUpperCase()).toPromise());
     });
     it("mapFailure is a nop on successful futures", async () => {
         assert.deepEqual(5, await Future.ok(5).mapFailure(_ => "oops"));