Comment 40 for bug 1259292

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to sahara (master)

Reviewed: https://review.openstack.org/165271
Committed: https://git.openstack.org/cgit/openstack/sahara/commit/?id=cb480679a1f75f558be414f85143c11b2406a31b
Submitter: Jenkins
Branch: master

commit cb480679a1f75f558be414f85143c11b2406a31b
Author: Jacob Bin Wang <email address hidden>
Date: Wed Mar 18 09:29:40 2015 +0800

    Fix order of arguments in assertEqual - Part1

    Some tests used incorrect order assertEqual(observed, expected).
    The correct order expected by testtools is:
        def assertEqual(self, expected, observed, message='')
    Changed to use the right order. Also if the order is wrong and
    expected value is None, change it to assertIsNone instead.

    It requires big changes and some tests change frequently.
    So I would do it in series of small CR.
    This part contains all changes except tests of plugins and service.

    Partial-Bug: #1259292

    Change-Id: Iefbb5f4b9efca35f7a2298f9f7f5b9de73fc260f