The GitHub Actions job "Comment commands" on texera.git/main has succeeded. Run started by GitHub user Ma77Ball (triggered by Ma77Ball).
Head commit for run: 8633188757e255f29cf43a9122aaef4dc7b8085a / Yicong Huang <[email protected]> test(frontend): rewrite sync-texera-model spec (#5194) ### What changes were proposed in this PR? `sync-texera-model.spec.ts` was carrying ~344 lines of dead test code: four `it()` blocks commented out against a `SyncTexeraModel` constructor that took an `OperatorGroup` third argument. `OperatorGroup` was removed from the codebase long ago, the constructor is now two-arg, and `SyncTexeraModel` no longer handles operator-delete events at all — so each commented test depended on a class that doesn't exist and exercised behavior that's been moved elsewhere. One nearby live test claimed to assert the operator-delete error path but never instantiated the SUT and made no assertions; it was vacuous. Drop the four commented blocks, their preceding doc blocks, the vacuous live test, and the now-orphaned `getJointOperatorValue` helper. Add a `describe("getOperatorLink", …)` that hits the static helper's two `throw` branches directly — the stream-handler tests can't reach them because `isValidJointLink` filters those cases out upstream. ### Any related issues, documentation, discussions? Closes #5193. ### How was this PR tested? `yarn ng test --watch=false --include=…` runs 9 tests, all green. `yarn lint` and `yarn format:ci` both clean. v8 line coverage of `sync-texera-model.ts` rises from 94 % (31/33) to 100 % (33/33); the spec itself shrinks 747 → 449 lines. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Opus 4.7 Report URL: https://github.com/apache/texera/actions/runs/26377229492 With regards, GitHub Actions via GitBox
