Added set default value when select 'wait for...' command (#678)

* Added set default value when select 'wait for...' command

* Added conditional statement to check empty string

* Added set value test about 'wait for' command

* Added set value validation test

Author: sapzape

packages/selenium-ide/src/neo/__test__/models/Command.spec.js

@@ -206,6 +206,20 @@ describe('Command', () => {
     expect(command.windowTimeout).toBe(jsRepresentation.windowTimeout)
     expect(command instanceof Command).toBeTruthy()
   })
+
+  it('should not overridden value when replace command', () => {
+    const command = new Command()
+    command.setCommand('assert text')
+    command.setValue('Hello World')
+    command.setCommand('wait for element present')
+    expect(command.value).toBe('Hello World')
+  })
+
+  it("should not set value for commands that do not start with 'wait for'", () => {
+    const command = new Command()
+    command.setCommand('click')
+    expect(command.value).toBe('')
+  })
 })
 
 describe('Commands enum', () => {

packages/selenium-ide/src/neo/models/Command/index.js

@@ -106,6 +106,10 @@ export default class Command {
     if (!this.canHaveTargets) {
       this.setTargets()
     }
+
+    if (this.command.indexOf('waitFor') > -1 && this.value === '') {
+      this.setValue('30000')
+    }
   }
 
   @action.bound