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

   ### What changes were proposed in this PR?
   
   Return `400 Bad Request` when the workflow name or description endpoint is 
called without the required workflow ID. Add positive coverage for existing 
metadata and negative coverage for the missing ID.
   
   Before: missing workflow ID returned `204 No Content`.
   After: missing workflow ID returns `400 Bad Request` with `wid is required`.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8141
   
   ### How was this PR tested?
   
   - `sbt "WorkflowExecutionService / Test / testOnly 
org.apache.texera.web.resource.dashboard.user.workflow.WorkflowResourceCoverSpec"`
   - `sbt scalafmtCheckAll`
   - `sbt "scalafixAll --check"`
   - Built with `bin/local-dev.sh up texera-web --build --json` and ran the 
generated distribution on localhost.
   - Verified both metadata endpoints without `wid` return 400 and both 
requests with `wid=7` still return 200 with the stored values.
   
   ### 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