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: Fri, 15 Jan 2016 18:11:59 -0000
Reply-to: Bug 1277104 <1277104@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

Reviewed:  https://review.openstack.org/266247
Committed: https://git.openstack.org/cgit/openstack/ironic-python-agent/commit/?id=2b07976b948d0e768e2b46c5b63ee454e5f8aeb2
Submitter: Jenkins
Branch:    master

commit 2b07976b948d0e768e2b46c5b63ee454e5f8aeb2
Author: Kan <kansks@xxxxxxxxxx>
Date:   Tue Jan 12 09:03:19 2016 +0000

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


** Changed in: ironic-python-agent
       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:
  In Progress
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