fix(replaceFeature): set featureState to NOT_LOADED when replaceFeature is dispatched

Author: jedwards1211

src/featureStatesReducer.js

@@ -16,7 +16,7 @@ export default function featureStatesReducer(
     [ADD_FEATURE]: (state, {meta: {id}}) => (state[id] ? state : {...state, [id]: 'NOT_LOADED'}),
     [LOAD_FEATURE]: (state, {meta: {id}}) => (state[id] && state[id] !== 'LOADED' ? {...state, [id]: 'LOADING'} : state),
     [INSTALL_FEATURE]: (state, {meta: {id}}) => (state[id] ? {...state, [id]: 'LOADED'} : state),
-    [REPLACE_FEATURE]: state => state,
+    [REPLACE_FEATURE]: (state, {meta: {id}}) => state[id] ? {...state, [id]: 'NOT_LOADED'} : state,
     [SET_FEATURE_STATE]: (state, {payload, meta: {id}}) => (state[id] ? {...state, [id]: payload} : state),
     [LOAD_INITIAL_FEATURES]: state => mapValues(state, fs => fs === 'LOADED' ? 'LOADING' : fs)
   })

test/featureStatesReducerTest.js

@@ -35,7 +35,7 @@ describe('featureStatesReducer', () => {
   it('handles replaceFeature correctly', () => {
     expect(reducer({}, replaceFeature('a', {}))).to.deep.equal({})
     expect(reducer({a: 'NOT_LOADED'}, replaceFeature('a', {}))).to.deep.equal({a: 'NOT_LOADED'})
-    expect(reducer({a: 'LOADED'}, replaceFeature('a', {}))).to.deep.equal({a: 'LOADED'})
+    expect(reducer({a: 'LOADED'}, replaceFeature('a', {}))).to.deep.equal({a: 'NOT_LOADED'})
   })
   it('handles setFeatureState correctly', () => {
     expect(reducer({}, setFeatureState('a', 'LOADING'))).to.deep.equal({})