Refactored test implementation to be compatible with PEP8

Author: userlocalhost

tests/acos_base_action_test_case.py

@@ -3,7 +3,6 @@
 import yaml
 
 from st2tests.base import BaseActionTestCase
-from urllib.parse import urlparse
 
 
 class ACOSBaseActionTestCase(BaseActionTestCase):
@@ -39,18 +38,19 @@ def get_mock_session(self, resp_data):
         whether sending requests from each test have expected destination endpoint
         and expected parameters.
         """
+        ACOS_API_AUTH_PATH = [
+            '/services/rest/v2.1/?format=json&method=authenticate',
+            '/axapi/v3/auth'
+        ]
         self._sending_requests = []
+
         def _save_sending_request(api_url, method, params):
             self._sending_requests.append({
                 'url': api_url,
                 'method': method,
                 'params': params,
             })
 
-        ACOS_API_AUTH_PATH = [
-            '/services/rest/v2.1/?format=json&method=authenticate',
-            '/axapi/v3/auth'
-        ]
         def _mock_sending_request(api_url, method, params):
             mock_resp = mock.Mock()
             if method == 'POST' and any([x in api_url for x in ACOS_API_AUTH_PATH]):
@@ -68,8 +68,10 @@ def _mock_sending_request(api_url, method, params):
 
         def get_side_effect(api_url, **kwargs):
             return _mock_sending_request(api_url, 'GET', kwargs)
+
         def post_side_effect(api_url, **kwargs):
             return _mock_sending_request(api_url, 'POST', kwargs)
+
         def delete_side_effect(api_url, **kwargs):
             return _mock_sending_request(api_url, 'DELETE', kwargs)
 

tests/test_slb_service_group.py

@@ -1,10 +1,8 @@
 import json
 import mock
-import re
 
 from acos_base_action_test_case import ACOSBaseActionTestCase
 from ax_action_runner import AXActionRunner
-from requests import Session
 from urllib.parse import urlparse
 
 RESP_SUCCESS = {'response': {'status': 'success'}}
@@ -243,7 +241,8 @@ def test_create_slb_service_group_for_acos(self, mock_session):
                 "name": "test-slb-service-group",
                 "protocol": "tcp",
                 "health-check-disable": 0,
-                "lb-method": "round-robin","stateless-auto-switch": 0
+                "lb-method": "round-robin",
+                "stateless-auto-switch": 0
             }
         }))
         self.assertEqual(self._sending_requests[0]['method'], 'POST')