[libcamera-devel] [PATCH v3] test: gstreamer: Add test for gstreamer single stream

[Laurent Pinchart]

On Tue, Aug 10, 2021 at 11:05:45AM +0100, Kieran Bingham wrote:
> Hi Vedant,
> 
> Thank you for this, I'm very glad to see gstreamer tests being introduced!
> 
> 
> On 10/08/2021 10:38, paul.elder at ideasonboard.com wrote:
> 
> <snip>
> 
> >> --- /dev/null
> >> +++ b/test/gstreamer/meson.build
> >> @@ -0,0 +1,15 @@
> >> +# SPDX-License-Identifier: CC0-1.0
> >> +
> >> +gstreamer_tests = [
> >> +    ['single_stream_test',   'gstreamer_single_stream_test.cpp'],
> >> +]
> >> +gstreamer_dep = dependency('gstreamer-1.0', required: true)
> 
> instead of required: true, we should depend upon whether we are built
> with gstreamer_support.
> 
> I think this would do it, but please check/test.
> 
> gstreamer_dep =
>  dependency('gstreamer-1.0', required: get_option('gstreamer'))
> 
> And then we would want to skip all tests from being built if there is no
> gstreamer:
> 
> 
> if not gstreamer_dep.found()gstallocator_dep.found()
>     subdir_done()
> endif

There a multiple dependencies for the gstreamer plugin. You can simply
use

if not gst_enabled
	subdir_done()
endif

> 
> I would put the dependency checking, and this subdir_done() addition at
> the beginning, before even listing the tests.
> 
> >> +
> >> +foreach t : gstreamer_tests
> >> +    exe = executable(t[0], t[1],
> >> +                     dependencies : [libcamera_private, gstreamer_dep],
> >> +                     link_with : test_libraries,
> >> +                     include_directories : test_includes_internal)
> >> +
> >> +    test(t[0], exe, suite : 'gstreamer', is_parallel : false)
> >> +endforeach
> >> diff --git a/test/meson.build b/test/meson.build
> >> index 3bceb5df..d0466f17 100644
> >> --- a/test/meson.build
> >> +++ b/test/meson.build
> >> @@ -11,6 +11,7 @@ subdir('libtest')
> >>  
> >>  subdir('camera')
> >>  subdir('controls')
> >> +subdir('gstreamer')
> >>  subdir('ipa')
> >>  subdir('ipc')
> >>  subdir('log')
-- 
Regards,

Laurent Pinchart