ilo_deploy driver has poor test coverage for continue_deploy
Bug #1402019 reported by
aeva black
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ironic |
Fix Released
|
Low
|
Anusha |
Bug Description
There is only one test handling an error path for the _continue_deploy method, and it merely asserts that an invalid initial state results in a LOG message being written out.
It would be better to assert that, when given a valid initial condition, any exceptions raised by methods called from _continue_deploy are handled appropriately.
affects: | hacking → ironic |
tags: | added: driver |
Changed in ironic: | |
status: | New → Triaged |
importance: | Undecided → Low |
tags: | added: low-hanging-fruit |
Changed in ironic: | |
assignee: | nobody → Anusha (anusha-iiitm) |
Changed in ironic: | |
milestone: | none → kilo-3 |
status: | Fix Committed → Fix Released |
Changed in ironic: | |
milestone: | kilo-3 → 2015.1.0 |
To post a comment you must log in.
Fix proposed to branch: master /review. openstack. org/142985
Review: https:/