[libcamera-devel] [PATCH 7/8] libcamera: test: remove test IPA and use dummy IPA instead

[Laurent Pinchart]

Hi Paul,

Thank you for the patch.

On Mon, May 27, 2019 at 06:35:39PM -0400, Paul Elder wrote:
> Use the dummy IPA for testing/sample IPA instead of the earlier test
> IPA. Remove the test IPA, and update tests and meson accordingly.

Nice, I like this. Please disregard my comment about using more
realistic names for the test module info in an earlier patch.

> Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
> ---
>  test/ipa/ipa_test.cpp |  8 ++++----
>  test/ipa/meson.build  | 10 ----------
>  2 files changed, 4 insertions(+), 14 deletions(-)
> 
> diff --git a/test/ipa/ipa_test.cpp b/test/ipa/ipa_test.cpp
> index f50880e..53e3a6e 100644
> --- a/test/ipa/ipa_test.cpp
> +++ b/test/ipa/ipa_test.cpp
> @@ -82,12 +82,12 @@ protected:
>  
>  		const struct IPAModuleInfo testInfo = {
>  			1,
> -			0, 9001,
> -			"bleep",
> -			"It's over nine thousand!",
> +			0, 1,
> +			"PipelineHandlerVimc",
> +			"Dummy IPA for Vimc",
>  		};
>  
> -		count += runTest("test/ipa/ipa-dummy-cpp.so", testInfo);
> +		count += runTest("src/ipa/ipa_dummy.so", testInfo);
>  
>  		if (count < 0)
>  			return TestFail;
> diff --git a/test/ipa/meson.build b/test/ipa/meson.build
> index 08ee95c..bca39fa 100644
> --- a/test/ipa/meson.build
> +++ b/test/ipa/meson.build
> @@ -1,13 +1,3 @@
> -ipa_modules_sources = [
> -    ['ipa-dummy-cpp', 'shared_test.cpp'],
> -]
> -
> -foreach m : ipa_modules_sources
> -    shared_library(m, name_prefix : '',
> -                   dependencies : libcamera_dep,
> -                   include_directories : test_includes_public)
> -endforeach
> -
>  ipa_test = [
>      ['ipa_test', 'ipa_test.cpp'],
>  ]

-- 
Regards,

Laurent Pinchart