← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1545042] [NEW] New i9n tests exceptions_captured context manager introduce a bug in failure reports

 

Public bug reported:

If test fails in a non-fatal for Selenium way and it's still able to
fetch page source/ browser log/ screenshot that data is not added to
TestCase reports because of lines like
https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/integration_tests/helpers.py#L127
where `content` gets thrown away once we return from block
https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/integration_tests/helpers.py#L106

To mitigate that we need to use a container instead of single reference.

** Affects: horizon
     Importance: High
     Assignee: Timur Sufiev (tsufiev-x)
         Status: In Progress

** Changed in: horizon
   Importance: Undecided => High

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1545042

Title:
  New i9n tests exceptions_captured context manager introduce a bug in
  failure reports

Status in OpenStack Dashboard (Horizon):
  In Progress

Bug description:
  If test fails in a non-fatal for Selenium way and it's still able to
  fetch page source/ browser log/ screenshot that data is not added to
  TestCase reports because of lines like
  https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/integration_tests/helpers.py#L127
  where `content` gets thrown away once we return from block
  https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/integration_tests/helpers.py#L106

  To mitigate that we need to use a container instead of single
  reference.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1545042/+subscriptions


Follow ups