Merge pull request #7500 from rachel-fenichel/update_webdriverio

chore(deps): update webdriverio to 8.16 and remove selenium-standalone

Author: rachel-fenichel

.github/workflows/browser_test.yml

@@ -45,10 +45,6 @@ jobs:
         if: runner.os == 'Linux'
         run: source ./tests/scripts/setup_linux_env.sh
 
-      - name: MacOS Test Setup
-        if: runner.os == 'macOS'
-        run: source ./tests/scripts/setup_osx_env.sh
-
       - name: Run Build
         run: npm run build
 

.github/workflows/build.yml

@@ -44,10 +44,6 @@ jobs:
         if: runner.os == 'Linux'
         run: source ./tests/scripts/setup_linux_env.sh
 
-      - name: MacOS Test Setup
-        if: runner.os == 'macOS'
-        run: source ./tests/scripts/setup_osx_env.sh
-
       - name: Run
         run: npm run test
 

package-lock.json

@@ -73,7 +73,6 @@
     "@microsoft/api-documenter": "^7.22.4",
     "@microsoft/api-extractor": "^7.29.5",
     "@typescript-eslint/eslint-plugin": "^6.1.0",
-    "@wdio/selenium-standalone-service": "^8.0.2",
     "async-done": "^2.0.0",
     "chai": "^4.2.0",
     "concurrently": "^8.0.1",
@@ -102,9 +101,8 @@
     "prettier": "3.0.3",
     "readline-sync": "^1.4.10",
     "rimraf": "^5.0.0",
-    "selenium-standalone": "^9.1.1",
     "typescript": "^5.0.2",
-    "webdriverio": "^8.0.5",
+    "webdriverio": "^8.16.7",
     "yargs": "^17.2.1"
   },
   "dependencies": {

package.json

@@ -41,7 +41,6 @@ async function driverSetup() {
         args: ['--allow-file-access-from-files'],
       },
     },
-    services: [['selenium-standalone']],
     logLevel: 'warn',
   };
 
@@ -367,8 +366,8 @@ async function connect(
   );
 
   const delta = {
-    x: targetLocation.x - draggedLocation.x,
-    y: targetLocation.y - draggedLocation.y,
+    x: Math.round(targetLocation.x - draggedLocation.x),
+    y: Math.round(targetLocation.y - draggedLocation.y),
   };
   if (mutatorBlockId) {
     await dragBlockSelector.dragAndDrop(delta);
@@ -525,7 +524,7 @@ async function getAllBlocks(browser) {
  *  - The workspace has a trash can, which means it has a second (hidden) flyout.
  * @param browser The active WebdriverIO Browser object.
  * @param xDelta How far to drag the flyout in the x direction. Positive is right.
- * @param yDelta How far to drag thte flyout in the y direction. Positive is down.
+ * @param yDelta How far to drag the flyout in the y direction. Positive is down.
  * @return A Promise that resolves when the actions are completed.
  */
 async function scrollFlyout(browser, xDelta, yDelta) {

tests/browser/test/test_setup.js

@@ -36,7 +36,6 @@ async function runCompileCheckInBrowser() {
       browserName: 'chrome',
     },
     logLevel: 'warn',
-    services: ['selenium-standalone']
   };
   // Run in headless mode on Github Actions.
   if (process.env.CI) {

tests/compile/webdriver.js

@@ -47,7 +47,6 @@ async function runGeneratorsInBrowser(outputDir) {
       },
     },
     logLevel: 'warn',
-    services: ['selenium-standalone']
   };
 
   // Run in headless mode on Github Actions.

tests/generators/webdriver.js

@@ -25,9 +25,6 @@ async function runMochaTestsInBrowser() {
         args: ['--allow-file-access-from-files'],
       },
     },
-    services: [
-      ['selenium-standalone'],
-    ],
     logLevel: 'warn',
   };