Skip to content

Conversation

@SunnyLee151064
Copy link
Contributor

@SunnyLee151064 SunnyLee151064 commented Jul 23, 2025

What this PR does / why we need it?

Add some uts for files in folder /multistream

Does this PR introduce any user-facing change?

No

How was this patch tested?

Signed-off-by: lwq <liwenquan5@huawei.com>
@codecov
Copy link

codecov bot commented Jul 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.54%. Comparing base (9a3bdf2) to head (ac8a06f).
⚠️ Report is 576 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1947      +/-   ##
==========================================
+ Coverage   60.61%   62.54%   +1.93%     
==========================================
  Files          75       78       +3     
  Lines        8127     8376     +249     
==========================================
+ Hits         4926     5239     +313     
+ Misses       3201     3137      -64     
Flag Coverage Δ
unittests 62.54% <100.00%> (+1.93%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@SunnyLee151064 SunnyLee151064 changed the title Add ut for multistream Add ut for files in /multistream Jul 23, 2025
@SunnyLee151064 SunnyLee151064 changed the title Add ut for files in /multistream [Test] Add ut for files in /multistream Jul 23, 2025
@wangxiyuan wangxiyuan merged commit ab7d5ac into vllm-project:main Jul 24, 2025
13 checks passed
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Sep 26, 2025
### What this PR does / why we need it?
Add some uts for files in folder /multistream

### Does this PR introduce _any_ user-facing change?
No

- vLLM version: v0.9.2
- vLLM main:
vllm-project/vllm@b77c7d3

Signed-off-by: lwq <liwenquan5@huawei.com>
Co-authored-by: lwq <liwenquan5@huawei.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
### What this PR does / why we need it?
Add some uts for files in folder /multistream

### Does this PR introduce _any_ user-facing change?
No

- vLLM version: v0.9.2
- vLLM main:
vllm-project/vllm@b77c7d3

Signed-off-by: lwq <liwenquan5@huawei.com>
Co-authored-by: lwq <liwenquan5@huawei.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants