]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
[3.5] bpo-31026: Fix test_dbm if dbm.ndbm is build with Berkeley DB. (GH-6632)
authorSerhiy Storchaka <storchaka@gmail.com>
Sun, 29 Apr 2018 12:45:03 +0000 (15:45 +0300)
committerLarry Hastings <larry@hastings.org>
Tue, 29 Oct 2019 05:56:33 +0000 (22:56 -0700)
(cherry picked from commit 70af06cdc4e8fbee0b9d7d46bdc193097d4bc71f)

Lib/test/test_dbm.py
Lib/test/test_dbm_ndbm.py

index 623d9929d58f78cb325145738330dcedf8de0c18..81857e6f8852586f2852dc70a7df3ecea83512d3 100644 (file)
@@ -39,8 +39,7 @@ def delete_files():
 
 
 class AnyDBMTestCase:
-    _dict = {'0': b'',
-             'a': b'Python:',
+    _dict = {'a': b'Python:',
              'b': b'Programming',
              'c': b'the',
              'd': b'way',
@@ -101,6 +100,20 @@ class AnyDBMTestCase:
         keys = self.keys_helper(f)
         f.close()
 
+    def test_empty_value(self):
+        if getattr(dbm._defaultmod, 'library', None) == 'Berkeley DB':
+            self.skipTest("Berkeley DB doesn't distinguish the empty value "
+                          "from the absent one")
+        f = dbm.open(_fname, 'c')
+        self.assertEqual(f.keys(), [])
+        f[b'empty'] = b''
+        self.assertEqual(f.keys(), [b'empty'])
+        self.assertIn(b'empty', f)
+        self.assertEqual(f[b'empty'], b'')
+        self.assertEqual(f.get(b'empty'), b'')
+        self.assertEqual(f.setdefault(b'empty'), b'')
+        f.close()
+
     def test_anydbm_access(self):
         self.init_db()
         f = dbm.open(_fname, 'r')
index 2291561defc4d77c91edcc36a73f63ec844ab322..463dd38f254ec3218a6c63f491fe68afeed51a52 100644 (file)
@@ -29,6 +29,20 @@ class DbmTestCase(unittest.TestCase):
         self.assertEqual(self.d[b'bytes'], b'data')
         self.d.close()
 
+    def test_empty_value(self):
+        if dbm.ndbm.library == 'Berkeley DB':
+            self.skipTest("Berkeley DB doesn't distinguish the empty value "
+                          "from the absent one")
+        self.d = dbm.ndbm.open(self.filename, 'c')
+        self.assertEqual(self.d.keys(), [])
+        self.d['empty'] = ''
+        self.assertEqual(self.d.keys(), [b'empty'])
+        self.assertIn(b'empty', self.d)
+        self.assertEqual(self.d[b'empty'], b'')
+        self.assertEqual(self.d.get(b'empty'), b'')
+        self.assertEqual(self.d.setdefault(b'empty'), b'')
+        self.d.close()
+
     def test_modes(self):
         for mode in ['r', 'rw', 'w', 'n']:
             try: