Change in osmo-ttcn3-hacks[master]: MSC/SMPP: fix RP-ACK expectations in TC_smpp_mo_sms

[fixeria]

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/16883 )


Change subject: MSC/SMPP: fix RP-ACK expectations in TC_smpp_mo_sms
......................................................................

MSC/SMPP: fix RP-ACK expectations in TC_smpp_mo_sms

The MSC shall not send RP-ACK before the response from ESME.

Change-Id: Ide1376cae8e75412039b7dc9f0b8bb390eab2280
Related: OS#3880
---
M msc/MSC_Tests.ttcn
1 file changed, 6 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/83/16883/1

diff --git a/msc/MSC_Tests.ttcn b/msc/MSC_Tests.ttcn
index 63cf2b1..9b015b1 100644
--- a/msc/MSC_Tests.ttcn
+++ b/msc/MSC_Tests.ttcn
@@ -2063,9 +2063,10 @@
 
 	/* Perform location update so IMSI is known + registered in MSC/VLR */
 	f_perform_lu();
-	f_establish_fully(EST_TYPE_MO_SMS);
 
-	f_mo_sms(spars);
+	/* MS/UE submits a MO SMS */
+	f_establish_fully(EST_TYPE_MO_SMS);
+	f_mo_sms_submit(spars);
 
 	var SMPP_PDU smpp;
 	var template SMPP_PDU tr_smpp := tr_SMPP(c_SMPP_command_id_deliver_sm, ESME_ROK);
@@ -2103,6 +2104,9 @@
 	[] SMPP.receive(tr_SMPP(c_SMPP_command_id_alert_notification, ESME_ROK)) { repeat; }
 	}
 
+	/* MSC terminates the SMS transaction with RP-ACK */
+	f_mo_sms_wait_rp_ack(spars);
+
 	f_expect_clear();
 }
 testcase TC_smpp_mo_sms() runs on MTC_CT {

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/16883
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ide1376cae8e75412039b7dc9f0b8bb390eab2280
Gerrit-Change-Number: 16883
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200115/f6746e61/attachment.htm>