Skip to content

fix(tests): align pytest coverage docs with required setup (#1259)#1373

Merged
qin-ctx merged 1 commit intovolcengine:mainfrom
chethanuk:fix/1259-pytest-cov-docs-mismatch
Apr 13, 2026
Merged

fix(tests): align pytest coverage docs with required setup (#1259)#1373
qin-ctx merged 1 commit intovolcengine:mainfrom
chethanuk:fix/1259-pytest-cov-docs-mismatch

Conversation

@chethanuk
Copy link
Copy Markdown
Contributor

Summary

  • document pytest-cov in the test README dependencies
  • keep the coverage command scoped to the intended Python test directories and preserve verbose output
  • address review feedback so the README matches actual pytest collection behavior

Validation

  • verified README diff is scoped to tests/README.md
  • confirmed documented install command includes pytest-cov
  • confirmed coverage command is scoped to tests/client tests/server tests/session tests/vectordb tests/misc tests/integration with term-missing output

Closes #1259

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

1259 - Fully compliant

Compliant requirements:

  • Fix the error when running test scripts due to unrecognized --cov arguments
  • Align test documentation with required setup
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🏅 Score: 100
🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@qin-ctx qin-ctx merged commit 320108f into volcengine:main Apr 13, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenViking project Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Bug]: 运行test目录下的脚本会报错

2 participants