diff options
| -rw-r--r-- | tests/artifactcache/config.py | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/tests/artifactcache/config.py b/tests/artifactcache/config.py index 46901937b..591cc0e03 100644 --- a/tests/artifactcache/config.py +++ b/tests/artifactcache/config.py @@ -1,11 +1,11 @@ # Pylint doesn't play well with fixtures and dependency injection from pytest # pylint: disable=redefined-outer-name -import pytest - import itertools import os +import pytest + from buildstream._artifactcache import ArtifactCacheSpec, ArtifactCache from buildstream._context import Context from buildstream._project import Project @@ -27,7 +27,13 @@ cache6 = ArtifactCacheSpec(url='https://example.com/cache6', push=True) # Generate cache configuration fragments for the user config and project config files. # -def configure_remote_caches(override_caches, project_caches=[], user_caches=[]): +def configure_remote_caches(override_caches, project_caches=None, user_caches=None): + if project_caches is None: + project_caches = [] + + if user_caches is None: + user_caches = [] + user_config = {} if len(user_caches) == 1: user_config['artifacts'] = { @@ -58,7 +64,7 @@ def configure_remote_caches(override_caches, project_caches=[], user_caches=[]): } project_config = {} - if len(project_caches) > 0: + if project_caches: if len(project_caches) == 1: project_config.update({ 'artifacts': { |