improvements to the tests, and added testing to the pipeline. Now a package won't publish if it doesn't test :)

.github/workflows/python-publish.yml

@@ -22,6 +22,9 @@ jobs:
       run: |
         python -m pip install --upgrade pip
         pip install setuptools wheel twine
+    - name: Test #TODO use a testing solution that won't be deprecated soon
+      run: |
+        python setup.py test
     - name: Build and publish
       env:
         TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }}

setup.py

@@ -7,6 +7,7 @@ setuptools.setup(
     name="vaultdb",
     version="0.0.1",
     version_config=True,
+    test_suite="test",
     setup_requires=['setuptools-git-versioning'],
     author="Beppe Vanrolleghem",
     author_email="beppe.vanrolleghem@gmail.com",

test/test_database.py

@@ -4,8 +4,17 @@ import os
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
 import src
 
-class MyTestCase(unittest.TestCase):
-    def test_psql(self):
+class DatabaseClassTests(unittest.TestCase):
+
+    def test_vault_connection_error(self):
+        #TODO give some "valid" url
+        self.fail()
+
+    def test_vault_non_existent_database(self):
+        #TODO test non existent_database
+        self.fail()
+
+    def test_psql_connection(self):
         #TODO exec psql connect
         print("hi")
         test = src.Database()
@@ -13,18 +22,24 @@ class MyTestCase(unittest.TestCase):
         self.assertEqual(test.valid, False)
 
     def test_psql_invalid_vault_url(self):
-        #TODO figure out how to fail unit test check
         try:
             src.Database(dbname="sdfk", vault_url="localhost", vault_port=8200, token="bla")
         except:
-            print("did it")
+            #TODO check the err msg
             return
+        else:
+            self.fail()
+
 
     def test_psql_invalid_vault_port(self):
         try:
             src.Database(dbname="sdfk", vault_url="localhost", vault_port="a", token="bla")
         except:
+            #TODO check the error message
             return
+        else:
+            self.fail()
+
 
     def test_psql_valid(self):
         print(src.Database(dbname="psql", vault_url="http://localhost", token="s.GoR2nisHPeKU1vOaw9hZ5L7h").get_creds())