Fixed condition to enable skipping parameter value tests.

[nmh] / test / mhshow / test-charset

diff --git a/test/mhshow/test-charset b/test/mhshow/test-charset
index 86729a6545a73f74895d3e98844946f92b536d71..b9bba9fa6e4b95dd3b497de7daf8d602060f7f75 100755 (executable)
--- a/test/mhshow/test-charset
+++ b/test/mhshow/test-charset
@@ -20,7 +20,19 @@ check_exit '-eq 1' mhshow -
 
 require_locale en_US.UTF-8 en_US.UTF8 en_US.utf-8 en_US.utf8
 
-test "$ICONV_ENABLED" -eq 0  &&  text_size=10  || text_size=11
+skip_param_value_checks=0
+if test "$ICONV_ENABLED" -eq 0; then
+    text_size=10
+    if ! iconv -l | grep EBCDIC-US >/dev/null; then
+        #### For unknown reasons, the parameter values checks
+        #### fail on the FreeBSD10 buildbot.  It doesn't
+        #### support EBCDIC-US, which is used by the checks,
+        #### so use that to detect whether to skip the check.
+        skip_param_value_checks=1
+    fi
+else
+    text_size=11
+fi
 
 expected=$MH_TEST_DIR/$$.expected
 actual=$MH_TEST_DIR/$$.actual
@@ -104,14 +116,9 @@ Content-Description: bogus quoted-printable data
 4 ÷ 2 = 2
 EOF
 
-if test "$ICONV_ENABLED" -eq 1; then
-  if iconv -l | grep EBCDIC-US >/dev/null; then
-    #### For unknown reasons, this test fails on the FreeBSD10
-    #### buildbot.  It doesn't support EBCDIC-US, so skip it
-    #### on that bot.
+if test $skip_param_value_checks -eq 0; then
     run_prog mhshow $msgnum > $actual 2>&1
     check "$expected" "$actual" : encoded parameter value
-  fi
 fi
 
 
@@ -133,8 +140,10 @@ MIME-Version: 1.0
 [ part  - text/plain -   0B  ]
 EOF
 
-run_prog mhshow $msgnum > $actual 2>&1
-check "$expected" "$actual"
+if test $skip_param_value_checks -eq 0; then
+    run_prog mhshow $msgnum > $actual 2>&1
+    check "$expected" "$actual"
+fi
 
 
 finish_test