[I] Add unit test coverage for the pyamber source and batch operator final methods [texera]

["aglinxinyuan (via GitHub)" <gi...@apache.org>]

aglinxinyuan opened a new issue, #8043:
URL: https://github.com/apache/texera/issues/8043

   ### Task Summary
   
   `core/models/operator.py` sits at 171 of 183 statements fully covered, and the gap includes two paths that matter more than the count suggests:
   
   - **`SourceOperator.on_finish`** (182–183) is the uncovered production path of *every* Python source UDF.
   - **`BatchOperator._process_batch`'s output conversion** (233–234, 241) has its `None` and non-DataFrame arms unexercised.
   
   The remaining five gap lines (153, 177, 259, 293, 515) are `@abstractmethod` bodies with no production `super()` call site, so 178/183 is the file's permanent ceiling — worth recording so nobody re-derives it.
   
   ### Task Type
   
   - [x] Testing / QA
   
   ### Was this issue authored using generative AI tooling?
   
   Generated-by: Claude Code (Opus 5)
   


-- 
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: notifications-unsubscribe@texera.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Add unit test coverage for the pyamber source and batch operator final methods [texera]

["aglinxinyuan (via GitHub)" <gi...@apache.org>]

aglinxinyuan closed issue #8043: Add unit test coverage for the pyamber source and batch operator final methods
URL: https://github.com/apache/texera/issues/8043


-- 
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: notifications-unsubscribe@texera.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org