yahoo-eng-team team mailing list archive

Thread
Date

[Bug 1277104] Re: wrong order of assertEquals args

To: yahoo-eng-team@xxxxxxxxxxxxxxxxxxx
From: OpenStack Infra <1277104@xxxxxxxxxxxxxxxxxx>
Date: Mon, 08 Feb 2016 15:42:28 -0000
Reply-to: Bug 1277104 <1277104@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

Reviewed:  https://review.openstack.org/266724
Committed: https://git.openstack.org/cgit/openstack/manila/commit/?id=dac72337f7efbd75e0eb269ad8e1fdd89a13e4b9
Submitter: Jenkins
Branch:    master

commit dac72337f7efbd75e0eb269ad8e1fdd89a13e4b9
Author: Yatin Kumbhare <yatinkumbhare@xxxxxxxxx>
Date:   Wed Jan 13 12:16:06 2016 +0530

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


** Changed in: manila
       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:
  Fix Released
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:
  Fix Released
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