aglinxinyuan opened a new issue, #7320:
URL: https://github.com/apache/texera/issues/7320

   ### Task Summary
   
   `files-uploader.component.ts` sits at 66.4% (29 missed). Its conflict dialog 
offers **Resume** and **Restart**, and the existing spec only ever takes the 
Resume half — so `Restart`, `Restart For All`, and the `markForceRestart` path 
are all untested.
   
   The two choices differ by exactly one observable: `item.restart`, which is 
what makes the uploader call the backend with `type=forceRestart` instead of 
continuing the existing multipart session. A Resume/Restart mix-up therefore 
silently resumes a session the user explicitly asked to discard, with nothing 
failing.
   
   Worth covering:
   
   - Restart sets the flag, and Resume leaves it unset. Written as a pair, so a 
`markForceRestart` call wrongly added to the Resume branch is caught too.
   - `Restart For All` applies the flag to files that never prompted — that is 
what distinguishes the latch from merely suppressing the dialog.
   - `Resume For All` must **not** set it, distinguishing the two latches from 
each other.
   - A non-conflicting file passes straight through with no dialog at all.
   
   The existing spec already captures modal configs into an array and clicks 
their footer buttons by label, so these are additions to that harness rather 
than new infrastructure.
   
   ### Task Type
   
   - [ ] Refactor / Cleanup
   - [ ] DevOps / Deployment / CI
   - [x] Testing / QA
   - [ ] Documentation
   - [ ] Performance
   - [ ] Other
   
   


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