fix(openapi): correct response validation for falsy objects

[amin-farjadi]

Issue number: closes #5887

Summary

Fix openapi response validation behaviour on Falsy return objects.

Changes

in openapi_validation.py, OpenAPIResponseValidationMiddleware class:

• behaviour of _handle_response method modified
• behaviour of _serialize_response method modified
• _serialize_response method renamed to _serialize_response_with_validation

in test_openapi_validation_middleware.py:

• remove the skip from test_validation_error_none_returned_non_optional_type to cover issue Bug: Open API Validation not validating response serialization when body is Falsy. #5887
• added test_validation_error_different_list_returned_non_optional_type and test_validate_list_response to cover additional scenarios

in test_http_resolver_pydantic.py:

• test_async_handler_with_validation marked to skip due to a separate issue

User experience

Now Falsy response will be validated againt the OpenAPI return field (is there is any).

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

❌ Patch coverage is 88.88889% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.63%. Comparing base (c4434b7) to head (6466bbf).

Files with missing lines	Patch %	Lines
...ls/event_handler/middlewares/openapi_validation.py	88.88%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #7990      +/-   ##
===========================================
- Coverage    96.73%   96.63%   -0.10%     
===========================================
  Files          278      278              
  Lines        13654    13656       +2     
  Branches      1086     1086              
===========================================
- Hits         13208    13197      -11     
- Misses         327      338      +11     
- Partials       119      121       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[amin-farjadi]

Hi @leandrodamascena is there anything I can do for this?

[dreamorosi]

Thanks for the PR and for proposing a fix.

If I understand the issue correctly, this fix might be more complex than necessary.

The root cause is that falsy values are being skipped due to if response.body and response.is_json().

A minimal fix would be:

- if response.body and response.is_json():
+ if response.is_json():

This single line change should fix both issues without more significant changes.

Do you think we could simplify the approach?