<p>pespin has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/21162">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">tests: Fix tdef_test on 32bit platforms<br><br>100 minutes = 6000000000 microseconds was too big to be stored in an<br>unsigned long in a 32bit platform, making the test print 4294967295<br>instead. Let's set a smaller value to have the test happy on 32 bits.<br><br>Change-Id: Ic0d009f00a69cee59f2d3fc0b40ecdc97d81c75c<br>---<br>M tests/tdef/tdef_test.c<br>M tests/tdef/tdef_test.ok<br>2 files changed, 9 insertions(+), 9 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/62/21162/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/tdef/tdef_test.c b/tests/tdef/tdef_test.c</span><br><span>index d874231..ede3082 100644</span><br><span>--- a/tests/tdef/tdef_test.c</span><br><span>+++ b/tests/tdef/tdef_test.c</span><br><span>@@ -38,7 +38,7 @@</span><br><span> static struct osmo_tdef tdefs[] = {</span><br><span>  { .T=1, .default_val=100, .desc="100s" },</span><br><span>  { .T=2, .default_val=100, .unit=OSMO_TDEF_MS, .desc="100ms" },</span><br><span style="color: hsl(0, 100%, 40%);">-        { .T=3, .default_val=100, .unit=OSMO_TDEF_M, .desc="100m" },</span><br><span style="color: hsl(120, 100%, 40%);">+        { .T=3, .default_val=50, .unit=OSMO_TDEF_M, .desc="50m" },</span><br><span>         { .T=4, .default_val=100, .unit=OSMO_TDEF_CUSTOM, .desc="100 potatoes" },</span><br><span> </span><br><span>      { .T=7, .default_val=50, .desc="Water Boiling Timeout", .min_val=20, .max_val=800 },  // default is .unit=OSMO_TDEF_S == 0</span><br><span>diff --git a/tests/tdef/tdef_test.ok b/tests/tdef/tdef_test.ok</span><br><span>index 1770968..4c97dab 100644</span><br><span>--- a/tests/tdef/tdef_test.ok</span><br><span>+++ b/tests/tdef/tdef_test.ok</span><br><span>@@ -12,12 +12,12 @@</span><br><span> osmo_tdef_get(2, m)      = 1</span><br><span> osmo_tdef_get(2, custom-unit)    = 100</span><br><span> osmo_tdef_get(2, us)   = 100000</span><br><span style="color: hsl(0, 100%, 40%);">-T3=100m</span><br><span style="color: hsl(0, 100%, 40%);">-osmo_tdef_get(3, s)      = 6000</span><br><span style="color: hsl(0, 100%, 40%);">-osmo_tdef_get(3, ms)      = 6000000</span><br><span style="color: hsl(0, 100%, 40%);">-osmo_tdef_get(3, m)    = 100</span><br><span style="color: hsl(0, 100%, 40%);">-osmo_tdef_get(3, custom-unit)      = 100</span><br><span style="color: hsl(0, 100%, 40%);">-osmo_tdef_get(3, us)       = 6000000000</span><br><span style="color: hsl(120, 100%, 40%);">+T3=50m</span><br><span style="color: hsl(120, 100%, 40%);">+osmo_tdef_get(3, s)       = 3000</span><br><span style="color: hsl(120, 100%, 40%);">+osmo_tdef_get(3, ms)    = 3000000</span><br><span style="color: hsl(120, 100%, 40%);">+osmo_tdef_get(3, m)  = 50</span><br><span style="color: hsl(120, 100%, 40%);">+osmo_tdef_get(3, custom-unit)     = 50</span><br><span style="color: hsl(120, 100%, 40%);">+osmo_tdef_get(3, us)      = 3000000000</span><br><span> T4=100custom-unit</span><br><span> osmo_tdef_get(4, s)        = 100</span><br><span> osmo_tdef_get(4, ms)   = 100</span><br><span>@@ -154,7 +154,7 @@</span><br><span> state=A T=0, no timeout</span><br><span>  --> A (configured as T1 100 s) rc=0;      state=A T=1, 100.000000 s remaining</span><br><span>  --> B (configured as T2 100 ms) rc=0;        state=B T=2, 1.000000 s remaining</span><br><span style="color: hsl(0, 100%, 40%);">- --> C (configured as T3 100 m) rc=0;       state=C T=3, 6000.000000 s remaining</span><br><span style="color: hsl(120, 100%, 40%);">+ --> C (configured as T3 50 m) rc=0;   state=C T=3, 3000.000000 s remaining</span><br><span>  --> D (configured as T4 100 custom-unit) rc=0;      state=D T=4, 100.000000 s remaining</span><br><span>  --> G (configured as T7 50 s) rc=0;  state=G T=7, 50.000000 s remaining</span><br><span>  --> H (configured as T8 300 s) rc=0;  state=H T=8, 300.000000 s remaining</span><br><span>@@ -183,5 +183,5 @@</span><br><span> - test disallowed transition:</span><br><span>  --> Z (no timer configured for this state) rc=0;      state=Z T=0, no timeout</span><br><span>  --> B (configured as T2 100 ms) rc=-1;   state=Z T=0, no timeout</span><br><span style="color: hsl(0, 100%, 40%);">- --> C (configured as T3 100 m) rc=-1;        state=Z T=0, no timeout</span><br><span style="color: hsl(120, 100%, 40%);">+ --> C (configured as T3 50 m) rc=-1;       state=Z T=0, no timeout</span><br><span>  --> D (configured as T4 100 custom-unit) rc=-1;  state=Z T=0, no timeout</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/21162">change 21162</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/libosmocore/+/21162"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ic0d009f00a69cee59f2d3fc0b40ecdc97d81c75c </div>
<div style="display:none"> Gerrit-Change-Number: 21162 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>