yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #08154
[Bug 1268480] [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: ceilometer
Importance: Undecided
Assignee: lizheming (lizheming-li)
Status: New
** Affects: glance
Importance: Undecided
Assignee: wanghong (w-wanghong)
Status: New
** Affects: keystone
Importance: Undecided
Assignee: wanghong (w-wanghong)
Status: New
** Affects: neutron
Importance: Undecided
Assignee: lizheming (lizheming-li)
Status: New
** Affects: nova
Importance: Undecided
Assignee: lizheming (lizheming-li)
Status: New
** Affects: python-novaclient
Importance: Undecided
Assignee: lizheming (lizheming-li)
Status: New
** Also affects: glance
Importance: Undecided
Status: New
** Also affects: keystone
Importance: Undecided
Status: New
** Also affects: neutron
Importance: Undecided
Status: New
** Also affects: cinder
Importance: Undecided
Status: New
** Also affects: ceilometer
Importance: Undecided
Status: New
** Changed in: nova
Assignee: (unassigned) => lizheming (lizheming-li)
** Changed in: cinder
Assignee: (unassigned) => lizheming (lizheming-li)
** Changed in: neutron
Assignee: (unassigned) => lizheming (lizheming-li)
** Changed in: ceilometer
Assignee: (unassigned) => lizheming (lizheming-li)
--
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/1268480
Title:
assertTrue(isinstance()) in tests should be replace with
assertIsInstance()
Status in OpenStack Telemetry (Ceilometer):
New
Status in OpenStack Image Registry and Delivery Service (Glance):
New
Status in OpenStack Identity (Keystone):
New
Status in OpenStack Neutron (virtual network service):
New
Status in OpenStack Compute (Nova):
New
Status in Python client library for 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/ceilometer/+bug/1268480/+subscriptions
Follow ups
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Sean McGinnis, 2016-09-27
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-05-31
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Sharat Sharma, 2016-05-16
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-05-15
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Sharat Sharma, 2016-05-12
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Sharat Sharma, 2016-05-12
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Sharat Sharma, 2016-05-12
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Sharat Sharma, 2016-05-12
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Dmitry Tantsur, 2016-04-10
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-04-09
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: sumit, 2016-03-07
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: sumit, 2016-03-07
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-02-13
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Dougal Matthews, 2016-01-27
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-25
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-25
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Dmitry Tantsur, 2016-01-18
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: sumit, 2016-01-17
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: sumit, 2016-01-17
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-16
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-11
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-11
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-11
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-11
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: LiuNanke, 2016-01-09
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: LiuNanke, 2016-01-09
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: LiuNanke, 2016-01-09
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: LiuNanke, 2016-01-09
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-08
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-08
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: algerwang, 2016-01-07
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: LiuNanke, 2016-01-06
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: yapeng Yang, 2016-01-06
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: yapeng Yang, 2016-01-06
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-06
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-05
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: LiuNanke, 2016-01-05
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Kan, 2016-01-05
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2016-01-04
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: LiuNanke, 2016-01-04
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Shuquan Huang, 2015-12-31
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Shuquan Huang, 2015-12-31
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: OpenStack Infra, 2015-12-30
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Shuquan Huang, 2015-12-30
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Shuquan Huang, 2015-12-30
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Shuquan Huang, 2015-12-30
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Shuquan Huang, 2015-12-30
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Doug Hellmann, 2015-12-03
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Bertrand Lallau, 2015-10-28
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Doug Hellmann, 2015-07-08
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Doug Hellmann, 2015-06-24
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Louis Taylor, 2014-09-21
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: wanghong, 2014-03-06
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Thierry Carrez, 2014-03-05
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Thierry Carrez, 2014-03-05
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Thierry Carrez, 2014-03-05
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Melanie Witt, 2014-03-05
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Dolph Mathews, 2014-02-03
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Thierry Carrez, 2014-01-22
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Thierry Carrez, 2014-01-22
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: lizheming, 2014-01-19
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: John Griffith, 2014-01-17
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: ZhiQiang Fan, 2014-01-17
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Liusheng, 2014-01-17
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: ZhiQiang Fan, 2014-01-17
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: Liusheng, 2014-01-17
-
[Bug 1268480] Re: assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: lizheming, 2014-01-13
-
[Bug 1268480] [NEW] assertTrue(isinstance()) in tests should be replace with assertIsInstance()
From: lizheming, 2014-01-13
References