Summary: | Move Selenium code to its own module | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | mtompset |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | |||
Bug Blocks: | 19243, 20757, 21479 | ||
Attachments: |
Bug 19802: Move Selenium code to t::lib::Selenium
Bug 19802: Add stack trace to ease debugging Bug 19802: [DO NOT PUSH] Generate screentshots Bug 19802: Move Selenium code to t::lib::Selenium Bug 19802: Add stack trace to ease debugging |
Description
Jonathan Druart
2017-12-12 22:15:02 UTC
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 |