Merge lp:~coreygoldberg/selenium-simple-test/native-to-synthesized into lp:selenium-simple-test
Proposed by
Corey Goldberg
Status: | Needs review |
---|---|
Proposed branch: | lp:~coreygoldberg/selenium-simple-test/native-to-synthesized |
Merge into: | lp:selenium-simple-test |
Diff against target: |
16 lines (+6/-0) 1 file modified
src/sst/browsers.py (+6/-0) |
To merge this branch: | bzr merge lp:~coreygoldberg/selenium-simple-test/native-to-synthesized |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Canonical ISD QA Team | Pending | ||
Review via email: mp+191985@code.launchpad.net |
Commit message
switched from native events to synthesized events
Description of the change
turned off native events (this enables JS synthesized events).
see attached bug for details on native vs. synthesized events.
To post a comment you must log in.
Unmerged revisions
- 438. By Corey Goldberg
-
revert changes except event model
- 437. By Corey Goldberg
-
disable native events and move sleep
> also moved sleep in browser launcher, so we don't always get a spurious "can't connect" message every time.
-1 on that, we can succeed without the explicit sleep (that's also what the selenium original implementation is doing), utils.is_ connectable will already introduce a delay when it tries to connect, no need for an additional one.
The "can't connect" you're seeing is a real one, it's not spurious. It may be that on jenkins the load is such that the first attempt never succeed but that's not true when working locally so we shouldn't slow down the happy path.
I don't have an opinion about switching the event model so I let others reply on that. As long as the new model has been tested under all envs we care about, whatever works !