Change in libosmo-netif[master]: tests: osmux_test: Hardcode h_output values set by random()

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Fri Oct 18 09:27:50 UTC 2019


laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-netif/+/15798 )

Change subject: tests: osmux_test: Hardcode h_output values set by random()
......................................................................

tests: osmux_test: Hardcode h_output values set by random()

osmux implementation randomizes those values. It seems build in OBS
sometimes provide different values than the ones expected in the test
result. Let's hardcode them to make sure we always have the same values
regarless of the random() implementation.

Values chosen are the one matching the current expected test output so
it doesn't need any change.

Change-Id: Icc553c83ddff41900ae3d5990a655c29c9073e01
---
M tests/osmux/osmux_test.c
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/tests/osmux/osmux_test.c b/tests/osmux/osmux_test.c
index 7fa9af7..b6a43fe 100644
--- a/tests/osmux/osmux_test.c
+++ b/tests/osmux/osmux_test.c
@@ -299,6 +299,9 @@
 
 	osmux_xfrm_output_init2(&h_output, 0x7000000, 98);
 	osmux_xfrm_output_set_tx_cb(&h_output, tx_cb, NULL);
+	/* These fields are set using random() */
+	h_output.rtp_seq = 9158;
+	h_output.rtp_timestamp = 1681692777;
 
 	/* If the test takes longer than 10 seconds, abort it */
 	alarm(10);

-- 
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/15798
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: Icc553c83ddff41900ae3d5990a655c29c9073e01
Gerrit-Change-Number: 15798
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191018/e294f784/attachment.htm>


More information about the gerrit-log mailing list