test: add test for failed fetching of repos

We currently only test the good case of fetching repos, however we also
need to test the error path. We add a check for that as well by mocking
the fetch_cmd implementation of a GitRepo. We further need to remove the
commit, as otherwise the short-circuit implementation added in 021e961
skips the fetching.

Signed-off-by: Felix Moessbauer <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>

Author: fmoessbauer

tests/test_commands.py

@@ -31,6 +31,7 @@
 from kas import kas
 from kas.libkas import TaskExecError, KasUserError, run_cmd
 from kas.attestation import file_digest_slow
+from kas.repos import RepoFetchError
 
 
 @pytest.mark.dirsfromenv
@@ -82,6 +83,21 @@ def test_checkout(monkeykas, tmpdir):
     assert not (kas_bd / 'downloads').exists()
     assert not (kas_bd / 'sstate-cache').exists()
 
+    # test re-fetching (remove commit as nothing is fetched otherwise)
+    with open('test.yml', 'r') as f:
+        yml = yaml.safe_load(f)
+    with open('test-no-commit.yml', 'w') as f:
+        del yml['repos']['kas_1.1']['commit']
+        yml['repos']['kas_1.1']['branch'] = 'master'
+        yaml.safe_dump(yml, f)
+    kas.kas(['checkout', '--update', 'test-no-commit.yml'])
+
+    with monkeykas.context() as mc:
+        from kas.repos import GitRepo
+        mc.setattr(GitRepo, "fetch_cmd", lambda _: ["false"])
+        with pytest.raises(RepoFetchError):
+            kas.kas(['checkout', '--update', 'test-no-commit.yml'])
+
 
 def test_invalid_checkout(monkeykas, tmpdir, capsys):
     tdir = str(tmpdir / 'test_commands')