Try to prevent this test from being flaky. We might need a sleep in here

which isn't as bad as it sounds.  The close() *should* raise an exception,
so if it didn't we should give more time to sync and really raise it.

Will backport.
This commit is contained in:
Neal Norwitz 2008-01-27 05:02:34 +00:00
parent 6e070814b2
commit 46c61b2c1d
1 changed files with 6 additions and 2 deletions

View File

@ -54,8 +54,12 @@ class ResourceTest(unittest.TestCase):
f.write("Y")
f.flush()
# On some systems (e.g., Ubuntu on hppa) the flush()
# doesn't cause the exception, but the close() does.
f.close()
# doesn't always cause the exception, but the close()
# does eventually. Try closing several times in
# an attempt to ensure the file is really synced and
# the exception raised.
for i in range(5):
f.close()
except IOError:
if not limit_set:
raise