cli.py: Change artifact checkout to pull implicitlyjennis/pull_implicitly_in_checkout

This makes it consistent with source checkout

1 files changed, 5 insertions, 7 deletions

diff --git a/tests/frontend/buildcheckout.py b/tests/frontend/buildcheckout.py
index c9a42239a..f0fa82151 100644
--- a/tests/frontend/buildcheckout.py
+++ b/tests/frontend/buildcheckout.py
@@ -1105,14 +1105,12 @@ def test_partial_artifact_checkout_fetch(cli, datafiles, tmpdir):
         os.unlink(objpath)
 
         # Verify that the build-only dependency is not (complete) in the local cache
-        result = cli.run(project=project, args=[
-            'artifact', 'checkout', build_elt,
-            '--directory', checkout_dir])
-        result.assert_main_error(ErrorDomain.STREAM, 'uncached-checkout-attempt')
+        assert cli.get_element_state(project, build_elt) != 'cached'
 
-        # Verify that the pull method fetches relevant artifacts in order to stage
+        # Verify that when we checkout, we implicitly pull the relevant
+        # artifacts in order to stage
         result = cli.run(project=project, args=[
-            'artifact', 'checkout', '--pull', build_elt,
+            'artifact', 'checkout', build_elt,
             '--directory', checkout_dir])
         result.assert_success()
 
@@ -1134,7 +1132,7 @@ def test_partial_checkout_fail(tmpdir, datafiles, cli):
         }})
 
         res = cli.run(project=project, args=[
-            'artifact', 'checkout', '--pull', build_elt, '--directory',
+            'artifact', 'checkout', build_elt, '--directory',
             checkout_dir])
         res.assert_main_error(ErrorDomain.STREAM, 'uncached-checkout-attempt')
         assert re.findall(r'Remote \((\S+)\) does not have artifact (\S+) cached', res.stderr)