FastDeploy icon indicating copy to clipboard operation
FastDeploy copied to clipboard

[CI] 【Hackathon 9th Sprint No.37】NO.37 功能模块单测补充

Open xunyoyo opened this issue 1 month ago • 2 comments

Motivation

NO.37 功能模块 fastdeploy/output/token_processor.py 单测补充

Modifications

add tests/output/test_token_processor.py

Usage or Command

tests/output/test_token_processor.py:

python -m coverage run -m unittest tests.output.test_token_processor \
&& python -m coverage report -m --include='fastdeploy/output/token_processor.py'

Accuracy Tests

tests/output/test_token_processor.py:

Stmts   Miss  Cover   Missing
--------------------------------------------------------------------
fastdeploy/output/token_processor.py     528     95    82%   238, 263, 312-331, 338-406, 472, 480-481, 488, 533, 537-538, 575-57
6, 583, 607-610, 616, 618-621, 675-676, 681, 721-723, 727, 753-754, 829, 836, 866-868, 871, 879, 881, 892-894, 900-902
--------------------------------------------------------------------
TOTAL                                    528     95    82%

Checklist

  • [x] Add at least a tag in the PR title.
    • Tag list: [[FDConfig],[APIServer],[Engine], [Scheduler], [PD Disaggregation], [Executor], [Graph Optimization], [Speculative Decoding], [RL], [Models], [Quantization], [Loader], [OP], [KVCache], [DataProcessor], [BugFix], [Docs], [CI], [Optimization], [Feature], [Benchmark], [Others], [XPU], [HPU], [GCU], [DCU], [Iluvatar], [Metax]]
    • You can add new tags based on the PR content, but the semantics must be clear.
  • [x] Format your code, run pre-commit before commit.
  • [x] Add unit tests. Please write the reason in this PR if no unit tests.
  • [x] Provide accuracy results.
  • [x] If the current PR is submitting to the release branch, make sure the PR has been submitted to the develop branch, then cherry-pick it to the release branch with the [Cherry-Pick] PR tag.

xunyoyo avatar Nov 15 '25 10:11 xunyoyo

Thanks for your contribution!

paddle-bot[bot] avatar Nov 15 '25 10:11 paddle-bot[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :warning: Please upload report for BASE (develop@8beb015). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #5059   +/-   ##
==========================================
  Coverage           ?   65.09%           
==========================================
  Files              ?      329           
  Lines              ?    41779           
  Branches           ?     6396           
==========================================
  Hits               ?    27195           
  Misses             ?    12552           
  Partials           ?     2032           
Flag Coverage Δ
GPU 65.09% <ø> (?)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov-commenter avatar Dec 12 '25 05:12 codecov-commenter

@sunlei1024 麻烦review~

xunyoyo avatar Dec 19 '25 04:12 xunyoyo

0.2⭐️ @luotao1

CSWYF3634076 avatar Dec 23 '25 02:12 CSWYF3634076