Last modified: 2013-12-02 16:06:00 UTC
As of today we have four tests that use the @make_selectable_line and @edit_user_page hooks, and that pass with or without the REUSE_BROWSER env variable. I suggest we set that env variable REUSE_BROWSER=true for bullets.feature indent_outdent.feature headings.feature general_markup.feature This will save a significant amount of time for each of the four VE builds, since the four tests will log in only four times per build instead of a total of twenty-one times without REUSE_BROWSER=true. Note that setting REUSE_BROWSER=true does not affect our ability to analyze test failures. For example, see the failure messages below for a run of the bullets test: ---- timed out after 10 seconds (Watir::Wait::TimeoutError) ./features/step_definitions/bullets_steps.rb:26:in `block (2 levels) in <top (required)>' ./features/step_definitions/bullets_steps.rb:25:in `/^a \# is added in front of input string in the diff view$/' features/bullets.feature:8:in `Then a <character> is added in front of input string in the diff view' timed out after 10 seconds (Watir::Wait::TimeoutError) ./features/step_definitions/bullets_steps.rb:39:in `block (2 levels) in <top (required)>' ./features/step_definitions/bullets_steps.rb:38:in `/^a \* is added in front of input string in the diff view$/' features/bullets.feature:8:in `Then a <character> is added in front of input string in the diff view' timed out after 10 seconds (Watir::Wait::TimeoutError) ./features/step_definitions/bullets_steps.rb:52:in `block (2 levels) in <top (required)>' ./features/step_definitions/bullets_steps.rb:51:in `/^a \#\# is added in front of input string in the diff view$/' features/bullets.feature:21:in `Then a <character> is added in front of input string in the diff view' timed out after 10 seconds (Watir::Wait::TimeoutError) ./features/step_definitions/bullets_steps.rb:61:in `block (2 levels) in <top (required)>' ./features/step_definitions/bullets_steps.rb:60:in `/^a \*\* is added in front of input string in the diff view$/' features/bullets.feature:21:in `Then a <character> is added in front of input string in the diff view'
I am **strongly against** reusing browsers between scenarios, for various reasons. If speed is our problem, we should run the tests in parallel.
As far as I remember we have agreed this will not be implemented. Please reopen if I misunderstood.