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

   ### What changes were proposed in this PR?
   
   Return 404 when the public-workflow query finds no record instead of 
dereferencing a null result. Add positive coverage for a public workflow and 
regression coverage for private and missing workflows.
   
   Before: private and missing workflows returned 500.
   
   After: private and missing workflows return 404, while a public workflow 
still returns 200.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8139
   
   ### How was this PR tested?
   
   - `sbt "WorkflowExecutionService / Test / testOnly 
org.apache.texera.web.resource.dashboard.user.workflow.WorkflowResourceCoverSpec"`
   - `sbt scalafmtCheckAll`
   - `sbt "scalafixAll --check"`
   - Built and launched `texera-web` from this worktree. Missing and private 
workflow requests returned 404. A temporarily public local workflow returned 
200 and was restored to private afterward.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Codex


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