validate:launcher: Add proper check for is_seekable

The seekable variable in media_info file is of type string. When checking if the file
is seekable using is_seekable, it just returns the string, resulting in it always being true.
It should actually be comparing the string and returning true or false based on comparison

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

validate: fast_forward: Calculate proper playback-time for scenario

In case of fast-forward scenario, the playback-time is not set properly
as per increase in the rate. This is resulting in short media files of duration
less that 15 seconds to fail.

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

codecanalyzer: fix typo leading to implicit decl warning at build-time

codecanalyzer: do not use g_error if abort is not desired

Use g_printerr() instead.

g_error() calls abort after outputting the message
so these blocks' return statements and free()s
were unreachable.

Aditionally, fix wrong void returns on non-void
function, drop trailing whitespace before newline and
add \n's as needed (default handler won't add one).

codecanalyzer: add missing includes for g_printf()

codecanalyzer: Update README URL in help dialog

Previous one was a 404

configure: fix typos

pad-monitor: More fixes of previous commit

You'd think in 2016 compilers could complain when assigning/comparing
different types of enums ... *sigh*.

tools: Fix relative track switching

I have no idea where that "-2" came from, but it was obviously wrong.
Just use modulo "total number of streams" to get the proper track id.

validate: Remove debugging left overs

pad-monitor: Add support for GstPadEventFullFunc

This ensures our sink pad event wrapper is properly called if the
element implement a GstPadEventFullFunc instead of a regular one.

Removes all stray "buffer received before segment" issues with
queue/multiqueue

validate: Fix testsuite

Use fake elements instead of real ones in our tests so that
we control exactly the number of issues generated.

Until now we were trying to hide extra issues with a probe dropping
events and buffers but since 2dfa548f3645844082c3db65d96d87255701b3ad
"pad: Append hooks instead of prepending to call them in the order they were added"
in core, hidding will not work.

g-i: fix init section to avoid compiler warnings

runner: Report criticals when the reporter is in smart mode

media-descriptor-writer: track running time of buffers

PTS and DTS can be deceiving as a change in segment can dramatically change
playback synchronization. Track the running-time as well to properly
get any change in synchronization

Add a toplevel configure script to build components at once

And this way respect https://github.com/cgwalters/build-api

media-check: enable 'full' for files which reference are also 'full'

If the reference file has frames information, enable it automatically
so that the comparison file also has frames to be used.

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

media-descriptor: check if frame data matches

When comparing media descriptors, also check if the frames
match

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

media-descriptor-writer: refactor getting the streamnode by a pad

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

media-descriptor-writer: remove condition from inside loop

Only enter the loop if it indeed has a change of doing something

docs: fix typo

Retrive -> retrieve

validate: Add some documentation and annotiations

validate: Add a 'smart' reporting details mode

Making sure to show all informations for critical issues, but be synthetic for others

scenario: Set the default action execution interval to 10ms

Having a default value of 0 meant that a g_idle_add loop was constantly
running, causing each test to use 100% cpu.

This is no longer required. Using a 10ms interval brings down cpu usage
to a sane value

codecanalyzer: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are
not getting free'd which results in memory leaks being reported.

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

validate: media-descriptor-writer: Don't create media info when stream info is not present.

When a file does not contain any stream info, then there is no need
to create the media info file as, it is not considered to be a valid file
and no validate checks are done for the same.
This skips unnecessary files like .txt, .dump files

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

validate: launcher: Set scenario manager config before discovering testsuites

validate: launcher: Print error when media-info files not present

When there are no media-info files present and --generate-media-info
option is not given, then it just fails without printing error.
Printing an error stating, use --generate-media-info if there are no
media info files.
When there are neither media files and media info files, print error
stating the same

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

validate: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.

And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.

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

validate: suppression: suppress 'uninitialised value of size 4' in aacdec

Suppress this error, until the logic in libav is fixed.

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

validate: Implement support to run tests inside gdb

Making debugging races leading to crashes easier to debug

validate: Minor documentation fix

validate: Remove 0.10 caps reference from the documentation

validate: Handle setting several scenarios

The user might have scenarios specific to a particular pipeline, and the
application might have several pipelines running and scenarios that
apply on specific pipeline. We have to handle that valid use case.

validate: Un-blacklist seeking HLS tests

They are reliable enough after 36b80edb7263118467dfcaee3923f7c964ae6bc8
in gst-plugins-base now.

validate: Add missing gir include path

Add missing gir include path for building with gst-uninstalled script

Differential Revision: https://phabricator.freedesktop.org/D461

validate-transcoding: trivial patch to change error from 0.10 to 1.0

While printing error in transcoding, gst-validate-transcoding-0.10 is being used.
Changing the same to 1.0

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

validate: Add support for prores

validate: fix double free

Summary:
Move variable declarations in the for block so we won't try re-free
tldir in case of early short circuiting of the 'for' code.

Depends on D348

Reviewers: thiblahute

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D349

validate: fix loading of full path scenario

Summary:
We were checking if the path was a full one but was using the
scenario_name instead of this path when trying to load the scenario.

Depends on D346

Reviewers: thiblahute

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D348

Update .arcconfig

validate:launcher: throw valgrind error only for definite loss

errors-for-leak-kinds should be set to definite, because almost every test case
, will have possibly lost memory, which may or may not be a leak.
And throwing error for all these cases doesn't seem to be correct.

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

validate: launcher: Fix media_check class name and add double quotes for valgrind logs

When creating the class names for media check, uri is being used,
instead of the path. Hence converting the uri using uri2path and creating
class name.
Add double quotes for valgrind logs, to support special characters like space

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

validate: media-check: Pass NULL instead of GError if not using it

If not using the GError being passed on to media descriptor, writer and parser,
simply pass NULL instead of GError.

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

validate: descriptor-writer: Handle NULL GError address and free GError during error cases

writer_new_discover() API should be able to accept NULL GError and in case of
error, if GError is passed on as parameter, it should be propagated, else it
should be free'd.

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

validate:tools: set locale to all and change argument to FILENAME

When file name consists of characters from other languages, say korean,
then it throws an error
Error initializing: Invalid byte sequence in conversion input

Hence setting locale to all to fix this.
And changing the media-info argument to type G_OPTION_ARG_FILENAME

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

validate: Print the return value at the end

Makes it easier to know if the test passed or failed.

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

validate: launcher: Support relative path for folder names

Even though relative paths are supported, right now it does not
work when we give the path as 'media/' present in the current directory.
Adding support for the same.

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

validate:launcher: Treat min-media-duration as a float

scenarios: Set min-media-duration on switch_audio_track

validate:scenario: Add a way to define a timeout for actions execution

Reviewers: Mathieu_Du

Differential Revision: https://phabricator.freedesktop.org/D271

validatessim: Stop using deprecated gst_segment_to_position()

Back to development

Release 1.6.0

validate: scenarios: set need clock sync for switching subtitle track

As soon as the track is changed, the pipeline state is set to NULL
by execution 'stop' action even if there is a 'playback-time' with 5sec.
If the AV sink is not synchronized,
audio fakesink and video fakesink has different position value.
When the validate request the position information of pipeline
to do 'stop' action, the audio fakesink response of the position query
with the bigger value than 5sec.

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

Release 1.5.90

Update common submodule

validate: Put the GstValidate testsuite version in the launcher config

This way testsuite implementation can have the information

validate/transcoding: Don't override the target state of the scenario when receiving BUFFERING=100%

If the scenario handles the states and wants to stay in PAUSED, it's not a
good idea to change the state to PLAYING when receiving BUFFERING=100%. This
caused a race condition in varios seeking tests, most often in the dash scrub
seeking test.

validate: Don't override the target state of the scenario when receiving BUFFERING=100%

If the scenario handles the states and wants to stay in PAUSED, it's not a
good idea to change the state to PLAYING when receiving BUFFERING=100%. This
caused a race condition in varios seeking tests, most often in the dash scrub
seeking test.

validate: Fix typo

validate/launcher: Blacklist some HLS seeking tests again

validate/launcher: Un-blacklist validate.dash.playback.reverse_playback.*

Fixed now

validate/launcher: Un-blacklist validate.hls.playback.reverse_playback.*

It apparently succeeds now.

validate/launcher: De-blacklist some HLS tests

validate/launcher: Blacklist some DASH tests

launcher: Handle override files for media files

And make sure to create a new dict for extra_env_vars when instanciating
GstValidateTest

launcher: Make sure MediaDescriptor is set in the GstValidateTest class itself

It is used there but was set in each and every subclasses

validate/launcher: Treat DASH like HLS in another place

validate: scenario: fix leak during error cases

When message_async is not called during error cases, needs_parsing GList is
not being freed resulting in leak. Hence free'ing the same in finalize.

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

validate: descriptor-writer: Handle error when stream info is not available

There is no check to see if stream info is available. This leads to
assertion error. Adding proper error messages for the same and reported
the same as a validate warning message.

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

validate: descriptor-writer: Print proper error message when discover fails

When discovering the files, there will be different kind of errors. If we print
the exact message, then it will be more helpful for user. Especially in the case
of missing plugins, displaying which plugin is missing as error message

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

validate:override-registry: fix memory leak

mutex is being initialized but not cleared.

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

validate: tools: transcoding error due to wrong condition check

when checking the restriction caps, not adding proper check, which
results in assertion error when calling gst_caps_from_string

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

validate:launcher: Fix documentation

Fix some trivial spelling mistakes in documentation
and document about --update-media-info.

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

validate:launcher: escape the characters to remove bad range in regex

When media file name consists of some special characters of the format
[b-a].mp3, then it fails with 'bad character range' error and exits.
call re.escape to escape the characters before using it in findall

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

validate:launcher: improve uri generation for --medias-path

When --medias-paths option is being used, right now we have to
specify the full path, like /home/user/gst/master/media/
But when inside master directory, would like to specify only
media/ and expect it to work. Using os.path.abspath and create uri based on that.
This way we can either just pass media/ or pass the full path as parameters.

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

validate:launcher: skip setting up test suite for --medias-paths

in validate.py, some mixer test generators are being added by default.
When passing --media-paths, i would not want to test these.
So instead of setting up the validate test suite, just call tester.register_defaults().

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

validate: scenarios: Change start time for reverse playback

Right now reverse playback happens till the beginning of the media file.
But for files which are longer than 150 seconds,
Timeout 'Hard timeout reached: 150 secs' error happens. So we should set the
start time within 150 seconds.

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

validate: launcher: Add a --force-sync option

Which should put the testsuite in a clean state (basically using git
reset --hard for git based testsuite for example)

validate: Enable flac reverse playback tests

This is now supported and works as expected.

validate: media-descriptor: Workaround file:// stream-id changing

file:// base stream-id will vary depending on the file path. As we
don't expect everyone to use the same absolute path to place the
validate testsuite, the resulting stream-id changes. Because of that,
we can't match the stream-id in the recorded file, hence cannot do
further check. We work around this by doing what filesink would do,
which is compute a SHA256 of the URI which we can use to first
validate the ID is prefixed like expected, and decide if we should
consider the stream IDs the same or not.

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

validate: media-descriptor: Fix reading seekable record

Casting the result of g_strmp0 to boolean won't make gboolean
value 0 or 1. We need proper 0 and 1 so we can use == comparision.

validate: media-descriptor: handle proper return values

while comparing the media descriptor with --expected-results, the return
values are not being handled properly, which results in wrong comparision

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

validate: media-descriptor: Add comment before ignored return value

As stated in the bug, this comparison failing is not a critical
error, warning is enough. Add a comment so nobody thinks it's a
coding error.

https://bugzilla.gnome.org/review?bug=748390

validate: media-descriptor: remove duplicate conditions

when comparing tags, two conditions in if an else if are same
the correct way is to first check if both are NULL and return.
changed the condition accordingly.

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

validate: media-descriptor: fix trivial spelling mistakes

replace comparse_stream with compare_streams

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

validate:tools: set locale to all and change argument to FILENAME

When file name consists of characters from other languages, say korean,
then it throws an error
Error initializing: Invalid byte sequence in conversion input

Hence setting locale to all to fix this.
And changing the media-info argument to type G_OPTION_ARG_FILENAME

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

validate: Add a method to get action->scenario in a thread safe way

API:
  gst_validate_action_get_scenario

validate-scenario: Use thread-safe GWeakRef

Since _set_done() is meant to be thread safe,
it can not be used with g_object_add_weak_pointer(),
instead, one must use GWeakRef. But since it is in the API,
document that fact and add a couple assertions to make sure
it doesn't get broken in the future.

validate-scenario: Use GLib functions to make sure GMainContext is used

scenario: Only modify the actions from the main thread

The action's content is not protected by a mutex, so only
modify it from the main thread.

gitignore: Add more generated files

validate-scenario: get duration from media_info if not able to query

In case of files, which don't have duration in header, baseparse
estimates the duration only after 1.5 seconds. But Async_done event
is sent before the duration is estimated, which results in error.
If duration query fails, getting the duration from the media-info being
passed through --set-media-info. If media-info is also not set,
printing an error message and throwing error.

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

validate:launcher: add quotes to the file path

When folder name contains space or other special characters,
it fails to recognise the same and error is thrown. Adding the path
inside  to recognise the same

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

validate:launcher: unquote the path to remove special characters

When getting path from url using, url2path, it is returning
special characters (%20 for space etc..), instead of returning
plain path. path.unquote replaces the same..

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

validate:launcher: return on error cases properly

When folder name contains spaces during --medias-paths, it does not
create the media info, but still it shows as passed.
Returing failed during this case

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

validate:launcher: handle file path in --medias-paths

right now --medias-paths accepts only directories. Added support to
accept file path as well.

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

validate:launcher: skip default media path for --media-paths option

when --media-paths is specified, then no need to check the default media.
And add Force argument to let testsuite force the inclusion of
default media directory.

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

validate:ssim: Fix calls to the converters

We were mixing them