diff options
| author | Vsevolod Stakhov <vsevolod@rspamd.com> | 2022-05-01 19:47:18 +0100 |
|---|---|---|
| committer | Vsevolod Stakhov <vsevolod@rspamd.com> | 2022-05-01 19:47:18 +0100 |
| commit | 25f1bddc7278b2537e5c2a04b1a59609f0e411ec (patch) | |
| tree | 100b6ae6411b335bf20b1c0c76a80e36a7e8d340 /src/libutil/cxx | |
| parent | 5c0c81f08cc75d6f8bde748265a6eb86ba904f3c (diff) | |
| download | rspamd-25f1bddc7278b2537e5c2a04b1a59609f0e411ec.tar.gz rspamd-25f1bddc7278b2537e5c2a04b1a59609f0e411ec.zip | |
[Minor] Improve test stability
Diffstat (limited to 'src/libutil/cxx')
| -rw-r--r-- | src/libutil/cxx/locked_file.cxx | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/src/libutil/cxx/locked_file.cxx b/src/libutil/cxx/locked_file.cxx index ca5600c5f..f392d9b4a 100644 --- a/src/libutil/cxx/locked_file.cxx +++ b/src/libutil/cxx/locked_file.cxx @@ -249,16 +249,20 @@ TEST_CASE("create and delete file") { CHECK(::access(fname.c_str(), R_OK) == 0); } // File must be deleted after this call - CHECK(::access(fname.c_str(), R_OK) == -1); - CHECK(errno == ENOENT); + auto ret = ::access(fname.c_str(), R_OK); + auto serrno = errno; + CHECK(ret == -1); + CHECK(serrno == ENOENT); // Create one more time { auto raii_locked_file = raii_locked_file::create_temp(fname.c_str(), O_RDONLY, 00600); CHECK(raii_locked_file.has_value()); CHECK(::access(fname.c_str(), R_OK) == 0); } - CHECK(::access(fname.c_str(), R_OK) == -1); - CHECK(errno == ENOENT); + ret = ::access(fname.c_str(), R_OK); + serrno = errno; + CHECK(ret == -1); + CHECK(serrno == ENOENT); } TEST_CASE("check lock") { @@ -272,8 +276,10 @@ TEST_CASE("check lock") { CHECK(::access(fname.c_str(), R_OK) == 0); } // File must be deleted after this call - CHECK(::access(fname.c_str(), R_OK) == -1); - CHECK(errno == ENOENT); + auto ret = ::access(fname.c_str(), R_OK); + auto serrno = errno; + CHECK(ret == -1); + CHECK(serrno == ENOENT); } } // TEST_SUITE |