Skip to content

Conversation

lxiao-coder
Copy link

@lxiao-coder lxiao-coder commented Nov 27, 2024

I am not sure if it is an issue happen to me only. But before the change, the test has error.
We need to mock session.enter as it is a content manager.

backend/app/backend_pre_start.py

with Session(db_engine) as session:
            # Try to create session to check if DB is awake
            session.exec(select(1))

@lxiao-coder lxiao-coder marked this pull request as draft November 27, 2024 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant