]> git.ipfire.org Git - thirdparty/asterisk.git/commit
res_rtp: Add unit tests for RTCP stats.
authorBen Ford <bford@digium.com>
Wed, 28 Aug 2019 19:25:57 +0000 (14:25 -0500)
committerBenjamin Keith Ford <bford@digium.com>
Tue, 10 Sep 2019 18:10:59 +0000 (13:10 -0500)
commit5dad4f4ea1bce2e823ed72e075b51d637673670f
tree886ee9c953ed5659fd2d28c3a217dc78dd3dd190
parent44068cc6bf802742ce8a0283351781bbcd9d7690
res_rtp: Add unit tests for RTCP stats.

Added unit tests for RTCP video stats. These tests include NACK, REMB,
FIR/FUR/PLI, SR/RR/SDES, and packet loss statistics. The REMB and FIR
tests are currently disabled due to a bug. We expect to receive a
compound packet, but the code sends this out as a single packet, which
the browser accepts, but makes Asterisk upset.

While writing these tests, I noticed an issue with NACK as well. Where
it is handling a received NACK request, it was reading in only the first
8 bits of following packets that were also lost. This has been changed
to the correct value of 16 bits.

Also made a minor fix to the data buffer unit test.

Change-Id: I56107c7411003a247589bbb6086d25c54719901b
include/asterisk/rtp_engine.h
main/rtp_engine.c
res/res_rtp_asterisk.c
tests/test_data_buffer.c
tests/test_res_rtp.c [new file with mode: 0644]