openstack
/
blazar
Code Issues Proposed changes

Use assertEqual(len(x), y) instead of assertTrue(len(x) == y) 

assertEqual() will return a nicer error if it fails.

Change-Id: I30c48fa17eee287bb5f42ed9b46e47f10bd39bb6
This commit is contained in:
Luong Anh Tuan 2018-07-06 11:04:34 +07:00 committed by Pierre Riteau
parent 8dd7a70a37
commit 0107cd72b6
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
blazar/tests/plugins/instances/test_instance_plugin.py
View File

@ -576,7 +576,7 @@ class TestVirtualInstancePlugin(tests.TestCase):
'removed': set(['host-id1', 'host-id2', 'host-id3'])}
ret = plugin.pickup_hosts(reservation['id'], values)
self.assertEqual(expect['added'], ret['added'])
self.assertTrue(len(ret['removed']) == 2)
self.assertEqual(len(ret['removed']), 2)
self.assertTrue(all([h in expect['removed'] for h in ret['removed']]))
query_params = {
'cpus': 1, 'memory': 1024, 'disk': 10,