♻️ Toleration falsey args in flow fix #202

packages/immutadot/src/flow/flow.js

@@ -1,4 +1,5 @@
 import { flatten } from 'util/array'
+import { isNil } from 'util/lang'
 
 /**
  * A function successively applying a list of functions.
@@ -11,14 +12,15 @@ import { flatten } from 'util/array'
 
 /**
  * Successively calls <code>fns</code>.<br/>
- * Each function is called with the result of the previous one.
+ * Each function is called with the result of the previous one.<br/>
+ * Falsey functions (<code>null</code>, <code>undefined</code> and <code>false</code>) are tolerated and will be skipped.
  * @memberof flow
  * @param {...(function|Array<function>)} args The functions to apply
  * @returns {flow.flowFunction} A function successively calling <code>fns</code>
  * @since 1.0.0
  */
 function flow(...args) {
-  const fns = flatten(args)
+  const fns = flatten(args).filter(fn => !isNil(fn) && fn !== false)
   return pObj => {
     const [result] = fns.reduce(
       ([obj, appliedPaths], fn) => [

packages/immutadot/src/flow/flow.spec.js

@@ -74,4 +74,37 @@ describe('flow.flow', () => {
       return output
     }, object, 'nested1.prop2', 'nested2.prop3', 'nested2.prop4')
   })
+
+  it('should skip falsey functions', () => {
+    immutaTest((input, path) => {
+      const output = flow(
+        null,
+        set(path, 'bar'),
+        undefined,
+        false,
+        [
+          null,
+          undefined,
+          false,
+        ],
+      )(input)
+      expect(output).toEqual({
+        nested: { prop: 'bar' },
+        other: {},
+      })
+      return output
+    }, {
+      nested: { prop: 'foo' },
+      other: {},
+    }, 'nested.prop')
+  })
+
+  it('should do nothing if empty flow', () => {
+    const input = {
+      nested: { prop: 'foo' },
+      other: {},
+    }
+    const output = flow()(input)
+    expect(output).toBe(input)
+  })
 })