New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for webdriver.io timeouts #258

Merged
merged 1 commit into from Oct 18, 2016

Conversation

Projects
None yet
2 participants
@easternbloc
Contributor

easternbloc commented Oct 14, 2016

This adds support for each possible timeouts option in webdriver.io

In your codecept.conf.js you can express it something like:

{
  ...
  WebDriverIO: {
    url: 'http://localhost:8080',
    browser: 'phantomjs',
    timeouts: {
      implicit: 10000,
     'page load': 20000
      script: 5000
    }
  }
}
Add support for webdriver.io timeouts
This adds support for each possible timeouts option in webdriver.io

In your codecept.conf.js you can express it something like:

```
{
  ...
  WebDriverIO: {
    url: 'http://localhost:8080',
    browser: 'phantomjs',
    timeouts: {
      implicit: 10000,
      page_load: 20000
      script: 5000
    }
  }
}
```
@DavertMik

This comment has been minimized.

Show comment
Hide comment
@DavertMik

DavertMik Oct 18, 2016

Member

Wow, thanks. Those options should be documented as well, I think I can do it.

Member

DavertMik commented Oct 18, 2016

Wow, thanks. Those options should be documented as well, I think I can do it.

@DavertMik DavertMik merged commit cf432f5 into Codeception:master Oct 18, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@easternbloc easternbloc deleted the easternbloc:feature/webdriver-io-timeouts branch Oct 18, 2016

@easternbloc

This comment has been minimized.

Show comment
Hide comment
@easternbloc

easternbloc Oct 18, 2016

Contributor

thanks!

Contributor

easternbloc commented Oct 18, 2016

thanks!

@DavertMik

This comment has been minimized.

Show comment
Hide comment
@DavertMik

DavertMik Oct 18, 2016

Member

I think this should options should be defined per tests, as well. Or inside the code...
It would be useful to define special timeout settings before some heavy pages or data to be loaded

Member

DavertMik commented Oct 18, 2016

I think this should options should be defined per tests, as well. Or inside the code...
It would be useful to define special timeout settings before some heavy pages or data to be loaded

@easternbloc

This comment has been minimized.

Show comment
Hide comment
@easternbloc

easternbloc Oct 18, 2016

Contributor

@DavertMik yeah although I think some of the timeouts once set run for the duration of the webdriver instance so not quite sure how that'd play out...

Contributor

easternbloc commented Oct 18, 2016

@DavertMik yeah although I think some of the timeouts once set run for the duration of the webdriver instance so not quite sure how that'd play out...

@DavertMik

This comment has been minimized.

Show comment
Hide comment
@DavertMik

DavertMik Oct 18, 2016

Member

Probably we need more methods to be added to do this, like:

I.setTimeout({ script: 5000 });
Member

DavertMik commented Oct 18, 2016

Probably we need more methods to be added to do this, like:

I.setTimeout({ script: 5000 });
@DavertMik

This comment has been minimized.

Show comment
Hide comment
@DavertMik

DavertMik Oct 18, 2016

Member

better to call this function another way for not to be confused with real setTimeout ;)

Member

DavertMik commented Oct 18, 2016

better to call this function another way for not to be confused with real setTimeout ;)

Nighthawk14 added a commit to Nighthawk14/CodeceptJS that referenced this pull request Nov 7, 2016

Update upstream (#2)
* fixed definition generation

* Fix def command (#223)

* Fix def command

Container does not have getTranslations() method, just translation().

* Fix def command

Fix typo in previous commit.

* updated version and changelog

* BeforeSuite and AfterSuite implement (#231)

* close tabs after test, close browser after all tests

* revert changes, fix afterAll

* experemental

* Implement BeforeSuite AfterSuite

* Remove temp changes

* remove formatting

* remove formatting

* remove Formatting

* Fix AfterSuite Hook

* Fix formatting

* updated docs for beforesuite

* version bump

* small fixes

* Update docs for beforeSuite/afterSuite and fix restart false (#240)

* close tabs after test, close browser after all tests

* revert changes, fix afterAll

* experemental

* Implement BeforeSuite AfterSuite

* Remove temp changes

* remove formatting

* remove formatting

* remove Formatting

* Fix AfterSuite Hook

* Fix formatting

* Update docs

* Update docs 2

* Update docs 3

* Fix "restart: false"

* version bump

* updated version, changelog

* updated docs

* Update attachFile.mustache

* Update SeleniumWebdriver.js

* Use global actor function in Page Objects docs (#251)

* Use global actor function in Page Objects docs

As suggested in https://github.com/Codeception/CodeceptJS/blob/master/CHANGELOG.md#022 , otherwise the example code might not work as expected.

* Update remaining usages of 'codeceptjs/actor'

* Fix attachFile on WebDriverIO helper (#254)

* Fix attachFile on WebDriverIO helper

* Improve check on returned element

* Update to helpers docs (#264)

* update to helpers docs

* updated docs

* fixed file uploads fro WebDriverIO (#265)

* Update helpers.md

* Update helpers.md

* Add support for webdriver.io timeouts (#258)

This adds support for each possible timeouts option in webdriver.io

In your codecept.conf.js you can express it something like:

```
{
  ...
  WebDriverIO: {
    url: 'http://localhost:8080',
    browser: 'phantomjs',
    timeouts: {
      implicit: 10000,
      page_load: 20000
      script: 5000
    }
  }
}
```

* do not throw error in dispatcher (#263)

* do not throw error, emit it!

fix issue Codeception#262

* remove emit error

* catch mocha fail

* Improved timeouts for WebdriverIO (#267)

* added timeouts to config section and defineTimeout method

* tiny typo

* fixed Nightmare bug: unable to locate and click element; async improv… (#268)

* fixed Nightmare bug: unable to locate and click element; async improvements

* added fix and test for Nightmare not located clicable element

* fix test generation (#260)

* using unlinkSync to prepare tests

* Added option to manually start the browser. (#255)

* Added manual option to WebDriverIO helper, to start client manualy.

* Added option to manualy start the browser.

* added config param in docs

* Generators fix (#270)

* added mkdirp for recursive directory creation

* init creates all required dirs

* Added config option descriptions for manualStart option (#271)

* added config option descriptions for manualStart option

* manual start renamed back

* Added moveCursorTo to Protractor, Nightmare, SeleniumWebdriver (#269)

* [SeleniumWebdriver][Protrctor][Nightmare] Added moveCursorTo and basic test

* added wait for hover test

* fixed moveCursorTo test

* another try to fix hover test

* fixed test

* another try for the test fix

* disabling hover test

* updated changelog

* updated docs

* fixed gt example

* updated mocha to fix security issue (#273)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment