laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/41736?usp=email )
Change subject: pySim.esim.saip.ProfileElementSequence: Update type annotations ......................................................................
pySim.esim.saip.ProfileElementSequence: Update type annotations
The type annotations didn't reflect reality in two cases.
Change-Id: Ib99c00a38bf009c63180b4a593d6cc796ff282d3 --- M pySim/esim/saip/__init__.py 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified
diff --git a/pySim/esim/saip/__init__.py b/pySim/esim/saip/__init__.py index c0f5340..5a71706 100644 --- a/pySim/esim/saip/__init__.py +++ b/pySim/esim/saip/__init__.py @@ -1789,7 +1789,7 @@ return None
@staticmethod - def peclass_for_path(path: Path) -> Optional[ProfileElement]: + def peclass_for_path(path: Path) -> Tuple[Optional[ProfileElement], Optional[templates.FileTemplate]]: """Return the ProfileElement class that can contain a file with given path.""" naa = ProfileElementSequence.naa_for_path(path) if naa: @@ -1822,7 +1822,7 @@ return ProfileElementTelecom, ft return ProfileElementGFM, None
- def pe_for_path(self, path: Path) -> Optional[ProfileElement]: + def pe_for_path(self, path: Path) -> Tuple[Optional[ProfileElement], Optional[templates.FileTemplate]]: """Return the ProfileElement instance that can contain a file with matching path. This will either be an existing PE within the sequence, or it will be a newly-allocated PE that is inserted into the sequence."""