My bad, I should have run all tests before pushing changes.

From what I gather, when pb.load_dataset() is used with duckdb, it's returning a live connection to the Ibis table. Hence a bunch of these tests failed, because I explicitly closed the connection.

The problem appears to be that Windows runs into permission errors due to maintaining that live connection beyond the function where that connection is defined. I'm not sure what an optimal solution is here - possibly converting the file into memory, and then closing the connection?

[EDIT]

Upon further investigation, part of the problem likely has to do with the test fixtures also returning live connections, for example:

@pytest.fixturedeftbl_dates_times_text_duckdb():file_path=pathlib.Path.cwd()/"tests"/"tbl_files"/"tbl_dates_times_text.ddb"withtempfile.TemporaryDirectory()astmp:fpath:Path=Path(tmp)/"tbl_dates_times_text.ddb"shutil.copy(file_path,fpath)returnibis.connect(f"duckdb://{fpath!s}").table("tbl_dates_times_text")

Part of the solution might be changing these fixtures also.

@matt-humphrey Are you saying that these tests need to be edited as well?

@gregorywaynepower yes, it's been a while so I can revise and identify exactly which tests need to be edited, but running on Windows I was running into problems with most tests with duckdb.