Comment 152 for bug 1259292

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

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

commit 8ce1530e21862b934faae9ffabf0ad879df0aae3
Author: Yusuke Hayashi <email address hidden>
Date: Sun Oct 4 03:43:17 2015 +0900

    Fix order of arguments in assertEqual

    Fix incorrect order assertEqual(observed, expected) as below.
      assertEqual(observed, expected) => assertEqual(expected, observed)

    Target of this patch:
      manila/tests/share/*

    Note:
    I also fix following asserts around above fixed parts.
      assertEqual(xx, True), assertEqual(True, xx) => assertTrue(xx)

    As for assertFalse,
    I do not convert assertEqual(xx, False) to assertFalse(xx)
    because assertFalse(None) does not raise exception.

    Change-Id: I56688105ab3b77e4f71bde472763a9c02c43664b
    Partial-Bug: #1259292