Added another explicit setting of this.paused = false, this time in abort. Helps fix part of an issue when trying to play a single test from the executing pane (instead all of the tests in the executing queue get run)

David Haeffner · David Haeffner · commit e7e62597c9bb · 2019-02-12T16:40:30.000+02:00
diff --git a/packages/selenium-ide/src/neo/stores/view/PlaybackState.js b/packages/selenium-ide/src/neo/stores/view/PlaybackState.js
@@ -235,6 +235,7 @@ class PlaybackState {
 
   @action.bound
   playTestOrResume() {
+    debugger
     if (this.paused) {
       return this.resume()
     } else if (!this.isPlaying) {
@@ -589,6 +590,7 @@ class PlaybackState {
       }
       this.stopPlayingGracefully()
     }
+    this.paused = false
     this.commandTarget.doCleanup({ isTestAborted: this.aborted })
   }
 

Original file line number	Diff line number	Diff line change
`@@ -235,6 +235,7 @@ class PlaybackState {`
`235`	`235`
`236`	`236`	`@action.bound`
`237`	`237`	`playTestOrResume() {`
	`238`	`+ debugger`
`238`	`239`	`if (this.paused) {`
`239`	`240`	`return this.resume()`
`240`	`241`	`} else if (!this.isPlaying) {`
`@@ -589,6 +590,7 @@ class PlaybackState {`
`589`	`590`	`}`
`590`	`591`	`this.stopPlayingGracefully()`
`591`	`592`	`}`
	`593`	`+ this.paused = false`
`592`	`594`	`this.commandTarget.doCleanup({ isTestAborted: this.aborted })`
`593`	`595`	`}`
`594`	`596`