[RFC PATCH v2 12/16] apps: lc-compliance: Add message to `GTEST_SKIP()`
Barnabás Pőcze
pobrn at protonmail.com
Tue Jan 14 19:22:47 CET 2025
Just like other gtest macros, `GTEST_SKIP()` returns an object
to which a formatted message can be added using the usual `<<`
stream operator. So use it instead of printing to `std::cout`.
Signed-off-by: Barnabás Pőcze <pobrn at protonmail.com>
---
src/apps/lc-compliance/helpers/capture.cpp | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/src/apps/lc-compliance/helpers/capture.cpp b/src/apps/lc-compliance/helpers/capture.cpp
index 91c4d4400..43db15d2d 100644
--- a/src/apps/lc-compliance/helpers/capture.cpp
+++ b/src/apps/lc-compliance/helpers/capture.cpp
@@ -26,10 +26,8 @@ void Capture::configure(StreamRole role)
{
config_ = camera_->generateConfiguration({ role });
- if (!config_) {
- std::cout << "Role not supported by camera" << std::endl;
- GTEST_SKIP();
- }
+ if (!config_)
+ GTEST_SKIP() << "Role not supported by camera";
if (config_->validate() != CameraConfiguration::Valid) {
config_.reset();
@@ -85,10 +83,8 @@ void CaptureBalanced::capture(unsigned int numRequests)
/* No point in testing less requests then the camera depth. */
if (buffers.size() > numRequests) {
- std::cout << "Camera needs " + std::to_string(buffers.size())
- + " requests, can't test only "
- + std::to_string(numRequests) << std::endl;
- GTEST_SKIP();
+ GTEST_SKIP() << "Camera needs " << buffers.size()
+ << " requests, can't test only " << numRequests;
}
queueCount_ = 0;
--
2.48.0
More information about the libcamera-devel
mailing list