To make it reusable easily
Created attachment 69756 [details] [review] Bug 19802: Move Selenium code to t::lib::Selenium To make it reusable easily. Test plan: The basic_workflow.t tests should still pass after this change.
Created attachment 70158 [details] [review] Bug 19802: Add stack trace to ease debugging
Created attachment 70159 [details] [review] Bug 19802: [DO NOT PUSH] Generate screentshots This needs to be improved, but let as it for now.
Created attachment 70733 [details] [review] Bug 19802: Move Selenium code to t::lib::Selenium To make it reusable easily. Test plan: The basic_workflow.t tests should still pass after this change. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 70734 [details] [review] Bug 19802: Add stack trace to ease debugging Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
prove before and after both pass, as expected. Code eyeballs well enough.
I am skipping QA, we need selenium tests to catch regressions.
Patches pushed to master for 18.05