validate: Add a GstValidateActionTypeFlag flag

Allowing us to define action types more in detail.
Keep backward compatibility, at least with the C API

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

validate: Add a GstValidatePipelineMonitor subclass

We had quite a bit of code dedicated to handled GstPipeline monitoring
inside GstValidateBinMonitor, cleanly split that code into a new object
type

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

validate:launcher: Force clock sync for some protocols

In HLS for example, not having clock sync might lead to races and failures
do not test that for now

validate: Don't fail getting master report from a ghostpad without target

validate: Print current position even if we do not know the rate

That could cause gst-validate-launcher to wrongly concider tests
as timeout

validate:scenario: Force clock sink for scenarios with a pause action

validate: Handle wrong paths when listing avalaible apps

User can make mistake or we can have an empty path.

validate: Do not check if first buffer running time is 0

It can perfectly not be 0, so it makes no sense to check that.

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

validate: Add audiomixer test to the default testsuite

pad-monitor: Give better details about segment mismatch issues

validate: Be more precise in issue type for wrong seqnum

Depending on the type of event where the bug occurs,
it is not the same issue type. That allows us to have
much precise reports, and better explain the user
where the issue stands.

validate: Fix a few annotation issues

validate: Add the notion of WAIT_MULTIPLIER for the wait action

Allowing the user to decide to wait more, or less, or even not wait
for the wait action to execute when running scenarios.

validate:scenario: Report an EXECUTION_ERROR on action execution failure

If the action type handles a better error report type, it should just
return TRUE, and report its issue itself.

validate: Do not exit when we can not discover a result file

Loggable.error actually exit the process, it is not what we want!

+ Avoid a backtrace

scenario: Allow set-property action to work much earlier

By default an action has no playback-time, this makes it actionable
immediatly.

When no playback-time is set on a set-property action, it will
be activated the moment the element is added in the pipeline.

validate-bin-monitor: Initialize local variable

Avoids segfaults when freeing them if they didn't get filled in

validate: update pre-commit hook.

+ Allows to run multiple pre-commit hooks.
+ Always relink the hooks on autogen.
+ Run pep8 on commited python files.

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

apps: gstvalidate.py: fix various pyflakes / uncaught pep8 issues.

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

validate-launcher: utils: fix various pyflakes / uncaught pep8 issues.

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

validate-launcher: main: fix various pyflakes / uncaught pep8 issues.

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

validate-launcher: loggable: fix various pyflakes / uncaught pep8 issues.

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

validate-launcher: baseclasses: fix various pyflakes / uncaught pep8 issues.

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

validate-launcher: pep8ify sources.

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

validate-utils: downgrade ERROR to DEBUG.

This function is called in places where it is legit for it
to return NULL.

launcher: add a way to specify an application directory.

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

launcher: Don't implement product-specific TestManagers.

This manager will be moved in GES.

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

validate-launcher: restructure filesystem

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

validate: Fix compiler warning about implicit enum type conversion

gst-validate-reporter.c:119:39: error: implicit conversion from enumeration type
      'GstValidateReportingDetails' to different enumeration type
      'GstValidateInterceptionReturn' [-Werror,-Wenum-conversion]
  GstValidateInterceptionReturn ret = GST_VALIDATE_SHOW_UNKNOWN;
                                ~~~   ^~~~~~~~~~~~~~~~~~~~~~~~~
gst-validate-reporter.c:124:11: error: implicit conversion from enumeration type
      'GstValidateReportingDetails' to different enumeration type
      'GstValidateInterceptionReturn' [-Werror,-Wenum-conversion]
    ret = iface->get_reporting_level (reporter);
        ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gst-validate-reporter.c:127:10: error: implicit conversion from enumeration type
      'GstValidateInterceptionReturn' to different enumeration type
      'GstValidateReportingDetails' [-Werror,-Wenum-conversion]
  return ret;
  ~~~~~~ ^~~

build: We install all headers system wide for now.

Will be fixed when the API is deemed stable enough

validate: rename GstValidateReportingLevel.

Removes the confusion with GstValidateReportLevel.

Modeled on GstDebugGraphDetails.

validate: Verify that elements always send a segment before pushing EOS

EOS is some kind of data flow and thus a segment event should always be
pushed before the EOS is sent

validate: Properly check that the seqnum of the EOS is always properly set

In the pipeline, an EOS should always have the same seqnum of the
previous SEGMENT event that was received. If the segment is the result
of a seek, it should always be the same as the seek seqnum too.

+ (Mathieu Duponchelle): fix reporting and concatenation tests.

validate: Add support for text based override files

Allowing user to easily determine the severity of issue
types in a config file

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

validate: Remove unused method

gst_media_descriptor_add_frame is not used anywhere

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

validate: Move GstStructure file parsing into utils

So it can be reused, at least in GstValidate.

validate: report: Simplify the issue ID registering using GQuarks

+ Remove unused issue types

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

validate: Check all buffers when we have the info from MediaDescriptor

We now check that each buffer is the expected one for each buffer that
come into the decoder.

+ Fix some minor leaks in test-utils

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

validate:media-descriptor-parser: Add a way to create from a string

So it is simple to make use of it from the testsuite

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

report: g_critical are CRITICAL issues!

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

validate: launcher: Fix printing of errors in final report

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

validate: Add the possibility to generate media infos with frame descs

+ Fix a little issue when the generation fails.

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

validate: MediaDescriptors: Add md5sum to buffer informations

In the media descriptor files, we now have the md5sum of the actual
content of encoded buffers so that we can check that the buffer content is
perfectly what is was supposed to be.

+ Fix the check of whether a frame is a keyframe in the string
  comparison (g_ascii_strcasecmp  return 0 if string matches)

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

validate: Move some method between GstMediaDescriptorParser and GstMediaDescriptor

So that method land where they actually belong.

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

validate: Add a way to pass a MediaDescriptor around monitors

And add an option in gst-validate so that the user can define what
media descriptor file to use.

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

validate:media-descriptor: Handle stream with no tags

It was segfaulting before.

validate: Add more files to gitignore

validate: Move some method between GstMediaDescriptorParser and GstMediaDescriptor

So that method land where they actually belong.

validate-report / reporter: rework the way we repeat issues.

+ runner: update reports count algorithm.

pad-monitor: mark the peer pad as EOS too.

When a sink pad gets EOS, its src pad monitor should also
be marked as EOS (helpful with issue concatenation).

validate-pad-monitor / runner: Check per-object reporting levels.

validate-report: Set conditions in which a report can't be 

validate-report: Add a reporting level field and setter.

validate-runner: implement synthetic report.

+ Fix criticals logic in validate_runner_printf
+ Update padmonitor tests
+ Split validate_report_printf function.

validate-runner: Implement REPORT_NONE for global reporting.

Yeah that was tough. Helpful already though, for example:
GST_VALIDATE_REPORT_LEVEL=none,x:all gst-validate src name=x ! sink
will only report issues reported by the source.

+ Add test.

tests: Check monitors correctly determine their reporting level.

+ [API] gst_validate_reporter_get_reporting_level

validate-runner / monitor: Let the user single out pads.

That's some pretty specific code but it should be helpful.
The following syntax can be used : element-name::pad-name.

+ Free return of gst_object_get_name.

validate-runner / reporter: Sanitize reports refcounting.

The previous code worked but was confusing, the runner didn't actually
take the ref it was releasing later.

+ Fix indentation.

tests: Test reports refcounts.

+ Set the element monitor on the element as qdata.

validate-monitor: Determine the reporting level at setup.

validate-runner: Add code to parse GST_VALIDATE_REPORT_LEVEL.

+ Extend the tests.
+ [API] gst_validate_runner_get_default_reporting_level
+ [API] gst_validate_runner_get_reporting_level_for_name

validate-runner: report-level initial work.

+ Defines reporting levels and document them.
+ Add API to get the default level.
+ fix indentation.
+ fix some typos.
+ Add the beginning of a reporting test.

validate-pad-monitor: concatenate issues.

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

The process is to check for a similar report in intercept_report on
the pads of the upstream element, set that report as the master report
of the intercepted report, and return REPORTER_KEEP instead
of REPORTER_REPORT.

test-utils: add a create_and_monitor element function.

validate-reporter: Add some methods

+ gst_validate_reporter_get_reports
+ gst_validate_reporter_get_reports_count

validate-report: Add the notion of master / shadow reports.

A master report is a report that has been detected by a monitor
to stem from the same issue. It thus contains a list of
"shadow reports" which it will browse when printing itself.

validate-report: Make the ref / unref functions safer.

tests/padmonitor: Correcly strdup the result of get_metadata.

The const pointer was becoming invalid after the first call to add_metadata,
and we ended up setting corrupted data on the second call.

validate-reporter: Add return value to intercept_report.

It will allow to drop, keep or report reports.

validate-pad-monitor: Reimplement reporter interface.

+ Do nothing there for now, except chain up.

validate-reporter: add gst_validate_reporter_get_report.

+ Add locking.

Back to development

Release 1.4.0

validate: Print position if it could properly be queried

Otherwize we will print meaningless garbage.

validate:launcher: Minor enhancement in the documentation

validate: Rename action type playback_time to playback-time

Keeping backward compatiblity with the old naming

validate: Rename gst_validate_add_action_type to gst_validate_register_action_type

The _register naming corresponds much better to what the method does
and makes it more similar to how we refer to this kind of action in
GStreamer.

It is a last minute API change, but that API should not change anymore
after 1.4 is released.

validate: Fix the addition of playback_time in the parameter types

validate: Rename --list-action-types to --inspect-action-type

Making clearer the meaning of the parameter and closer to the
usual naming in the GStreamer land.

validate: Add the 'flags' for the seek action type

This was always a mandatory field but was not documented

validate: mishandled pointer criticals

Free glist of criticals

Signed-off-by: Anuj Jaiswal <anuj.jaiswal@samsung.com>
https://bugzilla.gnome.org/show_bug.cgi?id=736313

validate:docs: Add documentation about the default testsuite

validate-runner: switch to using a GList for the reports.

+ Return a copy of that list in get_reports.
+ update tests.

validate-runner: Hide implementation.

gst-validate-runner: Add locking for the reports list.

Release 1.3.90

validate:launcher: Factorize code to get a MediaDescriptor name for classname

Instead of copy/pasting that code badly

validate/docs: Add location of Scenario/Action defines

validate: (performance issue)refactor to remove duplicate assignment

Signed-off-by: Anuj Jaiswal <anuj.jaiswal@samsung.com>
https://bugzilla.gnome.org/show_bug.cgi?id=736412

validate: docs: Always dist the launcher directoty

Fixing make distcheck

validate: Get the Runner reports in order of arrival

Making sure they are printed in the right order

validate: Start a testsuite

Currently implemented tests are:
  * Settup and cleanup on monitor is done properly
  * Some tests in the PadMonitor are done properly, namely:
        - Buffer before segment
        - Buffer outside segment
        - First buffer running time is always 0
        - The Demuxer flow aggregation is properly checked

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

validate/private: Avoid double typdef

Instead just include required (public and local) header

gst-validate-scenario.h:43:44: error: redefinition of typedef 'GstValidateActionParameter' is a C11 feature [-Werror,-Wtypedef-redefinition]

timeline: disable tooltip while scrolling

Hide some columns by default

columns: optimize cell data functions a little

Fix crash when copying row to clipboard

timeline: fix incorrect position after scrolling using the timeline

Apparently events are dropped internally, so the last position after you stop
dragging can be off.

models: store line offsets in arrays

Use pango markup instead of attributes

Attributes don't work from introspection, so this blocks porting to gtk3.

In MessageColumn, admit that multiple highlighters don't actually work.