7939: Use 100:: for bogus host check and add comment about API check.
authorPeter Amstutz <peter.amstutz@curoverse.com>
Mon, 14 Dec 2015 20:20:23 +0000 (15:20 -0500)
committerPeter Amstutz <peter.amstutz@curoverse.com>
Mon, 14 Dec 2015 20:20:23 +0000 (15:20 -0500)
services/fuse/arvados_fuse/command.py
services/fuse/tests/test_command_args.py

index 11935adb92c56eef8eb1d5559c45ae7948679024..cd81a5390a2bdf53807711174677a2bc8f1b47be 100644 (file)
@@ -153,6 +153,7 @@ class Mount(object):
             keep_params={
                 "block_cache": arvados.keep.KeepBlockCache(self.args.file_cache)
             })
+        # Do a sanity check that we have a working arvados host + token.
         self.api.users().current().execute()
 
     def _setup_mount(self):
index 2eb73b5a7ac59727a3360d8c8637231c033b0e95..bfefc674d7991dd80963a5d102a1bc757bc7bfb3 100644 (file)
@@ -218,7 +218,7 @@ class MountErrorTest(unittest.TestCase):
         self.assertEqual(1, ex.exception.code)
 
     def test_bogus_host(self):
-        arvados.config._settings["ARVADOS_API_HOST"] = "example.null"
+        arvados.config._settings["ARVADOS_API_HOST"] = "100::"
         with self.assertRaises(SystemExit) as ex:
             args = arvados_fuse.command.ArgumentParser().parse_args([self.mntdir])
             arvados_fuse.command.Mount(args, logger=self.logger).run()