diff options
author | Chris Wilson <chris+github@qwirx.com> | 2015-12-22 21:28:23 +0000 |
---|---|---|
committer | Chris Wilson <chris+github@qwirx.com> | 2015-12-22 21:28:23 +0000 |
commit | 9e20521b88d0520908d3194b5bfb9a4249072599 (patch) | |
tree | 8c6453045e985cc9da8ec8e1b323b1c2cb393da6 /test/backupstorefix | |
parent | 7ea37ed6e8a4f8ad1b3e9ca7bc75eba524a64648 (diff) |
Don't start and stop bbstored on Windows.
StopServer() does a hard terminate on Windows, which can leave temporary files
in place, causing problems later in the test.
Diffstat (limited to 'test/backupstorefix')
-rw-r--r-- | test/backupstorefix/testbackupstorefix.cpp | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/test/backupstorefix/testbackupstorefix.cpp b/test/backupstorefix/testbackupstorefix.cpp index 5e5a288e..a8cb759c 100644 --- a/test/backupstorefix/testbackupstorefix.cpp +++ b/test/backupstorefix/testbackupstorefix.cpp @@ -656,9 +656,12 @@ int test(int argc, const char *argv[]) BOX_INFO(" === Delete an entry for an object from dir, change that " "object to be a patch, check it's deleted"); { - // Temporarily stop the server, so it doesn't repair the - // refcount error + // Temporarily stop the server, so it doesn't repair the refcount error. Except + // on win32, where hard-killing the server can leave a lockfile in place, + // breaking the rest of the test. +#ifndef WIN32 TEST_THAT(StopServer()); +#endif // Open dir and find entry int64_t delID = getID("Test1/cannes/ict/metegoguered/oats"); @@ -762,9 +765,11 @@ int test(int argc, const char *argv[]) TEST_EQUAL(usage->GetBlocksInDirectories(), 56); } - // Start the server again, so testbackupstorefix.pl can - // run bbackupquery which connects to it. + // Start the server again, so testbackupstorefix.pl can run bbackupquery which + // connects to it. Except on win32, where we didn't stop it earlier. +#ifndef WIN32 TEST_THAT(StartServer()); +#endif // Check TEST_THAT(::system(PERL_EXECUTABLE |