test(integration): run arrow scenarios in CI instead of skipping#102
Conversation
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Verified: pyarrow floor matches pyproject's arrow extra; direct imports correctly replace importorskip so CI fails loudly; managed-catalog polling loop handles 404 retry / non-404 re-raise / timeout cleanly. LGTM.
super nit (not blocking): this removed the only consumer of the refresh_api fixture, so refresh_api and the RefreshApi import in tests/integration/conftest.py are now dead and could be cleaned up.
There was a problem hiding this comment.
Verified: pyarrow >= 14 floor matches the arrow extra in pyproject.toml, removed imports/fixtures are fully cleaned up, the ORDER BY makes the column/pagination asserts deterministic, and the managed-tables semantics changes are documented and still verify the load via the response. LGTM.
1 participant
Closes #100.
Adds
pyarrowtotest-requirements.txtand replacesimportorskipwith direct imports so the two arrow scenarios actually run in CI instead of silently skipping. Also correctsmanaged_tables_lifecycleto real managed-catalog semantics: drops therefreshstep (400 on a managed catalog) and pollsget_table_profilefor the async post-load sync.