flask-sqlalchemy icon indicating copy to clipboard operation
flask-sqlalchemy copied to clipboard

Bring up URI check for sqlite engine

Open Sparrow0hawk opened this issue 1 year ago • 0 comments

Changes

This PR brings up the check for a SQLite URI within the _apply_driver_defaults function to allow for earlier detection of if the database should run in-memory.

It adds a test to check that a database file is not created when using a SQLite URI filename for an in-memory database.

  • fixes #1314

Checklist:

  • [x] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
  • [ ] Add or update relevant docs, in the docs folder and in code.
  • [ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
  • [ ] Add .. versionchanged:: entries in any relevant code docs.
  • [x] Run pre-commit hooks and fix any issues.
  • [x] Run pytest and tox, no tests failed.

Sparrow0hawk avatar Feb 18 '24 20:02 Sparrow0hawk