The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-6604-42027e8c38d248a60cab32871887bc754ef311b5 has failed. Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run: e6aebcaf1780c23f6787bd05930b813b7f5ded44 / Meng Wang <[email protected]> test(frontend): extend UserWorkflowListItemComponent and BrowseSectionComponent coverage (#6604) ### What changes were proposed in this PR? Extends the existing specs for two components (both asserted little past creation) with unit tests for their previously-untested methods, per `frontend/TESTING.md`. **`UserWorkflowListItemComponent`** (codecov ~62%) — 11 new tests: - `getProjectIds` returns the entry's project-id set; `isLightColor` reports light vs dark hex colors. - `confirmUpdateWorkflowCustomName` / `confirmUpdateWorkflowCustomDescription` persist via `WorkflowPersistService`, update the workflow, and stop editing; the name falls back to the default when empty; both no-op when the workflow has no id. - `removeWorkflowFromProject` calls `UserProjectService` and prunes the entry's project ids; `onClickGetWorkflowExecutions` opens the execution-history modal; `onClickDownloadWorkfllow` delegates to `DownloadService` (and no-ops without an id). **`BrowseSectionComponent`** (codecov ~72%) — 4 new tests: - `initializeEntry` skips entries whose id is not a number and throws on an unexpected entity type. - `loadCoverImages` / `getCoverImage` build and cache `${apiEndpoint}/dataset/{id}/cover` for a dataset with a cover image, and fall back to the default background otherwise. No production code was changed. ### Any related issues, documentation, discussions? Closes #6590 ### How was this PR tested? Extended Vitest specs, run locally in `frontend/` (each whole file green, run 4× to confirm determinism; failure paths verified by breaking an assertion in each): ``` ng test --watch=false --include .../user-workflow-list-item.component.spec.ts # 15 passed ng test --watch=false --include .../browse-section.component.spec.ts # 9 passed eslint <specs> # clean prettier --check # clean ``` Services are stubbed (`vi.spyOn` / `useValue`), so the specs make no network calls. The list-item tests that mutate a workflow (rename, remove-from-project) use a freshly-constructed `DashboardEntry` per test so mutations cannot leak into the shared `testWorkflowEntries` fixture. ### 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/29771742597 With regards, GitHub Actions via GitBox
