Remove unnecesary validations

Signed-off-by: Pascal Hernandez <pascal.hernandez@nordicsemi.no>

Author: pascal-nordic

src/nrfcredstore/cli.py

@@ -89,8 +89,6 @@ def exec_cmd(args, credstore):
         ct = CredType[args.type]
         if credstore.delete(args.tag, ct):
             print(f'{ct.name} in secure tag {args.tag} deleted')
-        else:
-            raise RuntimeError('delete failed')
     elif args.subcommand=='generate':
         credstore.keygen(args.tag, args.file, args.attributes)
         print(f'New private key generated in secure tag {args.tag}')

src/nrfcredstore/credstore.py

@@ -26,9 +26,6 @@ def __init__(self, tag: int, type: int, sha: str):
         self.type = CredType(type)
         self.sha = sha
 
-def is_ok(response_lines):
-    return not any("ERROR" in line for line in response_lines) 
-
 class CredStore:
     def __init__(self, at_client):
         self.at_client = at_client
@@ -80,7 +77,7 @@ def write(self, tag: int, type: CredType, file: io.TextIOBase):
         if type == CredType.ANY:
             raise ValueError
         cert = file.read().rstrip()
-        return is_ok(self.at_client.at_command(f'AT%CMNG=0,{tag},{type.value},"{cert}"'))
+        return self.at_client.at_command(f'AT%CMNG=0,{tag},{type.value},"{cert}"')
 
     def delete(self, tag: int, type: CredType):
         """Delete a credential from the modem
@@ -90,7 +87,7 @@ def delete(self, tag: int, type: CredType):
 
         if type == CredType.ANY:
             raise ValueError
-        return is_ok(self.at_client.at_command(f'AT%CMNG=3,{tag},{type.value}'))
+        return self.at_client.at_command(f'AT%CMNG=3,{tag},{type.value}')
 
     def keygen(self, tag: int, file: io.BufferedIOBase, attributes: str = ''):
         """Generate a new private key and return a certificate signing request in DER format"""