fix(griffith-utils): rename mergeFunctions to sequence (#41)

packages/griffith-utils/src/__tests__/sequence.spec.js

@@ -0,0 +1,9 @@
+import sequence from '../sequence'
+
+test('sequence', () => {
+  const fn1 = jest.fn(() => 1)
+  const fn2 = jest.fn(() => 2)
+  expect(sequence(fn1, fn2)('foo', 'bar')).toBe(2)
+  expect(fn1).toHaveBeenCalledWith('foo', 'bar')
+  expect(fn2).toHaveBeenCalledWith('foo', 'bar')
+})

packages/griffith-utils/src/index.js

@@ -3,4 +3,4 @@ export {default as ua} from './ua'
 export {default as isMSESupported} from './isMSESupported'
 export {default as isHlsNativeSupported} from './isHlsNativeSupported'
 export {getGCD, reduce} from './math'
-export {default as mergeFunctions} from './mergeFunctions'
+export {default as sequence} from './sequence'

packages/griffith-utils/src/sequence.js

@@ -0,0 +1,3 @@
+export default function sequence(...fns) {
+  return (...args) => fns.reduce((_, fn) => fn(...args), null)
+}

packages/griffith/src/components/Hover/Hover.js

@@ -1,5 +1,6 @@
 import React from 'react'
-import {mergeFunctions} from 'griffith-utils'
+import {sequence} from 'griffith-utils'
+import noop from 'lodash/noop'
 
 export default class Hover extends React.Component {
   state = {
@@ -20,8 +21,8 @@ export default class Hover extends React.Component {
     return (
       <div
         {...rest}
-        onMouseEnter={mergeFunctions(this.handlePointerEnter, onMouseEnter)}
-        onMouseLeave={mergeFunctions(this.handlePointerLeave, onMouseLeave)}
+        onMouseEnter={sequence(this.handlePointerEnter, onMouseEnter || noop)}
+        onMouseLeave={sequence(this.handlePointerLeave, onMouseLeave || noop)}
       >
         {children(isHovered)}
       </div>

packages/griffith/src/components/Video/VideoWithMessage.js

@@ -1,7 +1,7 @@
 import React from 'react'
 import {EVENTS} from 'griffith-message'
-import mergeFunctions from 'griffith-utils/src/mergeFunctions'
-
+import {sequence} from 'griffith-utils'
+import noop from 'lodash/noop'
 import {InternalContext} from '../../contexts/Message'
 import {ObjectFitContext} from '../../contexts/ObjectFit'
 import {PositionContext} from '../../contexts/Position'
@@ -37,7 +37,7 @@ const VideoWithMessage = React.forwardRef((props, ref) => {
     eventMap.map(([eventName, key]) => {
       const handler = props[key]
       const emit = event => emitEvent(eventName, getMediaEventPayload(event))
-      newProps[key] = mergeFunctions(emit, handler)
+      newProps[key] = sequence(emit, handler || noop)
     })
 
     const updateVideoSizeOnLoadedMetadata = event => {
@@ -48,9 +48,9 @@ const VideoWithMessage = React.forwardRef((props, ref) => {
       }
     }
 
-    const newOnLoadedMetadata = mergeFunctions(
+    const newOnLoadedMetadata = sequence(
       updateVideoSizeOnLoadedMetadata,
-      props.onLoadedMetadata
+      props.onLoadedMetadata || noop
     )
 
     const {Video, ...otherProps} = props