<p style="white-space: pre-wrap; word-wrap: break-word;">While I agree that tests should be added at some point, the truth is that there's not much that can become tested (and hence broken) yet, since the PMM state is not really used for much nowadays (it still needs to be further developed and code added).</p><p style="white-space: pre-wrap; word-wrap: break-word;">So in a few works, imho it's impossible it becomes more broken than it was, and now we have a more solid code base to start implementing/fixing related bits...</p><p><a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15354">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-sgsn/+/15354">change 15354</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-sgsn/+/15354"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4c9cf8c27194817c56e8949af0205e1cc14af317 </div>
<div style="display:none"> Gerrit-Change-Number: 15354 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 02 Sep 2019 10:18:42 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>