CurtHagenlocher opened a new pull request, #467:
URL: https://github.com/apache/arrow-dotnet/pull/467

   ## What's Changed
   
   `FlightTests.EnsureCallRaisesRequestCancelled` cancelled its token with 
`CancelAfter(1)` and then awaited `Task.Delay(5)`, assuming the 1 ms 
cancellation would fire before the 5 ms delay completed. Nothing guarantees 
that ordering. On a loaded runner the first `ListFlights` call can start and 
complete before the token is cancelled, and the test fails with 
"Assert.Throws() Failure: No exception was thrown". This happened on macOS in 
the Verify job for #465, where the same test had passed earlier in the same job.
   
   The test now cancels the token before making any calls, so every call sees 
an already-cancelled token. What the test checks is unchanged: each client call 
surfaces `StatusCode.Cancelled`.
   
   Locally the test passed 25 of 25 runs, and the full Flight test suite passes.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   
   Co-Authored-By: Claude Opus 5.5 <[email protected]>
   


-- 
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]

Reply via email to