fix - Make sure bind does not mutate columns

Author: bebraw

CHANGELOG.md

@@ -1,3 +1,8 @@
+8.9.3 / 2017-02-09
+==================
+
+  * Fix - Make sure `bind` does not mutate columns.
+
 8.9.2 / 2017-02-09
 ==================
 

__tests__/bind-test.js

@@ -1,3 +1,4 @@
+import { cloneDeep } from 'lodash';
 import { bind } from '../src';
 
 describe('reactabular-column-extensions.bind', function () {
@@ -162,6 +163,41 @@ describe('reactabular-column-extensions.bind', function () {
     expect(result).toEqual(expected);
   });
 
+  it('does not mutate columns', function () {
+    const extensions = [
+      {
+        match({ cell }) {
+          return cell.ok;
+        },
+        evaluate({ cell: { ok } }) {
+          return {
+            cell: {
+              ok: ok + ok,
+              formatters: [
+                'b'
+              ]
+            }
+          };
+        }
+      }
+    ];
+    const columns = [
+      {
+        cell: {
+          ok: 'foo',
+          formatters: [
+            'a'
+          ]
+        }
+      }
+    ];
+    const expected = cloneDeep(columns);
+
+    bind(extensions)(columns);
+
+    expect(columns).toEqual(expected);
+  });
+
   it('works with multiple evaluators', function () {
     const extensions = [
       {

src/bind.js

@@ -23,7 +23,7 @@ function bindColumn({
     return match(col) && evaluate(col);
   });
 
-  return mergeWith(column, ...matches, (obj, src) => {
+  return mergeWith({}, column, ...matches, (obj, src) => {
     if (isArray(obj)) {
       return obj.concat(src);
     }