Elioooon commented on PR #4017: URL: https://github.com/apache/iggy/pull/4017#issuecomment-5504598010
@ethanlin01x Thanks for the review. Added the requested coverage in `a4aab32`: - numeric stream/topic IDs exercise both create and delete - surviving partition IDs are asserted after deletion - zero-count create and delete are rejected - deleting more partitions than the topic contains is rejected - missing stream and missing topic are covered for both operations Revalidated with: - `cargo fmt --check` - `foreign/python/.venv/bin/ruff check foreign/python/tests/test_partition.py` - `foreign/python/.venv/bin/ruff format --check foreign/python/tests/test_partition.py` - local `iggy-server --with-default-root-credentials --fresh` plus `foreign/python/.venv/bin/pytest -q foreign/python/tests/test_partition.py` (5 passed) -- 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]
