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

Commit

Permalink
fix(cli): Make the total timeout configurable.
Browse files Browse the repository at this point in the history
  • Loading branch information
Adam Shannon authored and juliemr committed Oct 8, 2013
1 parent 7d74184 commit e34a4ab
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
3 changes: 2 additions & 1 deletion lib/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ var config = {
seleniumArgs: [],
seleniumPort: null,
seleniumAddress: null,
allScriptsTimeout: 11000,
capabilities: {
'browserName': 'chrome'
},
Expand Down Expand Up @@ -146,7 +147,7 @@ var startJasmineTests = function() {
withCapabilities(config.capabilities).build();

driver.getSession().then(function(session) {
driver.manage().timeouts().setScriptTimeout(11000);
driver.manage().timeouts().setScriptTimeout(config.allScriptsTimeout);

id = session.getId();

Expand Down
4 changes: 4 additions & 0 deletions referenceConf.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,10 @@ exports.config = {
// seleniumAddress: 'http://localhost:4444/wd/hub'
seleniumAddress: null,

// The timeout for each script run on the browser. This should be longer
// than the maximum time your application needs to stabilize between tasks.
allScriptsTimeout = 11000,

This comment has been minimized.

Copy link
@cristifilip

cristifilip Oct 16, 2013

isn't it allScriptsTimeout: 11000, not allScriptsTimeout = 11000 ?

This comment has been minimized.

Copy link
@juliemr

juliemr Oct 16, 2013

Member

This has been fixed now :)


// ----- What tests to run -----
//
// Spec patterns are relative to the location of this config.
Expand Down

0 comments on commit e34a4ab

Please sign in to comment.