Remove options

Author: thomashohn

lib/helper/WebDriver.js

@@ -614,11 +614,6 @@ class WebDriver extends Helper {
         delete this.options.capabilities.hostname
         delete this.options.capabilities.port
         delete this.options.capabilities.path
-        if (this.options.devtoolsProtocol) {
-          if (!['chrome', 'chromium'].includes(this.options.browser.toLowerCase()))
-            throw Error('The devtools protocol is only working with Chrome or Chromium')
-          this.options.automationProtocol = 'devtools'
-        }
         this.browser = await webdriverio.remote(this.options)
       }
     } catch (err) {
@@ -649,10 +644,8 @@ class WebDriver extends Helper {
       this.browser.capabilities.platformName = this.browser.capabilities.platformName.toLowerCase()
     }
 
-    if (this.options.automationProtocol) {
-      this.puppeteerBrowser = await this.browser.getPuppeteer()
-      this.page = (await this.puppeteerBrowser.pages())[0]
-    }
+    this.puppeteerBrowser = await this.browser.getPuppeteer()
+    this.page = (await this.puppeteerBrowser.pages())[0]
 
     return this.browser
   }
@@ -1143,10 +1136,6 @@ class WebDriver extends Helper {
     assertElementExists(res, field, 'Field')
     const elem = usingFirstElement(res)
     highlightActiveElement.call(this, elem)
-    if (this.options.automationProtocol) {
-      const curentValue = await elem.getValue()
-      return elem.setValue(curentValue + value.toString())
-    }
     return elem.addValue(value.toString())
   }
 
@@ -1159,9 +1148,6 @@ class WebDriver extends Helper {
     assertElementExists(res, field, 'Field')
     const elem = usingFirstElement(res)
     highlightActiveElement.call(this, elem)
-    if (this.options.automationProtocol) {
-      return elem.setValue('')
-    }
     return elem.clearValue(getElementId(elem))
   }
 
@@ -1231,7 +1217,7 @@ class WebDriver extends Helper {
     const el = usingFirstElement(res)
 
     // Remote Upload (when running Selenium Server)
-    if (this.options.remoteFileUpload && !this.options.automationProtocol) {
+    if (this.options.remoteFileUpload) {
       try {
         this.debugSection('File', 'Uploading file to remote server')
         file = await this.browser.uploadFile(file)
@@ -2593,9 +2579,6 @@ class WebDriver extends Helper {
   async switchTo(locator) {
     this.browser.isInsideFrame = true
     if (Number.isInteger(locator)) {
-      if (this.options.automationProtocol) {
-        return this.browser.switchToFrame(locator + 1)
-      }
       return this.browser.switchToFrame(locator)
     }
     if (!locator) {
@@ -2741,12 +2724,6 @@ class WebDriver extends Helper {
    * {{> setGeoLocation }}
    */
   async setGeoLocation(latitude, longitude) {
-    if (!this.options.automationProtocol) {
-      console.log(`setGeoLocation deprecated:
-    * This command is deprecated due to using deprecated JSON Wire Protocol command. More info: https://webdriver.io/docs/api/jsonwp/#setgeolocation
-    * Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration`)
-      return
-    }
     this.geoLocation = { latitude, longitude }
 
     await this.browser.call(async () => {
@@ -2762,12 +2739,6 @@ class WebDriver extends Helper {
    *
    */
   async grabGeoLocation() {
-    if (!this.options.automationProtocol) {
-      console.log(`grabGeoLocation deprecated:
-    * This command is deprecated due to using deprecated JSON Wire Protocol command. More info: https://webdriver.io/docs/api/jsonwp/#getgeolocation
-    * Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration`)
-      return
-    }
     if (!this.geoLocation) return 'No GeoLocation is set!'
     return this.geoLocation
   }
@@ -2818,12 +2789,6 @@ class WebDriver extends Helper {
    * {{> flushNetworkTraffics }}
    */
   flushNetworkTraffics() {
-    if (!this.options.automationProtocol) {
-      console.log(
-        '* Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration',
-      )
-      return
-    }
     this.requests = []
   }
 
@@ -2834,12 +2799,6 @@ class WebDriver extends Helper {
    * {{> stopRecordingTraffic }}
    */
   stopRecordingTraffic() {
-    if (!this.options.automationProtocol) {
-      console.log(
-        '* Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration',
-      )
-      return
-    }
     this.page.removeAllListeners('request')
     this.recording = false
   }
@@ -2852,12 +2811,6 @@ class WebDriver extends Helper {
    *
    */
   async startRecordingTraffic() {
-    if (!this.options.automationProtocol) {
-      console.log(
-        '* Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration',
-      )
-      return
-    }
     this.flushNetworkTraffics()
     this.recording = true
     this.recordedAtLeastOnce = true
@@ -2890,12 +2843,6 @@ class WebDriver extends Helper {
    * {{> grabRecordedNetworkTraffics }}
    */
   async grabRecordedNetworkTraffics() {
-    if (!this.options.automationProtocol) {
-      console.log(
-        '* Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration',
-      )
-      return
-    }
     return grabRecordedNetworkTraffics.call(this)
   }
 
@@ -2906,12 +2853,6 @@ class WebDriver extends Helper {
    * {{> seeTraffic }}
    */
   async seeTraffic({ name, url, parameters, requestPostData, timeout = 10 }) {
-    if (!this.options.automationProtocol) {
-      console.log(
-        '* Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration',
-      )
-      return
-    }
     await seeTraffic.call(this, ...arguments)
   }
 
@@ -2923,12 +2864,6 @@ class WebDriver extends Helper {
    *
    */
   dontSeeTraffic({ name, url }) {
-    if (!this.options.automationProtocol) {
-      console.log(
-        '* Switch to devtools protocol to use this command by setting devtoolsProtocol: true in the configuration',
-      )
-      return
-    }
     dontSeeTraffic.call(this, ...arguments)
   }
 }