yahoo-eng-team team mailing list archive

Thread
Date

[Bug 1325906] Re: Huge test case for Ironic Nova driver should be split

To: yahoo-eng-team@xxxxxxxxxxxxxxxxxxx
From: Sean Dague <sean@xxxxxxxxx>
Date: Sat, 20 Feb 2016 10:23:26 -0000
Reply-to: Bug 1325906 <1325906@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

** Changed in: nova
       Status: Confirmed => Won't Fix

-- 
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 Ironic:
  Won't Fix
Status in OpenStack Compute (nova):
  Won't Fix

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