tests/check/pipelines/simple-launch-lines.c: Add timeout to _get_state() so we see...

Original commit message from CVS:
* tests/check/pipelines/simple-launch-lines.c: (run_pipeline):
Add timeout to _get_state() so we see which pipeline it is
that causes trouble on the gen64 build bot.

diff --git a/ChangeLog b/ChangeLog
index f6bd49d..a0a4b4d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-09-27  Tim-Philipp Müller  <tim at centricular dot net>
+
+       * tests/check/pipelines/simple-launch-lines.c: (run_pipeline):
+         Add timeout to _get_state() so we see which pipeline it is
+         that causes trouble on the gen64 build bot.
+
 2006-09-27  Wim Taymans  <wim@fluendo.com>
 
        * gst-libs/gst/rtp/gstbasertpdepayload.c:

diff --git a/tests/check/pipelines/simple-launch-lines.c b/tests/check/pipelines/simple-launch-lines.c
index 325a8e1..083aae8 100644 (file)
--- a/tests/check/pipelines/simple-launch-lines.c
+++ b/tests/check/pipelines/simple-launch-lines.c
@@ -58,9 +58,12 @@ run_pipeline (GstElement * pipe, const gchar * descr,
 
   fail_if (gst_element_set_state (pipe, GST_STATE_PLAYING) ==
       GST_STATE_CHANGE_FAILURE, "Could not set pipeline %s to playing", descr);
-  ret = gst_element_get_state (pipe, NULL, NULL, GST_CLOCK_TIME_NONE);
-  if (ret != GST_STATE_CHANGE_SUCCESS) {
-    g_critical ("Couldn't set pipeline to PLAYING");
+  ret = gst_element_get_state (pipe, NULL, NULL, 10 * GST_SECOND);
+  if (ret == GST_STATE_CHANGE_ASYNC) {
+    g_critical ("Pipeline '%s' failed to go to PLAYING fast enough", descr);
+    goto done;
+  } else if (ret != GST_STATE_CHANGE_SUCCESS) {
+    g_critical ("Pipeline '%s' failed to go into PLAYING state", descr);
     goto done;
   }