laforge submitted this change.

View Change

Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved
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(-)

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 change 41032. To unsubscribe, or for help writing mail filters, visit settings.

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 <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>