From 31e22e251436e759f9a0e67404cbcfdb22a23f50 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Tim-Philipp=20M=C3=BCller?= Date: Thu, 23 Feb 2006 17:39:20 +0000 Subject: [PATCH] tests/check/: ... and add some tests for the base source EOS stuff. Original commit message from CVS: * tests/check/Makefile.am: * tests/check/libs/basesrc.c: (eos_event_counter), (basesrc_eos_events_pull), (basesrc_eos_events_push), (basesrc_eos_events_push_live_op), (basesrc_eos_events_pull_live_op), (gst_basesrc_suite), (main): ... and add some tests for the base source EOS stuff. --- ChangeLog | 9 ++ tests/check/Makefile.am | 5 +- tests/check/libs/basesrc.c | 359 +++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 372 insertions(+), 1 deletion(-) create mode 100644 tests/check/libs/basesrc.c diff --git a/ChangeLog b/ChangeLog index 56c8392..aab4cc0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,14 @@ 2006-02-23 Tim-Philipp Müller + * tests/check/Makefile.am: + * tests/check/libs/basesrc.c: (eos_event_counter), + (basesrc_eos_events_pull), (basesrc_eos_events_push), + (basesrc_eos_events_push_live_op), (basesrc_eos_events_pull_live_op), + (gst_basesrc_suite), (main): + ... and add some tests for the base source EOS stuff. + +2006-02-23 Tim-Philipp Müller + * tests/check/gst/gstutils.c: (test_buffer_probe_n_times): Test case originally showed the problem fixed below, but was then amended. Add checks back at the place diff --git a/tests/check/Makefile.am b/tests/check/Makefile.am index 4ef3f83..15f233a 100644 --- a/tests/check/Makefile.am +++ b/tests/check/Makefile.am @@ -66,6 +66,7 @@ check_PROGRAMS = \ $(PARSE_CHECKS) \ pipelines/stress \ libs/libsabi \ + libs/basesrc \ libs/controller \ libs/gdp \ libs/gstnetclientclock \ @@ -91,10 +92,12 @@ libs_gdp_CFLAGS = $(AM_CFLAGS) elements_fdsrc_CFLAGS=$(GST_OBJ_CFLAGS) $(CHECK_CFLAGS) -DTESTFILE=\"$(top_srcdir)/configure.ac\" +libs_basesrc_LDADD = \ + $(top_builddir)/libs/gst/base/libgstbase-@GST_MAJORMINOR@.la \ + $(LDADD) libs_controller_LDADD = \ $(top_builddir)/libs/gst/controller/libgstcontroller-@GST_MAJORMINOR@.la \ $(LDADD) - libs_gstnetclientclock_LDADD = \ $(top_builddir)/libs/gst/net/libgstnet-@GST_MAJORMINOR@.la \ $(LDADD) diff --git a/tests/check/libs/basesrc.c b/tests/check/libs/basesrc.c new file mode 100644 index 0000000..ba83244 --- /dev/null +++ b/tests/check/libs/basesrc.c @@ -0,0 +1,359 @@ +/* GStreamer + * + * some unit tests for GstBaseSrc + * + * Copyright (C) 2006 Tim-Philipp Müller + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Library General Public + * License as published by the Free Software Foundation; either + * version 2 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Library General Public License for more details. + * + * You should have received a copy of the GNU Library General Public + * License along with this library; if not, write to the + * Free Software Foundation, Inc., 59 Temple Place - Suite 330, + * Boston, MA 02111-1307, USA. + */ + +#ifdef HAVE_CONFIG_H +#include "config.h" +#endif +#include +#include +#include + +static void +eos_event_counter (GstObject * pad, GstEvent * event, guint * p_num_eos) +{ + fail_unless (event != NULL); + fail_unless (GST_IS_EVENT (event)); + + if (GST_EVENT_TYPE (event) == GST_EVENT_EOS) + *p_num_eos += 1; +} + +/* basesrc_eos_events_push_live_op: + * - make sure source does send an EOS event when operating in push + * mode and being set to READY explicitly (like one might with + * live sources) + */ +GST_START_TEST (basesrc_eos_events_push_live_op) +{ + GstStateChangeReturn state_ret; + GstElement *src, *sink, *pipe; + GstMessage *msg; + GstBus *bus; + GstPad *srcpad; + guint probe, num_eos = 0; + + pipe = gst_pipeline_new ("pipeline"); + sink = gst_element_factory_make ("fakesink", "sink"); + src = gst_element_factory_make ("fakesrc", "src"); + + g_assert (pipe != NULL); + g_assert (sink != NULL); + g_assert (src != NULL); + + fail_unless (gst_bin_add (GST_BIN (pipe), src) == TRUE); + fail_unless (gst_bin_add (GST_BIN (pipe), sink) == TRUE); + + fail_unless (gst_element_link (src, sink) == TRUE); + + g_object_set (sink, "can-activate-push", TRUE, NULL); + g_object_set (sink, "can-activate-pull", FALSE, NULL); + + g_object_set (src, "can-activate-push", TRUE, NULL); + g_object_set (src, "can-activate-pull", FALSE, NULL); + + /* set up event probe to count EOS events */ + srcpad = gst_element_get_pad (src, "src"); + fail_unless (srcpad != NULL); + + probe = gst_pad_add_event_probe (srcpad, + G_CALLBACK (eos_event_counter), &num_eos); + + bus = gst_element_get_bus (pipe); + + gst_element_set_state (pipe, GST_STATE_PLAYING); + state_ret = gst_element_get_state (pipe, NULL, NULL, -1); + fail_unless (state_ret == GST_STATE_CHANGE_SUCCESS); + + /* wait a second, then do controlled shutdown */ + g_usleep (GST_USECOND * 1); + + /* shut down source only (should send EOS event) ... */ + gst_element_set_state (src, GST_STATE_NULL); + state_ret = gst_element_get_state (src, NULL, NULL, -1); + fail_unless (state_ret == GST_STATE_CHANGE_SUCCESS); + + fail_unless (gst_element_set_locked_state (src, TRUE) == TRUE); + + /* ... and wait for the EOS message from the sink */ + msg = gst_bus_poll (bus, GST_MESSAGE_EOS | GST_MESSAGE_ERROR, -1); + fail_unless (msg != NULL); + fail_unless (GST_MESSAGE_TYPE (msg) != GST_MESSAGE_ERROR); + fail_unless (GST_MESSAGE_TYPE (msg) == GST_MESSAGE_EOS); + + /* should be exactly one EOS event */ + fail_unless (num_eos == 1); + + gst_element_set_state (pipe, GST_STATE_NULL); + gst_element_get_state (pipe, NULL, NULL, -1); + + /* make sure source hasn't sent a second one when going PAUSED => READY */ + fail_unless (num_eos == 1); + + gst_pad_remove_event_probe (srcpad, probe); + gst_object_unref (srcpad); + gst_message_unref (msg); + gst_object_unref (bus); + gst_object_unref (pipe); +} + +GST_END_TEST; + +/* basesrc_eos_events_push: + * - make sure source only sends one EOS when operating in push-mode, + * reaching the max number of buffers, and is then shut down. + */ +GST_START_TEST (basesrc_eos_events_push) +{ + GstStateChangeReturn state_ret; + GstElement *src, *sink, *pipe; + GstMessage *msg; + GstBus *bus; + GstPad *srcpad; + guint probe, num_eos = 0; + + pipe = gst_pipeline_new ("pipeline"); + sink = gst_element_factory_make ("fakesink", "sink"); + src = gst_element_factory_make ("fakesrc", "src"); + + g_assert (pipe != NULL); + g_assert (sink != NULL); + g_assert (src != NULL); + + fail_unless (gst_bin_add (GST_BIN (pipe), src) == TRUE); + fail_unless (gst_bin_add (GST_BIN (pipe), sink) == TRUE); + + fail_unless (gst_element_link (src, sink) == TRUE); + + g_object_set (sink, "can-activate-push", TRUE, NULL); + g_object_set (sink, "can-activate-pull", FALSE, NULL); + + g_object_set (src, "can-activate-push", TRUE, NULL); + g_object_set (src, "can-activate-pull", FALSE, NULL); + g_object_set (src, "num-buffers", 8, NULL); + + /* set up event probe to count EOS events */ + srcpad = gst_element_get_pad (src, "src"); + fail_unless (srcpad != NULL); + + probe = gst_pad_add_event_probe (srcpad, + G_CALLBACK (eos_event_counter), &num_eos); + + bus = gst_element_get_bus (pipe); + + gst_element_set_state (pipe, GST_STATE_PLAYING); + state_ret = gst_element_get_state (pipe, NULL, NULL, -1); + fail_unless (state_ret == GST_STATE_CHANGE_SUCCESS); + + msg = gst_bus_poll (bus, GST_MESSAGE_EOS | GST_MESSAGE_ERROR, -1); + fail_unless (msg != NULL); + fail_unless (GST_MESSAGE_TYPE (msg) != GST_MESSAGE_ERROR); + fail_unless (GST_MESSAGE_TYPE (msg) == GST_MESSAGE_EOS); + + /* should be exactly one EOS event */ + fail_unless (num_eos == 1); + + gst_element_set_state (pipe, GST_STATE_NULL); + gst_element_get_state (pipe, NULL, NULL, -1); + + /* make sure source hasn't sent a second one when going PAUSED => READY */ + fail_unless (num_eos == 1); + + gst_pad_remove_event_probe (srcpad, probe); + gst_object_unref (srcpad); + gst_message_unref (msg); + gst_object_unref (bus); + gst_object_unref (pipe); +} + +GST_END_TEST; + +/* basesrc_eos_events_pull_live_op: + * - make sure source doesn't send an EOS event when operating in + * pull mode and being set to READY explicitly (like one might with + * live sources) + */ +GST_START_TEST (basesrc_eos_events_pull_live_op) +{ + GstStateChangeReturn state_ret; + GstElement *src, *sink, *pipe; + GstPad *srcpad; + guint probe, num_eos = 0; + + pipe = gst_pipeline_new ("pipeline"); + sink = gst_element_factory_make ("fakesink", "sink"); + src = gst_element_factory_make ("fakesrc", "src"); + + g_assert (pipe != NULL); + g_assert (sink != NULL); + g_assert (src != NULL); + + fail_unless (gst_bin_add (GST_BIN (pipe), src) == TRUE); + fail_unless (gst_bin_add (GST_BIN (pipe), sink) == TRUE); + + fail_unless (gst_element_link (src, sink) == TRUE); + + g_object_set (sink, "can-activate-push", FALSE, NULL); + g_object_set (sink, "can-activate-pull", TRUE, NULL); + + g_object_set (src, "can-activate-push", FALSE, NULL); + g_object_set (src, "can-activate-pull", TRUE, NULL); + + /* set up event probe to count EOS events */ + srcpad = gst_element_get_pad (src, "src"); + fail_unless (srcpad != NULL); + + probe = gst_pad_add_event_probe (srcpad, + G_CALLBACK (eos_event_counter), &num_eos); + + gst_element_set_state (pipe, GST_STATE_PLAYING); + state_ret = gst_element_get_state (pipe, NULL, NULL, -1); + fail_unless (state_ret == GST_STATE_CHANGE_SUCCESS); + + /* wait a second, then do controlled shutdown */ + g_usleep (GST_USECOND * 1); + + /* shut down source only ... */ + gst_element_set_state (src, GST_STATE_NULL); + state_ret = gst_element_get_state (src, NULL, NULL, -1); + fail_unless (state_ret == GST_STATE_CHANGE_SUCCESS); + + fail_unless (gst_element_set_locked_state (src, TRUE) == TRUE); + + /* source shouldn't have sent any EOS event in pull mode */ + fail_unless (num_eos == 0); + + gst_element_set_state (pipe, GST_STATE_NULL); + gst_element_get_state (pipe, NULL, NULL, -1); + + /* make sure source hasn't sent an EOS when going PAUSED => READY either */ + fail_unless (num_eos == 0); + + gst_pad_remove_event_probe (srcpad, probe); + gst_object_unref (srcpad); + gst_object_unref (pipe); +} + +GST_END_TEST; + +/* basesrc_eos_events_pull: + * - makes sure source doesn't send EOS event when reaching the max. + * number of buffers configured in pull-mode + * - make sure source doesn't send EOS event either when being shut down + * (PAUSED => READY state change) after EOSing in pull mode + */ +GST_START_TEST (basesrc_eos_events_pull) +{ + GstStateChangeReturn state_ret; + GstElement *src, *sink, *pipe; + GstMessage *msg; + GstBus *bus; + GstPad *srcpad; + guint probe, num_eos = 0; + + pipe = gst_pipeline_new ("pipeline"); + sink = gst_element_factory_make ("fakesink", "sink"); + src = gst_element_factory_make ("fakesrc", "src"); + + g_assert (pipe != NULL); + g_assert (sink != NULL); + g_assert (src != NULL); + + fail_unless (gst_bin_add (GST_BIN (pipe), src) == TRUE); + fail_unless (gst_bin_add (GST_BIN (pipe), sink) == TRUE); + + fail_unless (gst_element_link (src, sink) == TRUE); + + g_object_set (sink, "can-activate-push", FALSE, NULL); + g_object_set (sink, "can-activate-pull", TRUE, NULL); + + g_object_set (src, "can-activate-push", FALSE, NULL); + g_object_set (src, "can-activate-pull", TRUE, NULL); + g_object_set (src, "num-buffers", 8, NULL); + + /* set up event probe to count EOS events */ + srcpad = gst_element_get_pad (src, "src"); + fail_unless (srcpad != NULL); + + probe = gst_pad_add_event_probe (srcpad, + G_CALLBACK (eos_event_counter), &num_eos); + + bus = gst_element_get_bus (pipe); + + gst_element_set_state (pipe, GST_STATE_PLAYING); + state_ret = gst_element_get_state (pipe, NULL, NULL, -1); + fail_unless (state_ret == GST_STATE_CHANGE_SUCCESS); + + msg = gst_bus_poll (bus, GST_MESSAGE_EOS | GST_MESSAGE_ERROR, -1); + fail_unless (msg != NULL); + fail_unless (GST_MESSAGE_TYPE (msg) != GST_MESSAGE_ERROR); + fail_unless (GST_MESSAGE_TYPE (msg) == GST_MESSAGE_EOS); + + /* source shouldn't have sent any EOS event in pull mode */ + fail_unless (num_eos == 0); + + gst_element_set_state (pipe, GST_STATE_NULL); + gst_element_get_state (pipe, NULL, NULL, -1); + + /* make sure source hasn't sent an EOS when going PAUSED => READY either */ + fail_unless (num_eos == 0); + + gst_pad_remove_event_probe (srcpad, probe); + gst_object_unref (srcpad); + gst_message_unref (msg); + gst_object_unref (bus); + gst_object_unref (pipe); +} + +GST_END_TEST; + + +Suite * +gst_basesrc_suite (void) +{ + Suite *s = suite_create ("GstBaseSrc"); + TCase *tc = tcase_create ("general"); + + suite_add_tcase (s, tc); + tcase_add_test (tc, basesrc_eos_events_pull); + tcase_add_test (tc, basesrc_eos_events_push); + tcase_add_test (tc, basesrc_eos_events_push_live_op); + tcase_add_test (tc, basesrc_eos_events_pull_live_op); + + return s; +} + +int +main (int argc, char **argv) +{ + int nf; + + Suite *s = gst_basesrc_suite (); + SRunner *sr = srunner_create (s); + + gst_check_init (&argc, &argv); + + srunner_run_all (sr, CK_NORMAL); + nf = srunner_ntests_failed (sr); + srunner_free (sr); + + return nf; +} -- 2.7.4