I see that this test was not ported directly -- since the avro code uses the same codepath as all the other listing tables for handling multiple files it is probably ok.

However, I think it is important coverage to maintain as the end to end coverage makes sure everything is hooked up correctly.

I think the issue is there is no way to setup these files using sqllogictest. Perhaps you can add a special sqllogictest test setup (to make the directory with multiple files) following this model:

https://github.com/apache/arrow-datafusion/blob/b578c5819fc05801f2972dd427fbc13cf4773ea8/datafusion/core/tests/sqllogictests/src/main.rs#L171

Considering that avro_query_multiple_files utilizes a temporary directory, could we maintain the temporary directory within SessionContext to facilitate the creation of necessary data during the setup phase?

🤔 I think adding something to SessionContext for testing only is likely not the best idea

Maybe we could return the TempDir alongside the SessionContext like:

struct TestContext {
  /// Context for running queries
  ctx: SessionContext,
  /// Temporary directory created and cleared at the end of the test
  tmpdir: TempDir
}

And then change

async fn context_for_test_file(relative_path: &Path) -> SessionContext {

to

async fn context_for_test_file(relative_path: &Path) -> TestContext {

Thank you for taking the time to review my PR 🙏. I have updated the code. Please take another look.