Skip to content

Commit

Permalink
[js] Fix linting (#13753)
Browse files Browse the repository at this point in the history
  • Loading branch information
pujagani authored Apr 1, 2024
1 parent 9c6ccdb commit 1f88231
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 10 deletions.
6 changes: 0 additions & 6 deletions javascript/node/selenium-webdriver/test/bidi/script_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -586,7 +586,6 @@ suite(
})

it('can access preload script properties', async function () {
const id = await driver.getWindowHandle()
const manager = await ScriptManager([], driver)

await manager.addPreloadScript('() => { window.preloadScriptFunction = () => window.baz = 42; }')
Expand All @@ -602,7 +601,6 @@ suite(
})

it('can add preload script to sandbox', async function () {
const id = await driver.getWindowHandle()
const manager = await ScriptManager([], driver)

await manager.addPreloadScript('() => { window.foo = 1; }')
Expand Down Expand Up @@ -638,7 +636,6 @@ suite(
})

it('can remove properties set by preload script', async function () {
const id = await driver.getWindowHandle()
const manager = await ScriptManager([], driver)

await manager.addPreloadScript('() => { window.foo = 42; }')
Expand All @@ -657,7 +654,6 @@ suite(
})

it('can remove preload script', async function () {
const id = await driver.getWindowHandle()
const manager = await ScriptManager([], driver)

let script = await manager.addPreloadScript("() => { window.foo='bar'; }")
Expand All @@ -682,7 +678,6 @@ suite(
})

it('cannot remove same preload script twice', async function () {
const id = await driver.getWindowHandle()
const manager = await ScriptManager([], driver)

let script = await manager.addPreloadScript("() => { window.foo='bar'; }")
Expand All @@ -695,7 +690,6 @@ suite(
})

it('can remove one of preload script', async function () {
const id = await driver.getWindowHandle()
const manager = await ScriptManager([], driver)

let script_1 = await manager.addPreloadScript("() => { window.bar='foo'; }")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,8 @@ const { Pages, suite } = require('../../lib/test')
const { Browser, By } = require('../..')
const Input = require('../../bidi/input')
const io = require('../../io')
const fs = require('node:fs')
const { ReferenceValue, RemoteReferenceType } = require('../../bidi/protocolValue')
const fs = require('fs')

suite(
function (env) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
const assert = require('assert')
require('../../lib/test/fileserver')
const firefox = require('../../firefox')
const { ignore, suite } = require('../../lib/test')
const { suite } = require('../../lib/test')
const { Browser } = require('../..')
const Storage = require('../../bidi/storage')
const fileserver = require('../../lib/test/fileserver')
Expand Down
5 changes: 3 additions & 2 deletions javascript/node/selenium-webdriver/test/driver_factory.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,11 @@
const fs = require('fs')
const os = require('os')
const path = require('path')
const { Builder, Browser } = require('../index')
const { Browser } = require('../index')
const { Environment } = require('../testing')
const chrome = require('../chrome')
const firefox = require('../firefox')
// eslint-disable-next-line node/no-missing-require
const { runfiles } = require('@bazel/runfiles')

function GetBrowserForTests() {
Expand Down Expand Up @@ -90,7 +91,7 @@ function GetBrowserForTests() {
break

case 'safari':
builder.forBrowser(webdriver.Browser.SAFARI)
builder.forBrowser(Browser.SAFARI)
break

default:
Expand Down

0 comments on commit 1f88231

Please sign in to comment.