lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39011?usp=email ) Change subject: PCU_Tests_SNS: sns_del: check if removed NS-VC still transmits ...................................................................... PCU_Tests_SNS: sns_del: check if removed NS-VC still transmits After removing a NS-VC via SNS-DEL, the NS-VC shouldn't receive any further NS PDUs. Related: OS#6611 Change-Id: Ied4367a519cf75291ff8766c9efebb0f8a12b11f --- M pcu/PCU_Tests_SNS.ttcn 1 file changed, 24 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/11/39011/1 diff --git a/pcu/PCU_Tests_SNS.ttcn b/pcu/PCU_Tests_SNS.ttcn index 263e205..5464374 100644 --- a/pcu/PCU_Tests_SNS.ttcn +++ b/pcu/PCU_Tests_SNS.ttcn @@ -324,13 +324,33 @@ } /* Test deleting IP endpoints at runtime */ -testcase TC_sns_del() runs on RAW_Test_CT { +testcase TC_sns_del(integer idx := 0, integer idx_del := 1, float tout := 20.0) runs on RAW_Test_CT { f_sns_bringup_1c1u(); g_handle_rx_alive := true; - f_outgoing_sns_del(idx_del := 1, w_sig := 0, w_user := 1, idx := 0); - /* FIXME: ensure we don't receive anything on just-deleted NS-VC anymore */ - setverdict(pass); + f_outgoing_sns_del(idx_del := idx_del, w_sig := 0, w_user := 1, idx := idx); + g_handle_rx_alive := false; + + /* A small grace period to prevent race conditions */ + timer Tgrace := 1.0; + Tgrace.start; + alt { + [] NSCP[idx_del].receive(t_NS_ALIVE) {} + [] Tgrace.timeout {} + } + + /* Fail on any NS PDUs */ + timer T := tout; + T.start; + alt { + [] ax_rx_fail_on_any_ns(idx_del) { + setverdict(fail, "Received unexpected NS-ALIVE ACK"); + } + [] T.timeout { + setverdict(pass); + } + } + f_clean_ns_codec(); } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/39011?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Ied4367a519cf75291ff8766c9efebb0f8a12b11f Gerrit-Change-Number: 39011 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>