Set default WebDriver path to / (webdriverio#4907)

* set default path to / * PR feedback * still use /wd/hub for cloud sessions

Author: christian-bromann

docs/ConfigurationFile.md

@@ -28,7 +28,7 @@ exports.config = {
  //
  hostname: 'localhost',
  port: 4444,
- path: '/wd/hub',
+ path: '/',
  // Protocol: http | https
  // protocol: 'http',
  //

docs/DriverBinaries.md

@@ -3,7 +3,7 @@ id: driverbinaries
 title: Driver Binaries
 ---
 
-WebdriverIO allows using driver binaries directly instead of services. 
+WebdriverIO allows using driver binaries directly instead of services.
 
 Here’s an example with `geckodriver`.
 
@@ -72,7 +72,7 @@ Start Geckodriver by running:
 /path/to/binary/geckodriver --port 4444
 ```
 
-For example, if you ran the `curl` command from above, you should have a `geckodriver` binary available in the current folder. 
+For example, if you ran the `curl` command from above, you should have a `geckodriver` binary available in the current folder.
 
 Start it like this:
 
@@ -84,28 +84,3 @@ Start it like this:
 This will start Geckodriver on `localhost:4444` with the WebDriver endpoint set to `/`.
 
 Keep this running in the background and open a new terminal window. Next step is to download WebdriverIO via NPM:
-
-### Configure the `path`
-
-If you are using driver’s binary directly (like `geckodriver` or `chromedriver`), the `path` to it is different from the expected default (which is `/wd/hub`). 
-
-To change that, we need to edit the `wdio.conf.js` file. 
-
-Add a section to specify the `path`:
-
-```json
- path: '/',
-```
-
-So the top of your file should look like:
-
-```json
-exports.config = {
- runner: 'local',
- //
- // Uncomment line below to override default path ('/wd/hub') for usage of driver binary directly, ex: chromedriver or geckodriver.
- path: '/',
-}
-```
-
-Be sure to save the file after your edits.

docs/GettingStarted.md

@@ -28,7 +28,7 @@ The `-y` will answer 'yes' to all the prompts, giving you a standard NPM project
 
 ### Install WebdriverIO CLI
 
-If you want to use WebdriverIO in your project for integration testing, we recommend using the test runner. It comes with lots of useful features that makes your life easier. 
+If you want to use WebdriverIO in your project for integration testing, we recommend using the test runner. It comes with lots of useful features that makes your life easier.
 
 Since WebdriverIO version 5, the testrunner is in the [`@wdio/cli`](https://www.npmjs.com/package/@wdio/cli) NPM package.
 
@@ -40,21 +40,19 @@ npm i --save-dev @wdio/cli
 
 ### Generate Configuration File
 
-Next, you’ll generate a configuration file to store your WebdriverIO settings. 
+Next, you’ll generate a configuration file to store your WebdriverIO settings.
 
 To do that, just run the configuration utility:
 
 ```sh
 ./node_modules/.bin/wdio config -y
 ```
 
-That's it! The configurator will install all required packages for you and create a config file called `wdio.conf.js`. 
-
-As we're using Geckodriver, we need to override the default `path` (which uses the Selenium's default of `/wd/hub`). Then, we'll be ready to create your first <dfn>spec file</dfn> (or “test file”).
+That's it! The configurator will install all required packages for you and create a config file called `wdio.conf.js`.
 
 ### Create Spec Files
 
-Now it's time to create your test file. You’re going to store all of your test files in a new folder. 
+Now it's time to create your test file. You’re going to store all of your test files in a new folder.
 
 Create the test folder like this:
 
@@ -101,7 +99,7 @@ Now save the file and return to your terminal.
 
 ### Start the Testrunner
 
-Now, time to run your tests! 
+Now, time to run your tests!
 
 To do so, just run:
 
@@ -113,6 +111,6 @@ Hurray! The test should pass, and you can start writing integration tests with W
 
 If you ran into any issues, reach out in our [Gitter Channel](https://gitter.im/webdriverio/webdriverio) and post the error you're seeing, and which of the above steps you’re stuck on.
 
-If you are interested in more in depth video on-boarding tutorials, feel free to check out our very own course called [learn.webdriver.io](https://learn.webdriver.io/?coupon=wdio). 
+If you are interested in more in depth video on-boarding tutorials, feel free to check out our very own course called [learn.webdriver.io](https://learn.webdriver.io/?coupon=wdio).
 
 Our community has also collected a lot of [boilerplate projects](BoilerplateProjects.md) that may help you to get started.

docs/Options.md

@@ -35,7 +35,7 @@ Default: `4444`
 Path to driver server endpoint.
 
 Type: `String`<br>
-Default: `/wd/hub`
+Default: `/`
 
 ### `queryParams`
 Query paramaters that are propagated to the driver server.

examples/wdio.conf.js

@@ -13,7 +13,7 @@ exports.config = {
  //
  hostname: 'localhost',
  port: 4444,
- path: '/wd/hub',
+ path: '/',
  // Protocol: http | https
  // protocol: 'http',
  //

packages/wdio-cli/src/commands/run.js

@@ -27,7 +27,7 @@ export const cmdArgs = {
  },
  path: {
  type: 'string',
- desc: 'path to WebDriver endpoints (default /wd/hub)'
+ desc: 'path to WebDriver endpoints (default "/")'
  },
  user: {
  alias: 'u',

packages/wdio-cli/src/constants.js

@@ -167,8 +167,8 @@ export const QUESTIONNAIRE = [{
 }, {
  type: 'input',
  name: 'path',
- message: 'What is the path to your Selenium standalone or grid server?',
- default: '/wd/hub',
+ message: 'What is the path to your browser driver or grid server?',
+ default: '/',
  when: /* istanbul ignore next */ (answers) => answers.backend.indexOf('own Selenium cloud') > -1
 }, {
  type: 'list',

packages/wdio-cli/src/templates/wdio.conf.tpl.ejs

@@ -24,16 +24,6 @@ exports.config = {
  path: '<%= answers.path %>',<% } else if (answers.services.includes('appium')) { %>
  port: 4723,<% }
 
- if(answers.services.includes('chromedriver')) { %>
- //
- // Override default path ('/wd/hub') for chromedriver service.
- path: '/',<% }
-
- if(answers.services.length === 0) { %>
- //
- // Uncomment line below to override default path ('/wd/hub') for usage of driver binary directly, ex: chromedriver or geckodriver.
- // path: '/',<% }
-
  // Protocol: http | https
  // protocol: 'http',
  if(answers.env_user && answers.env_key) { %>

packages/wdio-config/src/utils.js

@@ -1,6 +1,8 @@
 const DEFAULT_HOSTNAME = '127.0.0.1'
 const DEFAULT_PORT = 4444
 const DEFAULT_PROTOCOL = 'http'
+const DEFAULT_PATH = '/'
+const LEGACY_PATH = '/wd/hub'
 
 const REGION_MAPPING = {
  'us': '', // default endpoint
@@ -46,7 +48,7 @@ export function isCucumberFeatureWithLineNumber(spec) {
  * helper to detect the Selenium backend according to given capabilities
  */
 export function detectBackend (options = {}, isRDC = false) {
- let { port, hostname, user, key, protocol, region, headless } = options
+ let { port, hostname, user, key, protocol, region, headless, path } = options
 
  /**
  * browserstack
@@ -56,7 +58,8 @@ export function detectBackend (options = {}, isRDC = false) {
  return {
  protocol: protocol || 'https',
  hostname: hostname || 'hub-cloud.browserstack.com',
- port: port || 443
+ port: port || 443,
+ path: path || LEGACY_PATH
  }
  }
 
@@ -66,9 +69,10 @@ export function detectBackend (options = {}, isRDC = false) {
  */
  if (typeof user === 'string' && typeof key === 'string' && key.length === 32) {
  return {
- protocol: protocol || DEFAULT_PROTOCOL,
+ protocol: protocol || 'https',
  hostname: hostname || 'hub.testingbot.com',
- port: port || 80
+ port: port || 443,
+ path: path || LEGACY_PATH
  }
  }
 
@@ -88,7 +92,8 @@ export function detectBackend (options = {}, isRDC = false) {
  return {
  protocol: protocol || 'https',
  hostname: hostname || getSauceEndpoint(sauceRegion, isRDC),
- port: port || 443
+ port: port || 443,
+ path: path || LEGACY_PATH
  }
  }
 
@@ -115,7 +120,8 @@ export function detectBackend (options = {}, isRDC = false) {
  return {
  hostname: hostname || DEFAULT_HOSTNAME,
  port: port || DEFAULT_PORT,
- protocol: protocol || DEFAULT_PROTOCOL
+ protocol: protocol || DEFAULT_PROTOCOL,
+ path: path || DEFAULT_PATH
  }
 }
 

packages/wdio-config/tests/__mocks__/@wdio/config.js

@@ -30,7 +30,7 @@ export const validateConfig = jest.fn().mockImplementation(
  hostname: 'localhost',
  port: 4444,
  protocol: 'http',
- path: '/wd/hub',
+ path: '/',
  automationProtocol: 'webdriver'
  },
  config