laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
ts_51_011: fix unittest for EF.ADN

The unittest for EF.ADN can run with _test_de_encode. However, the original
test vectors seem to be from a card with a slightly larger record size, so
they need a bit of re-alignment

Related: OS#5714
Change-Id: I241792e66ee6167be6ddc076453344b6307d6265
---
M pySim/ts_51_011.py
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/pySim/ts_51_011.py b/pySim/ts_51_011.py
index 1ef6bb6..be8926c 100644
--- a/pySim/ts_51_011.py
+++ b/pySim/ts_51_011.py
@@ -131,13 +131,13 @@

# TS 51.011 Section 10.5.1
class EF_ADN(LinFixedEF):
- _test_decode = [
- ( '42204841203120536963ffffffff06810628560810ffffffffffffff',
+ _test_de_encode = [
+ ( '42204841203120536963FFFFFFFFFFFF06810628560810FFFFFFFFFFFFFF',
{ "alpha_id": "B HA 1 Sic", "len_of_bcd": 6, "ton_npi": { "ext": True, "type_of_number":
"unknown", "numbering_plan_id":
"isdn_e164" }, "dialing_nr":
"6082658001", "cap_conf_id": 255, "ext1_record_id": 255 }),
- ( '4B756E64656E626574726575756E67FFFFFF0791947112122721ffffffffffff',
+ ( '4B756E64656E626574726575756E67FF0791947112122721ffffffffffff',
{"alpha_id": "Kundenbetreuung", "len_of_bcd": 7, "ton_npi": {"ext": True, "type_of_number":
"international",
"numbering_plan_id": "isdn_e164"},

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

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I241792e66ee6167be6ddc076453344b6307d6265
Gerrit-Change-Number: 38199
Gerrit-PatchSet: 8
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>