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))].