Skip to content

fix(test): fix make test failed#2502

Merged
Aries-ckt merged 3 commits into
eosphoros-ai:mainfrom
yyhhyyyyyy:fix-make-test
Mar 21, 2025
Merged

fix(test): fix make test failed#2502
Aries-ckt merged 3 commits into
eosphoros-ai:mainfrom
yyhhyyyyyy:fix-make-test

Conversation

@yyhhyyyyyy
Copy link
Copy Markdown
Collaborator

Description

These are some test failed:

================================================================================ short test summary info =================================================================================
FAILED packages/dbgpt-core/src/dbgpt/util/tests/test_parameter_utils.py::test_extract_union_field_type - AssertionError: assert 'union_field' == 'str_with_default'
FAILED packages/dbgpt-core/src/dbgpt/util/tests/test_parameter_utils.py::test_python_type_hint_variations - AssertionError: assert 'string' == 'integer'
================================================================= 2 failed, 402 passed, 1 skipped, 24 warnings in 11.90s =================================================================

Now:

====================================================================== 404 passed, 1 skipped, 24 warnings in 11.71s ======================================================================

How Has This Been Tested?

make test

Snapshots:

image

@github-actions github-actions Bot added the fix Bug fixes label Mar 21, 2025
csunny
csunny previously approved these changes Mar 21, 2025
Copy link
Copy Markdown
Collaborator

@csunny csunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+

@fangyinc
Copy link
Copy Markdown
Member

It didn't work in my testing.

make test
. .venv.make/bin/activate && uv sync --active --all-packages \
        --extra "base" \
        --extra "proxy_openai" \
        --extra "rag" \
        --extra "storage_chromadb" \
        --extra "dbgpts" \
        --link-mode=copy
error: unexpected argument '--active' found

  tip: a similar argument exists: '--native-tls'

Usage: uv sync --native-tls

For more information, try '--help'.
make: *** [Makefile:27:.venv.make/.testenv] 错误 2

@yyhhyyyyyy
Copy link
Copy Markdown
Collaborator Author

@fangyinc This was an issue with the uv version, which has now been fixed.

Copy link
Copy Markdown
Member

@fangyinc fangyinc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Aries-ckt Aries-ckt merged commit 5a35023 into eosphoros-ai:main Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants