Merge pull request #443 from scotchka/master

warn user when SQLALCHEMY_DATABASE_URI is not set
pallets-eco · Feb 25, 2017 · 9eff8b6 · 9eff8b6
2 parents 6c1f4a2 + 206e9b9
commit 9eff8b6
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/flask_sqlalchemy/__init__.py b/flask_sqlalchemy/__init__.py
@@ -805,7 +805,13 @@ def init_app(self, app):
         of an application not initialized that way or connections will
         leak.
         """
-        app.config.setdefault('SQLALCHEMY_DATABASE_URI', 'sqlite://')
+        if 'SQLALCHEMY_DATABASE_URI' not in app.config:
+            warnings.warn(
+                'SQLALCHEMY_DATABASE_URI not set. Defaulting to '
+                '"sqlite:///:memory:".'
+            )
+
+        app.config.setdefault('SQLALCHEMY_DATABASE_URI', 'sqlite:///:memory:')
         app.config.setdefault('SQLALCHEMY_BINDS', None)
         app.config.setdefault('SQLALCHEMY_NATIVE_UNICODE', None)
         app.config.setdefault('SQLALCHEMY_ECHO', False)