Skip to content

refactor: add fixtures for test loggers and update database schema #10546

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

manojkumargogula
Copy link

@manojkumargogula manojkumargogula commented May 4, 2025

Fix CI build issues in test infrastructure

Relevant issues
Addresses pytest collection warnings in test classes
Resolves unknown pytest marker warnings
Fixes Python 3.10 compatibility issues

Pre-Submission checklist
[x] I have Added testing in the tests/litellm/ directory
[x] I have added a screenshot of my test passing locally
[x] My PR passes all unit tests on make test-unit
[x] My PR's scope is as isolated as possible, it only solves 1 specific problem

Type
🐛 Bug Fix
🧹 Refactoring
✅ Test

Changes
This PR addresses CI build failures by:

  • Converting test classes with init constructors to use fixtures instead, fixing PytestCollectionWarning
  • Registering the flaky marker in conftest.py to resolve PytestUnknownMarkWarning
  • Adding Python 3.10 compatibility for datetime.UTC imports
  • Increasing GitHub workflow timeout to prevent CI failures
  • These changes ensure tests run successfully in both local and CI environments without warnings.

Copy link

vercel bot commented May 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 4, 2025 0:19am

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant