Attention is currently required from: pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42493?usp=email )
Change subject: s1gw: f_ConnHdlr_pfcp_assoc_handler(): handle already-connected case ......................................................................
Patch Set 1:
(1 comment)
File s1gw/S1GW_Tests.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/42493/comment/67c67863_6d9b4... : PS1, Line 143: vc_conn := f_ConnHdlr_spawn(refers(f_ConnHdlr_pfcp_assoc_handler),
I wonder why do you need to spawn a new ConnHdlr to do some non-session related stuff. […]
`test_CT` is not connected to the PFCPEM, so they cannot communicate. Only `ConnHdlr` components can talk to the PFCPEM. This is why I am spawning a `ConnHdlr` here.
I could connect `test_CT` to the PFCPEM, but this would be the only reason to do that: we do not run test logic directly in `test_CT`. Also, a `ConnHdlr` can re-use the existing API to send and expect PFCP PDUs, so this is more convenient.