It can be a real pain to debug selenium errors, especially when it is not reproducible locally. It would be great to capture a screenshot when an element is not found on the page for instance.
Created attachment 75287 [details] [review] Bug 20757: Capture and upload screenshot on selenium errors It is a real pain to debug selenium errors, especially when it is not reproducible locally. This patch capture a screenshot when an error occurred and upload it using the excellent lut.im service provided by framasoft (We could host our own later). Test plan: Modify a selenium script to make it fails (search for find_element) You will see a stack trace followed by a link to framapic.org
delete-day=1 means unlimited, would be nicer to lower it (7 should be enough).
Created attachment 75465 [details] [review] Bug 20757: Capture and upload screenshot on selenium errors It is a real pain to debug selenium errors, especially when it is not reproducible locally. This patch capture a screenshot when an error occurred and upload it using the excellent lut.im service provided by framasoft (We could host our own later). Test plan: Modify a selenium script to make it fails (search for find_element) You will see a stack trace followed by a link to framapic.org Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 75772 [details] [review] Bug 20757: Capture and upload screenshot on selenium errors It is a real pain to debug selenium errors, especially when it is not reproducible locally. This patch capture a screenshot when an error occurred and upload it using the excellent lut.im service provided by framasoft (We could host our own later). Test plan: Modify a selenium script to make it fails (search for find_element) You will see a stack trace followed by a link to framapic.org Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Please add POD for capture
Created attachment 77182 [details] [review] Bug 20757: Add pod for ->capture
Awesome work all! Pushed to master for 18.11
Pushed to 18.05.x for 18.05.03. Although it's an enhancement, it's a developer only one and aids in identifying causes for test failures so I decided it was worthwhile backporting.
Depends on Bug 19802 not in 17.11.x