The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-6570-547d5051aade41c2fb0a38c6c0d714967a6c502f has failed. Run started by GitHub user mengw15 (triggered by mengw15).
Head commit for run: 8937f71c616ae2beb3606a43c8c38fe907380987 / Meng Wang <[email protected]> test(frontend): add unit test coverage for WorkflowResultExportService, WorkflowExecutionsService, and areOperatorSchemasEqual (#6570) ### What changes were proposed in this PR? Adds/extends Vitest coverage for three previously-thin logic-layer targets (28 tests total, all jsdom, no browser mode): - **`areOperatorSchemasEqual`** (`workspace/types/operator-schema.interface.ts`, was ~9%, no spec) — new pure-function spec (no TestBed). Covers identical schemas → `true`, and `false` for differences in `operatorType` / `operatorVersion` / nested `jsonSchema` fields / `jsonSchema` array order / each `additionalMetadata` field / input & output port count, `displayName`, `disallowMultiLinks`, and port array order. - **`WorkflowResultExportService`** (`workspace/service/workflow-result-export/`, was ~42%, creation-only spec) — extended with `resetFlags` (resets the export-availability flags to defaults), plus a new suite for the exported `WorkflowResultDownloadability` helper class: `getExportableOperatorIds` / `getBlockedOperatorIds` partition operators against the restricted map, and `getBlockingDatasets` returns the deduped union of blocking labels. The HTTP/side-effecting `performExport` / `exportWorkflowExecutionResult` paths are intentionally out of scope. - **`WorkflowExecutionsService`** (`dashboard/service/user/workflow-executions/`, was ~33%, `it.todo` stub) — replaced the stub with `HttpClientTestingModule` + `HttpTestingController` tests asserting method / URL / body / query params for `retrieveLatestWorkflowExecution`, `retrieveWorkflowExecutions` (with and without a status filter), `groupSetIsBookmarked`, `groupDeleteWorkflowExecutions`, `updateWorkflowExecutionsName`, and `retrieveWorkflowRuntimeStatistics`. All tests are deterministic (pure functions, synchronous `of(...)`/flushed HTTP, `httpMock.verify()` in `afterEach`). No production code was changed. ### Any related issues, documentation, discussions? Closes #6567 ### How was this PR tested? New/extended unit tests, run locally in `frontend/` (all green; each file's failure path was verified by breaking an assertion to confirm the suite goes red): ``` ng test --watch=false --include src/app/workspace/types/operator-schema.interface.spec.ts # 14 passed ng test --watch=false --include src/app/workspace/service/workflow-result-export/workflow-result-export.service.spec.ts # 7 passed ng test --watch=false --include src/app/dashboard/service/user/workflow-executions/workflow-executions.service.spec.ts # 7 passed eslint <specs> # clean prettier --check # 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/29721524554 With regards, GitHub Actions via GitBox
