yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #08378
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
** Changed in: neutron
Status: New => Invalid
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1268480
Title:
assertTrue(isinstance()) in tests should be replace with
assertIsInstance()
Status in OpenStack Telemetry (Ceilometer):
In Progress
Status in Cinder:
Opinion
Status in OpenStack Image Registry and Delivery Service (Glance):
Fix Committed
Status in OpenStack Dashboard (Horizon):
In Progress
Status in Ironic (Bare Metal Provisioning):
In Progress
Status in OpenStack Identity (Keystone):
In Progress
Status in OpenStack Neutron (virtual network service):
Invalid
Status in OpenStack Compute (Nova):
In Progress
Status in Python client library for Ceilometer:
In Progress
Status in Python client library for Cinder:
In Progress
Status in Python client library for Keystone:
In Progress
Status in Python client library for Nova:
In Progress
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/ceilometer/+bug/1268480/+subscriptions
References