You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@texera.apache.org by GitBox <gi...@apache.org> on 2026/07/07 06:12:10 UTC

[GH] (texera/gh-readonly-queue/main/pr-6128-6e94b2d3e5871705cb27b732962ee7661c33ca2e): Workflow run "Required Checks" failed!

The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-6128-6e94b2d3e5871705cb27b732962ee7661c33ca2e has failed.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
12455eea16d02a292db5088fc1b65453363f5f2e / Xinyuan Lin <xi...@uci.edu>
test(workflow-operator): add unit tests for LogicalOp and Python operator descriptor bases (#6128)

### What changes were proposed in this PR?

Adds unit-test coverage for the operator-descriptor base classes in
`common/workflow-operator`, which Codecov reports as partly uncovered:

- **`LogicalOpSpec`** (new) — exercises the `LogicalOp` base directly:
the `getPhysicalOp` `???` default (via a minimal subtype), the
reflection-based `hashCode`/`equals`, `toString`, and the default
`runtimeReconfiguration` `UnsupportedOperationException`.
- **`PythonOperatorDescriptorSpec`** (new) — exercises
`PythonOperatorDescriptor.getPhysicalOp` through in-spec subtypes: the
code-generation exception is embedded in the generated code rather than
thrown, a source op gets `PreferController`, and a one-to-one op gets no
location preference.
- **`PythonUDFSourceOpDescV2Spec`** (extended) — adds the blank
virtual-environment-name rejection, the trimmed `pveName`, and the
parallelizable/`suggestedWorkerNum` path when `workers > 1`.

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 *LogicalOpSpec *PythonOperatorDescriptorSpec *PythonUDFSourceOpDescV2Spec"
```

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])

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

With regards,
GitHub Actions via GitBox