fixeria submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved
tests/csd: add NT variants for TCH/F4.8 and TCH/F9.6

The existing tests are all for T (transparent mode).
Add NT (non-transparent mode) variants.

Change-Id: Ie335bc0623dd7e887a0b3b1c40d61153b84924b2
Related: OS#1572
---
M tests/csd/csd_test.c
M tests/csd/csd_test.err
2 files changed, 62 insertions(+), 0 deletions(-)

diff --git a/tests/csd/csd_test.c b/tests/csd/csd_test.c
index 13a6419..0ad611f 100644
--- a/tests/csd/csd_test.c
+++ b/tests/csd/csd_test.c
@@ -54,12 +54,24 @@
.csd_mode = LCHAN_CSD_M_T_9600,
},
{
+ .name = "TCH/F9.6 NT",
+ .lchan_type = GSM_LCHAN_TCH_F,
+ .tch_mode = GSM48_CMODE_DATA_12k0,
+ .csd_mode = LCHAN_CSD_M_NT,
+ },
+ {
.name = "TCH/F4.8",
.lchan_type = GSM_LCHAN_TCH_F,
.tch_mode = GSM48_CMODE_DATA_6k0,
.csd_mode = LCHAN_CSD_M_T_4800,
},
{
+ .name = "TCH/F4.8 NT",
+ .lchan_type = GSM_LCHAN_TCH_F,
+ .tch_mode = GSM48_CMODE_DATA_6k0,
+ .csd_mode = LCHAN_CSD_M_NT,
+ },
+ {
.name = "TCH/H4.8",
.lchan_type = GSM_LCHAN_TCH_H,
.tch_mode = GSM48_CMODE_DATA_6k0,
diff --git a/tests/csd/csd_test.err b/tests/csd/csd_test.err
index b862324..cf19aff 100644
--- a/tests/csd/csd_test.err
+++ b/tests/csd/csd_test.err
@@ -24,6 +24,31 @@
ff ff ff ff ff ff ff ff 3f 3f 3f 3f ff ff ff ff
ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+[i] Testing 'TCH/F9.6 NT' (bitnum=240)
+[i] csd_v110_rtp_encode() returns 160
+ 3f 3f 3f 3f bf bf bf bf ff 7f 7f 7f bf bf bf bf
+ ff 7f 7f 7f bf 3f 7f 7f bf bf bf bf ff 7f 7f 7f
+ bf bf bf bf ff 7f 7f 7f 3f 3f 3f 3f bf bf bf bf
+ ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f bf 7f 7f 7f
+ bf bf bf bf ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f
+ 3f 3f 3f 3f bf bf bf bf ff 7f 7f 7f bf bf bf bf
+ ff 7f 7f 7f bf bf 7f 7f bf bf bf bf ff 7f 7f 7f
+ bf bf bf bf ff 7f 7f 7f 3f 3f 3f 3f bf bf bf bf
+ ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f bf ff 7f 7f
+ bf bf bf bf ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f
+[i] csd_v110_rtp_decode() returns 240
+[i] Testing 'TCH/F9.6 NT' (IDLE)
+[i] csd_v110_rtp_encode() returns 160
+ 3f 3f 3f 3f ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff 3f 3f 3f 3f ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ 3f 3f 3f 3f ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff 3f 3f 3f 3f ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
[i] Testing 'TCH/F4.8' (bitnum=120)
[i] csd_v110_rtp_encode() returns 160
7f 7f 7f 7f 7f 7f 7f 7f ff 7f ff 7f ff 7f ff 7f
@@ -49,6 +74,31 @@
ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+[i] Testing 'TCH/F4.8 NT' (bitnum=120)
+[i] csd_v110_rtp_encode() returns 160
+ 7f 7f 7f 7f 7f 7f 7f 7f ff 7f ff 7f ff 7f ff 7f
+ ff ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 7f
+ ff ff 7f ff 7f ff 7f ff ff 7f 7f 7f 7f ff 7f ff
+ ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff
+ ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff
+ 7f 7f 7f 7f 7f 7f 7f 7f ff 7f ff 7f ff 7f ff 7f
+ ff ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 7f
+ ff ff 7f ff 7f ff 7f ff ff 7f 7f ff 7f ff 7f ff
+ ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff
+ ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff
+[i] csd_v110_rtp_decode() returns 120
+[i] Testing 'TCH/F4.8 NT' (IDLE)
+[i] csd_v110_rtp_encode() returns 160
+ 7f 7f 7f 7f 7f 7f 7f 7f ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ 7f 7f 7f 7f 7f 7f 7f 7f ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
+ ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
[i] Testing 'TCH/H4.8' (bitnum=240)
[i] csd_v110_rtp_encode() returns 160
3f 3f 3f 3f bf bf bf bf ff 7f 7f 7f bf bf bf bf

To view, visit change 38302. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ie335bc0623dd7e887a0b3b1c40d61153b84924b2
Gerrit-Change-Number: 38302
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>