rtomac/robotframework-selenium2library — Issue #19: 'Set Selenium Timeout' does not pass timeout variable to selenium
Author: emanlove
Created: 2011-12-16 23:14:20+00:00
State: open

It appears the 'Set Selenium Timeout' keyword does not actually set the timeout. The code as currently written

```python
def set_selenium_timeout(self, seconds):
# . . .
old_timeout = self.get_selenium_timeout()
self._timeout_in_secs = robot.utils.timestr_to_secs(seconds)
return old_timeout

def set_selenium_implicit_wait(self, seconds):
# . . .
```

and the code as I suspect it should be

```python
def set_selenium_timeout(self, seconds):
# . . .
old_timeout = self.get_selenium_timeout()
self._timeout_in_secs = robot.utils.timestr_to_secs(seconds)
for browser in self._cache.browsers:
browser.set_script_timeout(self._timeout_in_secs)
return old_timeout

def set_selenium_implicit_wait(self, seconds):
# . . .
```

Also it appears this feature is untested within the rf-s2l acceptance tests.