Yicong-Huang opened a new issue, #5193: URL: https://github.com/apache/texera/issues/5193
### Task Summary `sync-texera-model.spec.ts` keeps four `it()` blocks commented out at lines 148, 200, 602, and 687. Each was written against a `SyncTexeraModel` constructor that took a third `OperatorGroup` argument. That `OperatorGroup` class has since been removed, the constructor is now two-arg, and `SyncTexeraModel` itself no longer handles operator-delete events — every commented test references a dependency and a responsibility that the SUT no longer has. The commented blocks are not "TODO, will revive"; they are dead code masquerading as TODO and add ~1.1k characters of noise to the spec. While auditing this file the static `getOperatorLink` helper also turned out to have two uncovered `throw` branches (null source / null target), which are easy to add and bring `SyncTexeraModel` to 100% line coverage. Rewrite the spec: drop the dead commented tests and their helpers, add direct tests for `getOperatorLink`'s success and error branches. ### Task Type - [x] Testing / QA - [x] Refactor / Cleanup -- 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]
