TST: added test fo closing AlignDb

[CHANGED] remove effort to better handle garbage collection
    error as it was invalid

src/ensembl_tui/_align.py

@@ -288,6 +288,12 @@ def get_distinct(self, field: str) -> list[str]:
     def num_records(self) -> int:
         return self.conn.sql(f"SELECT COUNT(*) from {self._tables[0]}").fetchone()[0]
 
+    def close(self) -> None:
+        """closes duckdb and h5py storage"""
+        if self.gap_store:
+            self.gap_store.close()
+        self.conn.close()
+
 
 def get_alignment(
     align_db: AlignDb,

src/ensembl_tui/_storage_mixin.py

@@ -95,8 +95,6 @@ def close(self) -> None:
         """closes the hdf5 file"""
         # hdf5 dumps content to stdout if resource already closed, so
         # we trap that here, and capture expected exceptions raised in the process
-        if "open" not in locals():
-            return
         with (
             open(os.devnull, "w") as devnull,  # noqa: PTH123
             contextlib.redirect_stderr(devnull),

tests/test_align.py

@@ -590,3 +590,9 @@ def test_aligndb_post_init_failure(tmp_path):
     outfile.write_text("blah")
     with pytest.raises(OSError):
         eti_align.AlignDb(source="/non/existent/directory")
+
+
+def test_aligndb_close(db_align):
+    db_align.close()
+    with pytest.raises(duckdb.duckdb.ConnectionException):
+        db_align.num_records()