refactor test

Signed-off-by: Chris Snow <[email protected]>

Author: snowch

DEVELOPING.md

@@ -110,7 +110,7 @@ tox -e py27 -- tests/library/client_test.py:TestCreateFromProperties.test_create
 Aim for 100% test coverage to ensure library will work with all specified python versions.
 
 ```
-coverage erase && coverage run --source hpecp setup.py test && coverage report -m
+coverage erase && coverage run --source hpecp,bin setup.py test && coverage report -m
 ```
 
 

tests/cli/cli_test.py

@@ -120,10 +120,7 @@ def mocked_requests_get(*args, **kwargs):
             )
         raise RuntimeError("Unhandle GET request: " + args[0])
 
-    @patch("requests.post", side_effect=mocked_requests_post)
-    @patch("requests.get", side_effect=mocked_requests_get)
-    def test_example(self, mock_post, mock_get):
-
+    def setUp(self):
         file_data = dedent(
             """[default]
                         api_host = 127.0.0.1
@@ -135,17 +132,29 @@ def test_example(self, mock_post, mock_get):
                         password = admin123"""
         )
 
-        tmp = tempfile.NamedTemporaryFile(delete=True)
-        try:
-            tmp.write(file_data.encode("utf-8"))
-            tmp.flush()
+        self.tmpFile = tempfile.NamedTemporaryFile(delete=True)
+        self.tmpFile.write(file_data.encode("utf-8"))
+        self.tmpFile.flush()
+
+        sys.path.insert(0, os.path.abspath("../../"))
+        from bin import cli
+
+        self.cli = cli
+        self.cli.HPECP_CONFIG_FILE = self.tmpFile.name
 
-            cli.HPECP_CONFIG_FILE = tmp.name
+    def tearDown(self):
+        self.tmpFile.close()
 
-            hpecp = cli.CLI()
-            hpecp.k8scluster.list()
+    def test_autocomplete_bash(self):
+
+        hpecp = self.cli.CLI()
+        hpecp.autocomplete.bash()
+
+    # TODO move this to tests/library/k8s_cluster_test.py
+    @patch("requests.post", side_effect=mocked_requests_post)
+    @patch("requests.get", side_effect=mocked_requests_get)
+    def test_k8scluster_list(self, mock_post, mock_get):
 
-            self.assertTrue(True)
+        hpecp = self.cli.CLI()
+        hpecp.k8scluster.list()
 
-        finally:
-            tmp.close()