notmuch/test/T560-lib-error.sh, branch 0.21thread-based email index, search, and tagginghttps://git.notmuchmail.org/git/notmuch/atom?h=0.212015-09-04T11:22:42Ztest: move backup_database and restore_database to library2015-09-04T11:22:42ZDavid Bremnerdavid@tethera.net2015-06-07T15:01:59Zurn:sha1:9ee29ad6f0805dc55e63f8a04a68a1f55d0c382b
The plan is to re-use them in testing the same errors from the CLI
test: make script exit (1) if it "fails" to source (.) a file2015-08-07T19:56:39ZTomi Ollilatomi.ollila@iki.fi2015-08-06T09:13:36Zurn:sha1:02a2eeb427d6b424029f6e5e5ddad4c6ec987741
The files (test) scripts source (with builtin command `.`) provides
information which the scripts depend, and without the `source` to
succeed allowing script to continue may lead to dangerous situations
(e.g. rm -rf "${undefined_variable}"/*).
At the end of all source (.) lines construct ' || exit 1' was added;
In our case the script script will exit if it cannot find (or read) the
file to be sourced. Additionally script would also exits if the last
command of the sourced file exited nonzero.
lib: reject relative paths in n_d_{create,open}_verbose2015-06-12T05:34:50ZDavid Bremnerdavid@tethera.net2015-06-08T06:02:22Zurn:sha1:32fd74b7aa9c24ec77f8c59d09f89e0535bf64cd
There are many places in the notmuch code where the path is assumed to be absolute. If someone (TM) wants a project, one could remove these assumptions. In the mean time, prevent users from shooting themselves in the foot.
Update test suite mark tests for this error as no longer broken, and
also convert some tests that used relative paths for nonexistent
directories.
lib: add NOTMUCH_STATUS_PATH_ERROR2015-06-12T05:34:47ZDavid Bremnerdavid@tethera.net2015-06-10T05:58:44Zurn:sha1:b59ad1a9cc6ea03764b1cd3d038920581ac5a9c4
The difference with FILE_ERROR is that this is for things that are
wrong with the path before looking at the disk.
Add some 3 tests; two broken as a reminder to actually use this new
code.
test: move nonexistent directory to something under notmuch's control.2015-04-08T14:43:55ZDavid Bremnerdavid@tethera.net2015-04-07T12:10:15Zurn:sha1:294bb6de20de7d9c83a45dca22888d557ece588a
Apparently some systems actually have a directory called /nonexist[ae]nt.
It's hard to fathom a good reason for that, but oh well. As long as we
don't create such a directory inside the notmuch source tree, the new
version should be more robust.
test: be consistent about spelling `nonexistent'2015-04-08T14:43:55ZDavid Bremnerdavid@tethera.net2015-04-08T14:31:43Zurn:sha1:2e9ecb611a7f698808add756b2ddcbdfcd8b7c8b
Apparently most of the misspellings are my fault.
test: make one error output test more robust.2015-03-29T07:50:01ZDavid Bremnerdavid@tethera.net2015-03-29T07:39:46Zurn:sha1:703c8f9511c6f96023fb180b0e9d831e762befc9
Since notmuch_database_status_string can return NULL, passing it
directly to fputs is not a good idea.
test: add more error reporting tests2015-03-28T23:34:15ZDavid Bremnerdavid@tethera.net2015-03-28T23:24:38Zurn:sha1:6a99505625785022d490207bdf6e3e29ed156c6a
This second half of the error reporting tests actually uses the function
notmuch_database_status_string to retrieve the last logged error
lib: replace almost all fprintfs in library with _n_d_log2015-03-28T23:34:15ZDavid Bremnerdavid@tethera.net2014-12-26T16:25:35Zurn:sha1:736ac26407914425a9c94e86616225292cf716dd
This is not supposed to change any functionality from an end user
point of view. Note that it will eliminate some output to stderr. The
query debugging output is left as is; it doesn't really fit with the
current primitive logging model. The remaining "bad" fprintf will need
an internal API change.
test: add error reporting tests2015-03-28T23:34:15ZDavid Bremnerdavid@tethera.net2015-03-14T08:50:55Zurn:sha1:83298fa0f8e7717a260c1f88edd7bf545709562d
This first half of the tests is all that don't need to retrieve the
the error string explicitly from the notmuch database structure.