yahoo-eng-team team mailing list archive

[Yikun Jiang <yikunkero@xxxxxxxxx>]

Public bug reported:

The test code in [1][2] uses `raise` statement without any parameter. It
is not a valid python construct if used outside of an `expect` block.

The test does not fail on this as this codepath never executed.


Also see the reference bug[3].

[1]
https://github.com/openstack/nova/blob/a7994ec3559cede764a6ff04f7fddc5a5b6d63ea/nova/tests/unit/compute/test_compute_mgr.py#L5851

[2]
https://github.com/openstack/nova/blob/a7994ec3559cede764a6ff04f7fddc5a5b6d63ea/nova/tests/unit/compute/test_compute_mgr.py#L5872

[3]https://bugs.launchpad.net/nova/+bug/1754409

** Affects: nova
     Importance: Undecided
         Status: New


** Tags: testing

** Tags added: testing

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

Title:
  Invalid raise in test_compute_mgr

Status in OpenStack Compute (nova):
  New

Bug description:
  The test code in [1][2] uses `raise` statement without any parameter.
  It is not a valid python construct if used outside of an `expect`
  block.

  The test does not fail on this as this codepath never executed.

  
  Also see the reference bug[3].

  [1]
  https://github.com/openstack/nova/blob/a7994ec3559cede764a6ff04f7fddc5a5b6d63ea/nova/tests/unit/compute/test_compute_mgr.py#L5851

  [2]
  https://github.com/openstack/nova/blob/a7994ec3559cede764a6ff04f7fddc5a5b6d63ea/nova/tests/unit/compute/test_compute_mgr.py#L5872

  [3]https://bugs.launchpad.net/nova/+bug/1754409

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