[libcamera-devel,v2,0/5] apps: lc-compliance: Add controls and properties test
mbox series

Message ID 20231223111023.668199-1-jacopo.mondi@ideasonboard.com
Headers show
Series
  • apps: lc-compliance: Add controls and properties test
Related show

Message

Jacopo Mondi Dec. 23, 2023, 11:10 a.m. UTC 
v2->v1:
- I forgot to move the newly added test in the test/ subdir

Add a test case to validate that mandatory properties and controls are
registered by a Camera.

Before adding a new test, a little refresh of the lc-compliance sources might
help. I collected patches from the list for a few minor fixes and re-organized
the source tree by splitting helpers/ and tests/ in dedicated sub-directory.

Finally, add a test for controls and properties.

The series is based on "[PATCH 0/2] libcamera: Add support for required
controls" on which the test depends.

Thanks
  j

Jacopo Mondi (3):
  apps: lc-compliance: Fix typo in comment
  apps: lc-compliance: Re-organize source directory
  apps: lc-compliance: Test for mandatory controls

Nícolas F. R. A. Prado (2):
  apps: lc-compliance: Fix source file ordering in meson.build
  apps: lc-compliance: Check that requests complete successfully

 .../capture.cpp}                              | 48 +++++----
 .../{simple_capture.h => helpers/capture.h}   | 14 +--
 src/apps/lc-compliance/meson.build            | 11 ++-
 .../{ => tests}/capture_test.cpp              |  9 +-
 .../lc-compliance/tests/controls_test.cpp     | 98 +++++++++++++++++++
 5 files changed, 146 insertions(+), 34 deletions(-)
 rename src/apps/lc-compliance/{simple_capture.cpp => helpers/capture.cpp} (76%)
 rename src/apps/lc-compliance/{simple_capture.h => helpers/capture.h} (76%)
 rename src/apps/lc-compliance/{ => tests}/capture_test.cpp (95%)
 create mode 100644 src/apps/lc-compliance/tests/controls_test.cpp

--
2.41.0