Testing method for LBaaS update status() is partially wrong
Bug #1212229 reported by
Evgeny Fedoruk
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
neutron |
Fix Released
|
Low
|
Eugene Nikanorov |
Bug Description
In testing class of the db_loadbalancer (\neutron\
is a method testing the update_status functionality, test_update_
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.assertFals
Changed in neutron: | |
status: | New → Confirmed |
importance: | Undecided → Low |
assignee: | nobody → Eugene Nikanorov (enikanorov) |
tags: | added: lbaas unittest |
Changed in neutron: | |
milestone: | none → havana-3 |
Changed in neutron: | |
status: | Fix Committed → Fix Released |
Changed in neutron: | |
milestone: | havana-3 → 2013.2 |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/41918
Review: https:/