mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-11-24 11:58:10 +01:00
27b25e45dc
Enigmail generates a "legacy-display" part when it sends encrypted mail with a protected Subject: header. This part is intended to display the Subject for mail user agents that are capable of decryption, but do not know how to deal with embedded protected headers. This part is the first child of a two-part multipart/mixed cryptographic payload within a cryptographic envelope that includes encryption (that is, it is not just a cleartext signed message). It uses Content-Type: text/rfc822-headers. That is: A └┬╴multipart/encrypted B ├─╴application/pgp-encrypted C └┬╴application/octet-stream * ╤ <decryption> D └┬╴multipart/mixed; protected-headers=v1 (cryptographic payload) E ├─╴text/rfc822-headers; protected-headers=v1 (legacy-display part) F └─╴… (actual message body) In discussions with jrollins, i've come to the conclusion that a legacy-display part should be stripped entirely from "notmuch show" and "notmuch reply" now that these tools can understand and interpret protected headers. You can tell when a message part is a protected header part this way: * is the payload (D) multipart/mixed with exactly two children? * is its first child (E) Content-Type: text/rfc822-headers? * does the first child (E) have the property protected-headers=v1? * do all the headers in the body of the first child (E) match the protected headers in the payload part (D) itself? If this is the case, and we already know how to deal with the protected header, then there is no reason to try to render the legacy-display part itself for the user. Furthermore, when indexing, if we are indexing properly, we should avoid indexing the text in E as part of the message body. 'notmuch reply' is an interesting case: the standard use of 'notmuch reply' will end up omitting all mention of protected Subject:. The right fix is for the replying MUA to be able to protect its headers, and for it to set them appropriately based on headers found in the original message. If a replying MUA is unable to protect headers, but still wants the user to be able to see the original header, a replying MUA that notices that the original message's subject differs from the proposed reply subject may choose to include the original's subject in the quoted/attributed text. (this would be a stopgap measure; it's not even clear that there is user demand for it) This test suite change indicates what we want to happen for this case (the tests are currently broken), and includes three additional TODO suggestions of subtle cases for anyone who wants to flesh out the test suite even further. (i believe all these cases should be already fixed by the rest of this series, but haven't had time to write the tests for the unusual cases) Signed-off-by: Daniel Kahn Gillmor <dkg@fifthhorseman.net> |
||
---|---|---|
.. | ||
corpora | ||
emacs-show.expected-output | ||
emacs-tree.expected-output | ||
emacs.expected-output | ||
smime | ||
test-databases | ||
test.expected-output | ||
valgrind | ||
.gitignore | ||
aggregate-results.sh | ||
arg-test.c | ||
atomicity.py | ||
database-test.c | ||
database-test.h | ||
emacs-address-cleaning.el | ||
emacs-attachment-warnings.el | ||
export-dirs.sh | ||
gen-threads.py | ||
ghost-report.cc | ||
gnupg-secret-key.asc | ||
gnupg-secret-key.NOTE | ||
hex-xcode.c | ||
json_check_nodes.py | ||
make-db-version.cc | ||
Makefile | ||
Makefile.local | ||
message-id-parse.c | ||
notmuch-test | ||
notmuch-test.h | ||
parse-time.c | ||
random-corpus.c | ||
README | ||
smtp-dummy.c | ||
symbol-test.cc | ||
T000-basic.sh | ||
T010-help-test.sh | ||
T020-compact.sh | ||
T030-config.sh | ||
T040-setup.sh | ||
T050-new.sh | ||
T060-count.sh | ||
T070-insert.sh | ||
T080-search.sh | ||
T090-search-output.sh | ||
T095-address.sh | ||
T100-search-by-folder.sh | ||
T110-search-position-overlap-bug.sh | ||
T120-search-insufficient-from-quoting.sh | ||
T130-search-limiting.sh | ||
T140-excludes.sh | ||
T150-tagging.sh | ||
T160-json.sh | ||
T170-sexp.sh | ||
T180-text.sh | ||
T190-multipart.sh | ||
T200-thread-naming.sh | ||
T205-author-naming.sh | ||
T210-raw.sh | ||
T220-reply.sh | ||
T230-reply-to-sender.sh | ||
T240-dump-restore.sh | ||
T250-uuencode.sh | ||
T260-thread-order.sh | ||
T270-author-order.sh | ||
T280-from-guessing.sh | ||
T290-long-id.sh | ||
T300-encoding.sh | ||
T310-emacs.sh | ||
T320-emacs-large-search-buffer.sh | ||
T330-emacs-subject-to-filename.sh | ||
T340-maildir-sync.sh | ||
T350-crypto.sh | ||
T355-smime.sh | ||
T356-protected-headers.sh | ||
T357-index-decryption.sh | ||
T358-emacs-protected-headers.sh | ||
T360-symbol-hiding.sh | ||
T370-search-folder-coherence.sh | ||
T380-atomicity.sh | ||
T390-python.sh | ||
T395-ruby.sh | ||
T400-hooks.sh | ||
T410-argument-parsing.sh | ||
T420-emacs-test-functions.sh | ||
T430-emacs-address-cleaning.sh | ||
T440-emacs-hello.sh | ||
T450-emacs-show.sh | ||
T455-emacs-charsets.sh | ||
T460-emacs-tree.sh | ||
T470-missing-headers.sh | ||
T480-hex-escaping.sh | ||
T490-parse-time-string.sh | ||
T500-search-date.sh | ||
T510-thread-replies.sh | ||
T520-show.sh | ||
T530-upgrade.sh | ||
T550-db-features.sh | ||
T560-lib-error.sh | ||
T570-revision-tracking.sh | ||
T580-thread-search.sh | ||
T585-thread-subquery.sh | ||
T590-libconfig.sh | ||
T590-thread-breakage.sh | ||
T600-named-queries.sh | ||
T610-message-property.sh | ||
T620-lock.sh | ||
T630-emacs-draft.sh | ||
T640-database-modified.sh | ||
T650-regexp-query.sh | ||
T660-bad-date.sh | ||
T670-duplicate-mid.sh | ||
T680-html-indexing.sh | ||
T690-command-line-args.sh | ||
T700-reindex.sh | ||
T710-message-id.sh | ||
T720-emacs-attachment-warnings.sh | ||
T720-lib-lifetime.sh | ||
T730-emacs-forwarding.sh | ||
T740-body.sh | ||
T750-gzip.sh | ||
T750-user-header.sh | ||
test-lib-common.sh | ||
test-lib-FREEBSD.sh | ||
test-lib.el | ||
test-lib.sh | ||
test-verbose |
Notmuch test suite ================== This directory contains the test suite for notmuch. When fixing bugs or enhancing notmuch, you are strongly encouraged to add tests in this directory to cover what you are trying to fix or enhance. Prerequisites ------------- The test system itself requires: - bash(1) version 4.0 or newer Without bash 4.0+ the tests just refuse to run. Some tests require external dependencies to run. Without them, they will be skipped, or (rarely) marked failed. Please install these, so that you know if you break anything. - GNU tar(1) - dtach(1) - emacs(1) - emacsclient(1) - gdb(1) - gpg(1) - python(1) If your system lacks these tools or have older, non-upgradable versions of these, please (possibly compile and) install these to some other path, for example /usr/local/bin or /opt/gnu/bin. Then prepend the chosen directory to your PATH before running the tests. e.g. env PATH=/opt/gnu/bin:$PATH make test For FreeBSD you need to install latest gdb from ports or packages and provide path to it in TEST_GDB environment variable before executing the tests, native FreeBSD gdb does not not work. If you install coreutils, which provides GNU versions of basic utils like 'date' and 'base64' on FreeBSD, the test suite will use these instead of the native ones. This provides robustness against portability issues with these system tools. Most often the tests are written, reviewed and tested on Linux system so such portability issues arise from time to time. Running Tests ------------- The easiest way to run tests is to say "make test", (or simply run the notmuch-test script). Either command will run all available tests. Alternately, you can run a specific subset of tests by simply invoking one of the executable scripts in this directory, (such as ./T*-search.sh, ./T*-reply.sh, etc). Note that you will probably want "make test-binaries" before running individual tests. The following command-line options are available when running tests: --debug:: This may help the person who is developing a new test. It causes the command defined with test_debug to run. --immediate:: This causes the test to immediately exit upon the first failed test. --valgrind:: Execute notmuch with valgrind and exit with status 126 on errors (just like regular tests, this will only stop the test script when running under -i). Valgrind errors go to stderr, so you might want to pass the -v option, too. Since it makes no sense to run the tests with --valgrind and not see any output, this option implies --verbose. For convenience, it also implies --tee. --tee:: In addition to printing the test output to the terminal, write it to files named 't/test-results/$TEST_NAME.out'. As the names depend on the tests' file names, it is safe to run the tests with this option in parallel. Certain tests require precomputed databases to complete. You can fetch these databases with make download-test-databases If you do not download the test databases, the relevant tests will be skipped. When invoking the test suite via "make test" any of the above options can be specified as follows: make test OPTIONS="--verbose" You can choose an emacs binary (and corresponding emacsclient) to run the tests in one of the following ways. TEST_EMACS=my-emacs TEST_EMACSCLIENT=my-emacsclient make test TEST_EMACS=my-emacs TEST_EMACSCLIENT=my-emacsclient ./T*-emacs.sh make test TEST_EMACS=my-emacs TEST_EMACSCLIENT=my-emacsclient Some tests may require a c compiler. You can choose the name and flags similarly to with emacs, e.g. make test TEST_CC=gcc TEST_CFLAGS="-g -O2" Parallel Execution ------------------ If either the moreutils or GNU "parallel" utility is available all tests will be run in parallel. If the NOTMUCH_TEST_SERIALIZE variable is non-null all tests will be executed sequentially. Quiet Execution --------------- Normally, when new script starts and when test PASSes you get a message printed on screen. This printing can be disabled by setting the NOTMUCH_TEST_QUIET variable to a non-null value. Message on test failures and skips are still printed. Skipping Tests -------------- If, for any reason, you need to skip one or more tests, you can do so by setting the NOTMUCH_SKIP_TESTS variable to the name of one or more sections of tests. For example: $ NOTMUCH_SKIP_TESTS="search reply" make test Even more fine-grained skipping is possible by appending a test number (or glob pattern) after the section name. For example, the first search test and the second reply test could be skipped with: $ NOTMUCH_SKIP_TESTS="search.1 reply.2" make test Note that some tests in the existing test suite rely on previous test items, so you cannot arbitrarily skip any test and expect the remaining tests to be unaffected. Currently we do not consider skipped tests as build failures. For maximum robustness, when setting up automated build processes, you should explicitly skip tests, rather than relying on notmuch's detection of missing prerequisites. In the future we may treat tests unable to run because of missing prerequisites, but not explicitly skipped by the user, as failures. Writing Tests ------------- The test script is written as a shell script. It is to be named as Tddd-testname.sh where 'ddd' is three digits and 'testname' the "bare" name of your test. Tests will be run in order the 'ddd' part determines. The test script should start with the standard "#!/usr/bin/env bash" and an assignment to variable 'test_description', like this: #!/usr/bin/env bash test_description='xxx test (option --frotz) This test exercises the "notmuch xxx" command when given the option --frotz.' Source 'test-lib.sh' -------------------- After assigning test_description, the test script should source test-lib.sh like this: . ./test-lib.sh || exit 1 This test harness library does the following things: - If the script is invoked with command line argument --help (or -h), it shows the test_description and exits. - Creates a temporary directory with default notmuch-config and a mail store with a corpus of mail, (initially, 50 early messages sent to the notmuch list). This directory is test/tmp.<test-basename>. The path to notmuch-config is exported in NOTMUCH_CONFIG environment variable and mail store path is stored in MAIL_DIR variable. - Defines standard test helper functions for your scripts to use. These functions are designed to make all scripts behave consistently when command line arguments --verbose (or -v), --debug (or -d), and --immediate (or -i) is given. End with test_done ------------------ Your script will be a sequence of tests, using helper functions from the test harness library. At the end of the script, call 'test_done'. Test harness library -------------------- There are a handful helper functions defined in the test harness library for your script to use. test_begin_subtest <message> Set the test description message for a subsequent test_expect_* invocation (see below). test_expect_success <script> This takes a string as parameter, and evaluates the <script>. If it yields success, test is considered successful. test_expect_code <code> <script> This takes two strings as parameter, and evaluates the <script>. If it yields <code> exit status, test is considered successful. test_subtest_known_broken Mark the current test as broken. Such tests are expected to fail. Unlike the normal tests, which say "PASS" on success and "FAIL" on failure, these will say "FIXED" on success and "BROKEN" on failure. Failures from these tests won't cause -i (immediate) to stop. A test must call this before any test_expect_* function. test_expect_equal <output> <expected> This is an often-used convenience function built on top of test_expect_success. It uses the message from the last test_begin_subtest call, so call before calling test_expect_equal. This function generates a successful test if both the <output> and <expected> strings are identical. If not, it will generate a failure and print the difference of the two strings. test_expect_equal_file <file1> <file2> Identical to test_expect_equal, except that <file1> and <file2> are files instead of strings. This is a much more robust method to compare formatted textual information, since it also notices whitespace and closing newline differences. test_expect_equal_json <output> <expected> Identical to test_expect_equal, except that the two strings are treated as JSON and canonicalized before equality testing. This is useful to abstract away from whitespace differences in the expected output and that generated by running a notmuch command. test_debug <script> This takes a single argument, <script>, and evaluates it only when the test script is started with --debug command line argument. This is primarily meant for use during the development of a new test script. test_emacs <emacs-lisp-expressions> This function executes the provided emacs lisp script within emacs. The script can be a sequence of emacs lisp expressions, (that is, they will be evaluated within a progn form). Emacs stdout and stderr is not available, the common way to get output is to save it to a file. There are some auxiliary functions useful in emacs tests provided in test-lib.el. Do not use `setq' for setting variables in Emacs tests because it affects other tests that may run in the same Emacs instance. Use `let' instead so the scope of the changed variables is limited to a single test. test_emacs_expect_t <emacs-lisp-expressions> This function executes the provided emacs lisp script within emacs in a manner similar to 'test_emacs'. The expressions should return the value `t' to indicate that the test has passed. If the test does not return `t' then it is considered failed and all data returned by the test is reported to the tester. test_done Your test script must have test_done at the end. Its purpose is to summarize successes and failures in the test script and exit with an appropriate error code. There are also a number of notmuch-specific auxiliary functions and variables which are useful in writing tests: generate_message Generates a message with an optional template. Most tests will actually prefer to call add_message. See below. add_message Generate a message and add it to the database (by calling "notmuch new"). It is sufficient to simply call add_message with no arguments if you don't care about the content of the message. If more control is needed, arguments can be provide to specify many different header values for the new message. See the documentation within test-lib.sh or refer to many example calls within existing tests. add_email_corpus This function should be called at the beginning of a test file when a test needs to operate on a non-empty body of messages. It will initialize the mail database to a known state of 50 sample messages, (culled from the early history of the notmuch mailing list). notmuch_counter_reset $notmuch_counter_command notmuch_counter_value These allow to count how many times notmuch binary is called. notmuch_counter_reset() function generates a script that counts how many times it is called and resets the counter to zero. The function sets $notmuch_counter_command variable to the path to the generated script that should be called instead of notmuch to do the counting. The notmuch_counter_value() function prints the current counter value. There are also functions which remove various environment-dependent values from notmuch output; these are useful to ensure that test results remain consistent across different machines. notmuch_search_sanitize notmuch_show_sanitize notmuch_show_sanitize_all notmuch_json_show_sanitize All these functions should receive the text to be sanitized as the input of a pipe, e.g. output=`notmuch search "..." | notmuch_search_sanitize`