- Notifications
You must be signed in to change notification settings - Fork 1k
[RFC]: Unit test coverage improvement #1298
Copy link
Copy link
Closed
Labels
feature requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Metadata
Metadata
Assignees
Labels
feature requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Note: Setup test environment: https://vllm-ascend.readthedocs.io/en/latest/developer_guide/contribution/testing.html
Motivation
This issue attempt to reduce the gap of unit tests to cover the code. There is a brief architecture of ut in
tests/ut/already. We need to add more to cover all the code in vllm-ascend, and there are several principles to follow:vllm_ascendtest_unittestframework, make good use of mockPlease refer to the official doc on contributing and testing to develop, thanks!
Unit tests need to add
test_attention.pywill drop latertest_draft_model_runner.py@wangyanhui-cmss will drop latertest_model_runner.pywill drop latertest_multi_step_runner.pywill drop latertest_multi_step_worker.pywill drop latertest_worker.py@zhanghw0354 will drop later