From 0aba00d6931c38974d4ed9e1c6130e367f910e48 Mon Sep 17 00:00:00 2001 From: Vu Cong Tuan Date: Tue, 13 Jun 2017 09:56:19 +0700 Subject: [PATCH] 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 --- .../tests/scenario/congress_datasources/test_murano.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/congress_tempest_tests/tests/scenario/congress_datasources/test_murano.py b/congress_tempest_tests/tests/scenario/congress_datasources/test_murano.py index 97ba480e8..63b257426 100644 --- a/congress_tempest_tests/tests/scenario/congress_datasources/test_murano.py +++ b/congress_tempest_tests/tests/scenario/congress_datasources/test_murano.py @@ -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])