pespin submitted this change.

View Change


Approvals: osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified
mgw: TC_conn_timeout Fix timeout left set upon test exit

The test mentioned in a comment that the value is reset upon each test
start, but that's false.

Change-Id: I9447fa6c6b3103e05fa3ee756ee572ddecb69006
---
M mgw/MGCP_Test.ttcn
1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn
index 73591b2..0cedb0e 100644
--- a/mgw/MGCP_Test.ttcn
+++ b/mgw/MGCP_Test.ttcn
@@ -2562,7 +2562,7 @@

f_init(ep);
log("Setting conn-timeout to 1s");
- f_vty_config(MGWVTY, "mgcp", "conn-timeout 1"); /* reset in f_init_vty() */
+ f_vty_config(MGWVTY, "mgcp", "conn-timeout 1");

log("Sending RTP data for 1.5s");
flow := valueof(t_RtpFlow(mp_local_ipv4, mp_remote_ipv4, 111, "GSM-HR-08/8000/1"));
@@ -2587,6 +2587,9 @@
f_sleep(0.2);
f_rtpem_conn_refuse_verify(RTPEM[0]);

+ log("Setting conn-timeout back to 0 (disabled)");
+ f_vty_config(MGWVTY, "mgcp", "conn-timeout 0");
+
setverdict(pass);
}


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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I9447fa6c6b3103e05fa3ee756ee572ddecb69006
Gerrit-Change-Number: 34549
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged