Attention is currently required from: daniel, fixeria, laforge, osmith.

pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41525?usp=email )

Change subject: stp: Introduce tests to validate inaccessible SP scenario ......................................................................

Patch Set 4:

(2 comments)

File stp/STP_Tests_IPA.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41525/comment/a9d2bcd3_06c59... : PS3, Line 655: f_sleep(1.0);

...

STP discards the message to be forwarded, since its destination is not accessible. […]

It's impossible the message gets anywhere because there's no conn1 up/connected where it can forward the message to. So for now this tests simply validates the osmo-stp doesn't crash unexpectedly, which is already good since I run into this issue.

File stp/STP_Tests_M3UA.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/41525/comment/03f8bdef_9621b... : PS3, Line 204: [] M3UA[idx].receive(t_M3UA_RecvFrom(?)) -> value rx

I am fine with the Hearbeats, it's actually the two `M3UA[idx]. […]

As mentioned, the 2 receives accomplish different purposes, so I want to keep them.