tests: Fix all remaining references to bst pushchandan/fix-bst-pull-docs

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

4 files changed, 12 insertions, 12 deletions

diff --git a/tests/artifactcache/junctions.py b/tests/artifactcache/junctions.py
index cf655a93b..9b7ebe2ec 100644
--- a/tests/artifactcache/junctions.py
+++ b/tests/artifactcache/junctions.py
@@ -57,7 +57,7 @@ def test_push_pull(cli, tmpdir, datafiles):
         project_set_artifacts(project, share.repo)
         project_set_artifacts(base_project, base_share.repo)
 
-        # Now try bst push
+        # Now try bst artifact push
         result = cli.run(project=project, args=['artifact', 'push', '--deps', 'all', 'target.bst'])
         assert result.exit_code == 0
 
diff --git a/tests/frontend/pull.py b/tests/frontend/pull.py
index de12e1d6b..2a1006ea2 100644
--- a/tests/frontend/pull.py
+++ b/tests/frontend/pull.py
@@ -86,7 +86,7 @@ def test_push_pull_all(cli, tmpdir, datafiles):
 
 # Tests that:
 #
-#  * `bst push` (default targets) pushes all built elements to configured 'push' cache
+#  * `bst artifact push` (default targets) pushes all built elements to configured 'push' cache
 #  * `bst artifact pull` (default targets) downloads everything from cache after local deletion
 #
 @pytest.mark.datafiles(DATA_DIR + '_world')
@@ -177,7 +177,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 artifact push --remote` pushes to the given remote, not one from the config
 #  * `bst artifact pull --remote` pulls from the given remote
 #
 @pytest.mark.datafiles(DATA_DIR)
@@ -200,7 +200,7 @@ def test_push_pull_specific_remote(cli, tmpdir, datafiles):
             'artifacts': {'url': bad_share.repo, 'push': True},
         })
 
-        # Now try `bst push` to the good_share.
+        # Now try `bst artifact push` to the good_share.
         result = cli.run(project=project, args=[
             'artifact', 'push', 'target.bst', '--remote', good_share.repo
         ])
diff --git a/tests/frontend/push.py b/tests/frontend/push.py
index 061607446..7b4e944d4 100644
--- a/tests/frontend/push.py
+++ b/tests/frontend/push.py
@@ -64,8 +64,8 @@ def assert_not_shared(cli, share, project, element_name):
 
 # Tests that:
 #
-#  * `bst push` fails if there are no remotes configured for pushing
-#  * `bst push` successfully pushes to any remote that is configured for pushing
+#  * `bst artifact push` fails if there are no remotes configured for pushing
+#  * `bst artifact push` successfully pushes to any remote that is configured for pushing
 #
 @pytest.mark.datafiles(DATA_DIR)
 def test_push(cli, tmpdir, datafiles):
@@ -87,7 +87,7 @@ def test_push(cli, tmpdir, datafiles):
             result = cli.run(project=project, args=['artifact', 'push', 'target.bst'])
             result.assert_main_error(ErrorDomain.STREAM, None)
 
-            # Configure bst to pull but not push from a cache and run `bst push`.
+            # Configure bst to pull but not push from a cache and run `bst artifact push`.
             # This should also fail.
             cli.configure({
                 'artifacts': {'url': share1.repo, 'push': False},
@@ -95,7 +95,7 @@ def test_push(cli, tmpdir, datafiles):
             result = cli.run(project=project, args=['artifact', 'push', 'target.bst'])
             result.assert_main_error(ErrorDomain.STREAM, None)
 
-            # Configure bst to push to one of the caches and run `bst push`. This works.
+            # Configure bst to push to one of the caches and run `bst artifact push`. This works.
             cli.configure({
                 'artifacts': [
                     {'url': share1.repo, 'push': False},
@@ -122,7 +122,7 @@ def test_push(cli, tmpdir, datafiles):
             assert_shared(cli, share2, project, 'target.bst')
 
 
-# Tests that `bst push --deps all` pushes all dependencies of the given element.
+# Tests that `bst artifact push --deps all` pushes all dependencies of the given element.
 #
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_all(cli, tmpdir, datafiles):
@@ -156,7 +156,7 @@ def test_push_all(cli, tmpdir, datafiles):
             }
         })
 
-        # Now try bst push all the deps
+        # Now try bst artifact push all the deps
         result = cli.run(project=project, args=[
             'artifact', 'push', 'target.bst',
             '--deps', 'all'
diff --git a/tests/integration/pullbuildtrees.py b/tests/integration/pullbuildtrees.py
index 5eb835cd3..bdbcad911 100644
--- a/tests/integration/pullbuildtrees.py
+++ b/tests/integration/pullbuildtrees.py
@@ -129,7 +129,7 @@ def test_pullbuildtrees(cli2, tmpdir, datafiles):
         assert share2.has_artifact('test', element_name, cli2.get_element_key(project, element_name))
         default_state(cli2, tmpdir, share1)
 
-        # Assert that bst push will automatically attempt to pull a missing buildtree
+        # Assert that bst artifact push will automatically attempt to pull a missing buildtree
         # if pull-buildtrees is set, however as share3 is the only defined remote and is empty,
         # assert that no element artifact buildtrees are pulled (no available remote buildtree) and thus the
         # artifact cannot be pushed.
@@ -144,7 +144,7 @@ def test_pullbuildtrees(cli2, tmpdir, datafiles):
         assert element_name not in result.get_pushed_elements()
         assert not share3.has_artifact('test', element_name, cli2.get_element_key(project, element_name))
 
-        # Assert that if we add an extra remote that has the buildtree artfact cached, bst push will
+        # Assert that if we add an extra remote that has the buildtree artfact cached, bst artifact 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 artifact pull.