mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-11-25 12:28:09 +01:00
test: Quote $output in calls to test_expect_equal
Previously, many tests in emacs-subject-to-filename didn't quote the $output argument to test_expect_equal. As a result, if $output was empty, test_expect_equal would be passed only one argument and would abort the entire test script. By quoting the argument, we ensure test_expect_equal will always receive two arguments.
This commit is contained in:
parent
7611a72be2
commit
dba1f6e432
1 changed files with 9 additions and 9 deletions
|
@ -60,55 +60,55 @@ test_begin_subtest "filename #1"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
"just a subject line")'
|
"just a subject line")'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"just-a-subject-line"'
|
test_expect_equal "$output" '"just-a-subject-line"'
|
||||||
|
|
||||||
test_begin_subtest "filename #2"
|
test_begin_subtest "filename #2"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
" [any] [prefixes are ] [removed!] from the subject")'
|
" [any] [prefixes are ] [removed!] from the subject")'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"from-the-subject"'
|
test_expect_equal "$output" '"from-the-subject"'
|
||||||
|
|
||||||
test_begin_subtest "filename #3"
|
test_begin_subtest "filename #3"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
" leading and trailing space ")'
|
" leading and trailing space ")'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"leading-and-trailing-space"'
|
test_expect_equal "$output" '"leading-and-trailing-space"'
|
||||||
|
|
||||||
test_begin_subtest "filename #4"
|
test_begin_subtest "filename #4"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
"!# leading ()// &%, and in between_and_trailing garbage ()(&%%")'
|
"!# leading ()// &%, and in between_and_trailing garbage ()(&%%")'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"-leading-and-in-between_and_trailing-garbage"'
|
test_expect_equal "$output" '"-leading-and-in-between_and_trailing-garbage"'
|
||||||
|
|
||||||
test_begin_subtest "filename #5"
|
test_begin_subtest "filename #5"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
"ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz.-_01234567890")'
|
"ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz.-_01234567890")'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz.-_01234567890"'
|
test_expect_equal "$output" '"ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz.-_01234567890"'
|
||||||
|
|
||||||
test_begin_subtest "filename #6"
|
test_begin_subtest "filename #6"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
"sequences of ... are squashed and trailing are removed ...")'
|
"sequences of ... are squashed and trailing are removed ...")'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"sequences-of-.-are-squashed-and-trailing-are-removed"'
|
test_expect_equal "$output" '"sequences-of-.-are-squashed-and-trailing-are-removed"'
|
||||||
|
|
||||||
test_begin_subtest "filename #7"
|
test_begin_subtest "filename #7"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
"max length test" 1)'
|
"max length test" 1)'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"m"'
|
test_expect_equal "$output" '"m"'
|
||||||
|
|
||||||
test_begin_subtest "filename #8"
|
test_begin_subtest "filename #8"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
"max length test /&(/%&/%%&¤%¤" 20)'
|
"max length test /&(/%&/%%&¤%¤" 20)'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"max-length-test"'
|
test_expect_equal "$output" '"max-length-test"'
|
||||||
|
|
||||||
test_begin_subtest "filename #9"
|
test_begin_subtest "filename #9"
|
||||||
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
output=$(test_emacs '(notmuch-wash-subject-to-filename
|
||||||
"[a prefix] [is only separated] by [spaces], so \"by\" is not okay!")'
|
"[a prefix] [is only separated] by [spaces], so \"by\" is not okay!")'
|
||||||
)
|
)
|
||||||
test_expect_equal $output '"by-spaces-so-by-is-not-okay"'
|
test_expect_equal "$output" '"by-spaces-so-by-is-not-okay"'
|
||||||
|
|
||||||
# test notmuch-wash-subject-to-patch-filename (subject)
|
# test notmuch-wash-subject-to-patch-filename (subject)
|
||||||
test_begin_subtest "patch filename #1"
|
test_begin_subtest "patch filename #1"
|
||||||
|
|
Loading…
Reference in a new issue