Skip to content
This repository has been archived by the owner on Sep 3, 2020. It is now read-only.

remove duplicate chromedriver check #137

Merged
merged 1 commit into from
Feb 2, 2015

Conversation

EndangeredMassa
Copy link
Collaborator

There were two checks for a local chromedriver. The first was not presenting the fancy error. Removing it allows the second check to do so. It will now look something like:

Preparing chrome...

/home/smassa/source/testium/lib/cli/console.js:112
      throw error;
            ^
Could not find required files for running selenium.
       - message: ENOENT, stat '/home/smassa/source/testium/bin/chromedriver'

You can provide your own version of selenium via ~/.testiumrc:

\```
[selenium]
jar = /path/to/selenium.jar
; For running tests in chrome:
chromedriver = /path/to/chromedriver
\```

testium can also download these files for you,
just execute the following before running your test suite:

$ ./node_modules/.bin/testium --download-selenium

testium will download selenium and chromedriver into this directory:
/home/smassa/source/testium/bin
Error: ENOENT, stat '/home/smassa/source/testium/bin/chromedriver'

(ignore the faux-escaped triple-`)


Addresses: #133

@jkrems
Copy link
Contributor

jkrems commented Feb 2, 2015

LGTM

EndangeredMassa added a commit that referenced this pull request Feb 2, 2015
…-check

remove duplicate chromedriver check
@EndangeredMassa EndangeredMassa merged commit 671a037 into master Feb 2, 2015
@EndangeredMassa EndangeredMassa deleted the remove-dupe-chromedriver-check branch February 2, 2015 22:37
@EndangeredMassa
Copy link
Collaborator Author

published in 2.6.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants