From: Jianpeng Ma Date: Fri, 16 Jan 2015 08:14:17 +0000 (+0800) Subject: test: Using different filename for different test case. X-Git-Tag: v0.93~261^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F3385%2Fhead;p=ceph.git test: Using different filename for different test case. Some test case use tmp file to test.But they used same file and create in the same directory. If we do in parallel, it will cause error. So different test case use own their tmp file. Signed-off-by: Jianpeng Ma --- diff --git a/src/test/bufferlist.cc b/src/test/bufferlist.cc index 8f6a87524bcc1..787ce7e429774 100644 --- a/src/test/bufferlist.cc +++ b/src/test/bufferlist.cc @@ -37,6 +37,9 @@ #include "sys/stat.h" #define MAX_TEST 1000000 +#define FILENAME "bufferlist" + +static char cmd[128]; TEST(Buffer, constructors) { bool ceph_buffer_track = get_env_bool("CEPH_BUFFER_TRACK"); @@ -150,15 +153,16 @@ TEST(Buffer, constructors) { EXPECT_THROW(buffer::create_zero_copy(len, -1, NULL), buffer::error_code); unsigned zc_len = 4; - ::unlink("testfile"); - EXPECT_EQ(0, ::system("echo ABC > testfile")); - int fd = ::open("testfile", O_RDONLY); + ::unlink(FILENAME); + snprintf(cmd, sizeof(cmd), "echo ABC > %s", FILENAME); + EXPECT_EQ(0, ::system(cmd)); + int fd = ::open(FILENAME, O_RDONLY); bufferptr ptr(buffer::create_zero_copy(zc_len, fd, NULL)); EXPECT_EQ(zc_len, ptr.length()); if (ceph_buffer_track) EXPECT_EQ(zc_len, (unsigned)buffer::get_total_alloc()); ::close(fd); - ::unlink("testfile"); + ::unlink(FILENAME); } #endif if (ceph_buffer_track) @@ -178,14 +182,15 @@ class TestRawPipe : public ::testing::Test { protected: virtual void SetUp() { len = 4; - ::unlink("testfile"); - EXPECT_EQ(0, ::system("echo ABC > testfile")); - fd = ::open("testfile", O_RDONLY); + ::unlink(FILENAME); + snprintf(cmd, sizeof(cmd), "echo ABC > %s", FILENAME); + EXPECT_EQ(0, ::system(cmd)); + fd = ::open(FILENAME, O_RDONLY); assert(fd >= 0); } virtual void TearDown() { ::close(fd); - ::unlink("testfile"); + ::unlink(FILENAME); } int fd; unsigned len; @@ -295,21 +300,21 @@ TEST_F(TestRawPipe, buffer_list_read_fd_zero_copy) { } TEST_F(TestRawPipe, buffer_list_write_fd_zero_copy) { - ::unlink("testfile_out"); + ::unlink(FILENAME); bufferlist bl; EXPECT_EQ(0, bl.read_fd_zero_copy(fd, len)); EXPECT_TRUE(bl.can_zero_copy()); - int out_fd = ::open("testfile_out", O_RDWR|O_CREAT|O_TRUNC, 0600); + int out_fd = ::open(FILENAME, O_RDWR|O_CREAT|O_TRUNC, 0600); EXPECT_EQ(0, bl.write_fd_zero_copy(out_fd)); struct stat st; memset(&st, 0, sizeof(st)); - EXPECT_EQ(0, ::stat("testfile_out", &st)); + EXPECT_EQ(0, ::stat(FILENAME, &st)); EXPECT_EQ(len, st.st_size); char buf[len + 1]; EXPECT_EQ((int)len, safe_read(out_fd, buf, len + 1)); EXPECT_EQ(0, memcmp(buf, "ABC\n", len)); ::close(out_fd); - ::unlink("testfile_out"); + ::unlink(FILENAME); } #endif // CEPH_HAVE_SPLICE @@ -1874,14 +1879,16 @@ TEST(BufferList, hexdump) { TEST(BufferList, read_file) { std::string error; bufferlist bl; - ::unlink("testfile"); + ::unlink(FILENAME); EXPECT_EQ(-ENOENT, bl.read_file("UNLIKELY", &error)); - EXPECT_EQ(0, ::system("echo ABC > testfile ; chmod 0 testfile")); + snprintf(cmd, sizeof(cmd), "echo ABC > %s ; chmod 0 %s", FILENAME, FILENAME); + EXPECT_EQ(0, ::system(cmd)); if (getuid() != 0) - EXPECT_EQ(-EACCES, bl.read_file("testfile", &error)); - EXPECT_EQ(0, ::system("chmod +r testfile")); - EXPECT_EQ(0, bl.read_file("testfile", &error)); - ::unlink("testfile"); + EXPECT_EQ(-EACCES, bl.read_file(FILENAME, &error)); + snprintf(cmd, sizeof(cmd), "chmod +r %s", FILENAME); + EXPECT_EQ(0, ::system(cmd)); + EXPECT_EQ(0, bl.read_file(FILENAME, &error)); + ::unlink(FILENAME); EXPECT_EQ((unsigned)4, bl.length()); std::string actual(bl.c_str(), bl.length()); EXPECT_EQ("ABC\n", actual); @@ -1889,36 +1896,37 @@ TEST(BufferList, read_file) { TEST(BufferList, read_fd) { unsigned len = 4; - ::unlink("testfile"); - EXPECT_EQ(0, ::system("echo ABC > testfile")); + ::unlink(FILENAME); + snprintf(cmd, sizeof(cmd), "echo ABC > %s", FILENAME); + EXPECT_EQ(0, ::system(cmd)); int fd = -1; bufferlist bl; EXPECT_EQ(-EBADF, bl.read_fd(fd, len)); - fd = ::open("testfile", O_RDONLY); + fd = ::open(FILENAME, O_RDONLY); EXPECT_EQ(len, (unsigned)bl.read_fd(fd, len)); EXPECT_EQ(CEPH_PAGE_SIZE - len, bl.buffers().front().unused_tail_length()); EXPECT_EQ(len, bl.length()); ::close(fd); - ::unlink("testfile"); + ::unlink(FILENAME); } TEST(BufferList, write_file) { - ::unlink("testfile"); + ::unlink(FILENAME); int mode = 0600; bufferlist bl; EXPECT_EQ(-ENOENT, bl.write_file("un/like/ly", mode)); bl.append("ABC"); - EXPECT_EQ(0, bl.write_file("testfile", mode)); + EXPECT_EQ(0, bl.write_file(FILENAME, mode)); struct stat st; memset(&st, 0, sizeof(st)); - ::stat("testfile", &st); + ::stat(FILENAME, &st); EXPECT_EQ((unsigned)(mode | S_IFREG), st.st_mode); - ::unlink("testfile"); + ::unlink(FILENAME); } TEST(BufferList, write_fd) { - ::unlink("testfile"); - int fd = ::open("testfile", O_WRONLY|O_CREAT|O_TRUNC, 0600); + ::unlink(FILENAME); + int fd = ::open(FILENAME, O_WRONLY|O_CREAT|O_TRUNC, 0600); bufferlist bl; for (unsigned i = 0; i < IOV_MAX * 2; i++) { bufferptr ptr("A", 1); @@ -1928,9 +1936,9 @@ TEST(BufferList, write_fd) { ::close(fd); struct stat st; memset(&st, 0, sizeof(st)); - ::stat("testfile", &st); + ::stat(FILENAME, &st); EXPECT_EQ(IOV_MAX * 2, st.st_size); - ::unlink("testfile"); + ::unlink(FILENAME); } TEST(BufferList, crc32c) { diff --git a/src/test/objectstore/chain_xattr.cc b/src/test/objectstore/chain_xattr.cc index 73dfc3b656df1..7e2e69371292f 100644 --- a/src/test/objectstore/chain_xattr.cc +++ b/src/test/objectstore/chain_xattr.cc @@ -29,9 +29,10 @@ #include #define LARGE_BLOCK_LEN CHAIN_XATTR_MAX_BLOCK_LEN + 1024 +#define FILENAME "chain_xattr" TEST(chain_xattr, get_and_set) { - const char* file = "testfile"; + const char* file = FILENAME; ::unlink(file); int fd = ::open(file, O_CREAT|O_WRONLY|O_TRUNC, 0700); const string user("user."); @@ -148,7 +149,7 @@ TEST(chain_xattr, get_and_set) { } TEST(chain_xattr, listxattr) { - const char* file = "testfile"; + const char* file = FILENAME; ::unlink(file); int fd = ::open(file, O_CREAT|O_WRONLY|O_TRUNC, 0700); const string user("user."); @@ -196,7 +197,7 @@ int main(int argc, char **argv) { g_ceph_context->_conf->set_val("log_to_stderr", "false"); g_ceph_context->_conf->apply_changes(NULL); - const char* file = "testfile"; + const char* file = FILENAME; int x = 1234; int y = 0; int tmpfd = ::open(file, O_CREAT|O_WRONLY|O_TRUNC, 0700);