Pau Espin Pedrol has uploaded this change for review. (
https://gerrit.osmocom.org/14240
Change subject: ggsn: Add tests to verify retransmit queue
......................................................................
ggsn: Add tests to verify retransmit queue
Change-Id: I8d937d0af9af8462418b174a54022655f9d77a5f
---
M ggsn_tests/GGSN_Tests.ttcn
M ggsn_tests/expected-results.xml
2 files changed, 35 insertions(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks
refs/changes/40/14240/1
diff --git a/ggsn_tests/GGSN_Tests.ttcn b/ggsn_tests/GGSN_Tests.ttcn
index 9577a60..dc9d07c 100644
--- a/ggsn_tests/GGSN_Tests.ttcn
+++ b/ggsn_tests/GGSN_Tests.ttcn
@@ -300,16 +300,25 @@
T_default.stop;
}
- function f_pdp_ctx_del(PdpContext ctx, template BIT1 teardown_ind) runs
on GT_CT {
+ function f_pdp_ctx_del(PdpContext ctx, template BIT1 teardown_ind, OCT1
expect_causevalue := '80'O) runs on GT_CT {
var Gtp1cUnitdata ud;
var default d;
+ var OCT4 expect_teid;
+
+ /* 3GPP TS 29.060 sec 7.3.6 specifies TEID used in response
+ message with cause value "Non existent" shall be zero. */
+ if (expect_causevalue == 'C0'O) {
+ expect_teid := '00000000'O;
+ } else {
+ expect_teid := ctx.teic;
+ }
f_send_gtpc(ts_GTPC_DeletePDP(g_peer_c, g_c_seq_nr,
ctx.teic_remote, ctx.nsapi, teardown_ind));
T_default.start;
d := activate(pingpong());
alt {
- [] GTPC.receive(tr_GTPC_MsgType(g_peer_c,
deletePDPContextResponse, ctx.teic)) -> value ud {
- if
(ud.gtpc.gtpc_pdu.deletePDPContextResponse.cause.causevalue == '80'O) {
+ [] GTPC.receive(tr_GTPC_MsgType(g_peer_c,
deletePDPContextResponse, expect_teid)) -> value ud {
+ if
(ud.gtpc.gtpc_pdu.deletePDPContextResponse.cause.causevalue ==
expect_causevalue) {
setverdict(pass);
} else {
setverdict(fail);
@@ -1244,6 +1253,27 @@
f_pdp_ctx_del(ctxA, '1'B);
}
+ /* Send a duplicate echo req. osmo-ggsn maintains a queue for sent
+ responses (60 seconds): If same delete req is sent and duplicate is
+ detected, saved duplicate response should be sent back. */
+ testcase TC_act_deact_retrans_duplicate() runs on GT_CT {
+ f_init();
+ var PdpContext ctx := valueof(t_DefinePDP(f_rnd_imsi('26242'H),
'1234'O, c_ApnInternet, valueof(t_EuaIPv4Dyn)));
+ f_pdp_ctx_act(ctx);
+ f_pdp_ctx_del(ctx, '1'B);
+ /* leave some time in between to make sure retransmit response
queue keeps packets for a while */
+ f_sleep(1.0);
+ /* g_c_seq_nr was increased during f_pdp_ctx_del(), we want a
+ duplicate. If it was not a duplicate, osmo-ggsn would answer
+ with a failure since that PDP ctx was already deleted. */
+ g_c_seq_nr := g_c_seq_nr - 1;
+ f_pdp_ctx_del(ctx, '1'B);
+
+ /* Now send a new pdp ctx del (increased seqnum). It should
fail with cause "non-existent": */
+ var OCT1 cause_nonexistent := 'C0'O;
+ f_pdp_ctx_del(ctx, '1'B, cause_nonexistent);
+ }
+
control {
execute(TC_pdp4_act_deact());
execute(TC_pdp4_act_deact_ipcp());
@@ -1269,5 +1299,6 @@
execute(TC_pdp46_clients_interact());
execute(TC_echo_req_resp());
+ execute(TC_act_deact_retrans_duplicate());
}
}
diff --git a/ggsn_tests/expected-results.xml b/ggsn_tests/expected-results.xml
index 081325e..c2defea 100644
--- a/ggsn_tests/expected-results.xml
+++ b/ggsn_tests/expected-results.xml
@@ -19,4 +19,5 @@
<testcase classname='GGSN_Tests' name='TC_pdp6_act_deact_gtpu_access'
time='MASKED'/>
<testcase classname='GGSN_Tests' name='TC_pdp6_clients_interact'
time='MASKED'/>
<testcase classname='GGSN_Tests' name='TC_echo_req_resp' time='MASKED'/>
+ <testcase classname='GGSN_Tests' name='TC_act_deact_retrans_duplicate'
time='MASKED'/>
</testsuite>
--
To view, visit https://gerrit.osmocom.org/14240
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d937d0af9af8462418b174a54022655f9d77a5f
Gerrit-Change-Number: 14240
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol <[email protected]>
