yahoo-eng-team team mailing list archive

["Dmitry \"Divius\" Tantsur" <divius.inside@xxxxxxxxx>]

Should be fixed in Nova now - driver is no longer in Ironic tree

** Also affects: nova
   Importance: Undecided
       Status: New

** Summary changed:

- Huge test case for Nova driver should be split
+ Huge test case for Ironic Nova driver should be split

** Changed in: ironic
       Status: In Progress => Won't Fix

** Tags added: ironic

-- 
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/1325906

Title:
  Huge test case for Ironic Nova driver should be split

Status in OpenStack Bare Metal Provisioning Service (Ironic):
  Won't Fix
Status in OpenStack Compute (Nova):
  New

Bug description:
  Currently, tests for Ironic Nova driver consist of one giant test
  case. That is not convenient from mocking point of view, as it
  requires different set of mocks to be places on every method. It is
  also hard to reason about. Splitting into several meaningful test
  cases will allow to have mock decorators mostly at class-levels, which
  will lead to less cluttered code. It will also simplify review for
  Nova folks.

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