fix: correct binding of overwritten functions

there was 1 closure missing, everything was broken 😞
eins78 · Oct 23, 2015 · dad6f47 · dad6f47
1 parent e986f0f
commit dad6f47
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/lib/callWithNewObject.js b/src/lib/callWithNewObject.js
@@ -1,3 +1,5 @@
-export default (fun, ...args) => {
-  return fun.bind(undefined, Object.create(null), ...args)
+export default (func) => {
+  return function () {
+    return func.bind(undefined, Object.create(null)).apply(undefined, arguments)
+  }
 }
diff --git a/test/tests.js b/test/tests.js
@@ -44,7 +44,7 @@ test('activeLodash', (t) => {
   })
 
   t.test('merge', (t) => {
-    t.plan(2)
+    t.plan(3)
 
     t.looseEqual(
       f.merge(
@@ -64,6 +64,9 @@ test('activeLodash', (t) => {
     let o = {s: 3}
     f.merge(o, {b: 2})
     t.deepEqual(o, {s: 3}, 'does not mutate the arguments')
+
+    // test for stupidity – makes sure it works more than 1 time 😕
+    t.deepEqual(f.merge({b: 2}, {c: 3}), {b: 2, c: 3}, 'does bind correctly')
   })
 
   t.test('presence', (assert) => {