laforge has submitted this change. (
https://gerrit.osmocom.org/c/pysim/+/42468?usp=email )
Change subject: pySim/EF.SMSP: add an additional de_encode test for EF_SMSP
......................................................................
pySim/EF.SMSP: add an additional de_encode test for EF_SMSP
Let's add another testvector where we test what happens when we populate
none of the fields except for the tp_sc_addr.
Related: SYS#7765
Change-Id: I12b600ab17d1acfdddaffe6006095acf1a4228c9
---
M pySim/ts_51_011.py
1 file changed, 10 insertions(+), 0 deletions(-)
Approvals:
laforge: Looks good to me, but someone else must approve
fixeria: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py
index 57cfc0c..86d0192 100644
--- a/pySim/ts_51_011.py
+++ b/pySim/ts_51_011.py
@@ -261,6 +261,16 @@
"numbering_plan_id":
"reserved_for_extension" },
"call_number": "" },
"tp_pid": b"\x00", "tp_dcs": b"\x00", "tp_vp_minutes": 1440 } ),
+ (
'fffffffffffffffffffffffffffffffffffffffffffffffffdffffffffffffffffffffffff07919403214365f7ffffffffffffff',
+ { "alpha_id": "", "parameter_indicators": { "tp_dest_addr": False,
"tp_sc_addr": True,
+ "tp_pid":
False, "tp_dcs": False, "tp_vp": False },
+ "tp_dest_addr": { "length": 255, "ton_npi": { "ext": True,
"type_of_number": "reserved_for_extension",
+ "numbering_plan_id":
"reserved_for_extension" },
+ "call_number": "" },
+ "tp_sc_addr": { "length": 7, "ton_npi": { "ext": True,
"type_of_number": "international",
+ "numbering_plan_id":
"isdn_e164" },
+ "call_number": "49301234567" },
+ "tp_pid": b"\xff", "tp_dcs": b"\xff", "tp_vp_minutes": 635040 } ),
]
_test_no_pad = True
class ValidityPeriodAdapter(Adapter):
--
To view, visit https://gerrit.osmocom.org/c/pysim/+/42468?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I12b600ab17d1acfdddaffe6006095acf1a4228c9
Gerrit-Change-Number: 42468
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
