From 36d62298b20bec1f052fac04fd3011511cc29226 Mon Sep 17 00:00:00 2001 From: Paul Elder Date: Sat, 13 Jul 2019 04:34:36 +0900 Subject: test: logging: move logging tests to a subdirectory Since there are two logging tests now, move them to their own subdirectory. Update meson as necessary. Signed-off-by: Paul Elder Reviewed-by: Laurent Pinchart --- test/log/log_api.cpp | 151 ++++++++++++++++++++++++++++++++++++++++++++++ test/log/log_process.cpp | 153 +++++++++++++++++++++++++++++++++++++++++++++++ test/log/meson.build | 13 ++++ 3 files changed, 317 insertions(+) create mode 100644 test/log/log_api.cpp create mode 100644 test/log/log_process.cpp create mode 100644 test/log/meson.build (limited to 'test/log') diff --git a/test/log/log_api.cpp b/test/log/log_api.cpp new file mode 100644 index 00000000..33622f84 --- /dev/null +++ b/test/log/log_api.cpp @@ -0,0 +1,151 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later */ +/* + * Copyright (C) 2019, Google Inc. + * + * log.cpp - log API test + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include + +#include "log.h" +#include "test.h" + +using namespace std; +using namespace libcamera; + +LOG_DEFINE_CATEGORY(LogAPITest) + +class LogAPITest : public Test +{ +protected: + void doLogging() + { + logSetLevel("LogAPITest", "DEBUG"); + LOG(LogAPITest, Info) << "good 1"; + + logSetLevel("LogAPITest", "WARN"); + LOG(LogAPITest, Info) << "bad"; + + logSetLevel("LogAPITest", "ERROR"); + LOG(LogAPITest, Error) << "good 3"; + LOG(LogAPITest, Info) << "bad"; + + logSetLevel("LogAPITest", "WARN"); + LOG(LogAPITest, Warning) << "good 5"; + LOG(LogAPITest, Info) << "bad"; + } + + int verifyOutput(istream &is) + { + list goodList = { 1, 3, 5 }; + string line; + while (getline(is, line)) { + if (goodList.empty()) { + cout << "Too many log lines" << endl; + return TestFail; + } + + unsigned int digit = line.back() - '0'; + unsigned int expect = goodList.front(); + goodList.pop_front(); + if (digit != expect) { + cout << "Incorrect log line" << endl; + return TestFail; + } + } + + if (!goodList.empty()) { + cout << "Too few log lines" << endl; + return TestFail; + } + + return TestPass; + } + + int testFile() + { + int fd = open("/tmp", O_TMPFILE | O_RDWR, S_IRUSR | S_IWUSR); + if (fd < 0) { + cerr << "Failed to open tmp log file" << endl; + return TestFail; + } + + char path[32]; + snprintf(path, sizeof(path), "/proc/self/fd/%u", fd); + + if (logSetFile(path) < 0) { + cerr << "Failed to set log file" << endl; + return TestFail; + } + + doLogging(); + + char buf[1000]; + memset(buf, 0, sizeof(buf)); + lseek(fd, 0, SEEK_SET); + if (read(fd, buf, sizeof(buf)) < 0) { + cerr << "Failed to read tmp log file" << endl; + return TestFail; + } + close(fd); + + istringstream iss(buf); + return verifyOutput(iss); + } + + int testStream() + { + stringstream log; + /* Never fails, so no need to check return value */ + logSetStream(&log); + + doLogging(); + + return verifyOutput(log); + } + + int testTarget() + { + logSetTarget(LoggingTargetNone); + logSetLevel("LogAPITest", "DEBUG"); + LOG(LogAPITest, Info) << "don't crash please"; + + if (!logSetTarget(LoggingTargetFile)) + return TestFail; + + if (!logSetTarget(LoggingTargetStream)) + return TestFail; + + return TestPass; + } + + int run() override + { + int ret = testFile(); + if (ret != TestPass) + return TestFail; + + ret = testStream(); + if (ret != TestPass) + return TestFail; + + ret = testTarget(); + if (ret != TestPass) + return TestFail; + + return TestPass; + } +}; + +TEST_REGISTER(LogAPITest) diff --git a/test/log/log_process.cpp b/test/log/log_process.cpp new file mode 100644 index 00000000..36d25b27 --- /dev/null +++ b/test/log/log_process.cpp @@ -0,0 +1,153 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later */ +/* + * Copyright (C) 2019, Google Inc. + * + * log_process.cpp - Logging in isolated child process test + */ + +#include +#include +#include +#include +#include +#include +#include +#include + +#include +#include +#include +#include + +#include "log.h" +#include "process.h" +#include "test.h" +#include "utils.h" + +using namespace std; +using namespace libcamera; + +static const string message("hello from the child"); + +LOG_DEFINE_CATEGORY(LogProcessTest) + +class LogProcessTestChild +{ +public: + int run(int status, int num) + { + usleep(50000); + + string logPath = "/tmp/libcamera.worker.test." + + to_string(num) + ".log"; + if (logSetFile(logPath.c_str()) < 0) + return TestSkip; + + LOG(LogProcessTest, Warning) << message; + + return status; + } +}; + +class LogProcessTest : public Test +{ +protected: + int init() + { + random_device random; + num_ = random(); + logPath_ = "/tmp/libcamera.worker.test." + + to_string(num_) + ".log"; + + proc_.finished.connect(this, &LogProcessTest::procFinished); + return 0; + } + + int run() + { + EventDispatcher *dispatcher = CameraManager::instance()->eventDispatcher(); + Timer timeout; + + int exitCode = 42; + vector args; + args.push_back(to_string(exitCode)); + args.push_back(to_string(num_)); + int ret = proc_.start("/proc/self/exe", args); + if (ret) { + cerr << "failed to start process" << endl; + return TestFail; + } + + timeout.start(200); + while (timeout.isRunning()) + dispatcher->processEvents(); + + if (exitStatus_ != Process::NormalExit) { + cerr << "process did not exit normally" << endl; + return TestFail; + } + + if (exitCode_ == TestSkip) + return TestSkip; + + if (exitCode_ != exitCode) { + cerr << "exit code should be " << exitCode + << ", actual is " << exitCode_ << endl; + return TestFail; + } + + int fd = open(logPath_.c_str(), O_RDONLY, S_IRUSR); + if (fd < 0) { + cerr << "failed to open tmp log file" << endl; + return TestFail; + } + + char buf[200]; + memset(buf, 0, sizeof(buf)); + if (read(fd, buf, sizeof(buf)) < 0) { + cerr << "Failed to read tmp log file" << endl; + return TestFail; + } + close(fd); + + string str(buf); + if (str.find(message) == string::npos) + return TestFail; + + return TestPass; + } + + void cleanup() + { + unlink(logPath_.c_str()); + } + +private: + void procFinished(Process *proc, enum Process::ExitStatus exitStatus, int exitCode) + { + exitStatus_ = exitStatus; + exitCode_ = exitCode; + } + + Process proc_; + Process::ExitStatus exitStatus_; + string logPath_; + int exitCode_; + int num_; +}; + +/* + * Can't use TEST_REGISTER() as single binary needs to act as both + * parent and child processes. + */ +int main(int argc, char **argv) +{ + if (argc == 3) { + int status = std::stoi(argv[1]); + int num = std::stoi(argv[2]); + LogProcessTestChild child; + return child.run(status, num); + } + + return LogProcessTest().execute(); +} diff --git a/test/log/meson.build b/test/log/meson.build new file mode 100644 index 00000000..95f6c1a2 --- /dev/null +++ b/test/log/meson.build @@ -0,0 +1,13 @@ +log_test = [ + ['log_api', 'log_api.cpp'], + ['log_process', 'log_process.cpp'], +] + +foreach t : log_test + exe = executable(t[0], t[1], + dependencies : libcamera_dep, + link_with : test_libraries, + include_directories : test_includes_internal) + + test(t[0], exe, suite : 'log') +endforeach -- cgit v1.2.1