<p style="white-space: pre-wrap; word-wrap: break-word;">It's a bit of a philosophy question. I typically try to not test too many things in one patch.  For example, if the patch is about testing paging load reporting, then that test should only test whatever is required in that context, and not verify if paging blocks are send on AGCH, as that's "out of scope".  It's sometimes difficult to draw the line, and this is not a strict rule.  I'm just explaining my rationale why I didn't add such checks before.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But we can merge this one.  Just in the future, keep it in mind that it's typically better to have more specific test cases rather than single test cases that test tons of thing in one.</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/10726">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10726">change 10726</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/10726"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I839e75ece05166518bf7132acd3017434b3d3bc2 </div>
<div style="display:none"> Gerrit-Change-Number: 10726 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 31 Aug 2018 16:31:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>