Thibault Saunier
c469f9b2a4
pad-monitor: Check if channel-mask is present only if channels > 2
...
As it is not a mandatory field otherwize
https://bugzilla.gnome.org/show_bug.cgi?id=708499
2013-09-21 00:34:05 +02:00
Thibault Saunier
42857faaae
scenario: Add GST_VALIDATE_SCENARIOS_PATH environment variable
...
So you can specify the PATHS where to look for scenario files
2013-09-21 00:32:52 +02:00
Thiago Santos
e3e463c0cc
validate-scenario: track position query results closer
...
Always keep probing the pipeline for the current position and compare
with the latest requested seek segment to detect if the seek boundaries
are being respected
2013-09-20 15:18:35 -03:00
Thibault Saunier
b4a711c568
scenario: Make it possible to define mandatory fields
...
And give a descrpition for actions
2013-09-17 18:38:31 -03:00
Vincent Penquerc'h
ccc3f8d696
scenario: add a track switch command, and an audio track switch test
...
The "switch-track" command can be used to switch tracks. The "type"
argument selects which track type to change (can be "audio", "video",
or "text"). The "index" argument selects which track of this type
to use: it can be either a number, which will be the Nth track of
the given type, or a number with a "+" or "-" prefix, which means
a relative change (eg, "+1" means "next track", "-1" means "previous
track").
Conflicts:
validate/gst/validate/gst-validate-scenario.c
2013-09-17 18:38:31 -03:00
Thibault Saunier
f0ef3803dd
scenario: Make it possible to register action parsing funcs before init
2013-09-16 18:49:31 -03:00
Thibault Saunier
8a9e9a7fd7
data: Port remaning scenario files to new format
...
And add support to user declared timestamps -1.0 as GST_CLOCK_TIME_NONE
2013-09-13 12:03:44 -03:00
Thibault Saunier
ea1f0a64f2
scenario: Rework scenarios to be: 1- Simpler to write them, 2- extendible
...
Make the scenario files a list of GstStructure-s as strings
2013-09-09 19:31:21 -03:00
Thibault Saunier
711946879e
bin-monitor: Add a way to specify pipelines on which to set scenarios
...
When used with LD_PRELOAD, the application might use various pipelines
for several different thing, we need to make it possible to spcify a
specific pipeline (or set of pipelines) on which to run the scenario.
The format is in the form of:
scenario_name:pipelinename_pattern*
2013-09-09 19:07:32 -03:00
Thibault Saunier
5cea134655
scenario: Try to run scenarios in development first
2013-09-09 19:01:44 -03:00
Edward Hervey
746b6f41b2
tools: Update .gitignore for tools move
2013-09-09 15:36:15 +02:00
Thiago Santos
24b6284d17
pad-monitor: avoid false positives when a seek fails
...
Remove the expected seqnums for events when a seek fails, preventing
false positives at the final report
2013-09-05 16:15:40 -03:00
Vincent Penquerc'h
fefc5ffb41
monitor-preload: schedule a report printout at exit
...
Conflicts:
tools/gst-validate.c
2013-09-05 13:40:41 -03:00
Thiago Santos
789ea6e9d5
pad-monitor: fix typo on macro usage
...
Pass the correct variable to macro
2013-09-05 11:47:21 -03:00
Thiago Santos
1860712098
pad-monitor: allow flushing flow returns when pad is flushing
...
It should always be acceptable to return GST_FLOW_FLUSHING when the
pad is flushing
2013-09-05 11:46:46 -03:00
Thiago Santos
a1775e75e5
pad-monitor: removing bad check
...
Elements are allowed to accumulate segments, they don't have to push
1:1 segments as they receive
2013-09-04 08:51:54 -03:00
Thiago Santos
25c55501a0
pad-monitor: buffer timestamp ranges check
...
Improve buffer timestamp range check:
* Only do it for encoders or decoders
* Audio has an acceptable tolerance of 100ms
To do this, keep track of the caps on the pad and store
if it is dealing with audio or video
2013-09-03 15:35:36 -03:00
Thiago Santos
26cff77b72
pad-monitor: move caps check to common event handling
...
Allows both src and sink pad to keep track of the current caps, but
the duplicated caps check is still only applied to sink pads as
src pads can push the same caps multiple times when it isn't linked
2013-09-03 15:17:05 -03:00
Thiago Santos
bd0f19c105
pad-monitor: fix reference handling for expired events list
2013-09-02 20:41:35 -03:00
Thiago Santos
2e216818bd
Replacing mentions of qa with validate
2013-09-02 13:22:51 -03:00
Edward Hervey
cc65145eb1
pad-monitor: Check if iterator exists before trying to use it
2013-09-02 12:18:07 -03:00
Edward Hervey
788054bda7
pad-monitor: make debug log more readable
...
Use pad as the debug object to make logs more meaningful.
Also adds a FIXME note
2013-09-02 12:15:24 -03:00
Edward Hervey
fb58e16c5a
element-monitor: protect agains elements that have no klass
2013-09-02 12:11:25 -03:00
Thiago Santos
ea903da1c7
pad-monitor: use activate-mode function to detect when to clear pad data
...
Clear as much as a flush-stop when pad is deactivated
2013-09-02 12:05:46 -03:00
Edward Hervey
d2edb59a24
WIP: pad-monitor: Fix serialized event order check
2013-09-02 10:47:32 -03:00
Thiago Santos
fef4eba9a2
pad-monitor: also track eos event that should be emitted after a seek
...
When seeking out of the media file length, the element should push an
EOS with the same seqnum of the seek event
2013-09-02 10:46:55 -03:00
Thiago Santos
390b54f40d
scenario: add missing space
2013-09-02 10:46:42 -03:00
Edward Hervey
78e1cb82da
validate: prettify output of results
...
Makes the result a bit more readable than a compact multi-line list.
FIXME: Figure out how to print the description of the issues (which can
spawn multiple lines) in a nice way.
2013-09-02 10:03:07 -03:00
Thiago Santos
adca72b069
scenario: add missing line break after print
2013-08-29 14:27:34 -03:00
Thiago Santos
713da78201
pad-monitor: avoid tracking tag events
...
Tag events are hard to track and check if properly serialized because
they mutate too much inside elements. There is no reliable way currently
to match a tag event pushed into an element and another tag event
leaving the element (other than if the pointers are actually the same).
2013-08-29 14:26:05 -03:00
Thiago Santos
1c874d175e
pad-monitor: only do combined return checks for demuxers
...
Seems like the only place that gstreamer elements should really
care about it
2013-08-29 11:48:33 -03:00
Thiago Santos
a426198916
pad-monitor: add two useful macros for readability
...
Avoids using long macros and having to check for pad-monitor parent
existance
2013-08-29 11:47:58 -03:00
Vincent Penquerc'h
f5f4635d0e
media-info: add a track switching test
...
This test will find the first input selector with more than one
sink pad, and cycle through them till it gets back to the original
one. Five seconds between switches. The test checks that some data
was sent from the input selector when each of the sink pads was
selected.
2013-08-29 10:43:03 -03:00
Thibault Saunier
f893bb7534
scenario: Print on stdout when we seek
2013-08-28 19:14:31 -03:00
Thibault Saunier
1dcfaea37f
validate: Report an issue result of query state that position > duration
2013-08-28 19:00:58 -03:00
Thibault Saunier
78d2fde632
scenario: Execute action whenever we pass the expected position
...
We know are sequential so whenever the wanted position is passed we
should execute the action.
This avoid issue with the tolerance when we have high rate playback
2013-08-28 18:55:55 -03:00
Thiago Santos
1dd93bea2a
licenses: improving licensing info on all files
2013-08-28 16:58:11 -03:00
Thiago Santos
5f261269f8
pad-monitor: fix typo when acessing parents data
2013-08-28 16:49:07 -03:00
Thiago Santos
103bf74768
pad-monitor: reset buffer timestamp data after a flush
...
As the pad/element also clears its internal state
2013-08-27 18:26:14 -03:00
Thiago Santos
49d39be022
pad-monitor: also track flush events on probes
2013-08-27 16:16:08 -03:00
Thiago Santos
41e0bf1e91
tools: moving applications from gst/validate to tools
...
Keeps the CLI applications separate from the libs files
2013-08-27 12:04:31 -03:00
Vincent Penquerc'h
aa814d0189
gst-validate-scenario: fix scenario listing missing installed ones
...
Only scenarii in the current directory or the user's home directory
were being listed.
2013-08-27 11:52:11 -03:00
Vincent Penquerc'h
69597d9f5e
gst-validate-transcoding: fix help text to refer to URIs as URIs
...
Referring to them as files is confusing, as you'll try to use files
and not URIs.
2013-08-27 11:52:10 -03:00
Thiago Santos
2ba098976a
pad-monitor: Move repeated caps to test only on sinkpads
...
Testing on source pads can lead to false positives when pads are
unlinked. The caps event is sticky and will be pushed again later
when another buffer/event is pushed, leading to an acceptable
situation to push the caps twice.
2013-08-27 11:48:00 -03:00
Thiago Santos
b591f11b15
pad-monitor: add another acceptable flow return combination scenarios
...
A demuxer knows when to return EOS after samples are over, so it is
ok for it to return even when all src pads returned OK
2013-08-26 20:33:23 -03:00
Thiago Santos
a06a9038a4
pad-monitor: improve serialized event checks
...
If the event was already found at the first position of the array, it
shouldn't be searched on the rest of it.
This removes lots of false positives.
2013-08-26 20:33:23 -03:00
Thiago Santos
1a54b40c78
pad-monitor: fix aggregate flow return check for error situations
...
Flow flushing must be returned upstream to indicate an error situation
downstream
2013-08-26 20:33:22 -03:00
Thiago Santos
185d10bf79
gst-validate: print error message when starting the pipeline fails
...
Instead of just exiting silently
2013-08-26 20:33:22 -03:00
Edward Hervey
e48eadad75
pad-monitor: New check for duplicate caps event
...
We shouldn't get/push twice caps that are identical
2013-08-26 17:09:04 -03:00
Thiago Santos
104cccdfc0
media-info: avoid glib assert
2013-08-23 17:26:51 -03:00
Edward Hervey
4f3e1de776
report: Avoid repeating long macros
...
Makes the code a bit more readable and compact
2013-08-23 11:38:15 +02:00
Edward Hervey
60a2dfaf72
validate-report: Fix critical flag handling
...
criticals are warnings/issues also
warnings are issues also
2013-08-23 11:10:57 +02:00
Thibault Saunier
0609de7156
scenario: Have GstClockTime as second (in double) inside scenario files
...
Making it easier to read
2013-08-22 14:13:07 -03:00
Thibault Saunier
ff3e435e94
data: Add a Backward and Forward seeking scenario
2013-08-22 13:55:08 -03:00
Thibault Saunier
96ca875c56
validate: Set return value of apps to -1 only if a critical issues was reported
...
Conflicts:
gst/validate/gst-validate-transcoding.c
gst/validate/gst-validate.c
2013-08-22 13:54:39 -03:00
Thibault Saunier
375a47aa0d
transcoding: Make sure to initialize Gst before parsing options
...
Avoiding to break the help
2013-08-22 13:06:01 -03:00
Thibault Saunier
40e3b5881a
transcoding: Connect to the bus signals watch as the main watch might already be connected
2013-08-22 12:08:14 -03:00
Thibault Saunier
161e0cee12
scenario: Start monitoring the position only when the pipeline starts playing
...
Otherwize seeking with a playback_time=0 won't work properly
2013-08-22 12:06:29 -03:00
Thibault Saunier
f9ddb6f59e
scenario: Fix negative rate management
...
Properly parse the it has a gdouble and set the stop position of the seek as
seeked_position if the rate is negative
+ Add some debug
2013-08-22 12:05:18 -03:00
Thibault Saunier
ddd00741fd
scenario: Actions order in xml file is the order in which they must be executed
...
When seeking we might want to execute seeks at a playback time inferior than previous
seek, so we need to be able to define the order in which actions have to be
executed, the simplest way is to just concider that actions are always
order in the XML files.
+ Add some more debugs
Conflicts:
gst/validate/gst-validate-scenario.c
2013-08-22 12:04:08 -03:00
Thibault Saunier
15b3049764
scenario: Rename the seeks list to actions, and initialize action to 0 when allocating
2013-08-22 12:02:49 -03:00
Thibault Saunier
2102648c5c
scenario: Rename scenario xml files extension to .scenario
2013-08-22 12:02:02 -03:00
Thibault Saunier
fc61dcf1ed
validate: Connect to the bus signals watch as the main watch might already be connected
2013-08-22 12:01:44 -03:00
Thibault Saunier
de57b1455a
validate: Add a way to list avalaible scenarios
...
Conflicts:
gst/validate/gst-validate-transcoding.c
2013-08-22 12:00:04 -03:00
Thiago Santos
123bdea93a
gst-validate-transcoding: add signal handling and issues printing
...
Update to have the same features as gst-validate.
1) Handle interrupts properly, with the additional of having the
'eos-on-shutdown' argument that sends EOS to the pipeline. This is
very useful for transcoding processes to finish correctly.
2) Print issues on the end of application
2013-08-22 10:35:50 -03:00
Thiago Santos
db44d3aa27
gst-validate: add interrupt handler
...
Handle interrupt properly to still print issues when exiting
2013-08-22 10:35:34 -03:00
Edward Hervey
ec6abff9dd
pad-monitor: Fix source pad probe handling
...
type is a bitmask and not an enum
2013-08-21 13:26:27 -03:00
Thiago Santos
df228c5729
gst-validate: fix documentation after debug category changes
2013-08-21 13:10:42 -03:00
Edward Hervey
de930073fc
validate-reporter: More comprehensive debug message
...
Some issues don't have any arguments, so put the full details in.
2013-08-21 18:00:16 +02:00
Edward Hervey
529576e69b
validate: Only use one debugging category: validate
...
There's no point in having a different debug category per file, you
can filter it by source filename if you *really* want that.
2013-08-21 17:59:51 +02:00
Thiago Santos
7e5f458e63
gst-validate: print issues at the end
...
And improve documentation about usage
2013-08-21 12:14:22 -03:00
Thiago Santos
044b21521b
media-check: add results file comparison
...
Adds a new expected-results argument to receive a file that is used
as a base for comparison with the new results. In case differences are
found, the application will print those issues.
2013-08-21 11:03:19 -03:00
Thiago Santos
549bcc1fd5
reporter: do not print issues to stdout
2013-08-20 17:10:44 -03:00
Thiago Santos
36e5c626af
media-info: fix playback tests
...
They weren't waiting for the pipeline to properly change state
before sending seek events, that would cause some events to
return TRUE even if they were not handled
2013-08-20 15:44:10 -03:00
Thiago Santos
387f7c1b63
media-check: return nonzero if a test failed
2013-08-20 15:42:54 -03:00
Thiago Santos
1713442fd3
media-info: add playback and reverse-playback tests
...
The tests are very simple as they only write the first error they
found during playback. If no error is set, an empty string is
printed.
The playback pipeline isn't monitored with validate monitors for now
2013-08-20 13:24:31 -03:00
Thiago Santos
3f91779c32
rename: gst-validate-file-check -> gst-validate-media-check
...
It not only validates files, takes any URI
2013-08-20 11:43:06 -03:00
Thiago Santos
577ad5c127
media-info: add stream topology parsing
...
Currently it only saves/loads the main type, but all topology is
already being parsed for future use
2013-08-20 11:41:15 -03:00
Thiago Santos
3d93eb78f1
media-info: add duration and seekable entries
...
Add duration entry in ns and seekable as a boolean to a new group
'media-info'
2013-08-19 16:52:12 -03:00
Thiago Santos
1c32bbbac3
media-info: replacing file-checker with a simpler media-info struct
...
This struct stores information about a media and tests run on it. It
also has a few helper functions that allows storing the results to a
file and loading it back.
Instead of having the file-checker object that would compare the
extracted values from the file to expected results set to its properties,
the media-info will store the values and it will be possible to compare
old media-info with new media-info from the same file. This allows
tracking improvements and regressions on different gstreamer versions.
Right now, the media-info is very tiny and doesn't store much info, only
the uri and the file size in bytes, but it will receive more additions in
the upcoming commits for storing duration, media topology, seekability and
playback information.
2013-08-19 16:38:13 -03:00
Edward Hervey
6a3070a1e4
.gitignore: Update for 1.0 and cleanup
2013-08-16 15:15:51 +02:00
Edward Hervey
15ea3bd8aa
all: Enable more C warnings at build time
...
And fix the issues:
* Proper forward declaration
* static functions marked properly
* absolute includes
* declaration order
2013-08-16 15:13:54 +02:00
Edward Hervey
3e5c921217
reporter: Fix proper debug message output partially
...
In order for the special gstreamer print argument handler to be used
you can't use g_strdup_printf. You need to pass it the actual va_list.
2013-08-16 14:30:02 +02:00
Edward Hervey
501be30379
pad-monitor: Handle case where internal pad iterator is NULL
...
Can happen with inputselector
2013-08-16 14:30:02 +02:00
Edward Hervey
eca452978a
pad-monitor: Don't use signal that doesn't exist
...
Note that we should just ensure we always get the pads from the parent
2013-08-16 14:30:01 +02:00
Edward Hervey
511ab5e1d8
pad-monitor: Update raw audio caps checks
2013-08-16 14:30:01 +02:00
Edward Hervey
5a659f205d
pad-monitor: Fix locking issues
...
We were taking locks twice.
Also add debugging info when taking/releasing locks to help further similar issues
2013-08-16 14:30:01 +02:00
Edward Hervey
f3bd5ac0c0
file-checker: GstEncodingProfile is a GObject in 1.0
2013-08-16 14:30:01 +02:00
Thiago Santos
25688f4e50
gst-validate: port to 1.0
2013-08-15 01:46:27 -03:00
Thiago Santos
6df0b6c010
validade: add missing config.h includes
2013-08-14 20:03:43 -03:00
Thiago Santos
ee9f1ad9b2
validate: add init function
...
Adds an init() function that should be called before using the lib.
It takes care of calling all internal initializing functions in
gst-validete
2013-08-14 19:14:18 -03:00
Thiago Santos
3dea4388fd
tools: improve documentation
2013-08-14 18:04:23 -03:00
Thiago Santos
52002376ea
rename gst-qa -> gst-validate
2013-08-14 17:46:28 -03:00
Thiago Santos
ddc95f5b8c
pad-monitor: only do complete caps checks on setcaps
...
On get caps it is acceptable to have missing fields to simplify caps
negotiation
2013-08-14 16:00:50 -03:00
Thiago Santos
848c86712b
qa-preload: split to separate lib
...
It should only be used separately, otherwise it will wrap around any
pipeline from applications linking with gstqa
2013-08-13 13:40:48 -03:00
Thiago Santos
4f11a05baf
file-check: add reverse-playback test
...
Adds a test that checks if reverse playback works without errors
2013-08-13 13:28:40 -03:00
Edward Hervey
737a2664f2
qa-reporter: Make debug message a bit more readable
...
By surrounding it with double quotes
2013-08-13 09:33:26 -03:00
Edward Hervey
5fcbf1026d
pad-monitor: 0.10 uses "channel-positions" field in audio caps
...
And it's an array, not a string
2013-08-13 09:33:26 -03:00
Edward Hervey
c8e2e3aa67
Makefile: Clean up for make distcheck
...
Directories, headers, files weren't properly disted
Also clean up the various CFLAGS/HEADERS/SOURCES variables and remove
ones that aren't needed.
2013-08-13 09:33:25 -03:00
Thiago Santos
4af966f5ee
pad-monitor: use correct variable for segment comparisons
...
Use the expected versus the received instead of using the received
twice.
2013-08-09 12:37:49 -03:00