Remove useless status_list parameter as the object is de facto in the more general revocation dict parameter

Author: oligatorr

pymdoccbor/mdoc/issuer.py

@@ -74,7 +74,6 @@ def new(
         validity: dict = None,
         devicekeyinfo: Union[dict, CoseKey, str] = None,
         cert_path: str = None,
-        status_list: dict = None,
         revocation: dict = None
     ):
         """
@@ -85,8 +84,7 @@ def new(
         :param validity: dict: validity info
         :param devicekeyinfo: Union[dict, CoseKey, str]: device key info
         :param cert_path: str: path to the certificate
-        :param status_list: dict: The status list to include in the mso of the mdoc
-        :param revocation: dict: The status list to include in the mso of the mdoc
+        :param revocation: dict: revocation status dict it may include status_list and identifier_list keys
 
         :return: dict: signed mdoc
         """
@@ -148,7 +146,6 @@ def new(
                 alg=self.alg,
                 kid=self.kid,
                 validity=validity,
-                status_list=status_list,
                 revocation=revocation
             )
 
@@ -159,7 +156,6 @@ def new(
                 alg=self.alg,
                 cert_path=cert_path,
                 validity=validity,
-                status_list=status_list,
                 revocation=revocation
             )
 

pymdoccbor/mso/issuer.py

@@ -43,7 +43,6 @@ def __init__(
         hsm: bool = False,
         private_key: Union[dict, CoseKey] = None,
         digest_alg: str = settings.PYMDOC_HASHALG,
-        status_list: dict = None,
         revocation: dict = None
     ) -> None:
         """
@@ -61,8 +60,7 @@ def __init__(
         :param hsm: bool: hardware security module
         :param private_key: Union[dict, CoseKey]: the signing key
         :param digest_alg: str: the digest algorithm
-        :param status_list: dict: the status list to include in the mso
-        :param revocation: dict: the status list to include in the mso
+        :param revocation: dict: revocation status dict to include in the mso, it may include status_list and identifier_list keys
         """
 
         if not hsm:
@@ -97,7 +95,6 @@ def __init__(
         self.alg = alg
         self.kid = kid
         self.validity = validity
-        self.status_list = status_list
         self.revocation = revocation
 
         alg_map = {"ES256": "sha256", "ES384": "sha384", "ES512": "sha512"}
@@ -206,8 +203,6 @@ def sign(
             },
             "digestAlgorithm": alg_map.get(self.alg)
         }
-        if self.status_list is not None:
-            payload.update({"status": self.status_list})
         if self.revocation is not None:
             payload.update({"status": self.revocation})
 

pymdoccbor/tests/test_02_mdoc_issuer.py

@@ -10,7 +10,7 @@
 from pymdoccbor.mdoc.issuer import MdocCborIssuer
 from pymdoccbor.mdoc.verifier import MdocCbor
 from pymdoccbor.mso.issuer import MsoIssuer
-from . pid_data import PID_DATA
+from pymdoccbor.tests.pid_data import PID_DATA
 
 
 PKEY = {
@@ -75,7 +75,7 @@ def test_mdoc_issuer():
             data=PID_DATA,
             devicekeyinfo=PKEY,
             validity=validity,
-            status_list=status_list
+            revocation=status_list
         )
 
     mdocp = MdocCbor()