fixeria submitted this change.

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

Objections:
  laforge: I would prefer this is not merged as is

BTS_Tests: f_est_rll_mo(): add waiting timeout (2s)

This helps to find where the problem is a lot quicker.

Change-Id: I71ab69d85f453964749270d970c55e6f577a73a1
---
M bts/BTS_Tests.ttcn
1 file changed, 5 insertions(+), 0 deletions(-)

diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index 0f55c54..6772cc5 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -7061,9 +7061,11 @@
 
 /* establish one Radio Link Layer via SABM -> UA. Use l3 for contention resolution */
 private function f_est_rll_mo(uint3_t sapi, RslLinkId link_id, octetstring l3) runs on ConnHdlr {
+	timer T := 2.0;
 	/* send SABM from MS -> BTS */
 	f_tx_lapdm(ts_LAPDm_SABM(sapi, cr_MO_CMD, true, l3), link_id);
 	/* expect RLL EST IND on Abis */
+	T.start;
 	alt {
 	[l3 != ''O] RSL.receive(tr_RSL_EST_IND(g_chan_nr, link_id, l3));
 	[l3 == ''O] RSL.receive(tr_RSL_EST_IND_NOL3(g_chan_nr, link_id));
@@ -7071,6 +7073,9 @@
 		Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, "Failing due to RSL_ERROR_IND");
 		}
 	[] RSL.receive { repeat; }
+	[] T.timeout {
+		Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail, "Timeout waiting for RLL EST IND");
+		}
 	}
 	/* expect UA from BTS -> MS */
 	f_l1_exp_lapdm(tr_LAPDm_UA(sapi, cr_MT_RSP, true, l3));

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