From: david-sarah <david-sarah@jacaranda.org>
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/architecture.txt?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":