Skip to content

Commit 284d7b5

Browse files
authored
Merge pull request #17 from getmarkus/cm-branch-15
refactor(tests): simplify client fixture by removing session override
2 parents 5ec0157 + 01852be commit 284d7b5

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

app/tests/conftest.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
from fastapi.testclient import TestClient
33
from sqlmodel import Session, delete
44

5-
from app.resource_adapters.persistence.sqlmodel.database import get_engine, get_db
5+
from app.resource_adapters.persistence.sqlmodel.database import get_engine
66
from app.resource_adapters.persistence.sqlmodel.issues import Issue
77
from main import app
88

@@ -19,11 +19,7 @@ def session_fixture():
1919

2020
@pytest.fixture(name="client")
2121
def client_fixture(session: Session):
22-
def get_session_override():
23-
return session
24-
25-
app.dependency_overrides[get_db] = get_session_override
2622

2723
with TestClient(app) as client:
2824
yield client
29-
app.dependency_overrides.clear()
25+

0 commit comments

Comments
 (0)