Remove the fully commented-out PythonWorkflowWorkerSpec
- Dominant language
- Scala
- Stars
- 314
- Forks
- 187
- Avg merge
- 1d 21h
- Merged PRs (30d)
- 214
Description
### Task Summary
`PythonWorkflowWorkerSpec.scala` contributes zero test suites — every line in it is commented out, including the `package` declaration, so it compiles to a licence header.
| File | Lines | Live lines |
| --- | ---: | ---: |
| `amber/.../engine/architecture/pythonworker/PythonWorkflowWorkerSpec.scala` | 201 | 0 |
The commented body targets APIs deleted long ago (`InitializeOperatorLogicHandler`, `UpdateInputLinkingHandler`, `OpenOperator`, `engine.common.Constants`, the old `PhysicalLink` shape), so it cannot be un-commented as-is. Live `PythonWorkflowWorker` coverage lives in the `@IntegrationTest` e2e specs, which this file does not participate in.
Pure deletion, no behaviour change: −201 lines.
### Task Type
- [x] Refactor / Cleanup
Contributor guide
Research direction
Open amber/.../engine/architecture/pythonworker/PythonWorkflowWorkerSpec.scala and confirm that all 201 lines are commented out and reference deleted APIs. Remove the file, then run the relevant test or build checks and verify that live PythonWorkflowWorker coverage remains in the @IntegrationTest end-to-end specs.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scala
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 90/100