<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">It's a bit of a philosophy question. I typically try to not test<br>too many things in one patch.  For example, if the patch is about<br>testing paging load reporting, then that test should only test<br>whatever is required in that context, and not verify if paging<br>blocks are send on AGCH, as that's "out of scope".  It's sometimes<br>difficult to draw the line, and this is not a strict rule.  I'm<br>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<br>that it's typically better to have more specific test cases rather<br>than single test cases that test tons of thing in one.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks for the hint. I will keep this in mind for next time.</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-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 12 Sep 2018 15:59:17 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>