summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNiklas Söderlund <niklas.soderlund@ragnatech.se>2019-03-05 23:49:03 +0100
committerNiklas Söderlund <niklas.soderlund@ragnatech.se>2019-03-14 15:59:09 +0100
commite44b63b3777df767d1e49f4e0ec2067b3638793f (patch)
treeb11a9eb95b3798b2702f000a2d0f59b87a23f497
parent86a1900657db9448a42593373f62fd7e9db3c6db (diff)
test: camera: Add setting of configuration test
Try to set the default configuration, a modified valid configuration and an invalid configuration. Signed-off-by: Niklas Söderlund <niklas.soderlund@ragnatech.se> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
-rw-r--r--test/camera/configuration_set.cpp91
-rw-r--r--test/camera/meson.build1
2 files changed, 92 insertions, 0 deletions
diff --git a/test/camera/configuration_set.cpp b/test/camera/configuration_set.cpp
new file mode 100644
index 00000000..dedb8500
--- /dev/null
+++ b/test/camera/configuration_set.cpp
@@ -0,0 +1,91 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/*
+ * Copyright (C) 2019, Google Inc.
+ *
+ * libcamera Camera API tests
+ */
+
+#include <iostream>
+
+#include "camera_test.h"
+
+using namespace std;
+
+namespace {
+
+class ConfigurationSet : public CameraTest
+{
+protected:
+ int run()
+ {
+ std::set<Stream *> streams = { *camera_->streams().begin() };
+ std::map<Stream *, StreamConfiguration> conf =
+ camera_->streamConfiguration(streams);
+ StreamConfiguration *sconf = &conf.begin()->second;
+
+ if (!configurationValid(streams, conf)) {
+ cout << "Failed to read default configuration" << endl;
+ return TestFail;
+ }
+
+ if (camera_->acquire()) {
+ cout << "Failed to acquire the camera" << endl;
+ return TestFail;
+ }
+
+ /* Test that setting the default configuration works. */
+ if (camera_->configureStreams(conf)) {
+ cout << "Failed to set default configuration" << endl;
+ return TestFail;
+ }
+
+ /*
+ * Test that configuring the camera fails if it is not
+ * acquired, this will also test release and reacquiring
+ * of the camera.
+ */
+ if (camera_->release()) {
+ cout << "Failed to release the camera" << endl;
+ return TestFail;
+ }
+
+ if (!camera_->configureStreams(conf)) {
+ cout << "Setting configuration on a camera not acquired succeeded when it should have failed"
+ << endl;
+ return TestFail;
+ }
+
+ if (camera_->acquire()) {
+ cout << "Failed to acquire the camera" << endl;
+ return TestFail;
+ }
+
+ /*
+ * Test that modifying the default configuration works. Doubling
+ * the default configuration of the VIMC camera is known to
+ * work.
+ */
+ sconf->width *= 2;
+ sconf->height *= 2;
+ if (camera_->configureStreams(conf)) {
+ cout << "Failed to set modified configuration" << endl;
+ return TestFail;
+ }
+
+ /*
+ * Test that setting an invalid configuration fails.
+ */
+ sconf->width = 0;
+ sconf->height = 0;
+ if (!camera_->configureStreams(conf)) {
+ cout << "Invalid configuration incorrectly accepted" << endl;
+ return TestFail;
+ }
+
+ return TestPass;
+ }
+};
+
+} /* namespace */
+
+TEST_REGISTER(ConfigurationSet);
diff --git a/test/camera/meson.build b/test/camera/meson.build
index 186ba211..ed483719 100644
--- a/test/camera/meson.build
+++ b/test/camera/meson.build
@@ -2,6 +2,7 @@
# They are not alphabetically sorted.
camera_tests = [
[ 'configuration_default', 'configuration_default.cpp' ],
+ [ 'configuration_set', 'configuration_set.cpp' ],
]
foreach t : camera_tests