tests/jitterbuffer: Major refactoring and cleanups
authorHavard Graff <havard.graff@gmail.com>
Wed, 14 Dec 2016 09:39:12 +0000 (10:39 +0100)
committerSebastian Dröge <sebastian@centricular.com>
Wed, 14 Dec 2016 13:00:37 +0000 (15:00 +0200)
commit0a81f71df5aa15627eb9f1756c92eabe5d3aa38c
tree5479b335828410fdf4c5497f9ea75b8bd342ca65
parent7446996b2af178a44670e890a66468bb80622e35
tests/jitterbuffer: Major refactoring and cleanups

* Changed PCMU->TEST for common macros
* Changed verify-functions (lost & rtx) into macros.
* Remove option to add marker-bit for test-buffers (not used anywhere)
* Add new push_test_buffer function that makes sure there are correlation
  between dts and the time on the clock. (classic test-mistake)
* Established a generic starting-point for tests with the
  construct_deterministic_initial_state function and use it where
  applicable, which removes lots of "boilerplate" everywhere.
* Add basic lost-event test
* Remove as much "magic constants" as possible.
* Remove 3 tests that no longer are testing anything that others don't,
  and was completely unmaintainable.
* Remove unnecessary use of the testclock
* Verify each test is testing what it actually says it does (and modify
  where it doesn't)

In general, make the tests much smaller, better, more maintainable and
readable.

https://bugzilla.gnome.org/show_bug.cgi?id=774409
tests/check/elements/rtpjitterbuffer.c