Commit 9a3e8ad2 authored by Jonny Lamb's avatar Jonny Lamb Committed by Mathieu Duponchelle

rtpulpfec: stop and start the harness when setting error-after

gstreamer!55 makes some changes to how the `error-after` counter works
which breaks this test. This change makes the test not rely on the
ability to alter `error-after` at runtime and explicitly stops and
starts the harness before pushing data.

An alternative would be to add another argument to
`harness_rtpulpfecdec` to set `error-after` on construction but that's
slightly more long-winded. so I went for this approach instead.

Fixes #532, even though that's already closed.
parent 306d5021
......@@ -515,7 +515,12 @@ GST_START_TEST (rtpulpfecdec_recovered_push_failed)
GstHarness *h = harness_rtpulpfecdec (3536077562, 100, 123);
RecoveredPacketInfo info = {.pt = 100,.ssrc = 3536077562,.seq = 36921 };
GList *expected = expect_recovered_packets (h, &info, 1);
// the harness is already PLAYING because there are no src pads, which
// means the error-after counter isn't set, so reset and start again.
gst_element_set_state (h->element, GST_STATE_NULL);
gst_harness_set (h, "identity", "error-after", 2, NULL);
gst_harness_play (h);
push_data (h, SAMPLE_ULPFEC0_FEC, sizeof (SAMPLE_ULPFEC0_FEC) - 1);
push_lost_event (h, 36921, 1111, 2222, FALSE);
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment