Skip to content

【Hackathon 9th No.84】Supplementary Unit Test for fastdeploy/reasoning #3570

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
luotao1 merged 3 commits into from
Sep 3, 2025
Merged

【Hackathon 9th No.84】Supplementary Unit Test for fastdeploy/reasoning #3570

luotao1 merged 3 commits into from
Sep 3, 2025

Conversation

@Echo-Nie
Copy link
Contributor

@Echo-Nie Echo-Nie commented Aug 23, 2025

Unit Tests for the fastdeploy/reasoning Module

Module Tested: fastdeploy/reasoning
Test File: test/reasoning/test_reasoning_parser.py
Test Content: Registration and retrieval functionality of the base class ReasoningParser

Test Case Coverage:

  1. Normal registration and retrieval of the parser class.
  2. KeyError thrown when attempting to register the same parser name without the force flag.
  3. TypeError thrown when attempting to register a class that is not a subclass of ReasoningParser.
  4. KeyError thrown when attempting to retrieve an unregistered parser.

Result:

/opt/conda/envs/python35-paddle120-env/lib/python3.10/site-packages/paddle/utils/cpp_extension/extension_utils.py:717: UserWarning: No ccache found. Please be aware that recompiling all source files may be required. You can download and install ccache from: https://github.com/ccache/ccache/blob/master/doc/INSTALL.md   
  warnings.warn(warning_message)
/opt/conda/envs/python35-paddle120-env/lib/python3.10/site-packages/_distutils_hack/__init__.py:30: UserWarning: Setuptools is replacing distutils. Support for replacing an already imported distutils is deprecated. In the future, this condition will fail. Register concerns at https://github.com/pypa/setuptools/issues/new?template=distutils-deprecation.yml   
  warnings.warn(
....
----------------------------------------------------------------------
Ran 4 tests in 0.001s

OK

@Echo-Nie
功能模块 fastdeploy/reasoning/ 单测补充
b6a33df 
测试内容：测试基类的注册、获取函数功能是否正常
@paddle-bot
Copy link

paddle-bot bot commented Aug 23, 2025

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor External developers label Aug 23, 2025
@Echo-Nie Echo-Nie changed the title 【Hackathon 9th No.84】Supplementary Unit Tests for fastdeploy/reasoning 【Hackathon 9th No.84】Supplementary Unit Test for fastdeploy/reasoning Aug 23, 2025
@luotao1 luotao1 mentioned this pull request Aug 25, 2025
Closed
@luotao1 luotao1 self-assigned this Aug 26, 2025
@luotao1
Copy link
Collaborator

luotao1 commented Aug 29, 2025

@luukunn

@luotao1 luotao1 merged commit f7a4bea into PaddlePaddle:develop Sep 3, 2025
23 of 27 checks passed
@Echo-Nie Echo-Nie deleted the reason branch September 3, 2025 11:28
qwes5s5 pushed a commit to qwes5s5/FastDeploy that referenced this pull request Sep 3, 2025
@Echo-Nie @luotao1
【Hackathon 9th No.84】Supplementary Unit Test for fastdeploy/reasoning (…
60d7456 
…PaddlePaddle#3570)

测试内容：测试基类的注册、获取函数功能是否正常

Co-authored-by: Tao Luo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luukunn luukunn luukunn approved these changes

Assignees

@luotao1 luotao1

@luukunn luukunn

Labels

contributor External developers PaddlePaddle Hackathon

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Echo-Nie @luotao1 @luukunn