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.
Reviewed: https:/ /review. openstack. org/165271 /git.openstack. org/cgit/ openstack/ sahara/ commit/ ?id=cb480679a1f 75f558be414f851 43c11b2406a31b
Committed: https:/
Submitter: Jenkins
Branch: master
commit cb480679a1f75f5 58be414f85143c1 1b2406a31b
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: Iefbb5f4b9efca3 5f7a2298f9f7f5b 9de73fc260f