Magnum

Use assertIsInstance instead of assertTrue(isinstance(a, b))

Bug #1510384 reported by wangqun
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Undecided
wangqun
Magnum mitaka-1

Bug Description

Developer should use assertIsInstance to replace the
assertTrue(isinstance(a, b)).

wangqun (bjwqun)
Changed in magnum:
assignee: nobody → wangqun (bjwqun)
Revision history for this message
wangqun (bjwqun) wrote :

https://review.openstack.org/#/c/239601/

venkatamahesh (venkatamaheshkotha)
Changed in magnum:
status: New → Confirmed
OpenStack Infra (hudson-openstack)
Changed in magnum:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/239601
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=87d6056846341d1b9b1087561768a7d1c99a9905
Submitter: Jenkins
Branch: master

commit 87d6056846341d1b9b1087561768a7d1c99a9905
Author: wangqun <email address hidden>
Date: Tue Oct 27 06:19:03 2015 +0000

    Use assertIsInstance instead of assertTrue(isinstance(a, b))

    Developer should use assertIsInstance to replace the
    assertTrue(isinstance(a, b)).

    Closes-Bug: #1510384
    Change-Id: I70f68a5810a6797c7b7e112b46ef2463907bba3e

Changed in magnum:
status: In Progress → Fix Committed
Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.