now made DB optional, added new test cases

Author: rhyselsmore

.travis.yml

@@ -6,8 +6,10 @@ python:
 env:
  - FLASK_VERSION=0.9 REDIS_VERSION=2.6.2
  - FLASK_VERSION=0.9 REDIS_VERSION=2.7.4
+ - FLASK_VERSION=0.9 REDIS_VERSION=2.10.3
  - FLASK_VERSION=0.10 REDIS_VERSION=2.6.2
  - FLASK_VERSION=0.10 REDIS_VERSION=2.7.4
+ - FLASK_VERSION=0.10 REDIS_VERSION=2.10.3
 install:
  - "pip install -r requirements.txt"
  - "pip install -I Flask==$FLASK_VERSION"
@@ -19,3 +21,5 @@ matrix:
       env: FLASK_VERSION=0.9 REDIS_VERSION=2.6.2
     - python: "3.3"
       env: FLASK_VERSION=0.9 REDIS_VERSION=2.7.4
+    - python: "3.3"
+      env: FLASK_VERSION=0.9 REDIS_VERSION=2.10.3

flask_redis.py

@@ -5,8 +5,6 @@
 
 class Redis(object):
 
-    converters = {'port': int}
-
     def __init__(self, app=None, config_prefix=None):
         """
         Constructor for non-factory Flask applications
@@ -29,13 +27,9 @@ def init_app(self, app):
         )
 
         self.app.config.setdefault(self.key('URL'), 'redis://localhost:6379')
-        self.app.config.setdefault(self.key('DATABASE'), 0)
 
         db = self.app.config.get(self.key('DATABASE'))
 
-        if not str(db).isdigit() or not isinstance(db, int):
-            raise ValueError('A valid DB must be supplied')
-
         self.connection = connection = RedisClass.from_url(
             self.app.config.get(self.key('URL')),
             db=db,

test_flask_redis.py

@@ -32,5 +32,15 @@ def test_custom_prefix(self):
         self.app.config['DB2_DATABASE'] = 1
         self.db2_redis.init_app(self.app)
 
+        self.db3_redis = Redis(config_prefix='DB3')
+        self.app.config['DB3_URL'] = "redis://localhost:6379"
+        self.db3_redis.init_app(self.app)
+
+        self.db4_redis = Redis(config_prefix='DB4')
+        self.app.config['DB4_URL'] = "redis://localhost:6379/5"
+        self.db4_redis.init_app(self.app)
+
         assert self.db1_redis.get('potato') is None
         assert self.db2_redis.get('potato') is None
+        assert self.db3_redis.get('potato') is None
+        assert self.db4_redis.get('potato') is None