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

   ### What changes were proposed in this PR?
   
   Map an unknown dashboard `resourceType` to `BadRequestException` instead of 
an unhandled `IllegalArgumentException`. Update the existing invalid-type 
regression coverage for the HTTP-facing exception.
   
   Before: unknown resource type returned 500.
   After: unknown resource type returns 400 with a specific message.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8143
   
   ### How was this PR tested?
   
   - `sbt "WorkflowExecutionService / Test / testOnly 
org.apache.texera.web.resource.dashboard.DashboardResourceSpec"`
   - `sbt scalafmtCheckAll`
   - `sbt "scalafixAll --check"`
   - Built with `bin/local-dev.sh up texera-web --build --json` and ran the 
generated distribution on localhost.
   - Verified `resourceType=bogus` returns 400 with `Unknown resource type: 
bogus` and `resourceType=workflow` still returns 200.
   
   ### 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