Edward Hervey [Wed, 19 Jul 2017 09:02:44 +0000 (11:02 +0200)]
validate: Cast GList data content before usage
Apart from code readability, it allows compilers to detect wrong usages,
such as the call to gst_validate_action_new() which was using the wrong
argument
Thibault Saunier [Tue, 18 Jul 2017 16:09:13 +0000 (12:09 -0400)]
validate:launcher: Error out if no testsuite could be loaded
Thibault Saunier [Thu, 13 Jul 2017 20:43:32 +0000 (16:43 -0400)]
validate: launcher: Namespace test name with the testsuite name
Also allowing users to pass test names directly
Thibault Saunier [Fri, 14 Jul 2017 00:17:51 +0000 (20:17 -0400)]
validate: launcher: Properly use TestsLauncher.list_test to load tests
Otherwise we might skip check_defined_tests.
Thibault Saunier [Tue, 18 Jul 2017 14:47:00 +0000 (10:47 -0400)]
validate: Remove some dead code
CID 1415457
Thibault Saunier [Tue, 18 Jul 2017 14:45:29 +0000 (10:45 -0400)]
validate: Plug a minor string leak
CID 1415459
Thibault Saunier [Tue, 18 Jul 2017 14:42:00 +0000 (10:42 -0400)]
validate: Don't create scenario on a monitor which has no target
Not very probable but avoids a potential NULL pointer dereferencing.
CID 1415460
Thibault Saunier [Tue, 18 Jul 2017 14:36:34 +0000 (10:36 -0400)]
validate: Always only use the first description in scenarios
Also pluging a leak of the descrption copied structure
CID 1415463
Thibault Saunier [Tue, 18 Jul 2017 14:27:49 +0000 (10:27 -0400)]
validate: ssim: Do not compare unsigned to < 0
CID 1415473
Thibault Saunier [Tue, 18 Jul 2017 14:23:31 +0000 (10:23 -0400)]
validate: Plug a string leak
CID 1415475
Thibault Saunier [Tue, 18 Jul 2017 14:19:23 +0000 (10:19 -0400)]
validate:ssim: Let user know when no file have been compared
Fixing a possible division by zero issue.
CID 1415482
Thibault Saunier [Tue, 18 Jul 2017 14:16:07 +0000 (10:16 -0400)]
validate: Add missing break statement
CID 1415485
Thibault Saunier [Tue, 18 Jul 2017 14:14:59 +0000 (10:14 -0400)]
Check g_file_set_contents() return value
CID 1415486
Thibault Saunier [Tue, 18 Jul 2017 14:12:07 +0000 (10:12 -0400)]
validate: Plug leak of copy of a va_list
CID 1415490
Thibault Saunier [Tue, 18 Jul 2017 14:07:34 +0000 (10:07 -0400)]
validate: Do not check NULL pointer uselessly
CID 141593
Thibault Saunier [Tue, 18 Jul 2017 14:05:02 +0000 (10:05 -0400)]
validate: Plug minor leak in issue creation error path
CID 1415494
Thibault Saunier [Tue, 18 Jul 2017 14:00:03 +0000 (10:00 -0400)]
validate: Remove some dead code
Next will never be NULL as `done` is always set to TRUE when next is
set.
CID 1415503
Thibault Saunier [Tue, 18 Jul 2017 13:51:16 +0000 (09:51 -0400)]
validate: monitor: Add missing break; statement
Fixes CID 1415500
Edward Hervey [Tue, 18 Jul 2017 13:50:35 +0000 (15:50 +0200)]
validate-scenario: Fix wrong return value
We were always returning ok ...
CID #1415484
Edward Hervey [Tue, 18 Jul 2017 13:48:01 +0000 (15:48 +0200)]
validate-scenario: Check g_file_set_contents() return value
CID #1415487
Edward Hervey [Tue, 18 Jul 2017 13:45:13 +0000 (15:45 +0200)]
validate-scenario: Fix copy/paste error
CID #1415502
Edward Hervey [Tue, 18 Jul 2017 13:43:26 +0000 (15:43 +0200)]
validate-scenario: Fix string usage
Use the string representation of the index if it *IS* present (and
not the opposite).
CID #1415506
Edward Hervey [Tue, 18 Jul 2017 13:38:04 +0000 (15:38 +0200)]
validate-utils: Add NULL check
Unlikely to be triggered. And fix typo at the same time
CID #1415464
Sebastian Dröge [Fri, 14 Jul 2017 07:36:49 +0000 (10:36 +0300)]
validate: Un-blacklist scrub_forward_seeking.op2b-mpeg2-wave_hd_mxf
It works now after various mxfdemux changes.
https://bugzilla.gnome.org/show_bug.cgi?id=764025
Edward Hervey [Thu, 13 Jul 2017 06:45:28 +0000 (08:45 +0200)]
gstvalidate: Re-allow tests that should be fixed
Edward Hervey [Wed, 12 Jul 2017 12:46:36 +0000 (14:46 +0200)]
check: Remove dead assignments
Thibault Saunier [Tue, 11 Jul 2017 14:11:33 +0000 (10:11 -0400)]
validate: launcher: Add some missing env variables in command to launch test
Tim-Philipp Müller [Fri, 7 Jul 2017 11:26:40 +0000 (12:26 +0100)]
meson: find python3 via python3 module
https://bugzilla.gnome.org/show_bug.cgi?id=783198
Thibault Saunier [Fri, 30 Jun 2017 16:32:56 +0000 (12:32 -0400)]
validate: Recalculate latency on LATENCY messages
Thibault Saunier [Fri, 30 Jun 2017 16:30:40 +0000 (12:30 -0400)]
validate:scenario: Allow setting properties by element factory name
Thibault Saunier [Fri, 30 Jun 2017 13:46:57 +0000 (09:46 -0400)]
validate:scenario: Enhance the set_property action to handle enum props
User needs to specify the enum value as a string, to be used
as with gst_util_set_object_arg.
Also enhance reporting and verify that the set value has actually
been taken into account.
Thibault Saunier [Fri, 30 Jun 2017 13:45:02 +0000 (09:45 -0400)]
validate: Use Gst printing utils in our reporting system
Allowing us to use GST_PTR_FORMAT and friends!
Thibault Saunier [Wed, 28 Jun 2017 19:54:13 +0000 (15:54 -0400)]
validate:launcher: Do not check ModuleNotFound exception
It is a subclass of ImportError and is avalaible only since 3.6
https://ci.gstreamer.net/job/pitivi-flatpak/626/console
Thibault Saunier [Wed, 28 Jun 2017 17:01:47 +0000 (13:01 -0400)]
validate:launcher: Disable seek with stop on RTSP streams
It is actually not supported
Thibault Saunier [Fri, 23 Jun 2017 20:20:01 +0000 (16:20 -0400)]
meson: Allow using glib as a subproject
Thibault Saunier [Thu, 22 Jun 2017 19:26:08 +0000 (15:26 -0400)]
validate:launcher: Avoid useless and expensive deep copies
Thibault Saunier [Thu, 22 Jun 2017 17:08:30 +0000 (13:08 -0400)]
validate:launcher: Speed up xml parsing using lxml if avalaible
Thibault Saunier [Thu, 22 Jun 2017 16:01:12 +0000 (12:01 -0400)]
validate: make swicthing subtitle track while paused require prerolling
Thibault Saunier [Thu, 22 Jun 2017 15:53:49 +0000 (11:53 -0400)]
validate: Fix the change_state_intensive scenario
The scenario was in no way certified that the pipeline was in PAUSED
state when starting on an stream that does not preroll
Thibault Saunier [Wed, 21 Jun 2017 18:36:33 +0000 (14:36 -0400)]
validate:scenario: Enhance playbin3 stream selection error message
Thibault Saunier [Tue, 20 Jun 2017 19:51:27 +0000 (15:51 -0400)]
validate: Add missing space in segment mismatch issue description
Thibault Saunier [Tue, 20 Jun 2017 14:43:54 +0000 (10:43 -0400)]
validate:launcher: Enhance command printing when using a server
So it can be copy pasted and work
Thibault Saunier [Tue, 20 Jun 2017 14:43:09 +0000 (10:43 -0400)]
validate:launcher: No need to use -validate as a tracer for RTSP server
It is now linked into the server app
Thibault Saunier [Fri, 16 Jun 2017 21:31:19 +0000 (17:31 -0400)]
validate:launcher: Fix launching testsuite with relative paths
Thibault Saunier [Tue, 13 Jun 2017 21:13:22 +0000 (17:13 -0400)]
validate: Add a plugin with potential extra checks
And add a way to check that a configured number of instances of a particular
element is used, this is useful to make sure for example that playing a
particular stream doesn't lead to several decoders being instanciated.
Thibault Saunier [Tue, 13 Jun 2017 21:11:40 +0000 (17:11 -0400)]
validate: Fix the way we check if _CONFIG is a list of structures
Thibault Saunier [Tue, 13 Jun 2017 20:15:20 +0000 (16:15 -0400)]
validate: overrides: Add a hook about newly added elements in a bin
Thibault Saunier [Tue, 13 Jun 2017 20:08:23 +0000 (16:08 -0400)]
validate: Attach overrides before calling monitor.setup()
Reynaldo H. Verdejo Pinochet [Thu, 8 Jun 2017 20:43:41 +0000 (13:43 -0700)]
validate: libs: video: improve ssim's action registrations
Thibault Saunier [Thu, 8 Jun 2017 16:30:22 +0000 (12:30 -0400)]
validate: Do not link the rtsp server against validatevideo
It is not needed
https://bugzilla.gnome.org/show_bug.cgi?id=783554
Thibault Saunier [Thu, 8 Jun 2017 16:16:24 +0000 (12:16 -0400)]
validate:launcher: Do not which(None), the rtsp-server command is unset if not avalaible
https://bugzilla.gnome.org/show_bug.cgi?id=783551
Thibault Saunier [Wed, 7 Jun 2017 20:18:59 +0000 (16:18 -0400)]
meson: Make dependency on rtsp-server really optionnal
Thibault Saunier [Wed, 7 Jun 2017 19:06:10 +0000 (15:06 -0400)]
validate:launcher: Handle test that can't be run in parralel
Thibault Saunier [Wed, 7 Jun 2017 16:29:17 +0000 (12:29 -0400)]
meson: Do not use path separator in test names
Avoiding warnings like:
WARNING: Target "elements/audioamplify" has a path separator in its name.
Thibault Saunier [Wed, 7 Jun 2017 13:16:45 +0000 (09:16 -0400)]
validate: Add 'our own' RTSP server implementation
Thibault Saunier [Tue, 6 Jun 2017 20:23:48 +0000 (16:23 -0400)]
validate:launcher: Handle optional tests
For example RTSP tests might not be avalaible if gst-rtsp-server-example-uri is not avalaible
Thibault Saunier [Tue, 6 Jun 2017 16:45:31 +0000 (12:45 -0400)]
validate:launcher: Print logfiles when printing test result
It was often annoying to check wrong logs because the result are
not printed near the reference to logs.
Thibault Saunier [Tue, 6 Jun 2017 16:42:57 +0000 (12:42 -0400)]
validate: Add a way to use config actions in GST_VALIDATE_CONFIG files
The synthax is:
core, action=action-name, param1=1, param2=param2
Thibault Saunier [Mon, 5 Jun 2017 15:35:43 +0000 (11:35 -0400)]
scenario: Fallback to using media_info to determine stream duration
In the case the query duration return CLOCK_TIME_NONE.
Thibault Saunier [Wed, 10 May 2017 11:12:18 +0000 (08:12 -0300)]
validate:launcher: Do not run scenarios that need prerolling on RTSP stream
Thibault Saunier [Fri, 28 Apr 2017 20:59:21 +0000 (17:59 -0300)]
validate: Do not print \r in actual files
Thibault Saunier [Fri, 5 Dec 2014 11:16:36 +0000 (12:16 +0100)]
validate: Implement RTSP support
Reynaldo H. Verdejo Pinochet [Wed, 7 Jun 2017 06:39:21 +0000 (23:39 -0700)]
validate: reporter: add doc entry for gst_validate_report()'s varargs
Avoids GTK-Doc parser warnings
Reynaldo H. Verdejo Pinochet [Wed, 7 Jun 2017 06:01:32 +0000 (23:01 -0700)]
validate: scenario: fix typo/grammar issues in function documentation
Reynaldo H. Verdejo Pinochet [Wed, 7 Jun 2017 03:56:29 +0000 (20:56 -0700)]
validate: actions: drop needless newline before feature-rank and wait
Stale new-lines messed the help output of gst-validate -t
Reynaldo H. Verdejo Pinochet [Wed, 7 Jun 2017 03:25:10 +0000 (20:25 -0700)]
validate: actions: add info on mandatory fields for set-property
Additionally, drop a comment that becomes redundant after adding this
info to the action description
Reynaldo H. Verdejo Pinochet [Wed, 7 Jun 2017 02:38:23 +0000 (19:38 -0700)]
validate: scenario: fix description for 'seek' and 'stop'
Reynaldo H. Verdejo Pinochet [Wed, 7 Jun 2017 01:50:47 +0000 (18:50 -0700)]
validate: fix documentation for gst_validate_init()
Thibault Saunier [Fri, 2 Jun 2017 20:51:21 +0000 (16:51 -0400)]
validate: Update win32 def file
Thibault Saunier [Thu, 1 Jun 2017 20:38:25 +0000 (16:38 -0400)]
validate: Make Reporter.runner a MT safe weak reference
It can be used in any thread!
Thibault Saunier [Wed, 31 May 2017 18:06:04 +0000 (14:06 -0400)]
validate:launcher: Add missing is_live implementation for FakeMediaDescriptor
Thibault Saunier [Tue, 30 May 2017 21:42:07 +0000 (17:42 -0400)]
validate: Fix json serialized object leaks
Thibault Saunier [Tue, 30 May 2017 20:15:19 +0000 (16:15 -0400)]
validate:launcher: Handle not redirecting valgrind output
Thibault Saunier [Tue, 30 May 2017 20:14:51 +0000 (16:14 -0400)]
validate:launcher: Add a way to pass arguments to the leak tracer
Thibault Saunier [Tue, 30 May 2017 20:13:08 +0000 (16:13 -0400)]
validate: Use GWeakRefs on monitor target and pipeline
Making it thread safe and more future proof (though having them point
to NULL might not be handled all around).
https://bugzilla.gnome.org/show_bug.cgi?id=782784
Thibault Saunier [Thu, 18 May 2017 13:21:41 +0000 (15:21 +0200)]
validate: monitor: Unref our weak reference to the pipeline
Thibault Saunier [Fri, 28 Apr 2017 21:02:05 +0000 (18:02 -0300)]
validate: Make accessing Scenario.pipeline thread safe
The fact that Scenario.pipeline was not accessible in a thread way lead
to the fact that all users had to take the unref the last pipeline ref
in the main thread, otherwise we were crying. This was an ugly
restriction which lead to issue when using scenario on gst-rtsp-server.
This break the API as this commit remove the GstValidateScenario.pipeline
field but it is worth it.
Edward Hervey [Thu, 25 May 2017 13:57:33 +0000 (15:57 +0200)]
scenarios: Add/Update scenarios for live contents
Edward Hervey [Thu, 25 May 2017 13:50:23 +0000 (15:50 +0200)]
validate: Implement Scenario.__repr__
Allows better debugging when looking at logs
Edward Hervey [Thu, 25 May 2017 11:55:52 +0000 (13:55 +0200)]
validate: Add live-related features to scenarios and medias
Note: The notion of "live" here is in the *content* sense and not in the
GStreamer sense.
Ex:
* A rtsp stream is always "live" in the GStreamer sense but might not always
provide live content.
* HLS/DASH streams are not "live" in the GStreamer sense but might
provide "live" content.
Some scenarios might:
* require live content
* not be compatible with live content
This patch adds two new properties for scenarios:
* live_content_required (default False) for scenarios that can only work with
live content.
* live_content_compatible (default False) for scenarios that can work with
both live and non-live content.
This patch adds support for reading a "live" property from stream_info
Víctor Manuel Jáquez Leal [Sat, 20 May 2017 10:26:31 +0000 (12:26 +0200)]
validate: g_object_newv() is deprecated
Since glib version 2.54, g_object_newv() is deprecated.
This patch changes that function with a simpler g_object_new(),
since no properties are set.
https://bugzilla.gnome.org/show_bug.cgi?id=782860
Edward Hervey [Fri, 5 May 2017 12:57:20 +0000 (14:57 +0200)]
validate-report: Don't leak GError
Edward Hervey [Fri, 5 May 2017 12:57:56 +0000 (14:57 +0200)]
validate-scenario: Don't end up with invalid action name
When replacing an action structure, also update the action name with
the (new) name from the new structure. Otherwise we end up with
a bogus name from the previous (deleted) structure.
Edward Hervey [Wed, 3 May 2017 15:57:05 +0000 (17:57 +0200)]
validate-scenario: Avoid invalid memory access
The name of the action comes directly (i.e. not copied) from the
contained GstStructure field. Therefore make sure to take that
name from the proper structure field (copied just before) and
not from an outside one.
Reynaldo H. Verdejo Pinochet [Thu, 4 May 2017 21:45:32 +0000 (14:45 -0700)]
validate: drop superfluous whitespace from cmd description
Scott D Phillips [Mon, 1 May 2017 20:51:48 +0000 (13:51 -0700)]
validate: remove const from outfolder
GOptionEntry's arg_data is of type gpointer which differs in
constness from const gchar*, so remove constness from outfolder.
This fixes a build issue with msvc.
https://bugzilla.gnome.org/show_bug.cgi?id=782031
Sebastian Dröge [Thu, 4 May 2017 15:59:42 +0000 (18:59 +0300)]
Back to development
Sebastian Dröge [Thu, 4 May 2017 15:59:14 +0000 (18:59 +0300)]
Back to development
Sebastian Dröge [Thu, 4 May 2017 12:48:44 +0000 (15:48 +0300)]
Release 1.12.0
Thibault Saunier [Sun, 30 Apr 2017 17:35:29 +0000 (14:35 -0300)]
launcher: Allow using the base launcher as a test manager
No reason to force people to subclass it in simple cases.
Thibault Saunier [Thu, 27 Apr 2017 19:45:00 +0000 (16:45 -0300)]
tools: Fix dotfile name
Sebastian Dröge [Thu, 27 Apr 2017 14:54:05 +0000 (17:54 +0300)]
Release 1.11.91
Thibault Saunier [Sun, 16 Aug 2015 11:32:04 +0000 (13:32 +0200)]
validate: Fix update_stop.scenario seek property naming
Thibault Saunier [Thu, 20 Apr 2017 13:02:58 +0000 (10:02 -0300)]
launcher:check: Ensure to set build dir before rebuilding
Edward Hervey [Tue, 11 Apr 2017 05:48:21 +0000 (07:48 +0200)]
baseclasses: Add method to set a specific list of scenarios
https://bugzilla.gnome.org/show_bug.cgi?id=781314
Reynaldo H. Verdejo Pinochet [Fri, 7 Apr 2017 20:49:32 +0000 (13:49 -0700)]
validate: drop unneeded break+concat in some string literals
Perform other related improvements while at it.
Thibault Saunier [Wed, 12 Apr 2017 14:07:44 +0000 (11:07 -0300)]
validate: Do not expect track switch to be synchronous for playbin3
And let following actions to be executed (setting the action as
INTERLACED) which will make sure the track switch happened at some
point. It means the user has to set the pipeline to PLAYING so we can
make it works but we do not have choice here I think
https://bugzilla.gnome.org/show_bug.cgi?id=781213
Sebastian Dröge [Mon, 10 Apr 2017 14:06:25 +0000 (17:06 +0300)]
Update meson.build version to 1.11.90 too
Edward Hervey [Mon, 10 Apr 2017 09:43:28 +0000 (11:43 +0200)]
validate-pad-monitor: Fix previous commit
Post-send handling was only meant for seek events
Edward Hervey [Mon, 10 Apr 2017 05:58:01 +0000 (07:58 +0200)]
pad-monitor: Safely store expected seek values
Protect the expected seek values with the same lock as the one
that will be used to read/validate the resulting segments and flush
values.
Avoids races with duplicated seeks (i.e. a seek that was already
sent and handled via another pad, such as in demuxers).
https://bugzilla.gnome.org/show_bug.cgi?id=781112