]> granicus.if.org Git - python/commitdiff
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)
committerGitHub <noreply@github.com>
Sun, 29 Apr 2018 12:45:03 +0000 (15:45 +0300)
Lib/test/test_dbm.py
Lib/test/test_dbm_ndbm.py

index fb8980780f1d7985b630e618cc4af5ae8fc34bcc..78c32c4c93d35c377d4fe85ac686a74a86724f5f 100644 (file)
@@ -38,8 +38,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',
@@ -109,6 +108,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 f921167ca463465f3696e99e54b21048e8ce361b..49b88f5cccefc8ead78ff876a0c6f1de23042172 100644 (file)
@@ -36,6 +36,20 @@ class DbmTestCase(unittest.TestCase):
         self.assertEqual(self.d[b'xxx'], b'foo')
         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: