aglinxinyuan opened a new pull request, #6098: URL: https://github.com/apache/texera/pull/6098
### What changes were proposed in this PR? Add unit test coverage for three `workflow-operator` executors, selected from the Codecov report. No production-code changes. | File | Codecov before | New coverage | | --- | --- | --- | | `IntervalJoinOpExec.scala` | 81% | extends the existing spec to cover the YEAR/MONTH/HOUR/MINUTE/SECOND timestamp interval-unit arms and the unsupported-key-type `WorkflowRuntimeException` | | `SleepOpExec.scala` | 0% | new spec: construction from a serialized desc + the pass-through `processTuple` (sleepTime 0) | | `CacheSourceOpExec.scala` | 0% | new spec covering the constructor's non-result-URI and non-vfs-URI rejections | The `CacheSourceOpExec` result-storage read path needs a live Iceberg catalog and stays in the amber integration suite. Likewise the `IntervalJoin` `case None` interval arm requires a JSON payload with the field absent (Jackson maps an explicit `None` to `Some(null)`), so it is left to integration. ### Any related issues, documentation, discussions? Follow-up to the review feedback on #6043: prioritize tests that fill uncovered code paths. ### How was this PR tested? - `sbt "WorkflowOperator/testOnly *IntervalOpExecSpec *SleepOpExecSpec *CacheSourceOpExecSpec"` — 19 tests, all green - `sbt "WorkflowOperator/Test/scalafmtCheck"` and `sbt "WorkflowOperator/scalafixAll --check"` — 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]
