Skip to content

fix: preserve empty dict handler output#532

Open
chenlichao wants to merge 1 commit into
runpod:mainfrom
chenlichao:fix/preserve-empty-dict-output
Open

fix: preserve empty dict handler output#532
chenlichao wants to merge 1 commit into
runpod:mainfrom
chenlichao:fix/preserve-empty-dict-output

Conversation

@chenlichao

Copy link
Copy Markdown

Summary

  • Preserve {} returned by serverless handlers as a valid output payload.
  • Add regression coverage for run_job() and the local runsync simulation path.

Why this helps

Test Plan

  • pytest tests/test_serverless/test_modules/test_job.py::TestJob::test_run_job_preserves_empty_dict_output tests/test_serverless/test_modules/test_fastapi.py::TestFastAPI::test_runsync -q --no-cov
  • git diff --check

Note: the same targeted pytest command without --no-cov ran both tests successfully, but the repository-wide coverage gate failed because only two focused tests were selected.

@capy-ai

capy-ai Bot commented Jul 7, 2026

Copy link
Copy Markdown

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Returning empty object from handler raises KeyError exception within framework

1 participant