diff --git a/entry_types/scrolled/package/spec/frontend/usePlayerState-spec.js b/entry_types/scrolled/package/spec/frontend/usePlayerState-spec.js index 23084d1903..23077778bc 100644 --- a/entry_types/scrolled/package/spec/frontend/usePlayerState-spec.js +++ b/entry_types/scrolled/package/spec/frontend/usePlayerState-spec.js @@ -238,15 +238,25 @@ describe('usePlayerState', () => { expect(nextState.unplayed).toBe(true); }); - it('is no longer unplayed after play action', () => { + it('is no longer unplayed after playing action', () => { const {result} = renderHookInEntry(() => usePlayerState()); const [,actions] = result.current; act(() => actions.play()); + act(() => actions.playing()); const [nextState,] = result.current; expect(nextState.unplayed).toBe(false); }); + it('stays unplayed after play action until playing', () => { + const {result} = renderHookInEntry(() => usePlayerState()); + const [,actions] = result.current; + act(() => actions.play()); + const [nextState,] = result.current; + + expect(nextState.unplayed).toBe(true); + }); + it('becomes unplayed again when video ends to make it return to its initial look', () => { const {result} = renderHookInEntry(() => usePlayerState()); const [,actions] = result.current; diff --git a/entry_types/scrolled/package/src/frontend/MediaPlayer/usePlayerState.js b/entry_types/scrolled/package/src/frontend/MediaPlayer/usePlayerState.js index 527f46532e..2496e37814 100644 --- a/entry_types/scrolled/package/src/frontend/MediaPlayer/usePlayerState.js +++ b/entry_types/scrolled/package/src/frontend/MediaPlayer/usePlayerState.js @@ -42,7 +42,6 @@ export function playerStateReducer(state, action){ isLoading: true, shouldPlay: true, playFailed: false, - unplayed: false, lastControlledVia: action.payload.via }; case PLAYING: @@ -50,6 +49,7 @@ export function playerStateReducer(state, action){ ...state, shouldPlay: true, isPlaying: true, + unplayed: false, userIdle: false }; case PLAY_AND_FADE_IN: