aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJia Tan <jiat0218@gmail.com>2023-07-14 21:30:25 +0800
committerJia Tan <jiat0218@gmail.com>2023-07-14 21:30:25 +0800
commit2c189bb00af73dc7ba1a67a9d274d5be03ee3a88 (patch)
tree381fc49fd0537d420f1cfac160c4882f93192198
parentTranslations: Update the Chinese (traditional) translation. (diff)
downloadxz-2c189bb00af73dc7ba1a67a9d274d5be03ee3a88.tar.xz
Tests: Improve feature testing for skipping.
Fixed a bug where test_compress_* would all fail if arm64 or armthumb filters were enabled for compression but arm was disabled. Since the grep tests only checked for "define HAVE_ENCODER_ARM", this would match on HAVE_ENCODER_ARM64 or HAVE_ENCODER_ARMTHUMB. Now the config.h feature test requires " 1" at the end to prevent the prefix problem. have_feature() was also updated for this even though there were known current bugs affecting it. This is just in case future features have a similar prefix problem.
Diffstat (limited to '')
-rwxr-xr-xtests/test_compress.sh4
-rwxr-xr-xtests/test_files.sh2
2 files changed, 3 insertions, 3 deletions
diff --git a/tests/test_compress.sh b/tests/test_compress.sh
index 61d20ce9..0aa81001 100755
--- a/tests/test_compress.sh
+++ b/tests/test_compress.sh
@@ -130,8 +130,8 @@ test_xz -4
test_filter()
{
- grep "define HAVE_ENCODER_$1" ../config.h > /dev/null || return
- grep "define HAVE_DECODER_$1" ../config.h > /dev/null || return
+ grep "define HAVE_ENCODER_$1 1" ../config.h > /dev/null || return
+ grep "define HAVE_DECODER_$1 1" ../config.h > /dev/null || return
shift
test_xz "$@" --lzma2=dict=64KiB,nice=32,mode=fast
}
diff --git a/tests/test_files.sh b/tests/test_files.sh
index 64d73081..e42af918 100755
--- a/tests/test_files.sh
+++ b/tests/test_files.sh
@@ -35,7 +35,7 @@ fi
EXIT_STATUS=0
have_feature()
{
- grep "define HAVE_$1" ../config.h > /dev/null && return 0
+ grep "define HAVE_$1 1" ../config.h > /dev/null && return 0
printf '%s: Skipping because HAVE_%s is not enabled\n' "$2" "$1"
EXIT_STATUS=77
return 1