Change in osmo-ttcn3-hacks[master]: hlr: stop on various failures

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

neels gerrit-no-reply at lists.osmocom.org
Mon Nov 25 20:08:02 UTC 2019


neels has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/16020 )

Change subject: hlr: stop on various failures
......................................................................

hlr: stop on various failures

I found some of the tests hard to analyse when geting failures, because they
don't stop the test on failure. Spread some 'mtc.stop' so that the test stops
at the failed message instead of carrying on.

Change-Id: I804aca84d0ccf4767a5c097cf6c882ccbd87c4e1
---
M hlr/HLR_Tests.ttcn
1 file changed, 15 insertions(+), 1 deletion(-)

Approvals:
  neels: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/hlr/HLR_Tests.ttcn b/hlr/HLR_Tests.ttcn
index f309575..f8d7483 100644
--- a/hlr/HLR_Tests.ttcn
+++ b/hlr/HLR_Tests.ttcn
@@ -425,15 +425,19 @@
 		}
 	[exp_fail] GSUP.receive(tr_GSUP_UL_ERR(imsi, ?)) -> value ret {
 		setverdict(fail, "Unexpected UL ERROR Cause");
+		mtc.stop;
 		}
 	[exp_fail] GSUP.receive(tr_GSUP_UL_RES(imsi)) -> value ret {
 		setverdict(fail, "Unexpected UL.res for unknown IMSI");
+		mtc.stop;
 		}
 	[exp_fail] GSUP.receive(tr_GSUP_ISD_REQ(imsi)) -> value ret {
 		setverdict(fail, "Unexpected ISD.req in error case");
+		mtc.stop;
 		}
 	[not exp_fail] GSUP.receive(tr_GSUP_UL_ERR(imsi, ?)) -> value ret {
 		setverdict(fail, "Unexpected UL ERROR");
+		mtc.stop;
 		}
 	[not exp_fail and not isd_done] GSUP.receive(tr_GSUP_ISD_REQ(imsi, msisdn)) -> value ret {
 		GSUP.send(ts_GSUP_ISD_RES(imsi));
@@ -540,12 +544,14 @@
 	alt {
 	[] GSUP.receive(tr_GSUP_PROC_SS_ERR(imsi, sid, ?, ?)) -> value ret {
 		setverdict(fail, "Unexpected PROC_SS ERROR Cause");
+		mtc.stop;
 		}
 	[not exp_ss] GSUP.receive(tr_GSUP_PROC_SS_RES(imsi, sid, state, omit)) -> value ret {
 		setverdict(pass);
 		}
 	[exp_ss] GSUP.receive(tr_GSUP_PROC_SS_RES(imsi, sid, state, omit)) -> value ret {
 		setverdict(fail, "Unexpected PROC_SS.res without SS IE");
+		mtc.stop;
 		}
 /*
 	[exp_ss] GSUP.receive(tr_GSUP_PROC_SS_RES(imsi, sid, state, decmatch facility)) -> value ret {
@@ -562,9 +568,13 @@
 			setverdict(pass);
 		} else {
 			setverdict(fail, "Unexpected PROC_SS.res with non-matching facility IE");
+			mtc.stop;
 		}
 		}
-	[] GSUP.receive { repeat; }
+	[] GSUP.receive {
+		setverdict(fail, "Unexpected GSUP");
+		mtc.stop;
+		}
 	[] T.timeout {
 		setverdict(fail, "Timeout waiting for PROC_SS response");
 		self.stop;
@@ -593,18 +603,22 @@
 		}
 	[exp_fail] GSUP.receive(tr_GSUP_CHECK_IMEI_ERR(imsi, ?)) -> value pdu {
 		setverdict(fail, "Unexpected CHECK IMEI ERROR Cause: ", pdu);
+		mtc.stop;
 		}
 	[exp_fail] GSUP.receive(tr_GSUP_CHECK_IMEI_RES(imsi, ?)) -> value pdu {
 		setverdict(fail, "Unexpected CHECK IMEI RES instead of ERR");
+		mtc.stop;
 		}
 	[not exp_fail] GSUP.receive(tr_GSUP_CHECK_IMEI_ERR(imsi, ?)) -> value pdu {
 		setverdict(fail, "Unexpected CHECK IMEI ERROR");
+		mtc.stop;
 		}
 	[not exp_fail] GSUP.receive(tr_GSUP_CHECK_IMEI_RES(imsi, result)) -> value pdu {
 		setverdict(pass);
 		}
 	[not exp_fail] GSUP.receive(tr_GSUP_CHECK_IMEI_RES(imsi, ?)) -> value pdu {
 		setverdict(fail, "Unexpected CHECK IMEI RES");
+		mtc.stop;
 		}
 	[] GSUP.receive { repeat; }
 	[] T.timeout {

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/16020
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: I804aca84d0ccf4767a5c097cf6c882ccbd87c4e1
Gerrit-Change-Number: 16020
Gerrit-PatchSet: 2
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191125/a6aedc93/attachment.htm>


More information about the gerrit-log mailing list