yahoo-eng-team team mailing list archive

[Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>]

** Changed in: neutron
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1212229

Title:
  Testing method for LBaaS update status() is partially wrong

Status in OpenStack Neutron (virtual network service):
  Fix Released

Bug description:
  In testing class of the db_loadbalancer (\neutron\tests\unit\db\loadbalancer\test_db_loadbalancer.py)
  is a method testing the update_status functionality, test_update_status().

  The last line of the test is checking that status description was cleared 
  after invoking the update_status() without description parameter.

  The problem is that assert checks the pool object (the original object after creation)
  instead of updated_pool object which was retrieved after the update_status() invocation.

  That causes the  last assert to always succeed.

  Suggestion for fix:
  Use the updated_pool object in assertFalse(), 
  e.g. self.assertFalse(updated_pool['status_description'])

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