The GitHub Actions job "Required Checks" on texera.git/main has succeeded. Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run: 1c3ec67977406fc1340d0f8e90d720ff71e7ae69 / Yicong Huang <[email protected]> test(pyamber): add unit tests for Operator base class (#4735) ### What changes were proposed in this PR? Adds pytest coverage for `amber/src/main/python/core/models/operator.py`. The `Operator` abstract base class had no dedicated spec. ### Any related issues, documentation, discussions? Closes #4733. Bug pinned in the spec with an explanatory comment (filed separately as a Bug issue): `SourceOperator` declares `__internal_is_source = True` at class level, but Python name-mangles that into `_SourceOperator__internal_is_source` while `Operator.is_source` reads `self._Operator__internal_is_source`. The two are different attributes, so a fresh `SourceOperator` subclass instance reports `is_source=False` until `ExecutorManager.initialize_executor` invokes the explicit setter — making the class-level declaration effectively dead code. ### How was this PR tested? ``` cd amber/src/main/python ruff check core/models/test_operator.py ruff format --check core/models/test_operator.py python -m pytest core/models/test_operator.py ``` ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (claude-opus-4-7) --------- Co-authored-by: Claude Opus 4.7 (1M context) <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/25271077126 With regards, GitHub Actions via GitBox
