_project.py: Remove unused artifacts parameter from load_elements()

buildstream/_pipeline.py

@@ -106,7 +106,7 @@ class Pipeline():
 
         profile_start(Topics.LOAD_PIPELINE, "_".join(t.replace(os.sep, '-') for t in targets))
 
-        elements = self._project.load_elements(targets, self._artifacts,
+        elements = self._project.load_elements(targets,
                                                rewritable=rewritable,
                                                fetch_subprojects=fetch_subprojects)
 

buildstream/_project.py

@@ -304,7 +304,6 @@ class Project():
     #
     # Args:
     #    targets (list): Target names
-    #    artifacts (ArtifactCache): Artifact cache
     #    rewritable (bool): Whether the loaded files should be rewritable
     #                       this is a bit more expensive due to deep copies
     #    fetch_subprojects (bool): Whether we should fetch subprojects as a part of the
@@ -313,7 +312,7 @@ class Project():
     # Returns:
     #    (list): A list of loaded Element
     #
-    def load_elements(self, targets, artifacts, *,
+    def load_elements(self, targets, *,
                       rewritable=False, fetch_subprojects=False):
         with self._context.timed_activity("Loading elements", silent_nested=True):
             meta_elements = self.loader.load(targets, rewritable=rewritable,

tests/artifactcache/pull.py

@@ -90,7 +90,7 @@ def test_pull(cli, tmpdir, datafiles):
         cas = context.artifactcache
 
         # Assert that the element's artifact is **not** cached
-        element = project.load_elements(['target.bst'], cas)[0]
+        element = project.load_elements(['target.bst'])[0]
         element_key = cli.get_element_key(project_dir, 'target.bst')
         assert not cas.contains(element, element_key)
 
@@ -132,7 +132,7 @@ def _test_pull(user_config_file, project_dir, artifact_dir,
     cas = context.artifactcache
 
     # Load the target element
-    element = project.load_elements([element_name], cas)[0]
+    element = project.load_elements([element_name])[0]
 
     # Manually setup the CAS remote
     cas.setup_remotes(use_config=True)
@@ -193,7 +193,7 @@ def test_pull_tree(cli, tmpdir, datafiles):
         cas = context.artifactcache
 
         # Assert that the element's artifact is cached
-        element = project.load_elements(['target.bst'], cas)[0]
+        element = project.load_elements(['target.bst'])[0]
         element_key = cli.get_element_key(project_dir, 'target.bst')
         assert cas.contains(element, element_key)
 

tests/artifactcache/push.py

@@ -69,7 +69,7 @@ def test_push(cli, tmpdir, datafiles):
         cas = context.artifactcache
 
         # Assert that the element's artifact is cached
-        element = project.load_elements(['target.bst'], cas)[0]
+        element = project.load_elements(['target.bst'])[0]
         element_key = cli.get_element_key(project_dir, 'target.bst')
         assert cas.contains(element, element_key)
 
@@ -111,7 +111,7 @@ def _test_push(user_config_file, project_dir, artifact_dir,
     cas = context.artifactcache
 
     # Load the target element
-    element = project.load_elements([element_name], cas)[0]
+    element = project.load_elements([element_name])[0]
 
     # Manually setup the CAS remote
     cas.setup_remotes(use_config=True)
@@ -168,7 +168,7 @@ def test_push_directory(cli, tmpdir, datafiles):
         cas = context.artifactcache
 
         # Assert that the element's artifact is cached
-        element = project.load_elements(['target.bst'], cas)[0]
+        element = project.load_elements(['target.bst'])[0]
         element_key = cli.get_element_key(project_dir, 'target.bst')
         assert cas.contains(element, element_key)