[libcamera-devel,v4,8/8] test: v4l2_device: Add format handling test

Message ID 20190227173837.6902-9-jacopo@jmondi.org
State Superseded
Headers show
Series
  • v4l2_(sub)dev: improvements and tests
Related show

Commit Message

Jacopo Mondi Feb. 27, 2019, 5:38 p.m. UTC
Add test for V4L2Device set and get format methods.

Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
---
 test/v4l2_device/meson.build      |  1 +
 test/v4l2_device/test_formats.cpp | 68 +++++++++++++++++++++++++++++++
 2 files changed, 69 insertions(+)
 create mode 100644 test/v4l2_device/test_formats.cpp

Patch

diff --git a/test/v4l2_device/meson.build b/test/v4l2_device/meson.build
index 9f7a7545ac9b..72b22a637a2f 100644
--- a/test/v4l2_device/meson.build
+++ b/test/v4l2_device/meson.build
@@ -6,6 +6,7 @@  v4l2_device_tests = [
   [ 'stream_on_off',      'stream_on_off.cpp' ],
   [ 'capture_async',      'capture_async.cpp' ],
   [ 'buffer_sharing',     'buffer_sharing.cpp' ],
+  [ 'test_formats',       'test_formats.cpp' ],
 ]
 
 foreach t : v4l2_device_tests
diff --git a/test/v4l2_device/test_formats.cpp b/test/v4l2_device/test_formats.cpp
new file mode 100644
index 000000000000..9fb1b5c73521
--- /dev/null
+++ b/test/v4l2_device/test_formats.cpp
@@ -0,0 +1,68 @@ 
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/*
+ * Copyright (C) 2019, Google Inc.
+ *
+ * libcamera V4L2 device format handling test
+ */
+
+#include <climits>
+#include <iostream>
+
+#include "v4l2_device.h"
+
+#include "v4l2_device_test.h"
+
+using namespace std;
+using namespace libcamera;
+
+class Format : public V4L2DeviceTest
+{
+protected:
+	int run();
+};
+
+int Format::run()
+{
+	V4L2DeviceFormat format = {};
+
+	int ret = capture_->getFormat(&format);
+	if (ret) {
+		cerr << "Failed to get format" << endl;
+		return TestFail;
+	}
+
+	format.width = UINT_MAX;
+	format.height = UINT_MAX;
+	ret = capture_->setFormat(&format);
+	if (ret) {
+		cerr << "Failed to set format: image resolution is invalid: "
+		     << "(UINT_MAX x UINT_MAX) but setFormat() should not fail."
+		     << endl;
+		return TestFail;
+	}
+
+	if (format.width == UINT_MAX || format.height == UINT_MAX) {
+		cerr << "Failed to update image format = (UINT_MAX x UINT_MAX)"
+		     << endl;
+		return TestFail;
+	}
+
+	format.width = 0;
+	format.height = 0;
+	ret = capture_->setFormat(&format);
+	if (ret) {
+		cerr << "Failed to set format: image resolution is invalid: "
+		     << "(0 x 0) but setFormat() should not fail."
+		     << endl;
+		return TestFail;
+	}
+
+	if (format.width == 0 || format.height == 0) {
+		cerr << "Failed to update image format = (0x0)" << endl;
+		return TestFail;
+	}
+
+	return TestPass;
+}
+
+TEST_REGISTER(Format);