summaryrefslogtreecommitdiff
path: root/test/stream
diff options
context:
space:
mode:
authorUmang Jain <umang.jain@ideasonboard.com>2022-08-30 13:17:24 +0530
committerUmang Jain <umang.jain@ideasonboard.com>2022-09-01 16:49:59 +0530
commitbb67630b1cb81ba18f34add9eda0f81927ed9c85 (patch)
treeb5233ee02c53d819202a12d207e34f687e0f0d8a /test/stream
parent4aa71c2cd26e672d04ea1c696de45f1c630062c3 (diff)
tests: stream: Add a colorspace adjustment test
ColorSpace can be adjusted based on the stream's pixelFormat being requested. Add a test to check the adjustment logic defined in ColorSpace::adjust(). Signed-off-by: Umang Jain <umang.jain@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Diffstat (limited to 'test/stream')
-rw-r--r--test/stream/meson.build1
-rw-r--r--test/stream/stream_colorspace.cpp96
2 files changed, 97 insertions, 0 deletions
diff --git a/test/stream/meson.build b/test/stream/meson.build
index 73608ffd..89f51c18 100644
--- a/test/stream/meson.build
+++ b/test/stream/meson.build
@@ -1,6 +1,7 @@
# SPDX-License-Identifier: CC0-1.0
stream_tests = [
+ ['stream_colorspace', 'stream_colorspace.cpp'],
['stream_formats', 'stream_formats.cpp'],
]
diff --git a/test/stream/stream_colorspace.cpp b/test/stream/stream_colorspace.cpp
new file mode 100644
index 00000000..1b7afe65
--- /dev/null
+++ b/test/stream/stream_colorspace.cpp
@@ -0,0 +1,96 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/*
+ * Copyright (C) 2022, Ideas on Board Oy.
+ *
+ * stream_colorspace.cpp - Stream colorspace adjustment test
+ */
+
+#include <iostream>
+
+#include <libcamera/camera.h>
+#include <libcamera/formats.h>
+#include <libcamera/stream.h>
+
+#include "test.h"
+
+using namespace libcamera;
+using namespace std;
+
+class TestCameraConfiguration : public CameraConfiguration
+{
+public:
+ TestCameraConfiguration()
+ : CameraConfiguration()
+ {
+ }
+
+ Status validate() override
+ {
+ return validateColorSpaces();
+ }
+};
+
+class StreamColorSpaceTest : public Test
+{
+protected:
+ int run()
+ {
+ TestCameraConfiguration config;
+
+ StreamConfiguration cfg;
+ cfg.size = { 640, 320 };
+ cfg.pixelFormat = formats::YUV422;
+ cfg.colorSpace = ColorSpace::Srgb;
+ config.addConfiguration(cfg);
+
+ StreamConfiguration &streamCfg = config.at(0);
+
+ /*
+ * YUV pixelformat with sRGB colorspace should have Y'CbCr encoding
+ * adjusted.
+ */
+ config.validate();
+ if (streamCfg.colorSpace->ycbcrEncoding == ColorSpace::YcbcrEncoding::None) {
+ cerr << "YUV format must have YCbCr encoding" << endl;
+ return TestFail;
+ }
+
+ /*
+ * For YUV pixelFormat, encoding should be picked up according
+ * to primaries and transfer function, if 'None' is specified.
+ */
+ streamCfg.pixelFormat = formats::YUV422;
+ streamCfg.colorSpace = ColorSpace(ColorSpace::Primaries::Rec2020,
+ ColorSpace::TransferFunction::Rec709,
+ ColorSpace::YcbcrEncoding::None,
+ ColorSpace::Range::Limited);
+ config.validate();
+ if (streamCfg.colorSpace->ycbcrEncoding != ColorSpace::YcbcrEncoding::Rec2020) {
+ cerr << "Failed to adjust colorspace Y'CbCr encoding according"
+ << " to primaries and transfer function" << endl;
+ return TestFail;
+ }
+
+ /* For RGB pixelFormat, Sycc colorspace should get adjusted to sRGB. */
+ streamCfg.pixelFormat = formats::RGB888;
+ streamCfg.colorSpace = ColorSpace::Sycc;
+ config.validate();
+ if (streamCfg.colorSpace != ColorSpace::Srgb) {
+ cerr << "RGB format's colorspace should be set to Srgb" << endl;
+ return TestFail;
+ }
+
+ /* Raw formats should always set colorspace to ColorSpace::Raw. */
+ streamCfg.pixelFormat = formats::SBGGR8;
+ streamCfg.colorSpace = ColorSpace::Rec709;
+ config.validate();
+ if (streamCfg.colorSpace != ColorSpace::Raw) {
+ cerr << "Raw format must always have Raw colorspace" << endl;
+ return TestFail;
+ }
+
+ return TestPass;
+ }
+};
+
+TEST_REGISTER(StreamColorSpaceTest)