<p>Holger Freyther has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/13076">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">resource: Introduce a base class for the modem<br><br>Extract IMSI, KI and authentication algorithm into a base class.<br><br>Change-Id: Id547cdcc241a307a2ea59b5fbac6b8d7a9d95639<br>---<br>M src/osmo_gsm_tester/modem.py<br>1 file changed, 21 insertions(+), 10 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester refs/changes/76/13076/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo_gsm_tester/modem.py b/src/osmo_gsm_tester/modem.py</span><br><span>index f9b827a..2f1b5a0 100644</span><br><span>--- a/src/osmo_gsm_tester/modem.py</span><br><span>+++ b/src/osmo_gsm_tester/modem.py</span><br><span>@@ -318,6 +318,22 @@</span><br><span> return self.property_is('Online', True)</span><br><span> </span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+class ModemBase(log.Origin, metaclass=ABCMeta):</span><br><span style="color: hsl(120, 100%, 40%);">+ """Base for everything about mobile/modem."""</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ def __init__(self, name, conf):</span><br><span style="color: hsl(120, 100%, 40%);">+ super().__init__(log.C_TST, name)</span><br><span style="color: hsl(120, 100%, 40%);">+ self._conf = conf</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ def imsi(self):</span><br><span style="color: hsl(120, 100%, 40%);">+ return self._conf.get('imsi')</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ def ki(self):</span><br><span style="color: hsl(120, 100%, 40%);">+ return self._conf.get('ki')</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ def auth_algo(self):</span><br><span style="color: hsl(120, 100%, 40%);">+ return self._conf.get('auth_algo', None)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> </span><br><span> class Modem(log.Origin):</span><br><span> 'convenience for ofono Modem interaction'</span><br><span>@@ -327,16 +343,14 @@</span><br><span> CTX_PROT_IPv46 = 'dual'</span><br><span> </span><br><span> def __init__(self, suite_run, conf):</span><br><span style="color: hsl(0, 100%, 40%);">- self.suite_run = suite_run</span><br><span style="color: hsl(0, 100%, 40%);">- self.conf = conf</span><br><span> self.syspath = conf.get('path')</span><br><span> self.dbuspath = get_dbuspath_from_syspath(self.syspath)</span><br><span style="color: hsl(0, 100%, 40%);">- super().__init__(log.C_TST, self.dbuspath)</span><br><span style="color: hsl(120, 100%, 40%);">+ super().__init__(self.dbuspath, conf)</span><br><span> self.dbg('creating from syspath %s' % self.syspath)</span><br><span> self.msisdn = None</span><br><span> self._ki = None</span><br><span> self._imsi = None</span><br><span style="color: hsl(0, 100%, 40%);">- self.run_dir = util.Dir(self.suite_run.get_test_run_dir().new_dir(self.name().strip('/')))</span><br><span style="color: hsl(120, 100%, 40%);">+ self.run_dir = util.Dir(suite_run.get_test_run_dir().new_dir(self.name().strip('/')))</span><br><span> self.sms_received_list = []</span><br><span> self.dbus = ModemDbusInteraction(self.dbuspath)</span><br><span> self.register_attempts = 0</span><br><span>@@ -423,7 +437,7 @@</span><br><span> self.dbg('got SIM properties', props)</span><br><span> self._imsi = props.get('SubscriberIdentity', None)</span><br><span> else:</span><br><span style="color: hsl(0, 100%, 40%);">- self._imsi = self.conf.get('imsi')</span><br><span style="color: hsl(120, 100%, 40%);">+ self._imsi = super().imsi()</span><br><span> if self._imsi is None:</span><br><span> raise log.Error('No IMSI')</span><br><span> return self._imsi</span><br><span>@@ -434,13 +448,10 @@</span><br><span> def ki(self):</span><br><span> if self._ki is not None:</span><br><span> return self._ki</span><br><span style="color: hsl(0, 100%, 40%);">- return self.conf.get('ki')</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">- def auth_algo(self):</span><br><span style="color: hsl(0, 100%, 40%);">- return self.conf.get('auth_algo', None)</span><br><span style="color: hsl(120, 100%, 40%);">+ return super().ki()</span><br><span> </span><br><span> def features(self):</span><br><span style="color: hsl(0, 100%, 40%);">- return self.conf.get('features', [])</span><br><span style="color: hsl(120, 100%, 40%);">+ return self._conf.get('features', [])</span><br><span> </span><br><span> def _required_ifaces(self):</span><br><span> req_ifaces = (I_NETREG,)</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/13076">change 13076</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/13076"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: osmo-gsm-tester </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Id547cdcc241a307a2ea59b5fbac6b8d7a9d95639 </div>
<div style="display:none"> Gerrit-Change-Number: 13076 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Holger Freyther <holger@freyther.de> </div>