<p>fixeria <strong>merged</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmocom-bb/+/14572">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, but someone else must approve
  fixeria: Looks good to me, approved
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">trx_toolkit/data_msg.py: tests: use random reference data<br><br>Having fn = 1024 and tn = 0 in all tests decreases the chances<br>to spot encoding / decoding bugs of higher or lower values.<br>Let's randomize the reference data before all the tests.<br><br>Change-Id: Id3c5be9faaf0bef727b975c7182098af0cec6e71<br>---<br>M src/target/trx_toolkit/data_msg.py<br>1 file changed, 14 insertions(+), 27 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/target/trx_toolkit/data_msg.py b/src/target/trx_toolkit/data_msg.py</span><br><span>index da109d9..5cfb745 100644</span><br><span>--- a/src/target/trx_toolkit/data_msg.py</span><br><span>+++ b/src/target/trx_toolkit/data_msg.py</span><br><span>@@ -432,10 +432,6 @@</span><br><span>    log.basicConfig(level = log.DEBUG,</span><br><span>           format = "[%(levelname)s] %(filename)s:%(lineno)d %(message)s")</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   # Common reference data</span><br><span style="color: hsl(0, 100%, 40%);">- fn = 1024</span><br><span style="color: hsl(0, 100%, 40%);">-       tn = 0</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>       # Generate two random bursts</span><br><span>         burst_l12trx_ref = []</span><br><span>        burst_trx2l1_ref = []</span><br><span>@@ -450,17 +446,18 @@</span><br><span>        log.info("Generating the reference messages")</span><br><span> </span><br><span>  # Create messages of both types</span><br><span style="color: hsl(0, 100%, 40%);">- msg_l12trx_ref = DATAMSG_L12TRX(fn = fn, tn = tn)</span><br><span style="color: hsl(0, 100%, 40%);">-       msg_trx2l1_ref = DATAMSG_TRX2L1(fn = fn, tn = tn)</span><br><span style="color: hsl(120, 100%, 40%);">+     msg_l12trx_ref = DATAMSG_L12TRX(burst = burst_l12trx_ref)</span><br><span style="color: hsl(120, 100%, 40%);">+     msg_trx2l1_ref = DATAMSG_TRX2L1(burst = burst_trx2l1_ref)</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   # Fill in message specific fields</span><br><span style="color: hsl(0, 100%, 40%);">-       msg_trx2l1_ref.rssi = -88</span><br><span style="color: hsl(0, 100%, 40%);">-       msg_l12trx_ref.pwr = 0x33</span><br><span style="color: hsl(0, 100%, 40%);">-       msg_trx2l1_ref.toa256 = -256</span><br><span style="color: hsl(120, 100%, 40%);">+  # Validate header randomization</span><br><span style="color: hsl(120, 100%, 40%);">+       for i in range(0, 100):</span><br><span style="color: hsl(120, 100%, 40%);">+               msg_l12trx_ref.rand_hdr()</span><br><span style="color: hsl(120, 100%, 40%);">+             msg_trx2l1_ref.rand_hdr()</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   # Specify the reference bursts</span><br><span style="color: hsl(0, 100%, 40%);">-  msg_l12trx_ref.burst = burst_l12trx_ref</span><br><span style="color: hsl(0, 100%, 40%);">- msg_trx2l1_ref.burst = burst_trx2l1_ref</span><br><span style="color: hsl(120, 100%, 40%);">+               assert(msg_l12trx_ref.validate())</span><br><span style="color: hsl(120, 100%, 40%);">+             assert(msg_trx2l1_ref.validate())</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   log.info("Validate header randomization: OK")</span><br><span> </span><br><span>  log.info("Encoding the reference messages")</span><br><span> </span><br><span>@@ -493,10 +490,10 @@</span><br><span>    log.info("Compare bursts: OK")</span><br><span> </span><br><span>         # Compare both parsed messages with the reference data</span><br><span style="color: hsl(0, 100%, 40%);">-  assert(msg_l12trx_dec.fn == fn)</span><br><span style="color: hsl(0, 100%, 40%);">- assert(msg_trx2l1_dec.fn == fn)</span><br><span style="color: hsl(0, 100%, 40%);">- assert(msg_l12trx_dec.tn == tn)</span><br><span style="color: hsl(0, 100%, 40%);">- assert(msg_trx2l1_dec.tn == tn)</span><br><span style="color: hsl(120, 100%, 40%);">+       assert(msg_l12trx_dec.fn == msg_l12trx_ref.fn)</span><br><span style="color: hsl(120, 100%, 40%);">+        assert(msg_trx2l1_dec.fn == msg_trx2l1_ref.fn)</span><br><span style="color: hsl(120, 100%, 40%);">+        assert(msg_l12trx_dec.tn == msg_l12trx_ref.tn)</span><br><span style="color: hsl(120, 100%, 40%);">+        assert(msg_trx2l1_dec.tn == msg_trx2l1_ref.tn)</span><br><span> </span><br><span>   log.info("Compare FN / TN: OK")</span><br><span> </span><br><span>@@ -507,16 +504,6 @@</span><br><span> </span><br><span>     log.info("Compare message specific data: OK")</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-     # Validate header randomization</span><br><span style="color: hsl(0, 100%, 40%);">- for i in range(0, 100):</span><br><span style="color: hsl(0, 100%, 40%);">-         msg_l12trx_ref.rand_hdr()</span><br><span style="color: hsl(0, 100%, 40%);">-               msg_trx2l1_ref.rand_hdr()</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-               assert(msg_l12trx_ref.validate())</span><br><span style="color: hsl(0, 100%, 40%);">-               assert(msg_trx2l1_ref.validate())</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-       log.info("Validate header randomization: OK")</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span>      # Bit conversation test</span><br><span>      usbits_ref = list(range(0, 256))</span><br><span>     sbits_ref = list(range(-127, 128))</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmocom-bb/+/14572">change 14572</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/osmocom-bb/+/14572"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmocom-bb </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Id3c5be9faaf0bef727b975c7182098af0cec6e71 </div>
<div style="display:none"> Gerrit-Change-Number: 14572 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>