[libcamera-devel] [PATCH v2 30/32] test: controls: control_value: Expand test to cover all control types

[Kieran Bingham]

On 06/03/2020 16:00, Laurent Pinchart wrote:
> The ControlValueTest hasn't been updated for a long time and is
> outdated. Improve it to support all control types, and test the type(),
> isArray() and toString() methods.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> ---
> Changes since v1:
> 
> - Move change to ControlValue::toString() to a separate patch
> ---
>  test/controls/control_value.cpp | 110 +++++++++++++++++++++++++-------
>  1 file changed, 88 insertions(+), 22 deletions(-)
> 
> diff --git a/test/controls/control_value.cpp b/test/controls/control_value.cpp
> index a1ffa842f29e..37f415302ff6 100644
> --- a/test/controls/control_value.cpp
> +++ b/test/controls/control_value.cpp
> @@ -19,46 +19,112 @@ class ControlValueTest : public Test
>  protected:
>  	int run()
>  	{
> -		ControlValue integer(1234);
> -		ControlValue boolean(true);
> +		/*
> +		 * None type.
> +		 */
> +		ControlValue value;
> +		if (!value.isNone() || value.isArray()) {
> +			cerr << "Empty value is non-null" << endl;
> +			return TestFail;
> +		}
>  
> -		/* Just a string conversion output test... no validation */
> -		cout << "Int: " << integer.toString()
> -		     << " Bool: " << boolean.toString()
> -		     << endl;
> +		/*
> +		 * Bool type.
> +		 */
> +		value.set(true);
> +		if (value.isNone() || value.isArray() ||
> +		    value.type() != ControlTypeBool) {
> +			cerr << "Control type mismatch after setting to bool" << endl;
> +			return TestFail;
> +		}
>  
> -		if (integer.get<int32_t>() != 1234) {
> -			cerr << "Failed to get Integer" << endl;
> +		if (value.get<bool>() != true) {
> +			cerr << "Control value mismatch after setting to bool" << endl;
>  			return TestFail;
>  		}
>  
> -		if (boolean.get<bool>() != true) {
> -			cerr << "Failed to get Boolean" << endl;
> +		if (value.toString() != "true") {
> +			cerr << "Control string mismatch after setting to bool" << endl;
>  			return TestFail;
>  		}
>  
> -		/* Test an uninitialised value, and updating it. */
> +		/*
> +		 * Integer8 type.
> +		 */
> +		value.set(static_cast<uint8_t>(42));
> +		if (value.isNone() || value.isArray() ||
> +		    value.type() != ControlTypeByte) {
> +			cerr << "Control type mismatch after setting to uint8_t" << endl;
> +			return TestFail;
> +		}
>  
> -		ControlValue value;
> -		if (!value.isNone()) {
> -			cerr << "Empty value is non-null" << endl;
> +		if (value.get<uint8_t>() != 42) {
> +			cerr << "Control value mismatch after setting to uint8_t" << endl;
>  			return TestFail;
>  		}
>  
> -		value.set<bool>(true);
> -		if (value.isNone()) {
> -			cerr << "Failed to set an empty object" << endl;
> +		if (value.toString() != "42") {
> +			cerr << "Control string mismatch after setting to uint8_t" << endl;
>  			return TestFail;
>  		}
>  
> -		if (value.get<bool>() != true) {
> -			cerr << "Failed to get Booleans" << endl;
> +		/*
> +		 * Integer32 type.
> +		 */
> +		value.set(0x42000000);
> +		if (value.isNone() || value.isArray() ||
> +		    value.type() != ControlTypeInteger32) {
> +			cerr << "Control type mismatch after setting to int32_t" << endl;
> +			return TestFail;
> +		}
> +
> +		if (value.get<int32_t>() != 0x42000000) {
> +			cerr << "Control value mismatch after setting to int32_t" << endl;
> +			return TestFail;
> +		}
> +
> +		if (value.toString() != "1107296256") {
> +			cerr << "Control string mismatch after setting to int32_t" << endl;
> +			return TestFail;
> +		}
> +
> +		/*
> +		 * Integer64 type.
> +		 */
> +		value.set(static_cast<int64_t>(-42));
> +		if (value.isNone() || value.isArray() ||
> +		    value.type() != ControlTypeInteger64) {
> +			cerr << "Control type mismatch after setting to int64_t" << endl;
> +			return TestFail;
> +		}
> +
> +		if (value.get<int64_t>() != -42) {
> +			cerr << "Control value mismatch after setting to int64_t" << endl;
> +			return TestFail;
> +		}
> +
> +		if (value.toString() != "-42") {
> +			cerr << "Control string mismatch after setting to int64_t" << endl;
> +			return TestFail;
> +		}
> +
> +		/*
> +		 * Float type.
> +		 */
> +		value.set(-0.42f);
> +		if (value.isNone() || value.isArray() ||
> +		    value.type() != ControlTypeFloat) {
> +			cerr << "Control type mismatch after setting to float" << endl;
> +			return TestFail;
> +		}
> +
> +		if (value.get<float>() != -0.42f) {
> +			cerr << "Control value mismatch after setting to float" << endl;
>  			return TestFail;
>  		}
>  
> -		value.set<int32_t>(10);
> -		if (value.get<int32_t>() != 10) {
> -			cerr << "Failed to get Integer" << endl;
> +		if (value.toString() != "-0.420000") {
> +			cerr << "Control string mismatch after setting to float" << endl;
>  			return TestFail;
>  		}
>  
> 

-- 
Regards
--
Kieran