Add support for anonymous gcs file source access

Author: nuwang

lib/galaxy/files/sources/googlecloudstorage.py

@@ -19,7 +19,9 @@ def _open_fs(self, user_context):
         root_path = props.pop('root_path', None)
         project = props.pop('project', None)
         args = {}
-        if props.get('token'):
+        if props.get('anonymous'):
+            args['client'] = Client.create_anonymous_client()
+        elif props.get('token'):
             args['client'] = Client(project=project, credentials=Credentials(**props))
         handle = GCSFS(bucket_name, root_path=root_path, retry=0, **args)
         return handle

packages/files/test-requirements.txt

@@ -1 +1,2 @@
 pytest
+fs-gcsfs

test/unit/files/gcsfs_file_sources_conf.yml

@@ -2,9 +2,10 @@
   id: test1
   doc: Test access to Google Cloud Storage.
   project: ${user.preferences['googlecloudstorage|project']}
-  bucket_name: ${user.preferences['googlecloudstorage|bucket_name']}
+  bucket_name: 'genomics-public-data'
   token_uri: "https://www.googleapis.com/oauth2/v4/token"
   client_id: ${user.preferences['googlecloudstorage|client_id']}
   client_secret: ${user.preferences['googlecloudstorage|client_secret']}
   token: ${user.preferences['googlecloudstorage|access_token']}
   refresh_token: ${user.preferences['googlecloudstorage|refresh_token']}
+  anonymous: true

test/unit/files/test_gcsfs.py

@@ -1,7 +1,5 @@
 import os
 
-import pytest
-
 from galaxy.files import ConfiguredFileSources, ConfiguredFileSourcesConfig
 from ._util import (
     assert_realizes_contains,
@@ -11,16 +9,7 @@
 SCRIPT_DIRECTORY = os.path.abspath(os.path.dirname(__file__))
 FILE_SOURCES_CONF = os.path.join(SCRIPT_DIRECTORY, "gcsfs_file_sources_conf.yml")
 
-skip_if_no_gcs_access_token = pytest.mark.skipif(
-    not os.environ.get('GALAXY_TEST_GCS_PROJECT')
-    or not os.environ.get('GALAXY_TEST_GCS_BUCKET')
-    or not os.environ.get('GALAXY_TEST_GCS_ACCESS_TOKEN')
-    or not os.environ.get('GALAXY_TEST_GCS_REFRESH_TOKEN'),
-    reason="GALAXY_TEST_GCS_ACCESS_TOKEN and related vars not set"
-)
-
 
-@skip_if_no_gcs_access_token
 def test_file_source():
     user_context = user_context_fixture()
     file_sources = _configured_file_sources()