laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/pysim/+/33452 )


Change subject: ts_31_103: enable encode tests for files containing sigle TLV IE
......................................................................

ts_31_103: enable encode tests for files containing sigle TLV IE

Now that we have fixed OS#6073 in the previous commit, we can enable
the so-far disabled encoder tests for EF.{DOMAIN,IMPU,IMPI} and
remove associated FIXMEs.

Change-Id: I79bfc5b77122907d6cc2f75605f9331b5e650286
---
M pySim/ts_31_103.py
1 file changed, 16 insertions(+), 6 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/52/33452/1

diff --git a/pySim/ts_31_103.py b/pySim/ts_31_103.py
index 4a66c0d..f1c8dd7 100644
--- a/pySim/ts_31_103.py
+++ b/pySim/ts_31_103.py
@@ -80,8 +80,7 @@

 # TS 31.103 Section 4.2.2
 class EF_IMPI(TransparentEF):
-    # FIXME: re-encode fails with "string encoding failed, expected unicode 
string"
-    _test_decode = [
+    _test_de_encode = [
         ( 
'803137333830303630303030303031303140696d732e6d6e633030302e6d63633733382e336770706e6574776f726b2e6f7267',
           { "nai": "[email protected]" } ),
     ]
@@ -94,8 +93,7 @@

 # TS 31.103 Section 4.2.3
 class EF_DOMAIN(TransparentEF):
-    # FIXME: re-encode fails with "string encoding failed, expected unicode 
string"
-    _test_decode = [
+    _test_de_encode = [
         ( 
'8021696d732e6d6e633030302e6d63633733382e336770706e6574776f726b2e6f7267',
           { "domain": "ims.mnc000.mcc738.3gppnetwork.org" } ),
     ]
@@ -108,8 +106,7 @@

 # TS 31.103 Section 4.2.4
 class EF_IMPU(LinFixedEF):
-    # FIXME: re-encode fails with "string encoding failed, expected unicode 
string"
-    _test_decode = [
+    _test_de_encode = [
         ( 
'80357369703a37333830303630303030303031303140696d732e6d6e633030302e6d63633733382e336770706e6574776f726b2e6f7267',
           { "impu": "sip:[email protected]" } 
),
     ]

--
To view, visit https://gerrit.osmocom.org/c/pysim/+/33452
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I79bfc5b77122907d6cc2f75605f9331b5e650286
Gerrit-Change-Number: 33452
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-MessageType: newchange

Reply via email to