After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 728368 - check: add GstHarness convenience API for unit tests
check: add GstHarness convenience API for unit tests
Status: RESOLVED DUPLICATE of bug 751916
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other All
: Normal enhancement
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on: 751916
Blocks: 738363
 
 
Reported: 2014-04-16 18:27 UTC by Tim-Philipp Müller
Modified: 2015-07-07 00:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fixed 'Unexpected critical/warning: [...] Got data flow before segment event' (1.01 KB, patch)
2014-08-26 11:49 UTC, Mirko Günther
none Details | Review

Description Tim-Philipp Müller 2014-04-16 18:27:54 UTC
Just a quick note that I'm working on integrating this (incl. docs and fix-ups).
Comment 1 Haakon Sporsheim (ieei) 2014-05-28 06:12:42 UTC
Tim: We are updating our implementation @ https://github.com/pexip/gstreamer
Note that we might rebase this repository.
Comment 2 Haakon Sporsheim (ieei) 2014-05-28 06:21:51 UTC
One annoying issue using GstCheck and GstTestClock (which GstHarness uses) is that since gst_deinit is called atexit, an element that does a clock_wait on the testclock will deadlock if a unit test fail.
Comment 3 Tim-Philipp Müller 2014-05-28 06:55:19 UTC
it would be helpful if you would make additional changes to gstharness as additional commits on top of what you have now (rather than git commit --amend + git push -f). Rebasing on top of master is not a problem of course.

Regarding gst_deinit(), that should be fixed in master, at least when using fork(), see exit() --> _exit() commit in libcheck.
Comment 4 mariuszb 2014-08-12 10:31:38 UTC
Hi guys, do you have a view on when GstHarness  is going to be on master?
Comment 5 Tim-Philipp Müller 2014-08-12 11:48:28 UTC
I plan to get to it this cycle. I will try to put my WIP branch up somewhere soon.
Comment 6 Mirko Günther 2014-08-26 11:49:57 UTC
Created attachment 284498 [details] [review]
fixed 'Unexpected critical/warning: [...] Got data flow before segment event'

It fixes an issue seen when I try to run the first example give during the presentation of 'GstHarness' last year on GStreamer conference.

I've also made a pull-request to the pexip-repo. Just to be sure ;)
Comment 7 Tim-Philipp Müller 2015-07-07 00:01:30 UTC

*** This bug has been marked as a duplicate of bug 751916 ***