Fix JS linting errors

shs96c · shs96c · commit 3c051755e355 · 2024-03-08T13:59:32.000Z
diff --git a/javascript/node/selenium-webdriver/bidi/addInterceptParameters.js b/javascript/node/selenium-webdriver/bidi/addInterceptParameters.js
@@ -30,7 +30,7 @@ class AddInterceptParameters {
   }
 
   urlPattern(pattern) {
-    if (!pattern instanceof UrlPattern) {
+    if (!(pattern instanceof UrlPattern)) {
       throw new Error(`Pattern must be an instance of UrlPattern. Received: '${pattern})'`)
     }
     this.#urlPatterns.push(Object.fromEntries(pattern.asMap()))
@@ -39,7 +39,7 @@ class AddInterceptParameters {
 
   urlPatterns(patterns) {
     patterns.forEach((pattern) => {
-      if (!pattern instanceof UrlPattern) {
+      if (!(pattern instanceof UrlPattern)) {
         throw new Error(`Pattern must be an instance of UrlPattern. Received:'${pattern}'`)
       }
       this.#urlPatterns.push(Object.fromEntries(pattern.asMap()))
@@ -48,7 +48,7 @@ class AddInterceptParameters {
   }
 
   urlStringPattern(pattern) {
-    if (!pattern instanceof String) {
+    if (!(pattern instanceof String)) {
       throw new Error(`Pattern must be an instance of String. Received:'${pattern}'`)
     }
 
@@ -58,7 +58,7 @@ class AddInterceptParameters {
 
   urlStringPatterns(patterns) {
     patterns.forEach((pattern) => {
-      if (!pattern instanceof String) {
+      if (!(pattern instanceof String)) {
         throw new Error(`Pattern must be an instance of String. Received:'${pattern}'`)
       }
       this.#urlPatterns.push({ type: 'string', pattern: pattern })
diff --git a/javascript/node/selenium-webdriver/bidi/network.js b/javascript/node/selenium-webdriver/bidi/network.js
@@ -97,7 +97,7 @@ class Network {
   }
 
   async addIntercept(params) {
-    if (!params instanceof AddInterceptParameters) {
+    if (!(params instanceof AddInterceptParameters)) {
       throw new Error(`Params must be an instance of AddInterceptParamenters. Received:'${params}'`)
     }
 
diff --git a/javascript/node/selenium-webdriver/bidi/urlPattern.js b/javascript/node/selenium-webdriver/bidi/urlPattern.js
@@ -15,8 +15,6 @@
 // specific language governing permissions and limitations
 // under the License.
 
-const { LocalValue } = require('./protocolValue')
-
 class UrlPattern {
   #map = new Map()
 
diff --git a/javascript/node/selenium-webdriver/lib/test/build.js b/javascript/node/selenium-webdriver/lib/test/build.js
@@ -133,7 +133,7 @@ exports.isDevMode = isDevMode
  * @return {!Build} The new build.
  * @throws {Error} If not running in dev mode.
  */
-exports.of = function (var_args) {
+exports.of = function (_) {
   // eslint-disable-line
   let targets = Array.prototype.slice.call(arguments, 0)
   return new Build(targets)
diff --git a/javascript/node/selenium-webdriver/test/bidi/add_intercept_parameters_test.js b/javascript/node/selenium-webdriver/test/bidi/add_intercept_parameters_test.js
@@ -19,7 +19,7 @@
 
 const assert = require('assert')
 const firefox = require('../../firefox')
-const { Browser, By, WebElement } = require('../../')
+const { Browser } = require('../../')
 const { suite } = require('../../lib/test')
 const Network = require('../../bidi/network')
 const { AddInterceptParameters } = require('../../bidi/addInterceptParameters')
diff --git a/javascript/node/selenium-webdriver/test/bidi/network_commands_test.js b/javascript/node/selenium-webdriver/test/bidi/network_commands_test.js
@@ -19,7 +19,7 @@
 
 const assert = require('assert')
 const firefox = require('../../firefox')
-const { Browser, By, WebElement } = require('../../')
+const { Browser, By } = require('../../')
 const { Pages, suite } = require('../../lib/test')
 const Network = require('../../bidi/network')
 const { AddInterceptParameters } = require('../../bidi/addInterceptParameters')
@@ -92,7 +92,7 @@ suite(
           await network.cancelAuth(event.request.request)
         })
         try {
-          const alert = await driver.wait(until.alertIsPresent(), 3000)
+          await driver.wait(until.alertIsPresent(), 3000)
           assert.fail('Alert should not be present')
         } catch (e) {
           assert.strictEqual(e.name, 'TimeoutError')

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ class AddInterceptParameters {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`urlPattern(pattern) {`
`33`		`- if (!pattern instanceof UrlPattern) {`
	`33`	`+ if (!(pattern instanceof UrlPattern)) {`
`34`	`34`	throw new Error(`Pattern must be an instance of UrlPattern. Received: '${pattern})'`)
`35`	`35`	`}`
`36`	`36`	`this.#urlPatterns.push(Object.fromEntries(pattern.asMap()))`
`@@ -39,7 +39,7 @@ class AddInterceptParameters {`
`39`	`39`
`40`	`40`	`urlPatterns(patterns) {`
`41`	`41`	`patterns.forEach((pattern) => {`
`42`		`- if (!pattern instanceof UrlPattern) {`
	`42`	`+ if (!(pattern instanceof UrlPattern)) {`
`43`	`43`	throw new Error(`Pattern must be an instance of UrlPattern. Received:'${pattern}'`)
`44`	`44`	`}`
`45`	`45`	`this.#urlPatterns.push(Object.fromEntries(pattern.asMap()))`
`@@ -48,7 +48,7 @@ class AddInterceptParameters {`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`urlStringPattern(pattern) {`
`51`		`- if (!pattern instanceof String) {`
	`51`	`+ if (!(pattern instanceof String)) {`
`52`	`52`	throw new Error(`Pattern must be an instance of String. Received:'${pattern}'`)
`53`	`53`	`}`
`54`	`54`
`@@ -58,7 +58,7 @@ class AddInterceptParameters {`
`58`	`58`
`59`	`59`	`urlStringPatterns(patterns) {`
`60`	`60`	`patterns.forEach((pattern) => {`
`61`		`- if (!pattern instanceof String) {`
	`61`	`+ if (!(pattern instanceof String)) {`
`62`	`62`	throw new Error(`Pattern must be an instance of String. Received:'${pattern}'`)
`63`	`63`	`}`
`64`	`64`	`this.#urlPatterns.push({ type: 'string', pattern: pattern })`
Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ class Network {`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`async addIntercept(params) {`
`100`		`- if (!params instanceof AddInterceptParameters) {`
	`100`	`+ if (!(params instanceof AddInterceptParameters)) {`
`101`	`101`	throw new Error(`Params must be an instance of AddInterceptParamenters. Received:'${params}'`)
`102`	`102`	`}`
`103`	`103`