stark256-spec commented on PR #3454:
URL: https://github.com/apache/iceberg-python/pull/3454#issuecomment-4665631258
Pushed in 7ab945a:
- Added D105 back to the pydocstyle ignore list (per the discussion above)
and removed the dead inline `# noqa: D105` comments.
- Added two integration tests in \`tests/integration/test_rest_catalog.py\`
against the real REST fixture (\`rest-page-size=1\`):
- \`test_list_tables_returns_pagination_list\` — creates a namespace +
tables, asserts \`list_tables\` returns a \`PaginationList\` containing all of
them
- \`test_list_namespaces_returns_pagination_list\` — same for
\`list_namespaces\`
Both set page size to 1 so if the fixture honors pagination, the lazy
multi-page fetch path gets exercised end-to-end; if it doesn't, the tests still
validate the `RestCatalog` → `PaginationList` wiring against a live server.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
