laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
tests: Add ProfileElementSD and ProfileElementSSD to test_constructor_encode

Change-Id: Idc6f37b487dfa8a69ac7a50a537cfc317113d501
---
M tests/test_esim_saip.py
1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/tests/test_esim_saip.py b/tests/test_esim_saip.py
index 704f513..4c1b936 100755
--- a/tests/test_esim_saip.py
+++ b/tests/test_esim_saip.py
@@ -66,7 +66,8 @@
def test_constructor_encode(self):
"""Test that DER-encoding of PE created by "empty" constructor works without raising exception."""
for cls in [ProfileElementMF, ProfileElementPuk, ProfileElementPin, ProfileElementTelecom,
- ProfileElementUSIM, ProfileElementISIM, ProfileElementAKA]:
+ ProfileElementUSIM, ProfileElementISIM, ProfileElementAKA, ProfileElementSD,
+ ProfileElementSSD]:
with self.subTest(cls.__name__):
pes = ProfileElementSequence()
inst = cls()

To view, visit change 37418. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: Idc6f37b487dfa8a69ac7a50a537cfc317113d501
Gerrit-Change-Number: 37418
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged