fix: add starlette/httpx to dev deps, guard test_openai_compat imports #2

2026-04-27T23:58:21-07:00

nsatoshi commented

2026-04-27 23:58:21 -07:00

Summary

Fixes the test_openai_compat.py collection error (ModuleNotFoundError: No module named starlette).

Changes:

Added starlette>=0.36 and httpx>=0.27 to dev dependencies in pyproject.toml
Added pytest.importorskip() guards for both starlette and fastapi in test_openai_compat.py — test file is skipped gracefully when those packages are missing instead of crashing the entire test collection
Fixed import sorting (ruff I001)

Result: 340 tests pass (was 325 with test_openai_compat excluded).

Review & Testing Checklist for Human

Run pip install -e ".[dev]" and verify starlette/httpx are pulled in
Run pytest tests/ without --ignore — all 340 tests should pass

Notes

The 20 warnings in test output are FastAPI deprecation warnings about on_event — pre-existing, not related to this change.

## Summary Fixes the `test_openai_compat.py` collection error (`ModuleNotFoundError: No module named starlette`). **Changes:** - Added `starlette>=0.36` and `httpx>=0.27` to `dev` dependencies in `pyproject.toml` - Added `pytest.importorskip()` guards for both `starlette` and `fastapi` in `test_openai_compat.py` — test file is skipped gracefully when those packages are missing instead of crashing the entire test collection - Fixed import sorting (ruff I001) **Result:** 340 tests pass (was 325 with test_openai_compat excluded). ## Review & Testing Checklist for Human - [ ] Run `pip install -e ".[dev]"` and verify starlette/httpx are pulled in - [ ] Run `pytest tests/` without `--ignore` — all 340 tests should pass ### Notes The 20 warnings in test output are FastAPI deprecation warnings about `on_event` — pre-existing, not related to this change.

nsatoshi added 1 commit 2026-04-27 23:58:22 -07:00

fix: add starlette/httpx to dev deps, guard test_openai_compat imports

Tests / test (3.10) (pull_request) Failing after 40s

Details

Tests / test (3.11) (pull_request) Failing after 36s

Details

Tests / test (3.12) (pull_request) Successful in 41s

Details

Tests / lint (pull_request) Successful in 34s

Details

Tests / docker (pull_request) Successful in 1m47s

Details

59d57cb2fb

- Add starlette>=0.36 and httpx>=0.27 to dev dependencies so
  test_openai_compat.py can run in dev environments
- Add pytest.importorskip guards for starlette and fastapi so the
  test file is skipped gracefully when those packages are missing
- Fix import sorting (ruff I001)

340 tests now pass (was 325 with test_openai_compat skipped).

Co-Authored-By: Nakamoto, S <defi@defi-oracle.io>

nsatoshi merged commit de97fd8ac9 into main

2026-04-28 01:19:13 -07:00

nsatoshi referenced this issue from a commit

2026-04-28 01:19:14 -07:00

Merge pull request 'fix: add starlette/httpx to dev deps, guard test_openai_compat imports' (#2) from devin/1777359479-fix-openai-compat-tests into main

nsatoshi referenced this issue from a commit

2026-04-28 01:32:11 -07:00

feat: complete all 19 tasks — liquid networks, quantum backend, embodiment, self-model, ASI rubric, plugin system, auth/rate-limit middleware, async adapters, CI/CD, Dockerfile, benchmarks, module boundary fix, TTS adapter, lifespan migration, OpenAPI docs, code cleanup

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: d-bis/FusionAGI#2