fix: exec returns array (not Match) in match(v, obj) with func conseq…

…uent
brigand · Dec 27, 2019 · 220399e · 220399e
1 parent 3f2db98
commit 220399e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/match.js b/src/match.js
@@ -44,7 +44,7 @@ function match(...values) {
     const maybeMatch = cond.exec(...values);
     if (maybeMatch && maybeMatch.values) {
       if (typeof consequent === 'function') {
-        result = new Match(consequent(...maybeMatch.values));
+        result = new Match(consequent(...maybeMatch));
       } else {
         result = new Match(consequent);
       }

diff --git a/src/match.test.js b/src/match.test.js
@@ -37,7 +37,7 @@ it(`works for simple conditions`, () => {
       [LenEq(1)]: 'len 1',
       [LenEq(2)]: 'len 2',
       [Test((x) => x === 'bar')]: 'is bar',
-      [LenEq(3)]: 'len 3',
+      [LenEq(3)]: () => 'len 3',
       [LenEq(4)]: 'len 4',
       _: 'default',
     }),
@@ -50,9 +50,9 @@ it(`works with exec arg passed to chain`, () => {
   const results = ['', 'f', 'fo', 'foo', 'bar', 'food'].map((word) =>
     match(word)
       .with(LenEq(1), 'len 1')
-      .with(LenEq(2), 'len 2')
+      .with(LenEq(2), () => 'len 2')
       .with(Eq('bar'), 'is bar')
-      .with(LenEq(3), 'len 3')
+      .with(LenEq(3), () => 'len 3')
       .with(LenEq(4), 'len 4')
       .any('default')
       .exec(),