[ARVADOS] updated: 8b93280fb2fae252e29263660e9591fcaa2d8722

Git user git at public.curoverse.com
Tue Apr 5 13:22:09 EDT 2016


Summary of changes:
 services/fuse/arvados_fuse/fusedir.py |  3 ++-
 services/fuse/tests/test_cache.py     | 10 ++++++++++
 2 files changed, 12 insertions(+), 1 deletion(-)

       via  8b93280fb2fae252e29263660e9591fcaa2d8722 (commit)
      from  55778ad90a9b869f514e729b3f85c12383f4e2a7 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 8b93280fb2fae252e29263660e9591fcaa2d8722
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Tue Apr 5 13:22:06 2016 -0400

    8712: Propagate return value of clear() from super method.  Test cache clearing
    collections with subdirs.

diff --git a/services/fuse/arvados_fuse/fusedir.py b/services/fuse/arvados_fuse/fusedir.py
index bfcf0ef..3f2bcd5 100644
--- a/services/fuse/arvados_fuse/fusedir.py
+++ b/services/fuse/arvados_fuse/fusedir.py
@@ -321,8 +321,9 @@ class CollectionDirectoryBase(Directory):
         src.flush()
 
     def clear(self, force=False):
-        super(CollectionDirectoryBase, self).clear(force)
+        r = super(CollectionDirectoryBase, self).clear(force)
         self.collection = None
+        return r
 
 
 class CollectionDirectory(CollectionDirectoryBase):
diff --git a/services/fuse/tests/test_cache.py b/services/fuse/tests/test_cache.py
index 7dab2c5..7aa0009 100644
--- a/services/fuse/tests/test_cache.py
+++ b/services/fuse/tests/test_cache.py
@@ -21,6 +21,11 @@ class TmpCollectionTest(IntegrationTest):
             cw = arvados.collection.Collection()
             f = cw.open("blurg%i" % i, "w")
             f.write("bloop%i" % i)
+
+            cw.mkdirs("dir%i" % i)
+            f = cw.open("dir%i/blurg" % i, "w")
+            f.write("dirbloop%i" % i)
+
             cw.save_new()
             pdh.append(cw.portable_data_hash())
         self.pool_test(self.mnt, pdh)
@@ -30,6 +35,11 @@ class TmpCollectionTest(IntegrationTest):
         for i,v in enumerate(pdh):
             j = os.path.join(mnt, "by_id", v, "blurg%i" % i)
             self.assertTrue(os.path.exists(j))
+            j = os.path.join(mnt, "by_id", v, "dir%i/blurg" % i)
+            self.assertTrue(os.path.exists(j))
+
         for i,v in enumerate(pdh):
             j = os.path.join(mnt, "by_id", v, "blurg%i" % i)
             self.assertTrue(os.path.exists(j))
+            j = os.path.join(mnt, "by_id", v, "dir%i/blurg" % i)
+            self.assertTrue(os.path.exists(j))

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list