← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1268479] [NEW] assertTrue(isinstance()) in tests should be replace with assertIsInstance()

 

Public bug reported:

some of tests use different method of assertTrue(isinstance(A, B)) or
assertEqual(type(A), B). The correct way is to use assertIsInstance(A,
B) provided by testtools

** Affects: nova
     Importance: Undecided
         Status: New

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

Title:
  assertTrue(isinstance()) in tests should be replace with
  assertIsInstance()

Status in OpenStack Compute (Nova):
  New

Bug description:
  some of tests use different method of assertTrue(isinstance(A, B)) or
  assertEqual(type(A), B). The correct way is to use assertIsInstance(A,
  B) provided by testtools

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


Follow ups

References