The GitHub Actions job "Required Checks" on texera.git/main has succeeded.
Run started by GitHub user github-merge-queue[bot] (triggered by 
github-merge-queue[bot]).

Head commit for run:
0467adea49c90e9feb4d0ca69de0f9f1d3f92fa8 / Xinyuan Lin <[email protected]>
test(workflow-operator): add unit tests for SQLSourceOpExec (#6877)

### What changes were proposed in this PR?

Adds `SQLSourceOpExecSpec` (27 tests) for a file that had **0%
coverage**. A small in-spec subclass supplies the three DB hooks
(`establishConn` returns an injected `Connection`,
`loadTableNames`/`addFilterConditions` are configurable), so the
query-building surface is exercised directly:
- `generateSqlQuery` across plain / filtered / LIMIT / LIMIT+OFFSET /
OFFSET-only / progressive combinations, and the `None` cases when the
limit is non-positive;
- `addBaseSelect`, `addLimit`, `addOffset`, `terminateSQL` fragments;
- `addBatchSlidingWindow` for LONG / TIMESTAMP / DOUBLE (incl. the
inclusive final window) and its no-batch-column and unsupported-type
errors;
- `batchAttributeToString` formatting per type;
- `hasNextQuery` bound transitions; `open()` table validation and
progressive-config rejection;
- `fetchBatchByBoundary` auto MIN/MAX probes and the `produceTuple`
iterator (parameter binding order, manual offset skipping, null columns,
result-set close-out) via ScalaMock JDBC doubles.

The descriptor JSON is produced from a real `PostgreSQLSourceOpDesc` so
the polymorphic `operatorType` tag resolves exactly as in production. No
source changes.

### Any related issues, documentation, discussions?

Closes #6874.

### How was this PR tested?

`sbt -java-home <jbr-17> "WorkflowOperator/testOnly
*SQLSourceOpExecSpec"` -> 27 succeeded, 0 failed. `Test/scalafmtCheck` +
`Test/scalafix --check` clean.

### Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Opus 4.8 [1M context])

Report URL: https://github.com/apache/texera/actions/runs/30145788572

With regards,
GitHub Actions via GitBox

Reply via email to