Make changes to del statement in conftest __del__() method of database connection

Author: LTan-101104

src/database/database_connection.py

@@ -36,12 +36,8 @@ def disconnect(self) -> None:
         if self.connection is not None:
             self.connection.close()
             self.connection = None
-
-    def __del__(self) -> None:
-        """Ensure the connection is closed when the object is deleted."""
-        self.disconnect()
-        if os.getenv("RUN_ENV") != "TEST":
-            print(f"Disconnected from {self.db_url}")
+            if os.getenv("RUN_ENV") != "TEST":
+                print(f"Disconnected from {self.db_url}")
 
     def is_connected(self) -> bool:
         """

tests/conftest.py

@@ -84,6 +84,9 @@ def preprocess_mock_data(
         return mem_db.fetch_query(query=query)
     except Exception as e:
         raise Exception("Exception at helper_filter_irrelevant_records") from e
+    finally:
+        if mem_db is not None:
+            mem_db.disconnect()
 
 
 # Get path to the temporary mock database file
@@ -114,4 +117,3 @@ def mock_data_frame(mock_data_path: str) -> Generator[pd.DataFrame]:
     finally:
         if mem_db is not None:
             mem_db.disconnect()
-            del mem_db