← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1277104] Re: wrong order of assertEquals args

 

Reviewed:  https://review.openstack.org/266819
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=b4db9c2644a04002232aee049315d89b2ac54bbe
Submitter: Jenkins
Branch:    master

commit b4db9c2644a04002232aee049315d89b2ac54bbe
Author: Yatin Kumbhare <yatinkumbhare@xxxxxxxxx>
Date:   Wed Jan 13 15:27:52 2016 +0530

    Fix params order in assertEqual
    
    Fix params order to correspond to real signature:
    assertEqual(expected, actual)
    
    Change-Id: I7888ca97afad663185a508002c6d9c62357e0cde
    Closes-Bug: #1277104


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1277104

Title:
  wrong order of assertEquals args

Status in Ceilometer:
  Fix Released
Status in Cinder:
  Fix Released
Status in Glance:
  Fix Released
Status in ironic-python-agent:
  Fix Released
Status in OpenStack Identity (keystone):
  Fix Released
Status in Manila:
  In Progress
Status in neutron:
  Fix Released
Status in oslo.messaging:
  Fix Released
Status in oslo.policy:
  Fix Released
Status in python-ceilometerclient:
  Fix Released
Status in python-glanceclient:
  Fix Released
Status in python-ironicclient:
  Fix Released
Status in python-novaclient:
  Fix Released
Status in python-openstackclient:
  Fix Released
Status in Python client library for Sahara:
  Fix Released
Status in python-solumclient:
  New
Status in python-swiftclient:
  Won't Fix
Status in python-troveclient:
  Fix Released
Status in Rally:
  Confirmed
Status in Trove:
  Fix Released

Bug description:
  Args of assertEquals method in ceilometer.tests are arranged in wrong order. In result when test fails it shows incorrect information about observed and actual data. It's found more than 2000 times.
  Right order of arguments is "expected, actual".

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