aglinxinyuan opened a new pull request, #6310:
URL: https://github.com/apache/texera/pull/6310

   ### What changes were proposed in this PR?
   
   Adds a unit test for `RetrieveWorkflowStateHandler.retrieveWorkflowState`. 
Using a real `CoordinatorProcessor` with a capturing output handler and a 
`CoordinatorAsyncRPCHandlerInitializer`, it invokes the handler and asserts the 
dispatched `PropagateEmbeddedControlMessage` request — covering target-operator 
collection, request construction, dispatch, and response-transform 
registration, for both an empty and a populated physical plan.
   
   ### Any related issues, documentation, discussions?
   
   Closes #6307. Part of the amber unit-test coverage effort toward 80% 
([Codecov](https://app.codecov.io/gh/apache/texera)).
   
   ### How was this PR tested?
   
   Added a new `RetrieveWorkflowStateHandlerSpec` (2 cases), a plain-JVM 
`AnyFlatSpec` — no ActorSystem, DB, Python, or storage. Verified locally on 
Java 17 (`WorkflowExecutionService/testOnly`, plus `scalafixAll --check` 
clean). Test-only change — no production code is modified.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Opus 4.8 [1M context])


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