Skip to content

Improve ResourceManagerV1 test coverage and style - #38

Merged
xunyoyo merged 2 commits into
fastdeploy/engine/sched/resource_manager_v1.pyfrom
codex/increase-test-coverage-and-fix-style-issues
Nov 27, 2025
Merged

xunyoyo merged 2 commits into
fastdeploy/engine/sched/resource_manager_v1.pyfrom
codex/increase-test-coverage-and-fix-style-issues

Conversation

@xunyoyo

@xunyoyo xunyoyo commented Nov 27, 2025

Copy link
Copy Markdown
Owner

Summary

  • add license header, shared helpers, and docstrings to ResourceManagerV1 tests while cleaning imports
  • expand ResourceManagerV1 test suite to cover multimodal boundary handling, preemption edge cases, and prefix cache errors
  • ensure stubs and monkeypatching reflect project expectations for style and xpu gating

Testing

  • pytest -c tests/cov_pytest.ini tests/engine/test_resource_manager_v1.py -vv
  • pytest -c tests/cov_pytest.ini --cov=fastdeploy.engine.sched.resource_manager_v1 --cov-report=term-missing tests/engine/test_resource_manager_v1.py -vv
  • pre-commit run --files fastdeploy/engine/sched/resource_manager_v1.py tests/engine/test_resource_manager_v1.py

Codex Task

@xunyoyo
xunyoyo merged commit 388c1fd into fastdeploy/engine/sched/resource_manager_v1.py Nov 27, 2025
1 check passed
@xunyoyo
xunyoyo deleted the codex/increase-test-coverage-and-fix-style-issues branch December 23, 2025 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant