From 8a65fa40c7dbd71d48b4c02889a6577104518a9b Mon Sep 17 00:00:00 2001 From: Matthew Waters Date: Fri, 29 Oct 2021 18:40:04 +1100 Subject: [PATCH] webrtc/tests: print the correct media idx on error Instead of the attribute index Part-of: --- .../gst-plugins-bad/tests/check/elements/webrtcbin.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/subprojects/gst-plugins-bad/tests/check/elements/webrtcbin.c b/subprojects/gst-plugins-bad/tests/check/elements/webrtcbin.c index a07f1fb074..0dd2c36f64 100644 --- a/subprojects/gst-plugins-bad/tests/check/elements/webrtcbin.c +++ b/subprojects/gst-plugins-bad/tests/check/elements/webrtcbin.c @@ -891,22 +891,22 @@ on_sdp_media_direction (struct test_webrtc *t, GstElement * element, if (g_strcmp0 (attr->key, "inactive") == 0) { fail_unless (have_direction == FALSE, - "duplicate/multiple directions for media %u", j); + "duplicate/multiple directions for media %u", i); have_direction = TRUE; fail_unless_equals_string (attr->key, expected_directions[i]); } else if (g_strcmp0 (attr->key, "sendonly") == 0) { fail_unless (have_direction == FALSE, - "duplicate/multiple directions for media %u", j); + "duplicate/multiple directions for media %u", i); have_direction = TRUE; fail_unless_equals_string (attr->key, expected_directions[i]); } else if (g_strcmp0 (attr->key, "recvonly") == 0) { fail_unless (have_direction == FALSE, - "duplicate/multiple directions for media %u", j); + "duplicate/multiple directions for media %u", i); have_direction = TRUE; fail_unless_equals_string (attr->key, expected_directions[i]); } else if (g_strcmp0 (attr->key, "sendrecv") == 0) { fail_unless (have_direction == FALSE, - "duplicate/multiple directions for media %u", j); + "duplicate/multiple directions for media %u", i); have_direction = TRUE; fail_unless_equals_string (attr->key, expected_directions[i]); } @@ -971,7 +971,7 @@ on_sdp_media_setup (struct test_webrtc *t, GstElement * element, if (g_strcmp0 (attr->key, "setup") == 0) { fail_unless (have_setup == FALSE, - "duplicate/multiple setup for media %u", j); + "duplicate/multiple setup for media %u", i); have_setup = TRUE; fail_unless_equals_string (attr->value, expected_setup[i]); }