| Commit message (Collapse) | Author | Age |
|---|
| |
|
|
| |
Use them in testbackupstorefix to reduce code duplication.
|
| |
|
|
|
| |
Fix a couple of compiler warnings, and don't continue the test if we failed
to load the configuration file.
|
| |
|
|
|
|
| |
Thanks to Jean-Yves Moulin for reporting this issue. When bbackupd is configured
to create locations whose names contain a slash, it was impossible to escape it,
and thus to enter the directory in bbackupquery to inspect or restore it.
|
| |
|
|
|
|
|
| |
This caused the issue reported by Brendon Baumgartner and described in my
email to the Box Backup list on Mon, 21 Apr 2014 at 18:44:38. If the directory
disappears then we used to try to send an empty attributes block to the server,
which is illegal.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
| |
Split single monolithic test into many smaller ones, which can be run
separately. Allows easier testing of new features in isolation.
Merged back changes from the test refactor branch to reduce diffs.
|
| |
|
|
|
| |
Reduces warnings about network operations with no timeout. May prevent some
hangs that might have occurred waiting forever for network data in the past.
|
| |
|
|
| |
Merged back changes from the test refactor branch to reduce diffs.
|
| |
|
|
|
|
| |
Makes way for more use of local protocol objects and protocol subclasses.
Merged back changes from the test refactor branch to reduce diffs.
|
| |
|
|
| |
Merged back changes from the test refactor branch to reduce diffs.
|
| |
|
|
|
|
| |
Hopefully all of these changes are inconsequential.
Merged back changes from the test refactor branch to reduce diffs.
|
| |
|
|
|
|
|
|
|
| |
Merged back changes from the test refactor branch to reduce diffs.
Fix wrong setting of log level in compare() function.
We want to only show errors if we expect mismatches, and show warnings if we
didn't expect mismatches. I had it the other way around before.
|
| |
|
|
|
|
|
| |
For some reason the timing changed and the test was no longer waiting for the
same length of time as bbackupd, causing it to fail.
Merged back changes from the test refactor branch to reduce diffs.
|
| | |
|
| |
|
|
|
|
|
|
| |
Similar to TEST_COMMAND_RETURNS_ERROR_OR, but allows any command or sequence
to cause the error, without direct access to the command message.
Refactor so that TEST_COMMAND_RETURNS_ERROR_OR calls TEST_PROTOCOL_ERROR_OR
to make the assertion on the last error seen by the protocol.
|
| |
|
|
| |
We don't really need these, so clean up by removing them.
|
| |
|
|
|
|
|
|
| |
Conflicts:
bin/bbackupd/BackupDaemon.cpp
bin/bbackupd/BackupDaemon.h
test/bbackupd/testbbackupd.cpp
|
| | |
|
| |
|
|
|
| |
It's incompatible with having a logger that logs everything, regardless of
the global log level.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
We pass a std::auto_ptr<SocketStream> to every Protocol subclass when we
construct it, and it takes control of this object. This reduces the risk of:
* accidentally reusing the same SocketStream for multiple Protocols
(it happened to me in testbackupstore);
* holding onto a reference to the SocketStream;
* allowing a locally-scoped SocketStream to go out of scope and be released
while still being referenced by a live Protocol.
|
| | |
|
| | |
|
| |
|
|
| |
It's much easier to debug these tests if we can step into them.
|
| |
|
|
| |
Fix previous changes by waiting for forked bbstored process during termination.
|
| |
|
|
|
|
|
| |
This should never have been a valid command, but was accepted before, and
silently ignored because it was treated as an empty command. This obscured
bad quoting of bbackupquery command-line arguments, allowing commands to
appear to work, but not do what you expected.
|
| | |
|
| |
|
|
|
| |
changes to it.
|
| | |
|
| |
|
|
|
|
|
| |
misinterpretation of char * arguments as bools. Use macros to
simplify test code. Test that locations not present when bbackupd
started will be detected and backed up if subsequently created.
|
| |
|
|
|
| |
Rename ProtocolObject to Message.
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
testbbackupd:
{{{
Waiting for internal daemon to scan spacetest/d1: ............ done.
Waiting for server to die (pid 6307): .............................. failed!
ERROR: **** TEST FAILURE: Condition [killed_server] failed at testbbackupd.cpp:718
ERROR: **** TEST FAILURE: Condition [stop_internal_daemon(pid)] failed at testbbackupd.cpp:1153
}}}
|
| |
|
|
|
|
|
|
|
|
| |
users wanting more detailed restore output.
Disable printing dots when logging at TRACE level is enabled.
Warn rather than failing to restore when the file attributes could not
be restored. (merges [2745], [2827]).
|
| | |
|
| |
|
|
|
| |
as the local destination name also.
|
| |
|
|
|
| |
Only show warning about diff test time taken if it's out of bounds.
|
| | |
|
| | |
|
| | |
|
| |
|
|
|
|
|
|
|
|
|
| |
to help with interpreting test output.
When running at Trace level, where extra output from bbackupd etc. is
common, log a single line at the start of waiting, instead of progress
dots, to help with interpreting output.
Show number of seconds waited for on Windows as well as Unixes.
|
| |
|
|
|
| |
64 bits.
|
| |
|
|
|
|
| |
waiting for its 90 second error timeout on slow machines, e.g.
FreeBSD VM, causing bbackupd test to fail.
|
| |
|
|
|
| |
provide TEST_EQUAL that only takes two arguments for simplicity.
|
| |
|
|
|
| |
system stops complaining that it can't delete them.
|
| | |
|
| | |
|
| |
|
|
|
|
| |
that we use for standard logging, to fix 64bit platform warnings
reported by Matt Brown.
|
| |
|
|
|
|
|
| |
testbbackupd.
Fix assignment of string constants to non-const char pointers.
|
