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

   ### What changes were proposed in this PR?
   
   Adds a unit test for `OperatorMetadataGenerator` covering its 
previously-untested error branch: `generateOperatorMetadata` throws a 
`RuntimeException` when passed a class that is not registered in 
`operatorTypeMap` (the existing metadata specs only exercise the happy path). 
The test uses the abstract `LogicalOp` base — which is not a registered 
concrete `@JsonTypeName` subtype — so the throw fires before any instantiation, 
needing no DB/actor.
   
   No production code is changed; this is test-only.
   
   ### Any related issues, documentation, discussions?
   
   Coverage gaps identified from the Codecov report for `apache/texera`.
   
   ### How was this PR tested?
   
   New ScalaTest spec, run locally:
   
   ```
   sbt "WorkflowOperator/testOnly *OperatorMetadataGeneratorSpec"
   ```
   
   Passes. `scalafmt` and `scalafixAll --check` are clean.
   
   ### 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