laforge has submitted this change. (
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41032?usp=email )
Change subject: s1ap_proxy_test: do more s1ap_proxy:fetch_erab/2
......................................................................
s1ap_proxy_test: do more s1ap_proxy:fetch_erab/2
Just like we do in test_e_rab_setup[_dup], let's test fetching
E-RAB info by UID in test_initial_context_setup[_dup] for the
sake of completeness and consistency.
Change-Id: I8e9ffc22be3bc352a8d3022b1c7b0c50699ff493
---
M test/s1ap_proxy_test.erl
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/test/s1ap_proxy_test.erl b/test/s1ap_proxy_test.erl
index c0ada57..97e01b6 100644
--- a/test/s1ap_proxy_test.erl
+++ b/test/s1ap_proxy_test.erl
@@ -379,6 +379,7 @@
?_assertMetricENB(?S1GW_CTR_S1AP_PROXY_OUT_PKT_FWD_ALL, 2 + 2),
?_assertMetricENB(?S1GW_CTR_S1AP_PROXY_OUT_PKT_FWD_PROC, 2),
?_assertMetricENB(?S1GW_CTR_S1AP_PROXY_OUT_PKT_FWD_UNMODIFIED, 2),
+ ?_assertMatch({ok, _}, s1ap_proxy:fetch_erab(Pid, {1, 5})),
?_assertMatch([_], s1ap_proxy:fetch_erab_list(Pid))].
@@ -392,6 +393,7 @@
?_assertMatch({forward, _}, s1ap_proxy:process_pdu(Pid,
InitCtxSetupRspIn)),
%% duplicate INITIAL CONTEXT SETUP REQUEST results in the PDU being
dropped
?_assertEqual({drop, InitCtxSetupReqIn}, s1ap_proxy:process_pdu(Pid,
InitCtxSetupReqIn)),
+ ?_assertMatch({ok, _}, s1ap_proxy:fetch_erab(Pid, {1, 5})),
?_assertMatch([_], s1ap_proxy:fetch_erab_list(Pid))].
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41032?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I8e9ffc22be3bc352a8d3022b1c7b0c50699ff493
Gerrit-Change-Number: 41032
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
