pespin has uploaded this change for review. (
https://gerrit.osmocom.org/c/libosmo-netif/+/30222 )
Change subject: tests/osmux: Add extra asserts to validate osmuxh eader is
pulled correctly
......................................................................
tests/osmux: Add extra asserts to validate osmuxh eader is pulled correctly
Change-Id: Ie2ccc22c53c33025c477236166ea97972c21ec3e
---
M tests/osmux/osmux_input_test.c
1 file changed, 5 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/22/30222/1
diff --git a/tests/osmux/osmux_input_test.c b/tests/osmux/osmux_input_test.c
index 5d1b84e..df02423 100644
--- a/tests/osmux/osmux_input_test.c
+++ b/tests/osmux/osmux_input_test.c
@@ -264,6 +264,7 @@
/* We expect 1 batch: */
osmuxh = osmux_xfrm_output_pull(batch_msg);
+ OSMO_ASSERT(osmuxh);
OSMO_ASSERT(osmuxh->ft == OSMUX_FT_VOICE_AMR);
/* Check CMR and Q values are the ones from the last message: */
OSMO_ASSERT(osmuxh->amr_f == 0);
@@ -357,6 +358,7 @@
/* We expect 1 batch: */
osmuxh = osmux_xfrm_output_pull(batch_msg);
+ OSMO_ASSERT(osmuxh);
/* Check seqnum is the one configured beforehand: */
OSMO_ASSERT(osmuxh->seq == 123);
@@ -425,6 +427,7 @@
/* We expect 1 batch: */
osmuxh = osmux_xfrm_output_pull(batch_msg);
+ OSMO_ASSERT(osmuxh);
/* Check seqnum is the one configured beforehand: */
OSMO_ASSERT(osmuxh->seq == 123);
osmux_pl = (uint8_t *)osmuxh + sizeof(*osmuxh);
@@ -503,6 +506,7 @@
/* We expect 1 batch: */
osmuxh = osmux_xfrm_output_pull(batch_msg);
+ OSMO_ASSERT(osmuxh);
/* Check seqnum is the one configured beforehand: */
OSMO_ASSERT(osmuxh->seq == 123);
/* Check four AMR payloads appear: */
@@ -587,6 +591,7 @@
/* We expect 1 batch: */
osmuxh = osmux_xfrm_output_pull(batch_msg);
+ OSMO_ASSERT(osmuxh);
/* Check seqnum is the one configured beforehand: */
OSMO_ASSERT(osmuxh->seq == (123 + *osmux_transmitted - 1));
/* Check four AMR payloads appear: */
--
To view, visit https://gerrit.osmocom.org/c/libosmo-netif/+/30222
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: Ie2ccc22c53c33025c477236166ea97972c21ec3e
Gerrit-Change-Number: 30222
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-MessageType: newchange
