Cole-Greer opened a new pull request, #3431: URL: https://github.com/apache/tinkerpop/pull/3431
`shouldTrackTransactionCountAccurately` was racing because commitTx() and rollbackTx() returned without consuming the HTTP response. With chunked transfer encoding, the client can receive response headers before the server completes transactionManager.destroy(). Wrapping in try-with-resources ensures the full response is read, guaranteeing server-side processing is complete before asserting the count. `shouldTimeoutIdleTransactionWithNoOperations` used a 1ms timeout, which could expire during the beginTx() HTTP round-trip itself, causing getFirstHeader(TRANSACTION_ID) to return null and NPE. Increased to 500ms — still well below the 1000ms sleep that validates timeout behavior. `PropertyDeserializationTests` and `GraphTraversalTests` were running concurrently with `@GraphComputerOnly` Gherkin scenarios. During OLAP computation, TinkerGraphComputer sets a graphComputerView on the shared TinkerGraph instance. Any concurrent read through TinkerVertex sees compute properties (like haltedTraversers) merged with real properties, causing spurious assertion failures and NPEs. Place all tests that hit the shared server into a single non-parallel xUnit collection (GremlinServerTests) so they never overlap with an active GraphComputer view. VOTE +1 -- 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]
