Mercurial > public > mercurial-scm > hg
diff tests/test-lrucachedict.py @ 39563:b31b01f93b11
util: properly copy lrucachedict instances
Previously, copy() only worked if the cache was full. We teach
copy() to only copy defined nodes.
Differential Revision: https://phab.mercurial-scm.org/D4498
| author | Gregory Szorc <gregory.szorc@gmail.com> |
|---|---|
| date | Thu, 06 Sep 2018 11:33:40 -0700 |
| parents | 067f7d2c7d60 |
| children | 5d75a3c16193 |
line wrap: on
line diff
--- a/tests/test-lrucachedict.py Thu Sep 06 11:27:25 2018 -0700 +++ b/tests/test-lrucachedict.py Thu Sep 06 11:33:40 2018 -0700 @@ -68,12 +68,28 @@ dc = d.copy() self.assertEqual(len(dc), 2) - # TODO this fails - return for key in ('a', 'b'): self.assertIn(key, dc) self.assertEqual(dc[key], 'v%s' % key) + self.assertEqual(len(d), 2) + for key in ('a', 'b'): + self.assertIn(key, d) + self.assertEqual(d[key], 'v%s' % key) + + d['c'] = 'vc' + del d['b'] + dc = d.copy() + self.assertEqual(len(dc), 2) + for key in ('a', 'c'): + self.assertIn(key, dc) + self.assertEqual(dc[key], 'v%s' % key) + + def testcopyempty(self): + d = util.lrucachedict(4) + dc = d.copy() + self.assertEqual(len(dc), 0) + def testcopyfull(self): d = util.lrucachedict(4) d['a'] = 'va'