← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1259292] Re: Some tests use assertEqual(observed, expected) , the argument order is wrong

 

Reviewed:  https://review.openstack.org/329787
Committed: https://git.openstack.org/cgit/openstack/python-glanceclient/commit/?id=c3de38ed530b8db77185e819af65574e35ebe134
Submitter: Jenkins
Branch:    master

commit c3de38ed530b8db77185e819af65574e35ebe134
Author: zhengyao1 <zheng.yao1@xxxxxxxxxx>
Date:   Wed Jun 15 16:42:42 2016 +0800

    Use correct order of arguments to assertEqual
    
    The correct order of arguments to assertEqual that is expected by
    testtools is (expected, observed).
    
    This patch fixes the inverted usage of arguments in some places
    that have cropped up since the last fix of this bug.
    
    Change-Id: If8c0dcb58496bc2fcf4c635f384522a1f7d2b2af
    Closes-Bug: #1259292


** Changed in: python-glanceclient
       Status: In Progress => Fix Released

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

Title:
  Some tests use assertEqual(observed, expected) , the argument order is
  wrong

Status in Astara:
  In Progress
Status in Barbican:
  In Progress
Status in Blazar:
  New
Status in Ceilometer:
  Invalid
Status in Cinder:
  Fix Released
Status in congress:
  Fix Released
Status in daisycloud-core:
  New
Status in Designate:
  Fix Released
Status in Freezer:
  In Progress
Status in Glance:
  Fix Released
Status in glance_store:
  Fix Released
Status in Higgins:
  New
Status in OpenStack Dashboard (Horizon):
  In Progress
Status in OpenStack Identity (keystone):
  Fix Released
Status in Magnum:
  Fix Released
Status in Manila:
  Fix Released
Status in Mistral:
  Fix Released
Status in Murano:
  Fix Released
Status in OpenStack Compute (nova):
  Won't Fix
Status in os-brick:
  In Progress
Status in python-ceilometerclient:
  Invalid
Status in python-cinderclient:
  Fix Released
Status in python-designateclient:
  Fix Committed
Status in python-glanceclient:
  Fix Released
Status in python-mistralclient:
  Fix Released
Status in python-solumclient:
  Fix Released
Status in Python client library for Zaqar:
  Fix Released
Status in Sahara:
  Fix Released
Status in sqlalchemy-migrate:
  New
Status in SWIFT:
  New
Status in tacker:
  New
Status in tempest:
  New
Status in zaqar:
  Fix Released

Bug description:
  The test cases will produce a confusing error message if the tests
  ever fail, so this is worth fixing.

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