]> granicus.if.org Git - python/commitdiff
Update mailbox.Maildir tests
authorPetri Lehtinen <petri@digip.org>
Sat, 12 Nov 2011 19:02:42 +0000 (21:02 +0200)
committerPetri Lehtinen <petri@digip.org>
Sat, 12 Nov 2011 19:14:53 +0000 (21:14 +0200)
Remove a sleep to fix transient test failures. Use skewfactor of -3 to
make it work on systems that have 1 second precision for time.time().

Closes #11999
Refs #13254

Lib/test/test_mailbox.py

index e0d8da29fea77de98a609ea58eae902c18ddff0a..007c419b9decaf53133566351d07b2e83efa5786 100644 (file)
@@ -815,7 +815,7 @@ class TestMaildir(TestMailbox):
         # skew factor to make _refresh think that the filesystem
         # safety period has passed and re-reading the _toc is only
         # required if mtimes differ.
-        self._box._skewfactor = -2
+        self._box._skewfactor = -3
 
         self._box._refresh()
         self.assertEqual(sorted(self._box._toc.keys()), sorted([key0, key1]))
@@ -908,7 +908,12 @@ class TestMaildir(TestMailbox):
         # refresh is done unconditionally if called for within
         # two-second-plus-a-bit of the last one, just in case the mbox has
         # changed; so now we have to wait for that interval to expire.
-        time.sleep(2.01 + self._box._skewfactor)
+        #
+        # Because this is a test, emulate sleeping. Instead of
+        # sleeping for 2 seconds, use the skew factor to make _refresh
+        # think that 2 seconds have passed and re-reading the _toc is
+        # only required if mtimes differ.
+        self._box._skewfactor = -3
 
         # Re-reading causes the ._toc attribute to be assigned a new dictionary
         # object, so we'll check that the ._toc attribute isn't a different
@@ -921,7 +926,8 @@ class TestMaildir(TestMailbox):
         self.assertFalse(refreshed())
 
         # Now, write something into cur and remove it.  This changes
-        # the mtime and should cause a re-read.
+        # the mtime and should cause a re-read. Note that "sleep
+        # emulation" is still in effect, as skewfactor is -3.
         filename = os.path.join(self._path, 'cur', 'stray-file')
         f = open(filename, 'w')
         f.close()