tests: Fix all remaining references to bst pull

As the `bst pull` is now called `bst artifact pull`, update the
remaining comments in testsuite that still refer to it as `bst pull`.

3 files changed, 11 insertions, 11 deletions

diff --git a/tests/artifactcache/junctions.py b/tests/artifactcache/junctions.py
index 6655a4f1d..cf655a93b 100644
--- a/tests/artifactcache/junctions.py
+++ b/tests/artifactcache/junctions.py
@@ -77,7 +77,7 @@ def test_push_pull(cli, tmpdir, datafiles):
         state = cli.get_element_state(base_project, 'target.bst')
         assert state != 'cached'
 
-        # Now try bst pull
+        # Now try bst artifact pull
         result = cli.run(project=project, args=['artifact', 'pull', '--deps', 'all', 'target.bst'])
         assert result.exit_code == 0
 
diff --git a/tests/frontend/pull.py b/tests/frontend/pull.py
index a2cbffa04..de12e1d6b 100644
--- a/tests/frontend/pull.py
+++ b/tests/frontend/pull.py
@@ -44,7 +44,7 @@ def assert_not_shared(cli, share, project, element_name):
 # Tests that:
 #
 #  * `bst build` pushes all build elements to configured 'push' cache
-#  * `bst pull --deps all` downloads everything from cache after local deletion
+#  * `bst artifact pull --deps all` downloads everything from cache after local deletion
 #
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_pull_all(cli, tmpdir, datafiles):
@@ -75,7 +75,7 @@ def test_push_pull_all(cli, tmpdir, datafiles):
         states = cli.get_element_states(project, all_elements)
         assert not any(states[e] == 'cached' for e in all_elements)
 
-        # Now try bst pull
+        # Now try bst artifact pull
         result = cli.run(project=project, args=['artifact', 'pull', '--deps', 'all', 'target.bst'])
         result.assert_success()
 
@@ -87,7 +87,7 @@ def test_push_pull_all(cli, tmpdir, datafiles):
 # Tests that:
 #
 #  * `bst push` (default targets) pushes all built elements to configured 'push' cache
-#  * `bst pull` (default targets) downloads everything from cache after local deletion
+#  * `bst artifact pull` (default targets) downloads everything from cache after local deletion
 #
 @pytest.mark.datafiles(DATA_DIR + '_world')
 def test_push_pull_default_targets(cli, tmpdir, datafiles):
@@ -125,7 +125,7 @@ def test_push_pull_default_targets(cli, tmpdir, datafiles):
         states = cli.get_element_states(project, all_elements)
         assert not any(states[e] == 'cached' for e in all_elements)
 
-        # Now try bst pull
+        # Now try bst artifact pull
         result = cli.run(project=project, args=['artifact', 'pull'])
         result.assert_success()
 
@@ -137,7 +137,7 @@ def test_push_pull_default_targets(cli, tmpdir, datafiles):
 # Tests that:
 #
 #  * `bst build` pushes all build elements ONLY to configured 'push' cache
-#  * `bst pull` finds artifacts that are available only in the secondary cache
+#  * `bst artifact pull` finds artifacts that are available only in the secondary cache
 #
 @pytest.mark.datafiles(DATA_DIR)
 def test_pull_secondary_cache(cli, tmpdir, datafiles):
@@ -166,7 +166,7 @@ def test_pull_secondary_cache(cli, tmpdir, datafiles):
         # Assert that the element is not cached anymore.
         assert cli.get_element_state(project, 'target.bst') != 'cached'
 
-        # Now try bst pull
+        # Now try bst artifact pull
         result = cli.run(project=project, args=['artifact', 'pull', 'target.bst'])
         result.assert_success()
 
@@ -178,7 +178,7 @@ def test_pull_secondary_cache(cli, tmpdir, datafiles):
 # Tests that:
 #
 #  * `bst push --remote` pushes to the given remote, not one from the config
-#  * `bst pull --remote` pulls from the given remote
+#  * `bst artifact pull --remote` pulls from the given remote
 #
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_pull_specific_remote(cli, tmpdir, datafiles):
@@ -269,7 +269,7 @@ def test_push_pull_non_strict(cli, tmpdir, datafiles):
         # Assert that the target is still waiting due to --no-strict
         assert cli.get_element_state(project, 'target.bst') == 'waiting'
 
-        # Now try bst pull
+        # Now try bst artifact pull
         result = cli.run(project=project, args=['artifact', 'pull', '--deps', 'all', 'target.bst'])
         result.assert_success()
 
@@ -344,7 +344,7 @@ def test_push_pull_cross_junction(cli, tmpdir, datafiles):
 
         assert cli.get_element_state(project, 'junction.bst:import-etc.bst') == 'buildable'
 
-        # Now try bst pull
+        # Now try bst artifact pull
         result = cli.run(project=project, args=['artifact', 'pull', 'junction.bst:import-etc.bst'])
         result.assert_success()
 
diff --git a/tests/integration/pullbuildtrees.py b/tests/integration/pullbuildtrees.py
index 1a89f776a..5eb835cd3 100644
--- a/tests/integration/pullbuildtrees.py
+++ b/tests/integration/pullbuildtrees.py
@@ -147,7 +147,7 @@ def test_pullbuildtrees(cli2, tmpdir, datafiles):
         # Assert that if we add an extra remote that has the buildtree artfact cached, bst push will
         # automatically attempt to pull it and will be successful, leading to the full artifact being pushed
         # to the empty share3. This gives the ability to attempt push currently partial artifacts to a remote,
-        # without exlipictly requiring a bst pull.
+        # without exlipictly requiring a bst artifact pull.
         cli2.configure({'artifacts': [{'url': share1.repo, 'push': False}, {'url': share3.repo, 'push': True}]})
         result = cli2.run(project=project, args=['--pull-buildtrees', 'artifact', 'push', element_name])
         assert "Attempting to fetch missing artifact buildtrees" in result.stderr