Use default for get ignore_exc

ignore_exc should treat an error as a cache miss and default specifies
what should be returned on a miss.

Fixes issue #350

2 files changed, 11 insertions, 1 deletions

diff --git a/pymemcache/client/base.py b/pymemcache/client/base.py
index 438ffe7..2fde2fa 100644
--- a/pymemcache/client/base.py
+++ b/pymemcache/client/base.py
@@ -1287,7 +1287,7 @@ class PooledClient(object):
                 return client.get(key, default)
             except Exception:
                 if self.ignore_exc:
-                    return None
+                    return default
                 else:
                     raise
 
diff --git a/pymemcache/test/test_client.py b/pymemcache/test/test_client.py
index 9938404..94c1251 100644
--- a/pymemcache/test/test_client.py
+++ b/pymemcache/test/test_client.py
@@ -381,6 +381,16 @@ class ClientTestMixin(object):
         result = client.get(b'key', default='foobar')
         assert result == 'foobar'
 
+    def test_get_ignore_exc_default(self):
+        client = self.make_client([b'INVALID DATA\r\n'], ignore_exc=True)
+        result = client.get(b'key', default='foobar')
+        assert result == 'foobar'
+
+        # Unit test for customized client (override _extract_value)
+        client = self.make_client([b'INVALID DATA\r\n'], ignore_exc=True)
+        result = client.get(b'key', default='foobar')
+        assert result == 'foobar'
+
     def test_get_found(self):
         client = self.make_client([
             b'STORED\r\n',