[Bug 752175] tests: remove usage of g_assert()

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=752175

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk
           Severity|normal                      |enhancement

--- Comment #1 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
But why? I mean, what's the point? This is unit tests... Consistency?

fail_*() admittedly has the advantage that you get a 'check point', so when it
fails the 'test xyz failed after line 1234' will be more accurate.

There are also cases where we definitely do not want to make such a change,
e.g. in inner loops of unit tests (e.g. I remember some unit tests did fail_*()
on a per-pixel basis for video tests and the log file was huuuuuuuge as a
result).

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.