fix: auto play when audio list changed if autoplayInitLoadPlayList is…

… false #154
lijinke666 · Aug 30, 2020 · 3f08c70 · 3f08c70
1 parent 2500d17
commit 3f08c70
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 0 deletions.
diff --git a/__tests__/setup.js b/__tests__/setup.js
@@ -7,6 +7,7 @@ global.document = window.document
 global.navigator = {
   userAgent: 'node.js',
 }
+global.URL.createObjectURL = jest.fn()
 if (typeof window !== 'undefined') {
   global.window.resizeTo = (width, height) => {
     global.window.innerWidth = width || global.window.innerWidth
@@ -33,6 +34,7 @@ if (typeof window !== 'undefined') {
   global.window.HTMLMediaElement.prototype.load = () => {}
   global.window.HTMLMediaElement.prototype.play = () => {}
   global.window.HTMLMediaElement.prototype.pause = () => {}
+  global.window.URL.createObjectURL = jest.fn()
 }
 
 Enzyme.configure({ adapter: new Adapter() })
diff --git a/__tests__/tests/player.test.js b/__tests__/tests/player.test.js
@@ -1346,4 +1346,14 @@ describe('<ReactJkMusicPlayer/>', () => {
     wrapper.update()
     expect(wrapper.state().theme).toEqual('light')
   })
+
+  it('should cannot set isAutoPlayWhenUserClicked to true if audioList updated', () => {
+    const wrapper = mount(
+      <ReactJkMusicPlayer autoPlay={false} clearPriorAudioLists />,
+    )
+    wrapper.setProps({
+      audioLists: [{ musicSrc: 'xxx', name: 'xxx' }],
+    })
+    expect(wrapper.state().isAutoPlayWhenUserClicked).toBeFalsy()
+  })
 })
diff --git a/src/index.js b/src/index.js
@@ -1841,6 +1841,7 @@ export default class ReactJkMusicPlayer extends PureComponent {
         this.audioListsPlay(currentPlay.id, true)
       }
     }
+    this.setState({ isAutoPlayWhenUserClicked: false })
   }
 
   playByIndex = (index) => {