aglinxinyuan opened a new pull request, #6135: URL: https://github.com/apache/texera/pull/6135
### What changes were proposed in this PR? Extends three existing source-operator specs in `common/workflow-operator` to cover Codecov-reported uncovered lines: - **`CSVScanSourceOpExecSpec`** — adds instance-side tests that drive `open()`/`produceTuple()`/`close()` over real temp CSV files (the previously-covered object helpers are left untouched): a header CSV emitting one tuple per row, `offset`+`limit` windowing, and graceful skipping of an unparseable row. - **`TwitterSearchSourceOpDescSpec`** — adds a `getPhysicalOp` test asserting the wired `OpExecWithClassName`, ports, and (by invoking the closure) the schema-propagation function. - **`TwitterFullArchiveSearchSourceOpDescSpec`** — adds the matching `getPhysicalOp` test (the existing `sourceSchema`/`operatorInfo`/round-trip tests already cover the rest). 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/extended ScalaTest specs, run locally: ``` sbt "WorkflowOperator/testOnly *CSVScanSourceOpExecSpec *TwitterSearchSourceOpDescSpec *TwitterFullArchiveSearchSourceOpDescSpec" ``` All pass. `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]
