gitgabrio commented on PR #3604:
URL: 
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3604#issuecomment-2271111168

   @elguardian @fjtirado
   Update. After debugging, it turns out that:
   1. the tests are actually wrongly written, because the http requests lacks 
required parameters (the ones marked as `NotNull` in the rest endpoints
   2. the springboot version actually behave in the right way, because it 
validate the parameters and, since they are invalid, throws an exception
   3. the quarkus version, on the other side, seems bugged, because a given 
"validator" is null, so the paraters are not validated, and invalid parameters 
(i.e. `null` where `NotNull` would be required) are allowed
   
   So:
   1. tests should be rewritten (both springboot and quarkus)
   2. quarkus side should be double checked/fixed to implement parameter 
checking
   
   ![Screenshot 2024-08-06 alle 08 58 
19](https://github.com/user-attachments/assets/46c4d3bd-befa-4c61-a631-f7b5be1a4606)
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to