Removing isPaused as it is not needed

Author: mattjburrows

app/actions/isPaused.js

@@ -1,12 +1,10 @@
 'use strict';
 
 import { IS_PLAYING } from '../actions/isPlaying';
-import { IS_PAUSED } from '../actions/isPaused';
 import { HAS_STOPPED } from '../actions/hasStopped';
 
 const initialState = {
   isPlaying: false,
-  isPaused: false,
   hasStopped: true
 };
 
@@ -18,20 +16,6 @@ function setIsPlayingState(isPlaying) {
   if (isPlaying === false) return newState;
 
   return Object.assign({}, newState, {
-    isPaused: false,
-    hasStopped: false
-  });
-}
-
-function setIsPausedState(isPaused) {
-  const newState = {
-    isPaused: isPaused
-  };
-
-  if (isPaused === false) return newState;
-
-  return Object.assign({}, newState, {
-    isPlaying: false,
     hasStopped: false
   });
 }
@@ -44,17 +28,14 @@ function setHasStoppedState(hasStopped) {
   if (hasStopped === false) return newState;
 
   return Object.assign({}, newState, {
-    isPlaying: false,
-    isPaused: false
+    isPlaying: false
   });
 }
 
 export function playbackReducer(state = initialState, action) {
   switch (action.type) {
     case IS_PLAYING :
       return Object.assign({}, state, setIsPlayingState(action.isPlaying));
-    case IS_PAUSED :
-      return Object.assign({}, state, setIsPausedState(action.isPaused));
     case HAS_STOPPED :
       return Object.assign({}, state, setHasStoppedState(action.hasStopped));
     default:

app/reducers/playback.js

@@ -2,7 +2,6 @@
 
 import assert from 'assert';
 import { IS_PLAYING } from '../../app/actions/isPlaying';
-import { IS_PAUSED } from '../../app/actions/isPaused';
 import { HAS_STOPPED } from '../../app/actions/hasStopped';
 import { playbackReducer } from '../../app/reducers/playback';
 
@@ -12,14 +11,13 @@ describe('playbackState', () => {
       assert.strictEqual(playbackReducer(undefined, '').isPlaying, false);
     });
 
-    it('{isPlaying: true, isPaused: false, hasStopped: false} when action.isPlaying is true', () => {
+    it('{isPlaying: true, hasStopped: false} when action.isPlaying is true', () => {
       const state = playbackReducer(undefined, {
         type: IS_PLAYING,
         isPlaying: true
       });
 
       assert.strictEqual(state.isPlaying, true);
-      assert.strictEqual(state.isPaused, false);
       assert.strictEqual(state.hasStopped, false);
     });
 
@@ -33,45 +31,18 @@ describe('playbackState', () => {
     });
   });
 
-  describe('isPaused property', () => {
-    it('defaults to false', () => {
-      assert.strictEqual(playbackReducer(undefined, '').isPaused, false);
-    });
-
-    it('{isPlaying: false, isPaused: true, hasStopped: false} when action.isPaused is true', () => {
-      const state = playbackReducer(undefined, {
-        type: IS_PAUSED,
-        isPaused: true
-      });
-
-      assert.strictEqual(state.isPlaying, false);
-      assert.strictEqual(state.isPaused, true);
-      assert.strictEqual(state.hasStopped, false);
-    });
-
-    it('{isPaused: false} when action.isPaused is false', () => {
-      const state = playbackReducer(undefined, {
-        type: IS_PAUSED,
-        isPaused: false
-      });
-
-      assert.strictEqual(state.isPaused, false);
-    });
-  });
-
   describe('hasStopped property', () => {
     it('defaults to true', () => {
       assert.strictEqual(playbackReducer(undefined, '').hasStopped, true);
     });
 
-    it('{isPlaying: false, isPaused: false, hasStopped: true} when action.hasStopped is true', () => {
+    it('{isPlaying: false, hasStopped: true} when action.hasStopped is true', () => {
       const state = playbackReducer(undefined, {
         type: HAS_STOPPED,
         hasStopped: true
       });
 
       assert.strictEqual(state.isPlaying, false);
-      assert.strictEqual(state.isPaused, false);
       assert.strictEqual(state.hasStopped, true);
     });