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
Reviewed: https:/ /review. openstack. org/231869 /git.openstack. org/cgit/ openstack/ manila/ commit/ ?id=8ce1530e218 62b934faae9ffab f0ad879df0aae3
Committed: https:/
Submitter: Jenkins
Branch: master
commit 8ce1530e21862b9 34faae9ffabf0ad 879df0aae3
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. al(observed, expected) => assertEqual( expected, observed)
assertEqu
Target of this patch: tests/share/ *
manila/
Note: al(xx, True), assertEqual(True, xx) => assertTrue(xx)
I also fix following asserts around above fixed parts.
assertEqu
As for assertFalse,
I do not convert assertEqual(xx, False) to assertFalse(xx)
because assertFalse(None) does not raise exception.
Change-Id: I56688105ab3b77 e4f71bde472763a 9c02c43664b
Partial-Bug: #1259292