<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20923">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20923/1/pcu/PCU_Tests_SNS.ttcn">File pcu/PCU_Tests_SNS.ttcn:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20923/1/pcu/PCU_Tests_SNS.ttcn@290">Patch Set #1, Line 290:</a> <code style="font-family:monospace,monospace">          /</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">should we really ignore all other PDUs?  If all we care about is a BVC-RESET in the opposite direction, then we should only ignore those.  Ignoring "any" packet from the peer has the risk of masking all kinds of failures, exactly why we want to have a test.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe turn the "ignore_other_pdu" into a "template PDU_NS ignore_rx_pdu" parameter.  If it is "omit", we don't ignore anything.  And if it is some other template, then we ignore anything matching the template specified by the caller - which in this case would be just a BVC-RESET, if I understand corectly.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20923">change 20923</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20923"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Id681749d75073c1d50a4b0a2e86f0a2dd0955b45 </div>
<div style="display:none"> Gerrit-Change-Number: 20923 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 27 Oct 2020 07:48:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>