Replaced use of snprintf() with memcpy()/memmove().

[nmh] / test / mhstore / test-mhstore

diff --git a/test/mhstore/test-mhstore b/test/mhstore/test-mhstore
index b92972b3e8dadab3b589e576a69c27bedfbe8084..d3f86b02591b45e377348760434dadd1dbfa235e 100755 (executable)
--- a/test/mhstore/test-mhstore
+++ b/test/mhstore/test-mhstore
@@ -16,68 +16,84 @@ fi
 
 setup_test
 
 
 setup_test
 

+check_exit '-eq 1' mhstore -
+

 expected="$MH_TEST_DIR/test-mhstore$$.expected"
 expected="$MH_TEST_DIR/test-mhstore$$.expected"

+expected2="$MH_TEST_DIR/test-mhstore$$.expected2"
+expected_err="$MH_TEST_DIR/test-mhmail$$.expected_err"

 actual="$MH_TEST_DIR/test-mhstore$$.actual"
 actual="$MH_TEST_DIR/test-mhstore$$.actual"

+actual_err="$MH_TEST_DIR/test-mhmail$$.actual_err"

 
 

-cd $MH_TEST_DIR
+cd "$MH_TEST_DIR"

 
 # check with no options and no current message
 
 # check with no options and no current message

+start_test 'with no options and no current message'

 run_test 'mhstore' 'mhstore: no cur message'
 
 # check with no options and current message
 run_test 'mhstore' 'mhstore: no cur message'
 
 # check with no options and current message

-folder +inbox 5 > /dev/null
-cat > $expected <<EOF
+start_test 'with no options and current message'
+folder +inbox 5 >/dev/null
+cat >"$expected" <<'EOF'

 This is message number 5
 EOF
 run_test 'mhstore' 'storing message 5 as file 5.txt'
 This is message number 5
 EOF
 run_test 'mhstore' 'storing message 5 as file 5.txt'

-check $expected 5.txt 'keep first'
+check "$expected" 5.txt 'keep first'

 
 # check with folder and current message
 
 # check with folder and current message

+start_test 'with folder and current message'

 run_test 'mhstore +inbox' 'storing message 5 as file 5.txt'
 run_test 'mhstore +inbox' 'storing message 5 as file 5.txt'

-check $expected 5.txt 'keep first'
+check "$expected" 5.txt 'keep first'

 
 # check with specified message
 
 # check with specified message

+start_test 'with specified message'

 run_test 'mhstore 5' 'storing message 5 as file 5.txt'
 run_test 'mhstore 5' 'storing message 5 as file 5.txt'

-check $expected 5.txt 'keep first'
+check "$expected" 5.txt 'keep first'

 
 # check -file
 
 # check -file

+start_test '-file'

 run_test "mhstore -file $MH_TEST_DIR/Mail/inbox/5" \
   "storing message $MH_TEST_DIR/Mail/inbox/5 as file 5.txt"
 run_test "mhstore -file $MH_TEST_DIR/Mail/inbox/5" \
   "storing message $MH_TEST_DIR/Mail/inbox/5 as file 5.txt"

-check $expected 5.txt 'keep first'
+check "$expected" 5.txt 'keep first'

 
 # check -file -
 
 # check -file -

-stored_contents=`mhstore -file - < $MH_TEST_DIR/Mail/inbox/5 2>&1 | \
+start_test '-file -'
+stored_contents=`run_prog mhstore -file - < $MH_TEST_DIR/Mail/inbox/5 2>&1 | \

   sed 's/.*as file //'`
   sed 's/.*as file //'`

-check $expected $stored_contents 'keep first'
+check "$expected" "$stored_contents" 'keep first'

 
 

-# check -outfile
-mhstore 5 -outfile "$actual" 2>&1 | grep -v '^storing' && \
-  echo unexpected output from test-mhstore check -outfile
-check $expected "$actual" 'keep first'
+# check -outfile and -noverbose
+start_test '-outfile and -noverbose'
+run_prog mhstore 5 -noverbose -outfile "$actual" >"$actual_err" 2>&1
+printf '' >"$expected_err"
+check "$expected" "$actual" 'keep first'
+check "$expected_err" "$actual_err"

 
 # check -outfile -
 
 # check -outfile -

-echo 'storing message 5 to stdout' >> "$expected"
-mhstore 5 -outfile - >"$actual" 2>&1
-check $expected "$actual"
+start_test '-outfile -'
+printf '%s\n' 'storing message 5 to stdout' >>"$expected"
+run_prog mhstore 5 -outfile - >"$actual" 2>&1
+check "$expected" "$actual"

 
 # check message number greater than highest
 
 # check message number greater than highest

+start_test 'message number greater than highest'

 run_test 'mhstore 11' "mhstore: message 11 doesn't exist"
 
 # check multiple messages
 run_test 'mhstore 11' "mhstore: message 11 doesn't exist"
 
 # check multiple messages

+start_test 'multiple messages'

 run_test 'mhstore 4 6' 'storing message 4 as file 4.txt
 storing message 6 as file 6.txt'
 run_test 'mhstore 4 6' 'storing message 4 as file 4.txt
 storing message 6 as file 6.txt'

-cat > $expected <<EOF
+cat >"$expected" <<'EOF'

 This is message number 4
 EOF
 This is message number 4
 EOF

-check $expected 4.txt
-cat > $expected <<EOF
+check "$expected" 4.txt
+cat >"$expected" <<'EOF'

 This is message number 6
 EOF
 This is message number 6
 EOF

-check $expected 6.txt
+check "$expected" 6.txt

 
 # Write message with a text/plain subpart.
 msgfile=`mhpath new`
 
 # Write message with a text/plain subpart.
 msgfile=`mhpath new`

-cat > $msgfile <<EOF
+cat >"$msgfile" <<'EOF'

 To: recipient@example.com
 From: sender@example.com
 Subject: mhlist test
 To: recipient@example.com
 From: sender@example.com
 Subject: mhlist test


@@ -114,185 +130,138 @@ This is the third text/plain part.
 ------- =_aaaaaaaaaa0
 Content-Type: text/plain; charset="iso-8859-1"; name="test4.txt"
 Content-Disposition: attachment; filename="test4.txt"
 ------- =_aaaaaaaaaa0
 Content-Type: text/plain; charset="iso-8859-1"; name="test4.txt"
 Content-Disposition: attachment; filename="test4.txt"

+Content-MD5: cMI1lB/LZ4jgVl3EbhdyWA==

 Content-Transfer-Encoding: base64
 
 Content-Transfer-Encoding: base64
 

-VGhpcyBpcyB0aGUgZm91cnRoIHRleHQvcGxhaW4gcGFydC4K
+VGhpcyBpcyB0aGUgZm91cnRoIHRleHQvcGxhaW4gcGFydC4NClRoaXMgdGVzdCBoYXMgbXVsdGlw
+bGUgbGluZXMuDQo=

 
 ------- =_aaaaaaaaaa0--
 EOF
 
 # check -part
 
 ------- =_aaaaaaaaaa0--
 EOF
 
 # check -part

-cat > $expected <<EOF
+start_test '-part'
+cat >"$expected" <<'EOF'

 This is the third text/plain part.
 EOF
 run_test 'mhstore last -part 3' 'storing message 11 part 3 as file 11.3.txt'
 This is the third text/plain part.
 EOF
 run_test 'mhstore last -part 3' 'storing message 11 part 3 as file 11.3.txt'

-check $expected 11.3.txt
+check "$expected" 11.3.txt

 
 # check -part of a subpart
 
 # check -part of a subpart

-cat > $expected <<EOF
+start_test '-part of a subpart'
+cat >"$expected" <<'EOF'

 This is the first text/plain part, in a subpart.
 EOF
 run_test 'mhstore last -part 1.1' \
          'storing message 11 part 1.1 as file 11.1.1.txt'
 This is the first text/plain part, in a subpart.
 EOF
 run_test 'mhstore last -part 1.1' \
          'storing message 11 part 1.1 as file 11.1.1.txt'

-check $expected 11.1.1.txt 'keep first'
+check "$expected" 11.1.1.txt 'keep first'

 
 # check -type
 
 # check -type

+start_test '-type'

 run_test 'mhstore last -type text/plain' \
   'storing message 11 part 1.1 as file 11.1.1.txt
 storing message 11 part 2 as file 11.2.txt
 storing message 11 part 3 as file 11.3.txt
 storing message 11 part 4 as file 11.4.txt'
 run_test 'mhstore last -type text/plain' \
   'storing message 11 part 1.1 as file 11.1.1.txt
 storing message 11 part 2 as file 11.2.txt
 storing message 11 part 3 as file 11.3.txt
 storing message 11 part 4 as file 11.4.txt'

-check $expected 11.1.1.txt
-cat > $expected <<EOF
+check "$expected" 11.1.1.txt
+cat >"$expected" <<'EOF'

 This is the second text/plain part.
 EOF
 This is the second text/plain part.
 EOF

-check $expected 11.2.txt
-cat > $expected <<EOF
+check "$expected" 11.2.txt
+cat >"$expected" <<'EOF'

 This is the third text/plain part.
 EOF
 This is the third text/plain part.
 EOF

-check $expected 11.3.txt
-cat > $expected <<EOF
+check "$expected" 11.3.txt
+cat >"$expected" <<'EOF'

 This is the fourth text/plain part.
 This is the fourth text/plain part.

+This test has multiple lines.

 EOF
 EOF

-check $expected 11.4.txt
+check "$expected" 11.4.txt

 
 # check -auto
 
 # check -auto

-cat > $expected <<EOF
+start_test '-auto'
+cat >"$expected" <<'EOF'

 This is the fourth text/plain part.
 This is the fourth text/plain part.

+This test has multiple lines.

 EOF
 run_test 'mhstore last -part 4 -auto' \
   'storing message 11 part 4 as file test4.txt'
 EOF
 run_test 'mhstore last -part 4 -auto' \
   'storing message 11 part 4 as file test4.txt'

-check $expected test4.txt 'keep first'
+check "$expected" test4.txt 'keep first'

 
 # check -noauto, the default
 
 # check -noauto, the default

+start_test '-noauto, the default'

 run_test 'mhstore last -part 4 -auto -noauto' \
   'storing message 11 part 4 as file 11.4.txt'
 run_test 'mhstore last -part 4 -auto -noauto' \
   'storing message 11 part 4 as file 11.4.txt'

-check $expected 11.4.txt 'keep first'
+check "$expected" 11.4.txt 'keep first'

 
 

-# check -verbose, which doesn't do anything
-run_test 'mhstore last -part 4 -verbose' \
+# check -verbose, the default
+start_test '-verbose, the default'
+run_test 'mhstore last -part 4 -noverbose -verbose' \

   'storing message 11 part 4 as file 11.4.txt'
   'storing message 11 part 4 as file 11.4.txt'

-check $expected 11.4.txt 'keep first'
+check "$expected" 11.4.txt 'keep first'

 
 

-# check -noverbose, the default
-run_test 'mhstore last -part 4 -verbose -noverbose' \
-  'storing message 11 part 4 as file 11.4.txt'
-check $expected 11.4.txt
+# check -noverbose
+start_test '-noverbose'
+run_test 'mhstore last -part 4 -noverbose' ''
+check "$expected" 11.4.txt

 
 # check that -auto obeys Content-Disposition header
 
 # check that -auto obeys Content-Disposition header

-cat > $expected <<EOF
+start_test 'that -auto obeys Content-Disposition header'
+cat >"$expected" <<'EOF'

 This is the first text/plain part, in a subpart.
 EOF
 run_test 'mhstore last -part 1.1 -auto' \
   'storing message 11 part 1.1 as file test1.txt'
 This is the first text/plain part, in a subpart.
 EOF
 run_test 'mhstore last -part 1.1 -auto' \
   'storing message 11 part 1.1 as file test1.txt'

-check $expected test1.txt
+check "$expected" test1.txt

 
 

-# check -check
-cat > $expected <<EOF
+# check -outfile - with -file
+start_test '-outfile - with -file'
+cat >"$expected" <<'EOF'

 This is the second text/plain part.
 EOF
 This is the second text/plain part.
 EOF

-run_test 'mhstore last -part 2 -check' \
-         'storing message 11 part 2 as file 11.2.txt'
-check $expected 11.2.txt 'keep first'
-
-# check -check with bad MD5 checksum
-sed 's/\(Content-MD5: \)kq+Hnc/\1BADBAD/' $msgfile > $MH_TEST_DIR/$$.tmp
-mv -f $MH_TEST_DIR/$$.tmp $msgfile
-run_test 'mhstore last -part 2 -check' \
-'storing message 11 part 2 as file 11.2.txt
-mhstore: content integrity suspect (digest mismatch) -- continuing
-         (content text/plain in message 11, part 2)'
-check $expected 11.2.txt 'keep first'
-
-# check -nocheck, the default
-run_test 'mhstore last -part 2 -check -nocheck' \
-         'storing message 11 part 2 as file 11.2.txt'
-check $expected 11.2.txt
-
-# check reassembly of message/partial messages (RFC 1521, 7.3.2)
-msgfile=`mhpath new`
-cat > $msgfile <<EOF
-To: recipient@example.com
-From: sender@example.com
-Subject: mhlist test
-MIME-Version: 1.0
-Content-Type: message/partial; id="test-mhstore message/partial test"; number=1
-
-Content-Type: text/plain; charset="iso-8859-1"
-
-This is part one of three.
-EOF
-
-msgfile=`mhpath new`
-cat > $msgfile <<EOF
-To: recipient@example.com
-From: sender@example.com
-Subject: mhlist test
-MIME-Version: 1.0
-Content-Type: message/partial; id="test-mhstore message/partial test"; number=2
-
-This is part two of three.
-EOF
-
-msgfile=`mhpath new`
-cat > $msgfile <<EOF
-To: recipient@example.com
-From: sender@example.com
-Subject: mhlist test
-MIME-Version: 1.0
-Content-Type: message/partial; id="test-mhstore message/partial test"; number=3;
-  total=3
-
-This is part three of three.
-EOF
-
-run_test 'mhstore last:3' \
-         'reassembling partials 12,13,14 to folder inbox as message 15'
-
-cat > $expected <<EOF
-This is part one of three.
-This is part two of three.
-This is part three of three.
-EOF
-run_test 'mhstore last' 'storing message 15 as file 15.txt'
-check $expected 15.txt
+run_prog mhstore -noverbose -file `mhpath 11` -part 2 -outfile - >"$actual" 2>&1
+check "$expected" "$actual"

 
 # cd into Mail directory because it gets cleaned up by the exit trap.
 cd Mail
 
 # check -clobber always
 
 # cd into Mail directory because it gets cleaned up by the exit trap.
 cd Mail
 
 # check -clobber always

-folder +inbox 7 > /dev/null
+start_test '-clobber always'
+folder +inbox 7 >/dev/null

 touch 7.txt
 touch 7.txt

-cat > $expected <<EOF
+cat >"$expected" <<'EOF'

 This is message number 7
 EOF
 run_test 'mhstore' 'storing message 7 as file 7.txt'
 This is message number 7
 EOF
 run_test 'mhstore' 'storing message 7 as file 7.txt'

-check $expected 7.txt 'keep first'
+check "$expected" 7.txt 'keep first'

 run_test 'mhstore -clobber ask -clobber always' \
          'storing message 7 as file 7.txt'
 run_test 'mhstore -clobber ask -clobber always' \
          'storing message 7 as file 7.txt'

-check $expected 7.txt 'keep first'
+check "$expected" 7.txt 'keep first'

 
 # check -clobber auto
 
 # check -clobber auto

+start_test '-clobber auto'

 touch 7.txt
 run_test 'mhstore -clobber auto' 'storing message 7 as file 7-1.txt'
 touch 7.txt
 run_test 'mhstore -clobber auto' 'storing message 7 as file 7-1.txt'

-check $expected 7-1.txt 'keep first'
+check "$expected" 7-1.txt 'keep first'

 touch 7-1.txt
 run_test 'mhstore -clobber auto' 'storing message 7 as file 7-2.txt'
 touch 7-1.txt
 run_test 'mhstore -clobber auto' 'storing message 7 as file 7-2.txt'

-check $expected 7-2.txt 'keep first'
+check "$expected" 7-2.txt 'keep first'

 
 # check -clobber suffix
 
 # check -clobber suffix

+start_test '-clobber suffix'

 run_test 'mhstore -clobber suffix' 'storing message 7 as file 7.txt.1'
 run_test 'mhstore -clobber suffix' 'storing message 7 as file 7.txt.1'

-check $expected 7.txt.1 'keep first'
+check "$expected" 7.txt.1 'keep first'

 touch 7.txt.1
 run_test 'mhstore -clobber suffix' 'storing message 7 as file 7.txt.2'
 touch 7.txt.1
 run_test 'mhstore -clobber suffix' 'storing message 7 as file 7.txt.2'

-check $expected 7.txt.2 'keep first'
+check "$expected" 7.txt.2 'keep first'

 
 # Don't check -clobber ask because it requires connection to a
 # terminal, and this test won't always be run with one.
 
 # check -clobber never.  Its exit status is the number of files not overwritten.
 
 # Don't check -clobber ask because it requires connection to a
 # terminal, and this test won't always be run with one.
 
 # check -clobber never.  Its exit status is the number of files not overwritten.

+start_test '-clobber never.  Its exit status is the number of files not overwritten.'

 run_test 'mhstore -clobber never' \
 run_test 'mhstore -clobber never' \

-         "mhstore: will not overwrite `pwd`/7.txt with -clobber never"
+         "mhstore: will not overwrite $MH_TEST_DIR/Mail/7.txt with -clobber never"

 set +e
 mhstore -clobber never >/dev/null 2>&1
 run_test "echo $?" 1
 set +e
 mhstore -clobber never >/dev/null 2>&1
 run_test "echo $?" 1


@@ -300,37 +269,56 @@ set -e
 
 cd ..
 
 
 cd ..
 

-# check with relative nmh-storage profile component
-storagedir=storagedir
+# check with short relative nmh-storage profile component
+start_test 'with short relative nmh-storage profile component'
+storagedir=s

 dir="$MH_TEST_DIR/Mail/inbox/$storagedir"
 mkdir "$dir"
 dir="$MH_TEST_DIR/Mail/inbox/$storagedir"
 mkdir "$dir"

-echo "nmh-storage: $storagedir" >> $MH
-cat > $expected <<EOF
+printf '%s\n' "nmh-storage: $storagedir" >>"$MH"
+cat >"$expected" <<'EOF'

 This is the fourth text/plain part.
 This is the fourth text/plain part.

+This test has multiple lines.

 EOF
 run_test 'mhstore 11 -part 4' \
 "storing message 11 part 4 as file $storagedir/11.4.txt"
 EOF
 run_test 'mhstore 11 -part 4' \
 "storing message 11 part 4 as file $storagedir/11.4.txt"

-check $expected "$dir/11.4.txt" 'keep first'
+check "$expected" "$dir/11.4.txt" 'keep first'
+rmdir "$dir"
+sed '$d' "$MH" >"$MH.new"
+mv "$MH.new" "$MH"
+
+# check with relative nmh-storage profile component
+start_test 'with relative nmh-storage profile component'
+storagedir=storagedir
+dir="$MH_TEST_DIR/Mail/inbox/$storagedir"
+mkdir "$dir"
+printf '%s\n' "nmh-storage: $storagedir" >>"$MH"
+run_test 'mhstore 11 -part 4' \
+"storing message 11 part 4 as file $storagedir/11.4.txt"
+check "$expected" "$dir/11.4.txt" 'keep first'

 
 # check with relative nmh-storage profile component and -auto
 
 # check with relative nmh-storage profile component and -auto

+start_test 'with relative nmh-storage profile component and -auto'

 run_test 'mhstore 11 -part 4 -auto' \
 "storing message 11 part 4 as file $storagedir/test4.txt"
 run_test 'mhstore 11 -part 4 -auto' \
 "storing message 11 part 4 as file $storagedir/test4.txt"

-check $expected "$dir/test4.txt" 'keep first'
+check "$expected" "$dir/test4.txt" 'keep first'

 
 # check with absolute nmh-storage profile component
 
 # check with absolute nmh-storage profile component

+start_test 'with absolute nmh-storage profile component'

 absstoragedir="$MH_TEST_DIR/$storagedir"
 mkdir "$absstoragedir"
 absstoragedir="$MH_TEST_DIR/$storagedir"
 mkdir "$absstoragedir"

-sed "s%\(nmh-storage: \)storagedir%\1$absstoragedir%" $MH > $MH.new
-mv -f $MH.new $MH
+sed "s%\(nmh-storage: \)storagedir%\1$absstoragedir%" "$MH" >"$MH.new"
+mv -f "$MH.new" "$MH"

 run_test 'mhstore 11 -part 4' \
 "storing message 11 part 4 as file $storagedir/11.4.txt"
 run_test 'mhstore 11 -part 4' \
 "storing message 11 part 4 as file $storagedir/11.4.txt"

-check $expected "$absstoragedir/11.4.txt" 'keep first'
+check "$expected" "$absstoragedir/11.4.txt" 'keep first'

 
 # check with absolute nmh-storage profile component and -auto
 
 # check with absolute nmh-storage profile component and -auto

+start_test 'with absolute nmh-storage profile component and -auto'

 run_test 'mhstore 11 -part 4 -auto' \
 "storing message 11 part 4 as file $storagedir/test4.txt"
 run_test 'mhstore 11 -part 4 -auto' \
 "storing message 11 part 4 as file $storagedir/test4.txt"

-check $expected "$absstoragedir/test4.txt"
+check "$expected" "$absstoragedir/test4.txt"

 rmdir "$absstoragedir"
 
 
 rmdir "$absstoragedir"
 
 

+finish_test

 exit $failed
 exit $failed