I think it would be good to move some of the tests to the "integration test" arrow_reader binary rather than as unit tests so the structure visibility is tested (the rules for tests compiled with --lib is different than external) clearer, but we can do that as a follow on PR

Originally posted by @alamb in #6637 (review)