-
-
Notifications
You must be signed in to change notification settings - Fork 387
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add init_memory_sqlite decorator #1657
Add init_memory_sqlite decorator #1657
Conversation
Pull Request Test Coverage Report for Build 9544850985Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build 9560727240Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build 9560904994Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build 9561639055Details
💛 - Coveralls |
Pull Request Test Coverage Report for Build 9562578561Details
💛 - Coveralls |
await Tortoise.init(db_url="sqlite://:memory:", modules={"models": ["__main__"]}) | ||
await Tortoise.generate_schemas() | ||
|
||
@init_memory_sqlite |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May be we should make it configurable? To let it pass another modules
value
Because if you allow only __main__
I am not sure it would be useful enough to make it part of contrib
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abondar I made a refactor to support models
argument.
Pull Request Test Coverage Report for Build 9645745307Details
💛 - Coveralls |
Description
Almost every file in examples include the following two lines:
Motivation and Context
How Has This Been Tested?
make ci
Checklist: