From: David Stainton <dstainton415@gmail.com>
Date: Thu, 23 Apr 2015 00:25:03 +0000 (+0100)
Subject: Fix _test_persistence AND remove superfluous print statements
X-Git-Url: https://git.rkrishnan.org/pf?a=commitdiff_plain;h=c5bef8f9565bf69d3d9f5c6851f1db3fd1ed5a2b;p=tahoe-lafs%2Ftahoe-lafs.git

Fix _test_persistence AND remove superfluous print statements
---

diff --git a/src/allmydata/frontends/drop_upload.py b/src/allmydata/frontends/drop_upload.py
index 382b6d10..0b757cb0 100644
--- a/src/allmydata/frontends/drop_upload.py
+++ b/src/allmydata/frontends/drop_upload.py
@@ -1,5 +1,5 @@
 
-import sys, os
+import sys, os, stat
 from collections import deque
 
 from twisted.internet import defer, reactor, task
@@ -89,12 +89,10 @@ class DropUploader(service.MultiService):
         assert self._db != None
         r = self._db.check_file(childpath)
         filecap = r.was_uploaded()
-        print "uploaded filecap %s" % (filecap,)
         if filecap is False:
             return True
 
     def _scan(self, localpath):
-        print "_scan"
         if not os.path.isdir(localpath):
             raise AssertionError("Programmer error: _scan() must be passed a directory path.")
         quoted_path = quote_local_unicode_path(localpath)
@@ -116,7 +114,6 @@ class DropUploader(service.MultiService):
             if islink:
                 self.warn("WARNING: cannot backup symlink %s" % quote_local_unicode_path(childpath))
             elif isdir:
-                print "isdir"
                 must_upload = self._check_db_file(childpath.decode('UTF-8'))
                 if must_upload:
                     self._append_to_deque(childpath)
@@ -124,10 +121,8 @@ class DropUploader(service.MultiService):
                 # recurse on the child directory
                 self._scan(childpath)
             elif isfile:
-                print "isfile %s" % (childpath,)
                 must_upload = self._check_db_file(childpath.decode('UTF-8'))
                 if must_upload:
-                    print "must_upload"
                     self._append_to_deque(childpath)
             else:
                 self.warn("WARNING: cannot backup special file %s" % quote_local_unicode_path(childpath))
@@ -231,18 +226,12 @@ class DropUploader(service.MultiService):
                 d.addCallback(_add_file)
                 def add_db_entry(filenode):
                     filecap = filenode.get_uri()
-                    print "filename %s filecap %s" % (path, filecap)
                     s = os.stat(path)
+                    size = s[stat.ST_SIZE]
+                    ctime = s[stat.ST_CTIME]
+                    mtime = s[stat.ST_MTIME]
                     self._db.did_upload_file(filecap, path.decode('UTF-8'),
-                                             s.st_mtime, s.st_ctime, s.st_size)
-
-                    #r = self._db.check_file(path.decode('UTF-8'))
-                    #was_uploaded = r.was_uploaded()
-                    #if was_uploaded:
-                    #    print "was_uploaded true"
-                    #else:
-                    #    print "was_uploaded false"
-
+                                             mtime, ctime, size)
                 d.addCallback(add_db_entry)
                 self._stats_provider.count('drop_upload.files_uploaded', 1)
                 return None
diff --git a/src/allmydata/test/test_drop_upload.py b/src/allmydata/test/test_drop_upload.py
index a8849035..d9021449 100644
--- a/src/allmydata/test/test_drop_upload.py
+++ b/src/allmydata/test/test_drop_upload.py
@@ -204,7 +204,6 @@ class DropUploadTestMixin(GridTestMixin, ShouldFailMixin, ReallyEqualMixin, NonA
         d.addCallback(self._cleanup)
 
         def _restart(ign):
-            print "in _restart"
             self.set_up_grid()
             self.client = self.g.clients[0]
             self.stats_provider = self.client.stats_provider