fix: preserve empty dict handler output - #535
Conversation
|
Capy auto-review is paused for this organization because the usage-cycle auto-review limit has been reached. Increase the limit or turn it off in billing settings to resume automatic reviews. |
|
Promptless prepared a documentation update related to this change. Triggered by runpod-python PR #535 Added a "Handler return values" section to the Serverless handler functions docs, explaining how a handler's return value maps to the job |
|
bumping this for a first review — one-line fix preserving empty dict handler output, no behavioral regression intended. |
|
Weekly bump — still waiting on a first review for this one-line fix preserving empty-dict handler output. No behavioral regression, happy to adjust. |
Summary
output: {}.Why this helps
KeyErrorexception within framework #459: handlers that intentionally return{}should complete successfully instead of causing local FastAPI simulation to raiseKeyError: 'output'.Test Plan
pytest tests/test_serverless/test_modules/test_fastapi.py -q --no-cov(8 passed)Note: a targeted run without
--no-covalso had all 8 tests pass, but failed the repository-wide coverage threshold because only one test file was selected.