From 1a4ea3847785af1a089fda4165b921bb66e1ce37 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Wed, 26 Dec 2018 12:58:08 +0800 Subject: [PATCH] test: write log file to current directory instead of writing log files to '/tmp', write them to current directory. by default, ctest runs the tests in ${CMAKE_CURRENT_BINARY_DIR}, so, in this case, the log files are located in build/src/test/. this ensure that current user has enough previledge to remove the log files. this also allows multiple instances of 'unittest_log' to run in parallel. See-also: http://tracker.ceph.com/issues/36737 Signed-off-by: Kefu Chai (cherry picked from commit a13a6f1d0b39a541896591cd04fc1980bb1acfee) Conflicts: src/log/test.cc: trivial resolution --- src/log/test.cc | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/log/test.cc b/src/log/test.cc index e11505af45611..9a433666052ec 100644 --- a/src/log/test.cc +++ b/src/log/test.cc @@ -19,7 +19,7 @@ TEST(Log, Simple) Log log(&subs); log.start(); - log.set_log_file("/tmp/foo"); + log.set_log_file("foo"); log.reopen_log_file(); log.set_stderr_level(5, -1); @@ -53,7 +53,7 @@ TEST(Log, ManyNoGather) subs.add(1, "foo", 1, 1); Log log(&subs); log.start(); - log.set_log_file("/tmp/big"); + log.set_log_file("big"); log.reopen_log_file(); for (int i=0; i