Fixed tests

Author: ckunki

exasol/saas/client/api_access.py

@@ -255,6 +255,10 @@ def still_exists() -> bool:
             raise DatabaseDeleteTimeout
 
     def _wait_for_delete(self, database_id: str):
+        # if http
+        # "status": 400,
+        # "message": "Operation is not allowed:The cluster is not in a proper state!",
+        # then wait a bit an retry
         deletable = [
             Status.RUNNING,
             Status.SCALING,
@@ -321,6 +325,7 @@ def database(
         try:
             db = self.create_database(name, idle_time=idle_time)
             yield db
+            wait_for_delete_clearance()
         finally:
             db_repr = f"{db.name} with ID {db.id}" if db else None
             if db and not keep:

test/integration/connection_test.py

@@ -4,7 +4,6 @@
 from exasol.saas.client.api_access import get_connection_params
 
 
-@pytest.mark.slow
 def test_get_connection_params_with_id(
     saas_host, saas_pat, saas_account_id, operational_saas_database_id, allow_connection
 ):
@@ -23,7 +22,6 @@ def test_get_connection_params_with_id(
         assert result == [(1,)]
 
 
-@pytest.mark.slow
 def test_get_connection_params_with_name(
     saas_host,
     saas_pat,

test/integration/databases_test.py

@@ -9,6 +9,7 @@
 from exasol.saas.client import PROMISING_STATES
 from exasol.saas.client.api_access import (
     get_connection_params,
+    timestamp_name,
 )
 
 
@@ -66,11 +67,3 @@ def test_get_connection(api_access, local_name):
         clusters = api_access.clusters(db.id)
         connection = api_access.get_connection(db.id, clusters[0].id)
         assert connection.db_username is not None and connection.port == 8563
-
-
-def test_dummy():
-    """
-    Dummy test to prevent empty set of test cases to be interpreted as
-    failure.
-    """
-    pass