Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

seleniumArgs should have a default #65

Closed
tarjei opened this issue Aug 30, 2013 · 0 comments
Closed

seleniumArgs should have a default #65

tarjei opened this issue Aug 30, 2013 · 0 comments

Comments

@tarjei
Copy link

tarjei commented Aug 30, 2013

If I try to run with a config like this:

exports.config = {
  // The address of a running selenium server.
  //seleniumAddress: 'http://localhost:4444/wd/hub',
  seleniumServerJar: './selenium/selenium-server-standalone-2.35.0.jar',
  chromeDriver: './selenium/chromedriver',

  // Capabilities to be passed to the webdriver instance.
  capabilities: {
    'browserName': 'chrome'
  },

I get an error:
/home/tarjei/htdocs/frisvold/selenium-tests/node_modules/protractor/lib/cli.js:115
config.seleniumArgs.push(
^
TypeError: Cannot call method 'push' of undefined

Adding seleniumArgs solves this. Wouldn't it be better to just default seleniumArgs to []?

@juliemr juliemr closed this as completed in 9f53118 Sep 5, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant