From: david-sarah Date: Wed, 10 Aug 2011 02:59:42 +0000 (-0700) Subject: test_drop_upload.py: add comment explaining why we don't use FilePath.setContent... X-Git-Url: https://git.rkrishnan.org/components/com_hotproperty/flags/module-simplejson.decoder.html?a=commitdiff_plain;h=f157b733676b33f578cd12020b1b63a825fa0ef4;p=tahoe-lafs%2Ftahoe-lafs.git test_drop_upload.py: add comment explaining why we don't use FilePath.setContent. refs #1429 --- diff --git a/src/allmydata/test/test_drop_upload.py b/src/allmydata/test/test_drop_upload.py index fa697877..ea21ff02 100644 --- a/src/allmydata/test/test_drop_upload.py +++ b/src/allmydata/test/test_drop_upload.py @@ -122,6 +122,8 @@ class DropUploadTestMixin(GridTestMixin, ShouldFailMixin, ReallyEqualMixin): else: path = filepath.FilePath(path_u.encode(get_filesystem_encoding())) + # We don't use FilePath.setContent() here because it creates a temporary file that + # is renamed into place, which causes events that the test is not expecting. f = open(path.path, "wb") try: if temporary and sys.platform != "win32":