fix: close leaked InMemoryCatalog in test_inspect.py catalog fixture

Fixes ResourceWarning unclosed database connections in test_inspect.py tests.
The local catalog fixture (lines 38-42) returned the InMemoryCatalog without
yielding or calling close(), leaving SQLite connections uncovered.

Changed the fixture to a generator that yields and closes the catalog on
teardown, matching the pattern established in conftest.py and elsewhere on
this branch for issue #2530.

All 4 tests in test_inspect.py now pass without ResourceWarnings.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Author: williamacostalora

tests/table/test_inspect.py

@@ -15,6 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
+from collections.abc import Iterator
 from pathlib import PosixPath
 
 import pyarrow as pa
@@ -36,10 +37,11 @@ def test_readable_bound_without_bound() -> None:
 
 
 @pytest.fixture
-def catalog(tmp_path: PosixPath) -> InMemoryCatalog:
+def catalog(tmp_path: PosixPath) -> Iterator[InMemoryCatalog]:
     cat = InMemoryCatalog("test.in_memory.catalog", warehouse=tmp_path.absolute().as_posix())
     cat.create_namespace("default")
-    return cat
+    yield cat
+    cat.close()
 
 
 def test_inspect_entries_and_files_render_empty_string_bound(catalog: InMemoryCatalog) -> None: