** 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

-- 
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help   : https://help.launchpad.net/ListHelp

Reply via email to