mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-11-23 11:28:13 +01:00
Merge branch 'release'
This commit is contained in:
commit
702635d5f6
3 changed files with 33 additions and 17 deletions
|
@ -404,14 +404,17 @@ static notmuch_status_t
|
||||||
_notmuch_database_link_message (notmuch_database_t *notmuch,
|
_notmuch_database_link_message (notmuch_database_t *notmuch,
|
||||||
notmuch_message_t *message,
|
notmuch_message_t *message,
|
||||||
notmuch_message_file_t *message_file,
|
notmuch_message_file_t *message_file,
|
||||||
bool is_ghost)
|
bool is_ghost,
|
||||||
|
bool is_new)
|
||||||
{
|
{
|
||||||
void *local = talloc_new (NULL);
|
void *local = talloc_new (NULL);
|
||||||
notmuch_status_t status;
|
notmuch_status_t status;
|
||||||
const char *thread_id = NULL;
|
const char *thread_id = NULL;
|
||||||
|
|
||||||
/* Check if the message already had a thread ID */
|
/* Check if the message already had a thread ID */
|
||||||
if (notmuch->features & NOTMUCH_FEATURE_GHOSTS) {
|
if (! is_new) {
|
||||||
|
thread_id = notmuch_message_get_thread_id (message);
|
||||||
|
} else if (notmuch->features & NOTMUCH_FEATURE_GHOSTS) {
|
||||||
if (is_ghost)
|
if (is_ghost)
|
||||||
thread_id = notmuch_message_get_thread_id (message);
|
thread_id = notmuch_message_get_thread_id (message);
|
||||||
} else {
|
} else {
|
||||||
|
@ -538,7 +541,7 @@ notmuch_database_index_file (notmuch_database_t *notmuch,
|
||||||
}
|
}
|
||||||
|
|
||||||
ret = _notmuch_database_link_message (notmuch, message,
|
ret = _notmuch_database_link_message (notmuch, message,
|
||||||
message_file, is_ghost);
|
message_file, is_ghost, is_new);
|
||||||
if (ret)
|
if (ret)
|
||||||
goto DONE;
|
goto DONE;
|
||||||
|
|
||||||
|
|
|
@ -114,12 +114,10 @@ test_expect_equal \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
|
||||||
# try inserting it with decryption, should appear as a single copy
|
# try inserting it with decryption, should appear as a single copy
|
||||||
# (note: i think thread id skips 4 because of duplicate message-id
|
|
||||||
# insertion, above)
|
|
||||||
test_begin_subtest "message cleartext is present with insert --decrypt=true"
|
test_begin_subtest "message cleartext is present with insert --decrypt=true"
|
||||||
notmuch insert --folder=sent --decrypt=true <<<"$contents"
|
notmuch insert --folder=sent --decrypt=true <<<"$contents"
|
||||||
output=$(notmuch search wumpus)
|
output=$(notmuch search wumpus | notmuch_search_sanitize)
|
||||||
expected='thread:0000000000000005 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 002 (encrypted inbox unread)'
|
expected='thread:XXX 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 002 (encrypted inbox unread)'
|
||||||
test_expect_equal \
|
test_expect_equal \
|
||||||
"$output" \
|
"$output" \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
@ -129,9 +127,9 @@ test_expect_equal \
|
||||||
test_begin_subtest 'tagging all messages'
|
test_begin_subtest 'tagging all messages'
|
||||||
test_expect_success 'notmuch tag +blarney "encrypted message"'
|
test_expect_success 'notmuch tag +blarney "encrypted message"'
|
||||||
test_begin_subtest "verify that tags have not changed"
|
test_begin_subtest "verify that tags have not changed"
|
||||||
output=$(notmuch search tag:blarney)
|
output=$(notmuch search tag:blarney | notmuch_search_sanitize)
|
||||||
expected='thread:0000000000000001 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 001 (blarney encrypted inbox)
|
expected='thread:XXX 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 001 (blarney encrypted inbox)
|
||||||
thread:0000000000000005 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 002 (blarney encrypted inbox unread)'
|
thread:XXX 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 002 (blarney encrypted inbox unread)'
|
||||||
test_expect_equal \
|
test_expect_equal \
|
||||||
"$output" \
|
"$output" \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
@ -140,14 +138,14 @@ test_expect_equal \
|
||||||
test_begin_subtest 'reindex old messages'
|
test_begin_subtest 'reindex old messages'
|
||||||
test_expect_success 'notmuch reindex --decrypt=true tag:encrypted and not property:index.decryption=success'
|
test_expect_success 'notmuch reindex --decrypt=true tag:encrypted and not property:index.decryption=success'
|
||||||
test_begin_subtest "reindexed encrypted message, including cleartext"
|
test_begin_subtest "reindexed encrypted message, including cleartext"
|
||||||
output=$(notmuch search wumpus)
|
output=$(notmuch search wumpus | notmuch_search_sanitize)
|
||||||
test_expect_equal \
|
test_expect_equal \
|
||||||
"$output" \
|
"$output" \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
|
||||||
# and the same search, but by property ($expected is untouched):
|
# and the same search, but by property ($expected is untouched):
|
||||||
test_begin_subtest "emacs search by property for both messages"
|
test_begin_subtest "emacs search by property for both messages"
|
||||||
output=$(notmuch search property:index.decryption=success)
|
output=$(notmuch search property:index.decryption=success | notmuch_search_sanitize)
|
||||||
test_expect_equal \
|
test_expect_equal \
|
||||||
"$output" \
|
"$output" \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
@ -156,7 +154,7 @@ test_expect_equal \
|
||||||
test_begin_subtest 'reindex in auto mode'
|
test_begin_subtest 'reindex in auto mode'
|
||||||
test_expect_success 'notmuch reindex tag:encrypted and property:index.decryption=success'
|
test_expect_success 'notmuch reindex tag:encrypted and property:index.decryption=success'
|
||||||
test_begin_subtest "reindexed encrypted messages, should not have changed"
|
test_begin_subtest "reindexed encrypted messages, should not have changed"
|
||||||
output=$(notmuch search wumpus)
|
output=$(notmuch search wumpus | notmuch_search_sanitize)
|
||||||
test_expect_equal \
|
test_expect_equal \
|
||||||
"$output" \
|
"$output" \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
@ -190,9 +188,9 @@ test_expect_equal \
|
||||||
|
|
||||||
# ensure that the tags remain even when we are dropping the cleartext.
|
# ensure that the tags remain even when we are dropping the cleartext.
|
||||||
test_begin_subtest "verify that tags remain without cleartext"
|
test_begin_subtest "verify that tags remain without cleartext"
|
||||||
output=$(notmuch search tag:blarney)
|
output=$(notmuch search tag:blarney | notmuch_search_sanitize)
|
||||||
expected='thread:0000000000000001 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 001 (blarney encrypted inbox)
|
expected='thread:XXX 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 001 (blarney encrypted inbox)
|
||||||
thread:0000000000000005 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 002 (blarney encrypted inbox unread)'
|
thread:XXX 2000-01-01 [1/1] Notmuch Test Suite; test encrypted message for cleartext index 002 (blarney encrypted inbox unread)'
|
||||||
test_expect_equal \
|
test_expect_equal \
|
||||||
"$output" \
|
"$output" \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
@ -201,7 +199,7 @@ test_begin_subtest "index cleartext without keeping session keys"
|
||||||
test_expect_success "notmuch reindex --decrypt=nostash tag:blarney"
|
test_expect_success "notmuch reindex --decrypt=nostash tag:blarney"
|
||||||
|
|
||||||
test_begin_subtest "Ensure that the indexed terms are present"
|
test_begin_subtest "Ensure that the indexed terms are present"
|
||||||
output=$(notmuch search wumpus)
|
output=$(notmuch search wumpus | notmuch_search_sanitize)
|
||||||
test_expect_equal \
|
test_expect_equal \
|
||||||
"$output" \
|
"$output" \
|
||||||
"$expected"
|
"$expected"
|
||||||
|
|
|
@ -2,10 +2,25 @@
|
||||||
test_description="duplicate message ids"
|
test_description="duplicate message ids"
|
||||||
. $(dirname "$0")/test-lib.sh || exit 1
|
. $(dirname "$0")/test-lib.sh || exit 1
|
||||||
|
|
||||||
|
test_require_external_prereq xapian-delve
|
||||||
|
|
||||||
add_message '[id]="duplicate"' '[subject]="message 1" [filename]=copy1'
|
add_message '[id]="duplicate"' '[subject]="message 1" [filename]=copy1'
|
||||||
add_message '[id]="duplicate"' '[subject]="message 2" [filename]=copy2'
|
add_message '[id]="duplicate"' '[subject]="message 2" [filename]=copy2'
|
||||||
|
|
||||||
add_message '[id]="duplicate"' '[subject]="message 0" [filename]=copy0'
|
add_message '[id]="duplicate"' '[subject]="message 0" [filename]=copy0'
|
||||||
|
|
||||||
|
test_begin_subtest 'at most 1 thread-id per xapian document'
|
||||||
|
db=${MAIL_DIR}/.notmuch/xapian
|
||||||
|
for doc in $(xapian-delve -1 -t '' "$db" | grep '^[1-9]'); do
|
||||||
|
xapian-delve -1 -r "$doc" "$db" | grep -c '^G'
|
||||||
|
done > OUTPUT.raw
|
||||||
|
sort -u < OUTPUT.raw > OUTPUT
|
||||||
|
cat <<EOF > EXPECTED
|
||||||
|
0
|
||||||
|
1
|
||||||
|
EOF
|
||||||
|
test_expect_equal_file EXPECTED OUTPUT
|
||||||
|
|
||||||
test_begin_subtest 'search: first indexed subject preserved'
|
test_begin_subtest 'search: first indexed subject preserved'
|
||||||
cat <<EOF > EXPECTED
|
cat <<EOF > EXPECTED
|
||||||
thread:XXX 2001-01-05 [1/1(3)] Notmuch Test Suite; message 1 (inbox unread)
|
thread:XXX 2001-01-05 [1/1(3)] Notmuch Test Suite; message 1 (inbox unread)
|
||||||
|
|
Loading…
Reference in a new issue