validate:scenario: Report disabling plugin issues

Summary:
+ typedef GstValidateActionReturn so it can be used in the introspection
+ Add GST_VALIDATE_EXECUTE_ACTION_ERROR_REPORTED which should be used
  to tell Validate that something wrong happened so the sub action
  won't be executed, but that it should not report an error itself
  as it has already been handled in the action function.

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D81

validate:scenario: Allow execution of disable-plugin as a config action

Summary:
And fix a bug where config actions were added to the list of action even
if they had already been executed

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D80

validate:scenarios: Set seek_forward_backward min-media-duration=45

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D79

validate:launcher: Set more env variable in the launcher command desc

Summary:
Adding if present:
  * LD_PRELOAD
  * DISPLAY
  * GST_VALIDATE_CONFIG
  * GST_VALIDATE_OVERRIDE

+  enhance the add_env_variable method to more easily set envvar from
current value

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D78

validate: Add a method to easily get plugin configuration

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D77

validate:plugins: Add support to all GApplication as a test apps

Summary:
Add a very simple plugin that will allow any GApplication to easily be
used with GstValidate using the LD_PRELOAD feature

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D75

validate: let structs_from_filename be exported.

Summary: It is useful for plugins too

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D74

validate:scenario: Update Action.repeat field when needed

Summary: And print the current repeat value of the action that have such a field

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D73

validate:scenario: Alway execute a 'quit' action on EOS

Summary: Making scenario more usable with LD_PRELOAD

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D72

validate:scenario: Set the main action structure in fill_structure

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D71

validate:launcher: Use GST_VALIDATE_SCENARIO envvar to set scenarios

Summary:
Instead of concidering all apps will have a --set-scenario argument
which is not going to be the case as soon as we run the tests through
LD_PRELOAD

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D70

validate:launcher: Move get_current_position from GstValidatePipelineTest to GstValidateTest

This is where it belongs

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D69

validate: fix typo in scenario file format docs

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

validate: scenario: fix caps leak

validate: media-descriptor: fix filenode->caps leak

validate: transcoding: don't create a second mainloop

validate: transcoding: don't leak the requested sinkpad from decodebin

validate: pad-monitor: fix caps leak

Don't create othercaps when early returning.

validate: GstValidateMediaCheckTest should inherit from GstValidateTest

validate:launcher: Install config.py for non-development mode

Running installed gst-validate-launcher aborted with the following error.

File "lib/gst-validate-launcher/python/launcher/baseclasses.py", line 28, in <module>
  import config
ImportError: No module named config

This is because config.py is added but not installed
in ba6d209b3fd062f4e6bd889f81f1213cc12339ec.

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

validate:launcher: Show timeout seconds for timeout result message

Current timeout message doesn't show how many seconds a test took and
it is timeouted by normal timeout or hard timeout.

This patch changes the message like following.

1. normal timeout
old : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Application timed out)
new : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Application timed out: 120 secs)

2. hard timeout
old : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Application timed out)
new : validate.http.playback.reverse_playback.raw_video_mov: Timeout (Hard timeout reached: 600 secs)

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

validate:launcher: Fix wrong test number with -j option

When '-j n' option is given, first n tests print test number 0.
This is caused by test_num part of 919db986052602dca452f05e284cfc857302d4f0.

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

validate:launcher: Avoid depending on PyGObject

Summary:
And rely on our knowledge of the configuration to figure out where the
suppression file has been installed

Reviewers: gdesmott

Differential Revision: http://phabricator.freedesktop.org/D61

validate: media-descriptor-writer: don't leak info and streaminfo

validate: media-check: don't leak output_file and expected_file

validate: check VALGRIND_ERROR_CODE in Test as well

We were doing it only in GstValidateTest which was overriding the default
implementation.

validate: don't increase hard_timeout is if it's None

Some tests, like the media check ones, have None as hard_timeout.

validate: increase VALGRIND_TIMEOUT_FACTOR

5 wasn't enough for my poor laptop.

validate: increase the normal timeout as well when using valgrind

validate: media-descriptor: fix caps leak

gst_pad_get_current_caps() returns a reffed caps.

also call gst_deinit() in media-check and transcoding

More valgrind friendly.

validate: raise an error if valgrind detected issues

Differential Revision: http://phabricator.freedesktop.org/D53

call gst_deinit() when we are done

More valgrind friendly.

validate: call gst_bus_remove_signal_watch()

We are supposed to call gst_bus_remove_signal_watch() for each gst_bus_add_signal_watch() call to prevent leaks.

validate: media-descriptor-parser: fix string leak

_set_content() doesn't actually consume @content so the caller is responsible
freeing it.

validate: pad-monitor: fix buffers list leak

validate: pad-monitor: fix caps leak

validate: fix a bunch of GstBus leaks

validate: store valgrind logs to its own file

validate: report: fix invalid read when destroying Report

Summary:
@report was invalid when we were trying to clear the mutex.

validate: scenario: remove weak pointer when destroying action

Free an invalid read when the scenario is destroyed after the action.

Differential Revision: http://phabricator.freedesktop.org/D44

validate: report: fix invalid read when destroying Report

@report was invalid when we were trying to clear the mutex.

validate: fix string arguments leaks

We are responsible of freeing the string arguments parsed by GOptionContext.

validate: scenario: don't borrow @structure in _fill_action()

@structure was borrowed in some code path and wasn't in some other. Make it
clearer, and fix a leak, by always copying it.

validate: override-registry: fix structs list leak

The list returned by _lines_get_strutures() needs to be deeply freed.

validate: scenario: fix scenarios leak

validate: reporter: fix message leak

validate: report: don't shadow the GString variable

We were leaking the GString as it's freed outside of the block.

validate: report: fix GStrv leak

validate: report: fix GStrv leak

We borrow the content of the GStrv but were leaking the array itself.

launcher: try using gst.supp as valgrind suppressions file

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

validate: install gst.supp

Will be used when running tests inside Valgrind.

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

launcher: add valgrind support

Add a --valgrind option to gst-validate-launcher to run the tests inside
Valgrind and tune GLib's memory allocator accordingly.

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

validate: move scenarios to validate/scenarios/

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

validate:launcher: Fix small typo

validate:launcher: Make sure to show apps specific options in the help

validate:launcher: Fix test number printing

validate:launcher Rename _other_testsuite_for_tester

To _check_tester_has_other_testsuite

validate:launcher: Cache all the tests in the runner

This way we do not have to re ask all the test managers
what tests should be run.

validate:launcher: Add a way to simply run validate default tests on uris

Summary:
This allows us to easily run all the scenarios on a particular file doing:

  $ gst-validate-launcher validate --validate-check-uri file:///some/media/file.webm

Reviewers: Mathieu_Du

Differential Revision: http://phabricator.freedesktop.org/D36

validate: keep executing actions even after linking up following execution

When linking actions execution without waiting on execution context, then
idle callback should keep being called so following action keep being
executed.

validate: Add more files to .gitignore

Differential Revision: http://phabricator.freedesktop.org/D34

validate:scenario: Rename get_position to execute_next_action

That function was wrongly called and did not correspond to what it
actually does.

Revert "validate: add non flushing seek support"

This reverts commit 3ff55dcc3119b39e7c86044159db8bce49a2dc3a.

Regressions on the test server, apparently linked to this patchset.

Revert "validate: use segments to detect success of flushing seeks too"

This reverts commit c47cc7ba90e96ffaefe201087428ef448670f3be.

Regressions on the test server, apparently linked to this patchset.

Revert "validate: expect a buffer with discontinuity after a seek"

This reverts commit 87064b6994e36203b6976d436feda809068f1497.

Regressions on the test server, apparently linked to this patchset.

validate:launcher: Make sure TIMEOUTs do not get converted to ERROR

This was a regression introduced in c0e3d2e4f190fc9627897cc3d3d016448cb5dbe9

validate: expect a buffer with discontinuity after a seek

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

validate: use segments to detect success of flushing seeks too

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

validate: add non flushing seek support

and a couple scenarios using them

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

validate:launcher: Do not forget to install apps/__init__.py

It is a python module that should be usable by external apps/testsuites

validate:scenario: Handle not mandatory action types

Summary:
There is currently no way to handle the fact that action types
might be handled only by a specific application but not handling
this action types would not cause any difference for the good execution
of the scenario as a whole

Differential Revision: http://phabricator.freedesktop.org/D33

validate:launcher: Better handle GST debug log outputs redirection

validate:launcher: First rely on the presence of criticals to set tests result

In the case of external applications they might not set their exist
code bases on the result of validate so we should rely on what
validates as to say first.

validate:report: Allow registering of issue types through the introspection

Fixing annotations and make GstValidateIssue refcounted

We break the ABI in that commit but I do not expect anyone to register
issue type outside GstValidate yet.

Add padding in the structures so we can avoid breaking the ABI again later.

validate:monitor: Do not requiere a GstObject as target

We can work with any GObject and that allows applications to write
monitors for other aspects too

validate:scenario: Alway execute sub action on action running SYNC

Move methods around to avoid needing on top prototypes

validate:docs: Rename gst-validate-launch to gst-validate-launcher

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

validate:scenario: Do not execute last sub action twice when ASYNC

validate:scenario: Properly print sub action as if they were main actions

validate:docs: Fix typos in Scenario File Format

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

validate:launcher Do not use git -C as it is relatively recent

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

validate: Build the preload so when possible

validate:launcher: Print the long help in less when possible

validate:launcher: Mention testsuite implementation in the help

validate: Define GST_PLUGIN_LDFLAGS as needed

validate: Determine development mode using git hash value

Development mode has been determined by whether the launcher is in git
repo
or not. This could be wrong when the launcher is installed to
subdirectory of other project's git repo, such as jhbuild. It is normal
to install compiled output to subdirectory of your jhbuild.

Changed logic gets the first commit hash of current git repo and
compares it with gst-devtools' the first commit hash.

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

validate:launcher: Fix typo s/FILE_EXTENDION/FILE_EXTENSION/g

validate:scenario: Fix GstValidateAction ABI adding a private structure

This way we can easily extend the structure and avoid needing using
a union and such

validate: Test buffer outside of received range.

Summary: As part of the preparation for a port to tracer.

Test Plan: This is a test, we won't test tests

Reviewers: tsaunier

Differential Revision: http://internal.opencreed.com:8888/D19

codecanalyzer: fix codec detection with git master

The names might be 'MPEG-2 (Simple Profile)' now.
Shouldn't really rely on codec name strings here
in the first place, but use caps instead.

validate: Properly notify user about missing plugins

This way it is clear in gst-validate-launcher that the failure is due
to a missing plugin

validate: Fix wrong sizeof usage

sizeof(int) is always <= sizeof(gpointer)

validate: launcher: Use cElementTree for XML parsing

Using cElementTree instead of ElementTree speeds up parsing of media
descriptor files.

The total time spent parsing XML files drops from ~0.64 s to ~0.24 s,
leading to faster initialisation times for gst-validate-launcher.

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

validate: Print actions directly from the scenario

Avoiding user to have to print them in each and every action type
implementation.

This requires adding some API to prepare actions before printing them.
Preparing action in that case mean parsing the values contained in the
GstStructure parsing equations and setting back the actual value
afterward

API:
  * GstValidatePrepateAction
  * gst_validate_action_type_set_prepare_function

validate: Fix build on windows

Check where libgstvalidate.dll is installed and use that base folder to
figure out where GstValidate plugins are installed

validate: Do not forget to dist _full variant of scrubing scenarios

validate: Create a gst-validate-types.h header where we define types

And include it from validate.h.

This way we avoid to need to typedef GstValidateAction twice, which is
a C11 feature

validate-report: Fix valist usage

a va_list always 'exists' (it's a struct). It therefore can't be NULL
(and can't be tested)

Just use the regular print variant where appropriate.

validate:scenario: Document locking

validate:scenario: Make get_position happen on idle

Summary:
- Add a way to force action to be executed in their own GSource dispatch, disabling chain action execution

API:
  GstValidateScenario::execute-on-idle property