allow one timeout between tries to avoid resource locking

Author: toddtarsi

packages/selenium-ide/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seleniumhq/selenium-ide",
-  "version": "4.0.0-alpha.21",
+  "version": "4.0.0-alpha.22",
   "private": true,
   "description": "Selenium IDE electron app",
   "author": "Todd <[email protected]>",
@@ -100,10 +100,10 @@
     "@mui/lab": "^5.0.0-alpha.72",
     "@mui/material": "^5.5.0",
     "@seleniumhq/get-driver": "^4.0.0-alpha.1",
-    "@seleniumhq/side-api": "^4.0.0-alpha.8",
+    "@seleniumhq/side-api": "^4.0.0-alpha.9",
     "@seleniumhq/side-model": "^4.0.0-alpha.2",
     "@seleniumhq/side-plugin-example": "^4.0.0-alpha.1",
-    "@seleniumhq/side-runtime": "^4.0.0-alpha.13",
+    "@seleniumhq/side-runtime": "^4.0.0-alpha.14",
     "dnd-core": "16.0.1",
     "electron-chromedriver": "^18.0.0",
     "electron-log": "^4.4.8",

packages/side-api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seleniumhq/side-api",
-  "version": "4.0.0-alpha.8",
+  "version": "4.0.0-alpha.9",
   "description": "Selenium IDE shared models",
   "author": "Todd Tarsi <[email protected]>",
   "homepage": "http://github.com/SeleniumHQ/selenium-ide",
@@ -19,7 +19,7 @@
     "@seleniumhq/browser-info": "^4.0.0-alpha.1",
     "@seleniumhq/get-driver": "^4.0.0-alpha.2",
     "@seleniumhq/side-model": "^4.0.0-alpha.2",
-    "@seleniumhq/side-runtime": "^4.0.0-alpha.13",
+    "@seleniumhq/side-runtime": "^4.0.0-alpha.14",
     "@types/electron": "1.6.10",
     "lodash": "^4.17.21"
   },

packages/side-plugin-example/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@seleniumhq/side-plugin-example",
   "private": true,
-  "version": "4.0.0-alpha.5",
+  "version": "4.0.0-alpha.6",
   "description": "Selenium IDE example plugin",
   "author": "Todd <[email protected]>",
   "homepage": "http://github.com/SeleniumHQ/selenium-ide",
@@ -16,7 +16,7 @@
     "watch": "tsc --watch"
   },
   "devDependencies": {
-    "@seleniumhq/side-runtime": "^4.0.0-alpha.13"
+    "@seleniumhq/side-runtime": "^4.0.0-alpha.14"
   },
   "repository": {
     "type": "git",

packages/side-runner/package.json

@@ -1,6 +1,6 @@
 {
   "name": "selenium-side-runner",
-  "version": "4.0.0-alpha.31",
+  "version": "4.0.0-alpha.32",
   "description": "Run Selenium IDE projects in cli",
   "repository": "https://github.com/SeleniumHQ/selenium-ide",
   "scripts": {
@@ -23,7 +23,7 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@seleniumhq/side-model": "^4.0.0-alpha.2",
-    "@seleniumhq/side-runtime": "^4.0.0-alpha.13",
+    "@seleniumhq/side-runtime": "^4.0.0-alpha.14",
     "commander": "^9.4.0",
     "glob": "^7.2.0",
     "jest": "^27.5.1",

packages/side-runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seleniumhq/side-runtime",
-  "version": "4.0.0-alpha.13",
+  "version": "4.0.0-alpha.14",
   "description": "Selenium IDE playback and execution",
   "author": "Tomer <[email protected]>",
   "homepage": "http://github.com/SeleniumHQ/selenium-ide",

packages/side-runtime/src/playback-tree/command-node.ts

@@ -128,6 +128,10 @@ export class CommandNode {
     }
   }
 
+  async pauseTimeout(timeout?: number): Promise<void> {
+    return new Promise((resolve) => setTimeout(resolve, timeout))
+  }
+
   async retryCommand(
     execute: () => Promise<unknown>,
     timeout: number
@@ -143,6 +147,7 @@ export class CommandNode {
     } catch (e) {
       this.handleTransientError(e, timeout)
       clearTimeout(expirationTimer)
+      await this.pauseTimeout()
       return this.retryCommand(execute, timeout)
     }
   }

packages/side-runtime/src/webdriver.ts

@@ -1308,16 +1308,15 @@ export default class WebDriverExecutor {
         }, timeout)
         const clearTimer = () => timer && clearTimeout(timer)
 
-        condition.then(
-          function (value: T) {
+        condition
+          .then((value: T) => {
             clearTimer()
             resolve(value)
-          },
-          function (error: Error) {
+          })
+          .catch((error: Error) => {
             clearTimer()
             reject(error)
-          }
-        )
+          })
       })
     }