X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/ddc1f6b0c54b72df395cfef26e0dc4eb8f65463d..f0fe7273c1851cb93e9edd58c0b60d3590b222ed:/services/fuse/tests/test_mount.py diff --git a/services/fuse/tests/test_mount.py b/services/fuse/tests/test_mount.py index 55fd0f99a1..379a1306d8 100644 --- a/services/fuse/tests/test_mount.py +++ b/services/fuse/tests/test_mount.py @@ -1,29 +1,38 @@ -import unittest import arvados import arvados_fuse as fuse -import threading -import time -import os +import glob +import json import llfuse -import tempfile +import os import shutil import subprocess -import glob +import sys +import tempfile +import threading +import time +import unittest + import run_test_server -import json class MountTestBase(unittest.TestCase): def setUp(self): self.keeptmp = tempfile.mkdtemp() os.environ['KEEP_LOCAL_STORE'] = self.keeptmp self.mounttmp = tempfile.mkdtemp() - run_test_server.run(False) + run_test_server.run() run_test_server.authorize_with("admin") - self.api = api = fuse.SafeApi(arvados.config) + self.api = fuse.SafeApi(arvados.config) - def tearDown(self): - run_test_server.stop() + def make_mount(self, root_class, **root_kwargs): + operations = fuse.Operations(os.getuid(), os.getgid()) + operations.inodes.add_entry(root_class( + llfuse.ROOT_INODE, operations.inodes, self.api, 0, **root_kwargs)) + llfuse.init(operations, self.mounttmp, []) + threading.Thread(None, llfuse.main).start() + # wait until the driver is finished initializing + operations.initlock.wait() + def tearDown(self): # llfuse.close is buggy, so use fusermount instead. #llfuse.close(unmount=True) count = 0 @@ -35,6 +44,13 @@ class MountTestBase(unittest.TestCase): os.rmdir(self.mounttmp) shutil.rmtree(self.keeptmp) + run_test_server.reset() + + def assertDirContents(self, subdir, expect_content): + path = self.mounttmp + if subdir: + path = os.path.join(path, subdir) + self.assertEqual(sorted(expect_content), sorted(os.listdir(path))) class FuseMountTest(MountTestBase): @@ -67,37 +83,31 @@ class FuseMountTest(MountTestBase): cw.start_new_file('thing8.txt') cw.write("data 8") + cw.start_new_stream('edgecases') + for f in ":/./../.../-/*/\x01\\/ ".split("/"): + cw.start_new_file(f) + cw.write('x') + + for f in ":/../.../-/*/\x01\\/ ".split("/"): + cw.start_new_stream('edgecases/dirs/' + f) + cw.start_new_file('x/x') + cw.write('x') + self.testcollection = cw.finish() self.api.collections().create(body={"manifest_text":cw.manifest_text()}).execute() def runTest(self): - # Create the request handler - operations = fuse.Operations(os.getuid(), os.getgid()) - e = operations.inodes.add_entry(fuse.CollectionDirectory(llfuse.ROOT_INODE, operations.inodes, self.api, 0, self.testcollection)) - - llfuse.init(operations, self.mounttmp, []) - t = threading.Thread(None, lambda: llfuse.main()) - t.start() - - # wait until the driver is finished initializing - operations.initlock.wait() - - # now check some stuff - d1 = os.listdir(self.mounttmp) - d1.sort() - self.assertEqual(['dir1', 'dir2', 'thing1.txt', 'thing2.txt'], d1) - - d2 = os.listdir(os.path.join(self.mounttmp, 'dir1')) - d2.sort() - self.assertEqual(['thing3.txt', 'thing4.txt'], d2) - - d3 = os.listdir(os.path.join(self.mounttmp, 'dir2')) - d3.sort() - self.assertEqual(['dir3', 'thing5.txt', 'thing6.txt'], d3) - - d4 = os.listdir(os.path.join(self.mounttmp, 'dir2/dir3')) - d4.sort() - self.assertEqual(['thing7.txt', 'thing8.txt'], d4) + self.make_mount(fuse.CollectionDirectory, collection=self.testcollection) + + self.assertDirContents(None, ['thing1.txt', 'thing2.txt', + 'edgecases', 'dir1', 'dir2']) + self.assertDirContents('dir1', ['thing3.txt', 'thing4.txt']) + self.assertDirContents('dir2', ['thing5.txt', 'thing6.txt', 'dir3']) + self.assertDirContents('dir2/dir3', ['thing7.txt', 'thing8.txt']) + self.assertDirContents('edgecases', + "dirs/:/_/__/.../-/*/\x01\\/ ".split("/")) + self.assertDirContents('edgecases/dirs', + ":/__/.../-/*/\x01\\/ ".split("/")) files = {'thing1.txt': 'data 1', 'thing2.txt': 'data 2', @@ -113,6 +123,24 @@ class FuseMountTest(MountTestBase): self.assertEqual(v, f.read()) +class FuseNoAPITest(MountTestBase): + def setUp(self): + super(FuseNoAPITest, self).setUp() + keep = arvados.keep.KeepClient(local_store=self.keeptmp) + self.file_data = "API-free text\n" + self.file_loc = keep.put(self.file_data) + self.coll_loc = keep.put(". {} 0:{}:api-free.txt\n".format( + self.file_loc, len(self.file_data))) + + def runTest(self): + self.make_mount(fuse.MagicDirectory) + self.assertDirContents(self.coll_loc, ['api-free.txt']) + with open(os.path.join( + self.mounttmp, self.coll_loc, 'api-free.txt')) as keep_file: + actual = keep_file.read(-1) + self.assertEqual(self.file_data, actual) + + class FuseMagicTest(MountTestBase): def setUp(self): super(FuseMagicTest, self).setUp() @@ -126,31 +154,22 @@ class FuseMagicTest(MountTestBase): self.api.collections().create(body={"manifest_text":cw.manifest_text()}).execute() def runTest(self): - # Create the request handler - operations = fuse.Operations(os.getuid(), os.getgid()) - e = operations.inodes.add_entry(fuse.MagicDirectory(llfuse.ROOT_INODE, operations.inodes, self.api, 0)) - - self.mounttmp = tempfile.mkdtemp() - - llfuse.init(operations, self.mounttmp, []) - t = threading.Thread(None, lambda: llfuse.main()) - t.start() - - # wait until the driver is finished initializing - operations.initlock.wait() - - # now check some stuff - d1 = os.listdir(self.mounttmp) - d1.sort() - self.assertEqual(['README'], d1) - - d2 = os.listdir(os.path.join(self.mounttmp, self.testcollection)) - d2.sort() - self.assertEqual(['thing1.txt'], d2) - - d3 = os.listdir(self.mounttmp) - d3.sort() - self.assertEqual([self.testcollection, 'README'], d3) + self.make_mount(fuse.MagicDirectory) + + mount_ls = os.listdir(self.mounttmp) + self.assertIn('README', mount_ls) + self.assertFalse(any(arvados.util.keep_locator_pattern.match(fn) or + arvados.util.uuid_pattern.match(fn) + for fn in mount_ls), + "new FUSE MagicDirectory lists Collection") + self.assertDirContents(self.testcollection, ['thing1.txt']) + self.assertDirContents(os.path.join('by_id', self.testcollection), + ['thing1.txt']) + mount_ls = os.listdir(self.mounttmp) + self.assertIn('README', mount_ls) + self.assertIn(self.testcollection, mount_ls) + self.assertIn(self.testcollection, + os.listdir(os.path.join(self.mounttmp, 'by_id'))) files = {} files[os.path.join(self.mounttmp, self.testcollection, 'thing1.txt')] = 'data 1' @@ -162,15 +181,7 @@ class FuseMagicTest(MountTestBase): class FuseTagsTest(MountTestBase): def runTest(self): - operations = fuse.Operations(os.getuid(), os.getgid()) - e = operations.inodes.add_entry(fuse.TagsDirectory(llfuse.ROOT_INODE, operations.inodes, self.api, 0)) - - llfuse.init(operations, self.mounttmp, []) - t = threading.Thread(None, lambda: llfuse.main()) - t.start() - - # wait until the driver is finished initializing - operations.initlock.wait() + self.make_mount(fuse.TagsDirectory) d1 = os.listdir(self.mounttmp) d1.sort() @@ -186,116 +197,148 @@ class FuseTagsTest(MountTestBase): class FuseTagsUpdateTest(MountTestBase): - def runRealTest(self): - operations = fuse.Operations(os.getuid(), os.getgid()) - e = operations.inodes.add_entry(fuse.TagsDirectory(llfuse.ROOT_INODE, operations.inodes, self.api, 0, poll_time=1)) - - llfuse.init(operations, self.mounttmp, []) - t = threading.Thread(None, lambda: llfuse.main()) - t.start() - - # wait until the driver is finished initializing - operations.initlock.wait() - - d1 = os.listdir(self.mounttmp) - d1.sort() - self.assertEqual(['foo_tag'], d1) - - self.api.links().create(body={'link': { - 'head_uuid': 'fa7aeb5140e2848d39b416daeef4ffc5+45', - 'link_class': 'tag', - 'name': 'bar_tag' + def tag_collection(self, coll_uuid, tag_name): + return self.api.links().create( + body={'link': {'head_uuid': coll_uuid, + 'link_class': 'tag', + 'name': tag_name, }}).execute() - time.sleep(1) - - d2 = os.listdir(self.mounttmp) - d2.sort() - self.assertEqual(['bar_tag', 'foo_tag'], d2) - - d3 = os.listdir(os.path.join(self.mounttmp, 'bar_tag')) - d3.sort() - self.assertEqual(['fa7aeb5140e2848d39b416daeef4ffc5+45'], d3) + def runTest(self): + self.make_mount(fuse.TagsDirectory, poll_time=1) - l = self.api.links().create(body={'link': { - 'head_uuid': 'ea10d51bcf88862dbcc36eb292017dfd+45', - 'link_class': 'tag', - 'name': 'bar_tag' - }}).execute() + self.assertIn('foo_tag', os.listdir(self.mounttmp)) + bar_uuid = run_test_server.fixture('collections')['bar_file']['uuid'] + self.tag_collection(bar_uuid, 'fuse_test_tag') time.sleep(1) + self.assertIn('fuse_test_tag', os.listdir(self.mounttmp)) + self.assertDirContents('fuse_test_tag', [bar_uuid]) - d4 = os.listdir(os.path.join(self.mounttmp, 'bar_tag')) - d4.sort() - self.assertEqual(['ea10d51bcf88862dbcc36eb292017dfd+45', 'fa7aeb5140e2848d39b416daeef4ffc5+45'], d4) + baz_uuid = run_test_server.fixture('collections')['baz_file']['uuid'] + l = self.tag_collection(baz_uuid, 'fuse_test_tag') + time.sleep(1) + self.assertDirContents('fuse_test_tag', [bar_uuid, baz_uuid]) self.api.links().delete(uuid=l['uuid']).execute() - time.sleep(1) - - d5 = os.listdir(os.path.join(self.mounttmp, 'bar_tag')) - d5.sort() - self.assertEqual(['fa7aeb5140e2848d39b416daeef4ffc5+45'], d5) + self.assertDirContents('fuse_test_tag', [bar_uuid]) class FuseSharedTest(MountTestBase): def runTest(self): - operations = fuse.Operations(os.getuid(), os.getgid()) - e = operations.inodes.add_entry(fuse.SharedDirectory(llfuse.ROOT_INODE, operations.inodes, self.api, 0, self.api.users().current().execute()['uuid'])) - - llfuse.init(operations, self.mounttmp, []) - t = threading.Thread(None, lambda: llfuse.main()) - t.start() - - # wait until the driver is finished initializing - operations.initlock.wait() - - d1 = os.listdir(self.mounttmp) - d1.sort() - self.assertIn('FUSE User', d1) - - d2 = os.listdir(os.path.join(self.mounttmp, 'FUSE User')) - d2.sort() - self.assertEqual(['FUSE Test Project', - 'collection #1 owned by FUSE', - 'collection #2 owned by FUSE' - ], d2) - - d3 = os.listdir(os.path.join(self.mounttmp, 'FUSE User', 'FUSE Test Project')) - d3.sort() - self.assertEqual(['Pipeline Template in FUSE Project.pipelineTemplate', - 'collection in FUSE project', - ], d3) - - with open(os.path.join( + self.make_mount(fuse.SharedDirectory, + exclude=self.api.users().current().execute()['uuid']) + + # shared_dirs is a list of the directories exposed + # by fuse.SharedDirectory (i.e. any object visible + # to the current user) + shared_dirs = os.listdir(self.mounttmp) + shared_dirs.sort() + self.assertIn('FUSE User', shared_dirs) + + # fuse_user_objs is a list of the objects owned by the FUSE + # test user (which present as files in the 'FUSE User' + # directory) + fuse_user_objs = os.listdir(os.path.join(self.mounttmp, 'FUSE User')) + fuse_user_objs.sort() + self.assertEqual(['Empty collection.link', # permission link on collection + 'FUSE Test Project', # project owned by user + 'collection #1 owned by FUSE', # collection owned by user + 'collection #2 owned by FUSE', # collection owned by user + 'pipeline instance owned by FUSE.pipelineInstance', # pipeline instance owned by user + ], fuse_user_objs) + + # test_proj_files is a list of the files in the FUSE Test Project. + test_proj_files = os.listdir(os.path.join(self.mounttmp, 'FUSE User', 'FUSE Test Project')) + test_proj_files.sort() + self.assertEqual(['collection in FUSE project', + 'pipeline instance in FUSE project.pipelineInstance', + 'pipeline template in FUSE project.pipelineTemplate' + ], test_proj_files) + + # Double check that we can open and read objects in this folder as a file, + # and that its contents are what we expect. + pipeline_template_path = os.path.join( self.mounttmp, 'FUSE User', 'FUSE Test Project', - 'Pipeline Template in FUSE Project.pipelineTemplate')) as f: + 'pipeline template in FUSE project.pipelineTemplate') + with open(pipeline_template_path) as f: j = json.load(f) - self.assertEqual("Pipeline Template in FUSE Project", j['name']) + self.assertEqual("pipeline template in FUSE project", j['name']) + # check mtime on template + st = os.stat(pipeline_template_path) + self.assertEqual(st.st_mtime, 1397493304) -class FuseHomeTest(MountTestBase): - def runTest(self): - operations = fuse.Operations(os.getuid(), os.getgid()) - e = operations.inodes.add_entry(fuse.ProjectDirectory(llfuse.ROOT_INODE, operations.inodes, self.api, 0, self.api.users().current().execute())) + # check mtime on collection + st = os.stat(os.path.join( + self.mounttmp, + 'FUSE User', + 'collection #1 owned by FUSE')) + self.assertEqual(st.st_mtime, 1391448174) - llfuse.init(operations, self.mounttmp, []) - t = threading.Thread(None, lambda: llfuse.main()) - t.start() - # wait until the driver is finished initializing - operations.initlock.wait() +class FuseHomeTest(MountTestBase): + def runTest(self): + self.make_mount(fuse.ProjectDirectory, + project_object=self.api.users().current().execute()) d1 = os.listdir(self.mounttmp) - d1.sort() self.assertIn('Unrestricted public data', d1) d2 = os.listdir(os.path.join(self.mounttmp, 'Unrestricted public data')) - d2.sort() - self.assertEqual(['GNU General Public License, version 3'], d2) + public_project = run_test_server.fixture('groups')[ + 'anonymously_accessible_project'] + found_in = 0 + found_not_in = 0 + for name, item in run_test_server.fixture('collections').iteritems(): + if 'name' not in item: + pass + elif item['owner_uuid'] == public_project['uuid']: + self.assertIn(item['name'], d2) + found_in += 1 + else: + # Artificial assumption here: there is no public + # collection fixture with the same name as a + # non-public collection. + self.assertNotIn(item['name'], d2) + found_not_in += 1 + self.assertNotEqual(0, found_in) + self.assertNotEqual(0, found_not_in) d3 = os.listdir(os.path.join(self.mounttmp, 'Unrestricted public data', 'GNU General Public License, version 3')) - d3.sort() self.assertEqual(["GNU_General_Public_License,_version_3.pdf"], d3) + + +class FuseUnitTest(unittest.TestCase): + def test_sanitize_filename(self): + acceptable = [ + "foo.txt", + ".foo", + "..foo", + "...", + "foo...", + "foo..", + "foo.", + "-", + "\x01\x02\x03", + ] + unacceptable = [ + "f\00", + "\00\00", + "/foo", + "foo/", + "//", + ] + for f in acceptable: + self.assertEqual(f, fuse.sanitize_filename(f)) + for f in unacceptable: + self.assertNotEqual(f, fuse.sanitize_filename(f)) + # The sanitized filename should be the same length, though. + self.assertEqual(len(f), len(fuse.sanitize_filename(f))) + # Special cases + self.assertEqual("_", fuse.sanitize_filename("")) + self.assertEqual("_", fuse.sanitize_filename(".")) + self.assertEqual("__", fuse.sanitize_filename(".."))