Merge pull request #9 from CodingZeal/features/whitelist-actions-for-…

…fsa-check

Allow whitelisting actions for flux-standard-action check

.nvmrc

@@ -1 +1 @@
-6.9.1
+6.10.3

src/__tests__/createReducer-spec.js

@@ -33,4 +33,27 @@ describe('createReducer', () => {
       () => reducer(state, invalidAction)
     ).toThrowError(NonStandardAction)
   })
+
+  describe('whitelisting non-flux standard actions', () => {
+    const reducerWithWhitelist = createReducer({}, {}, {
+      allowNonStandardActionIf: action => action.type === 'WHITELISTED'
+    })
+    const state = reducerWithWhitelist(undefined, nullAction)
+
+    test('does not complain about whitelisted actions', () => {
+      const whitelistedAction = { type: 'WHITELISTED', notPayload: {} }
+
+      expect(
+        () => reducerWithWhitelist(state, whitelistedAction)
+      ).not.toThrowError(NonStandardAction)
+    })
+
+    test('continues to complain about non-whitelisted actions', () => {
+      const invalidAction = { type: 'NOT WHITELISTED', notPayload: {} }
+
+      expect(
+        () => reducer(state, invalidAction)
+      ).toThrowError(NonStandardAction)
+    })
+  })
 })

src/createReducer.js

@@ -1,5 +1,5 @@
 import { isFSA } from 'flux-standard-action'
-import { identity, propOr } from 'ramda'
+import { always, identity, propOr } from 'ramda'
 
 export class NonStandardAction {
   constructor(action) {
@@ -11,8 +11,8 @@ export class NonStandardAction {
   }
 }
 
-const throwIfNotFSA = action => {
-  if (isFSA(action)) return
+const throwIfNotFSA = (action, isWhitelisted = always(false)) => {
+  if (isWhitelisted(action) || isFSA(action)) return
 
   throw new NonStandardAction(action)
 }
@@ -21,9 +21,9 @@ const throwIfNotFSA = action => {
 const isProduction = process.env.NODE_ENV === 'production'
 const ensureIsFSA = isProduction ? identity : throwIfNotFSA
 
-export default function createReducer(initialState, handlers) {
+export default function createReducer(initialState, handlers, options = {}) {
   return (state = initialState, action) => {
-    ensureIsFSA(action)
+    ensureIsFSA(action, options.allowNonStandardActionIf)
 
     return propOr(identity, action.type, handlers)(state, action)
   }