From 256d0a4098aa4c4e9f87db46cecbd66f693dd9bf Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Mon, 27 Apr 2020 21:04:06 +0300 Subject: test: Use float values for brightness, contrast and saturation Two tests use the brightness, contrast and saturation controls with integer failures. They were not updated by commit eff4b1aa01c1 which turned those controls into floats. This doesn't cause test failures as the control API converts the value types. For correctness, update the tests to use float values. Fixes: eff4b1aa01c1 ("libcamera: controls: Reorder and update description of existing controls") Signed-off-by: Laurent Pinchart Reviewed-by: Jacopo Mondi --- test/serialization/control_serialization.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'test/serialization') diff --git a/test/serialization/control_serialization.cpp b/test/serialization/control_serialization.cpp index 2989b527..11136524 100644 --- a/test/serialization/control_serialization.cpp +++ b/test/serialization/control_serialization.cpp @@ -42,9 +42,9 @@ protected: const ControlInfoMap &infoMap = camera_->controls(); ControlList list(infoMap); - list.set(controls::Brightness, 255); - list.set(controls::Contrast, 128); - list.set(controls::Saturation, 50); + list.set(controls::Brightness, 0.5f); + list.set(controls::Contrast, 1.2f); + list.set(controls::Saturation, 0.2f); /* * Serialize the control list, this should fail as the control -- cgit v1.2.1