Add check based on text

Signed-off-by: Sayali Gaikawad <[email protected]>

src/sign_workflow/signer_jar.py

@@ -48,4 +48,8 @@ def sign(self, artifact: str, basepath: Path, signature_type: str) -> None:
 
     def verify(self, filename: str) -> None:
         verify_cmd = ["jarsigner", "-verify", filename, "-verbose", "-certs", "-strict"]
-        self.git_repo.execute(" ".join(verify_cmd))
+        signature = self.git_repo.output(" ".join(verify_cmd))
+        if signature.find('jar verified') == -1:
+            raise ValueError(f"Cannot verify the signature for {filename}")
+
+

tests/tests_sign_workflow/test_signer_jar.py

@@ -10,7 +10,7 @@
 import unittest
 from pathlib import Path
 from unittest.mock import MagicMock, Mock, call, patch
-
+from git.git_repository import GitRepository
 from sign_workflow.signer_jar import SignerJar
 
 
@@ -57,5 +57,5 @@ def test_sign_command_for_overwrite(self, mock_repo: Mock) -> None:
     @patch("sign_workflow.signer.GitRepository")
     def test_signer_verify(self, mock_repo: Mock) -> None:
         signer = SignerJar(True)
-        signer.verify("/path/the-jar.jar")
-        mock_repo.assert_has_calls([call().execute("jarsigner -verify /path/the-jar.jar -verbose -certs -strict")])
+        signer.verify('/path/the-jar.jar')
+        mock_repo.assert_has_calls([call().output("jarsigner -verify /path/the-jar.jar -verbose -certs -strict")])