Vadim Yanitskiy has uploaded this change for review. ( 
https://gerrit.osmocom.org/12382


Change subject: tests/gsup_test.c: drop session IEs from MO-ForwardSM Error
......................................................................

tests/gsup_test.c: drop session IEs from MO-ForwardSM Error

Both session state and session ID IEs were left from the initial
version of Ibe325c64ae2d6c626b232533bb4cbc65fc2b5d71. There is
no need to use them (as we use SM-RP-MR), so let's claen up.

Change-Id: I0d910b87f15ffbc0aeeca9cb4fcbef32bdf3ef88
---
M tests/gsup/gsup_test.c
M tests/gsup/gsup_test.err
2 files changed, 4 insertions(+), 10 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/82/12382/1

diff --git a/tests/gsup/gsup_test.c b/tests/gsup/gsup_test.c
index b3a8c77..6c5d351 100644
--- a/tests/gsup/gsup_test.c
+++ b/tests/gsup/gsup_test.c
@@ -256,12 +256,6 @@
                0x25, /* OSMO_GSUP_MSGT_MO_FORWARD_SM_ERROR */
                TEST_IMSI_IE,

-               /* Session related IEs */
-               0x30, 0x04, /* Session ID */
-                       0xde, 0xad, 0xbe, 0xef,
-               0x31, 0x01, /* Session state (END) */
-                       0x03,
-
                /* SM related IEs */
                0x40, 0x01, /* SM-RP-MR (Message Reference) */
                        0xfa,
diff --git a/tests/gsup/gsup_test.err b/tests/gsup/gsup_test.err
index 236c38a..c81755e 100644
--- a/tests/gsup/gsup_test.err
+++ b/tests/gsup/gsup_test.err
@@ -58,8 +58,8 @@
   generated message: 2a 01 08 21 43 65 87 09 21 43 f5 40 01 fa 43 04 de ad be 
ef
   original message:  2a 01 08 21 43 65 87 09 21 43 f5 40 01 fa 43 04 de ad be 
ef
   IMSI:              123456789012345
-  generated message: 25 01 08 21 43 65 87 09 21 43 f5 30 04 de ad be ef 31 01 
03 40 01 fa 44 01 af
-  original message:  25 01 08 21 43 65 87 09 21 43 f5 30 04 de ad be ef 31 01 
03 40 01 fa 44 01 af
+  generated message: 25 01 08 21 43 65 87 09 21 43 f5 40 01 fa 44 01 af
+  original message:  25 01 08 21 43 65 87 09 21 43 f5 40 01 fa 44 01 af
   IMSI:              123456789012345
   generated message: 2c 01 08 21 43 65 87 09 21 43 f5 46 01 02
   original message:  2c 01 08 21 43 65 87 09 21 43 f5 46 01 02
@@ -84,7 +84,7 @@
   message 17: tested 33 truncations, 29 parse failures
   message 18: tested 44 truncations, 39 parse failures
   message 19: tested 20 truncations, 18 parse failures
-  message 20: tested 26 truncations, 22 parse failures
+  message 20: tested 17 truncations, 15 parse failures
   message 21: tested 14 truncations, 13 parse failures
 DLGSUP Stopping DLGSUP logging
   message 0: tested 2816 modifications, 510 parse failures
@@ -107,5 +107,5 @@
   message 17: tested 8448 modifications, 2053 parse failures
   message 18: tested 11264 modifications, 2307 parse failures
   message 19: tested 5120 modifications, 1031 parse failures
-  message 20: tested 6656 modifications, 1546 parse failures
+  message 20: tested 4352 modifications, 1030 parse failures
   message 21: tested 3584 modifications, 771 parse failures

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0d910b87f15ffbc0aeeca9cb4fcbef32bdf3ef88
Gerrit-Change-Number: 12382
Gerrit-PatchSet: 1
Gerrit-Owner: Vadim Yanitskiy <[email protected]>

Reply via email to