code clean up

JIRA: YARDSTICK-272

Change-Id: Id2f1b3d1beff8843700cdfcaaa1e8f496f445ed9
Signed-off-by: tjuyinkanglin <14_ykl@tongji.edu.cn>

6 files changed, 20 insertions, 20 deletions

diff --git a/tests/unit/benchmark/scenarios/availability/test_attacker_general.py b/tests/unit/benchmark/scenarios/availability/test_attacker_general.py
index d6488a9a7..643c1e7ac 100644
--- a/tests/unit/benchmark/scenarios/availability/test_attacker_general.py
+++ b/tests/unit/benchmark/scenarios/availability/test_attacker_general.py
@@ -32,7 +32,7 @@ class GeneralAttackerServiceTestCase(unittest.TestCase):
             'fault_type': 'general-attacker',
             'action_parameter':{'process_name':'nova_api'},
             'rollback_parameter':{'process_name':'nova_api'},
-            'key':'stop_service',
+            'key':'stop-service',
             'host': 'node1',
         }
 
diff --git a/tests/unit/benchmark/scenarios/availability/test_basemonitor.py b/tests/unit/benchmark/scenarios/availability/test_basemonitor.py
index 140841075..a20cf8187 100644
--- a/tests/unit/benchmark/scenarios/availability/test_basemonitor.py
+++ b/tests/unit/benchmark/scenarios/availability/test_basemonitor.py
@@ -23,7 +23,7 @@ class MonitorMgrTestCase(unittest.TestCase):
     def setUp(self):
         config = {
             'monitor_type': 'openstack-api',
-            'key' : 'service_status'
+            'key' : 'service-status'
         }
 
         self.monitor_configs = []
@@ -40,7 +40,7 @@ class MonitorMgrTestCase(unittest.TestCase):
     def test_MonitorMgr_getitem(self, mock_monitor):
         monitorMgr = basemonitor.MonitorMgr()
         monitorMgr.init_monitors(self.monitor_configs, None)
-        monitorIns = monitorMgr['service_status']
+        monitorIns = monitorMgr['service-status']
 
 class BaseMonitorTestCase(unittest.TestCase):
 
diff --git a/tests/unit/benchmark/scenarios/availability/test_baseoperation.py b/tests/unit/benchmark/scenarios/availability/test_baseoperation.py
index 8c341913f..d85f1e19f 100644
--- a/tests/unit/benchmark/scenarios/availability/test_baseoperation.py
+++ b/tests/unit/benchmark/scenarios/availability/test_baseoperation.py
@@ -22,7 +22,7 @@ class OperationMgrTestCase(unittest.TestCase):
     def setUp(self):
         config = {
             'operation_type': 'general-operation',
-            'key' : 'service_status'
+            'key' : 'service-status'
         }
 
         self.operation_configs = []
@@ -31,7 +31,7 @@ class OperationMgrTestCase(unittest.TestCase):
     def  test_all_successful(self, mock_operation):
         mgr_ins = baseoperation.OperationMgr()
         mgr_ins.init_operations(self.operation_configs, None)
-        operation_ins = mgr_ins["service_status"]
+        operation_ins = mgr_ins["service-status"]
         mgr_ins.rollback()
 
     def test_getitem_fail(self, mock_operation):
@@ -59,7 +59,7 @@ class BaseOperationTestCase(unittest.TestCase):
     def setUp(self):
         self.config = {
             'operation_type': 'general-operation',
-            'key' : 'service_status'
+            'key' : 'service-status'
         }
 
     def test_all_successful(self):
diff --git a/tests/unit/benchmark/scenarios/availability/test_director.py b/tests/unit/benchmark/scenarios/availability/test_director.py
index 887ddd631..06116725d 100644
--- a/tests/unit/benchmark/scenarios/availability/test_director.py
+++ b/tests/unit/benchmark/scenarios/availability/test_director.py
@@ -33,16 +33,16 @@ class DirectorTestCase(unittest.TestCase):
                     'key': "kill-process"}],
                 'monitors': [{
                     'monitor_type': "general-monitor",
-                    'key': "service_status"}],
+                    'key': "service-status"}],
                 'operations': [{
                     'operation_type': 'general-operation',
-                    'key' : 'service_status'}],
+                    'key' : 'service-status'}],
                 'resultCheckers': [{
                     'checker_type': 'general-result-checker',
                     'key' : 'process-checker',}],
                 'steps':[
                     {
-                        'actionKey': "service_status",
+                        'actionKey': "service-status",
                         'actionType': "operation",
                         'index': 1},
                     {
@@ -54,7 +54,7 @@ class DirectorTestCase(unittest.TestCase):
                         'actionType': "resultchecker",
                         'index': 3},
                     {
-                        'actionKey': "service_status",
+                        'actionKey': "service-status",
                         'actionType': "monitor",
                         'index': 4},
                     ]
@@ -69,12 +69,12 @@ class DirectorTestCase(unittest.TestCase):
 
     def test_director_all_successful(self, mock_checer, mock_opertion, mock_attacker, mock_monitor):
         ins = Director(self.scenario_cfg, self.ctx)
-        opertion_action = ins.createActionPlayer("operation", "service_status")
+        opertion_action = ins.createActionPlayer("operation", "service-status")
         attacker_action = ins.createActionPlayer("attacker", "kill-process")
         checker_action = ins.createActionPlayer("resultchecker", "process-checker")
-        monitor_action = ins.createActionPlayer("monitor", "service_status")
+        monitor_action = ins.createActionPlayer("monitor", "service-status")
 
-        opertion_rollback = ins.createActionRollbacker("operation", "service_status")
+        opertion_rollback = ins.createActionRollbacker("operation", "service-status")
         attacker_rollback = ins.createActionRollbacker("attacker", "kill-process")
         ins.executionSteps.append(opertion_rollback)
         ins.executionSteps.append(attacker_rollback)
diff --git a/tests/unit/benchmark/scenarios/availability/test_monitor_general.py b/tests/unit/benchmark/scenarios/availability/test_monitor_general.py
index 85487a574..b48434c99 100644
--- a/tests/unit/benchmark/scenarios/availability/test_monitor_general.py
+++ b/tests/unit/benchmark/scenarios/availability/test_monitor_general.py
@@ -31,18 +31,18 @@ class GeneralMonitorServiceTestCase(unittest.TestCase):
         self.context = {"node1": host}
         self.monitor_cfg = {
             'monitor_type': 'general-monitor',
-            'key': 'service_status',
+            'key': 'service-status',
             'host': 'node1',
             'monitor_time': 3,
             'parameter': {'serviceName': 'haproxy'},
-            'sla': {'max_recover_time': 1}
+            'sla': {'max_outage_time': 1}
         }
         self.monitor_cfg_noparam = {
             'monitor_type': 'general-monitor',
-            'key': 'service_status',
+            'key': 'service-status',
             'host': 'node1',
             'monitor_time': 3,
-            'sla': {'max_recover_time': 1}
+            'sla': {'max_outage_time': 1}
         }
 
     def test__monitor_general_all_successful(self, mock_open, mock_ssh):
diff --git a/tests/unit/benchmark/scenarios/availability/test_scenario_general.py b/tests/unit/benchmark/scenarios/availability/test_scenario_general.py
index c17edea45..bab9d62f1 100644
--- a/tests/unit/benchmark/scenarios/availability/test_scenario_general.py
+++ b/tests/unit/benchmark/scenarios/availability/test_scenario_general.py
@@ -29,14 +29,14 @@ class ScenarioGeneralTestCase(unittest.TestCase):
                     'key': "kill-process"}],
                 'monitors': [{
                     'monitor_type': "general-monitor",
-                    'key': "service_status"}],
+                    'key': "service-status"}],
                 'steps':[
                     {
                         'actionKey': "kill-process",
                         'actionType': "attacker",
                         'index': 1},
                     {
-                        'actionKey': "service_status",
+                        'actionKey': "service-status",
                         'actionType': "monitor",
                         'index': 2}]
             }
@@ -62,4 +62,4 @@ class ScenarioGeneralTestCase(unittest.TestCase):
         mock_obj.verify.return_value = False
         ins.director = mock_obj
         ins.run(None)
-        ins.teardown()
\ No newline at end of file
+        ins.teardown()