openstack
/
congress-tempest-plugin
Code Issues Proposed changes

Replace assertEqual([], items) with assertEmpty(items) 

Since assertEmpty() function has already been implemented in tempest,
let's use this function instead of generic assertEqual() function.
This change makes the code and the error messages to be more readable.
Therefore it improves maintainability a little bit.

Change-Id: I20879ed2e662040e2d5c843634f9f90f396c33b8
This commit is contained in:
Vu Cong Tuan 2017-06-13 09:56:19 +07:00
parent 40ee3cd674
commit b0814a5584
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/scenario/congress_datasources/test_murano.py
View File

@ -195,7 +195,7 @@ class TestMuranoDriver(manager_congress.ScenarioPolicyBase):
_create_rule(policy_name, rule6)
_create_rule(policy_name, rule7)
result = _simulate_policy(policy_name, sim_query1)
self.assertEqual([], result)
self.assertEmpty(result)
result = _simulate_policy(policy_name, sim_query2)
self.assertEqual('predeploy_error("env_uuid")', result[0])