GumpacG opened a new pull request, #3584: URL: https://github.com/apache/tinkerpop/pull/3584
# Increase gremlin-python test coverage Adds targeted unit tests for `gremlin-python` that exercise previously-untested library code: the explicit-transaction lifecycle (`Transaction` / `execute_in_tx`), the GraphBinary V4 serializer internals (`configure_pdt_registry` merge and `finalize_message` byte layout), the `Traversal` iteration protocol and `traversal.py` object model (`Traverser`, `TraversalStrategies`/`TraversalStrategy`, `P`/`TextP`, `AtomicInteger`, `GremlinLang` copy/convert), `DriverRemoteConnection`/`RemoteStrategy` wiring and `extract_request_options`, the SigV4 auth interceptor, the aiohttp read-timeout normalization (`ServerTimeoutError` -> `ReadTimeoutError`), GraphBinary V4 error branches (unknown type code, long overflow, naive datetime), and several `structure/graph.py` object-model / PDT-registry paths. Also adds one integration test for the server-enforced per-request `timeoutMillis`. **Tests only, no production code changes.** ## Coverage (gremlin-python: unit + integration + feature) | Metric | Baseline (master) | With added tests | |--------|-------------------|------------------| | Line | 88.1% | 90.0% | ## Coverage reports Baseline (master: b907ebbed0) and with-tests HTML reports were generated with `coverage.py` (unit + integration + radish): <img width="672" height="832" alt="Screenshot 2026-07-30 at 8 20 59 AM" src="https://github.com/user-attachments/assets/5cc36608-03db-445f-a603-3b1edf3b84bc"; /> With added tests: <img width="687" height="842" alt="Screenshot 2026-07-29 at 5 05 29 PM" src="https://github.com/user-attachments/assets/dfdac5b4-f921-4bec-a13d-6efd40b75d43"; /> Assisted-by: Kiro: Claude Opus 4.8 -- 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]
