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

Change subject: BSC_Tests: fix TC_mgwpool_keepalive_down_not_used
......................................................................

BSC_Tests: fix TC_mgwpool_keepalive_down_not_used

Current osmo-bsc master does not support the MGW keepalive yet, so
f_vty_mgw_cfg_keepalive() fails.  This leaves osmo-bsc in unclean
state and makes all LCLS testcases fail.  The problem is that before
calling it we also call f_vty_mgw_enable() and f_vty_mgw_block(),
but not calling their counterparts.

Rearrange the testcase to call f_vty_mgw_cfg_keepalive() first,
so that we fail early before calling f_vty_mgw_{enable,block}().

Change-Id: I6a94c441fe80a92c237c3c4a5481f2dac3376e35
Fixes: bd59842b6 "bsc: Introduce test TC_mgwpool_keepalive"
---
M bsc/BSC_Tests.ttcn
1 file changed, 20 insertions(+), 1 deletion(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, approved




diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index ba0858c..a9cfa03 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -12307,9 +12307,9 @@
        f_init(1, true, nr_mgw := 2);
        f_sleep(1.0);

+       f_vty_mgw_cfg_keepalive(0, req_intval := 1, req_endp := "null", 
rx_timeout := 3);
        f_vty_mgw_enable(1);
        f_vty_mgw_block(1, false);
-       f_vty_mgw_cfg_keepalive(0, req_intval := 1, req_endp := "null", 
rx_timeout := 3);

        vc_conn1 := f_start_handler_create(pars1);
        vc_conn2 := f_start_handler_create(pars2);

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/33376
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: I6a94c441fe80a92c237c3c4a5481f2dac3376e35
Gerrit-Change-Number: 33376
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to