mengw15 opened a new pull request, #6758:
URL: https://github.com/apache/texera/pull/6758

   ### What changes were proposed in this PR?
   
   Extends the existing `DatasetDetailComponent` spec to cover a few untested
   methods
   
(`frontend/src/app/dashboard/component/user/user-dataset/user-dataset-explorer/dataset-detail.component.ts`).
   
   > Note: most of the methods the issue lists are already covered by the 
existing
   > spec. `removeFromPendingQueue` / `refreshPendingChanges` are `private` and 
are
   > exercised indirectly through the upload/handler flows, so this PR focuses 
on
   > the remaining public gaps.
   
   3 tests cover:
   
   - `getUploadStatus` — maps the multipart upload status to a progress state 
(`active` / `exception` / `success`).
   - `onVersionFileTreeNodeSelected` — loads the selected node's content.
   - `trackByTask` — returns the task's file path.
   
   No production code was changed.
   
   ### Any related issues, documentation, discussions?
   
   Closes #6747
   
   ### How was this PR tested?
   
   Extended unit tests, run locally in `frontend/` (all green; the failure path 
was
   verified by breaking an assertion to confirm the suite goes red):
   
   ```
   ng test --watch=false --include 
src/app/dashboard/component/user/user-dataset/user-dataset-explorer/dataset-detail.component.spec.ts
   # Test Files 1 passed (1) | Tests 63 passed (63)
   prettier --write <spec>   # clean
   eslint  <spec>            # clean
   ```
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Opus 4.8 [1M context])
   


-- 
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]

Reply via email to