Merge pull request #190 from ngnpope/fix-delete-return

delete() return value now is 1 for "DELETED" and 0 for "NOT_FOUND" or server error.

Author: linsomniac

memcache.py

@@ -552,9 +552,9 @@ def delete(self, key, noreply=False):
             if noreply:
                 return 1
             line = server.readline()
-            if line and line.strip() in [b'DELETED', b'NOT_FOUND']:
+            if line and line.strip() == b'DELETED':
                 return 1
-            self.debuglog('delete expected DELETED or NOT_FOUND, got: {!r}'.format(line))
+            self.debuglog('delete expected DELETED, got: {!r}'.format(line))
         except OSError as msg:
             if isinstance(msg, tuple):
                 msg = msg[1]

tests/test_memcache.py

@@ -56,6 +56,8 @@ def test_delete(self):
         result = self.mc.delete("long")
         self.assertEqual(result, True)
         self.assertEqual(self.mc.get("long"), None)
+        result = self.mc.delete("<missing>")
+        self.assertEqual(result, False)
 
     def test_default(self):
         key = "default"