Merge pull request #596 from Jongkeun/improving-executeCommand

checking isDisabled before executeCommand

Author: corevo

packages/selenium-ide/src/neo/playback/playback-tree/command-node.js

@@ -55,18 +55,26 @@ export class CommandNode {
   isTerminal() {
     return (
       ControlFlowCommandChecks.isTerminal(this.command) ||
-      this.command.command === '' ||
-      this.command.command.startsWith('//')
+      this.command.command === ''
     )
   }
 
+  isDisabled() {
+    return this.command.command.startsWith('//')
+  }
+
   execute(commandExecutor, options, targetOverride) {
     if (this._isRetryLimit()) {
       return Promise.resolve({
         result:
           'Max retry limit exceeded. To override it, specify a new limit in the value input field.',
       })
     }
+    if (this.isDisabled()) {
+      return Promise.resolve(
+        this._executionResult(commandExecutor, { result: 'success' })
+      )
+    }
     return commandExecutor.beforeCommand(this.command).then(() => {
       return this._executeCommand(
         commandExecutor,