[libcamera-devel] [PATCH 3/5] test: camera: Remove test for bad Stream IDs

[Jacopo Mondi]

Hi Niklas,

On Wed, Apr 03, 2019 at 03:12:19AM +0200, Niklas Söderlund wrote:
> In preparation of reworking how a default configuration is retrieved
> from a camera remove test that stream IDs must be valid as the data type
> passed to Camera::streamConfiguration() will change. This change is in
> preparation for an invasive change.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

Reviewed-by: Jacopo Mondi <jacopo at jmondi.org>

> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  test/camera/configuration_default.cpp | 13 -------------
>  1 file changed, 13 deletions(-)
>
> diff --git a/test/camera/configuration_default.cpp b/test/camera/configuration_default.cpp
> index 53ee021d33ca39b1..856cd415f7a6aec1 100644
> --- a/test/camera/configuration_default.cpp
> +++ b/test/camera/configuration_default.cpp
> @@ -49,19 +49,6 @@ protected:
>  			return TestFail;
>  		}
>
> -		/*
> -		 * Test that asking for configuration for an array of bad streams
> -		 * returns an empty list of configurations.
> -		 */
> -		Stream *stream_bad = reinterpret_cast<Stream *>(0xdeadbeef);
> -		std::set<Stream *> streams_bad = { stream_bad };
> -		conf = camera_->streamConfiguration(streams_bad);
> -		if (!conf.empty()) {
> -			cout << "Failed to retrieve configuration for bad streams"
> -			     << endl;
> -			return TestFail;
> -		}
> -
>  		return TestPass;
>  	}
>  };
> --
> 2.21.0
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.libcamera.org/pipermail/libcamera-devel/attachments/20190403/3a93d114/attachment.sig>