Module Name: src Committed By: rillig Date: Wed Jun 16 10:19:23 UTC 2021
Modified Files: src/tests/usr.bin/xlint/lint1: t_integration.sh Log Message: tests/lint: clean up ATF test driver No functional change. To generate a diff of this commit: cvs rdiff -u -r1.50 -r1.51 src/tests/usr.bin/xlint/lint1/t_integration.sh Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/tests/usr.bin/xlint/lint1/t_integration.sh diff -u src/tests/usr.bin/xlint/lint1/t_integration.sh:1.50 src/tests/usr.bin/xlint/lint1/t_integration.sh:1.51 --- src/tests/usr.bin/xlint/lint1/t_integration.sh:1.50 Sun May 16 11:11:37 2021 +++ src/tests/usr.bin/xlint/lint1/t_integration.sh Wed Jun 16 10:19:23 2021 @@ -1,4 +1,4 @@ -# $NetBSD: t_integration.sh,v 1.50 2021/05/16 11:11:37 rillig Exp $ +# $NetBSD: t_integration.sh,v 1.51 2021/06/16 10:19:23 rillig Exp $ # # Copyright (c) 2008, 2010 The NetBSD Foundation, Inc. # All rights reserved. @@ -25,9 +25,10 @@ # POSSIBILITY OF SUCH DAMAGE. # -LINT1=/usr/libexec/lint1 +lint1=/usr/libexec/lint1 + +test_case_names= -Names= extract_flags() { @@ -59,47 +60,44 @@ check_lint1() local exp="${src%.c}.exp" local src_ln="${src%.c}.ln" local wrk_ln="${1%.c}.ln" - local flags="$(extract_flags "${src}")" + local flags="$(extract_flags "$src")" - if [ ! -f "${src_ln}" ]; then - src_ln="/dev/null" - wrk_ln="/dev/null" + if [ ! -f "$src_ln" ]; then + src_ln='/dev/null' + wrk_ln='/dev/null' fi - if [ -f "${exp}" ]; then + if [ -f "$exp" ]; then # shellcheck disable=SC2086 - atf_check -s not-exit:0 -o "file:${exp}" -e empty \ - ${LINT1} ${flags} "${src}" "${wrk_ln}" + atf_check -s not-exit:0 -o "file:$exp" -e empty \ + "$lint1" $flags "$src" "$wrk_ln" else # shellcheck disable=SC2086 atf_check -s exit:0 \ - ${LINT1} ${flags} "${src}" "${wrk_ln}" + "$lint1" $flags "$src" "$wrk_ln" fi - if [ "${src_ln}" != "/dev/null" ]; then - atf_check -o "file:${src_ln}" cat "${wrk_ln}" + if [ "$src_ln" != '/dev/null' ]; then + atf_check -o "file:$src_ln" cat "$wrk_ln" fi } test_case() { - local name="${1}"; shift - local descr="${*}" + local name="$1" - atf_test_case ${name} + atf_test_case "$name" eval "${name}_head() { - if [ \"${descr}\" ]; then - atf_set \"descr\" \"${descr}\" - fi - atf_set \"require.progs\" \"${LINT1}\" + atf_set 'require.progs' '$lint1' }" eval "${name}_body() { - check_lint1 ${name}.c + check_lint1 '$name.c' }" - Names="${Names} ${name}" + test_case_names="$test_case_names $name" } + test_case d_bltinoffsetof test_case d_c99_anon_struct test_case d_c99_anon_union @@ -126,7 +124,6 @@ test_case d_return_type test_case d_type_question_colon test_case d_typefun test_case d_typename_as_var - test_case d_c99_struct_init test_case d_c99_union_init1 test_case d_c99_union_init2 @@ -137,27 +134,22 @@ test_case d_nested_structs test_case d_packed_structs test_case d_pr_22119 test_case d_struct_init_nested - test_case d_cast_init test_case d_cast_init2 test_case d_cast_lhs - test_case d_gcc_func test_case d_c99_func - test_case d_gcc_variable_array_init test_case d_c9x_array_init test_case d_c99_decls_after_stmt test_case d_c99_decls_after_stmt3 test_case d_nolimit_init test_case d_zero_sized_arrays - test_case d_compound_literals1 test_case d_compound_literals2 test_case d_gcc_compound_statements1 test_case d_gcc_compound_statements2 test_case d_gcc_compound_statements3 - test_case d_cvt_in_ternary test_case d_cvt_constant test_case d_ellipsis_in_switch @@ -173,19 +165,16 @@ test_case d_type_conv2 test_case d_type_conv3 test_case d_incorrect_array_size test_case d_long_double_int - test_case emit test_case expr_range - +test_case feat_stacktrace test_case gcc_attribute test_case gcc_attribute_aligned test_case gcc_bit_field_types test_case gcc_init_compound_literal test_case gcc_typeof_after_statement - test_case op_colon -test_case feat_stacktrace test_case all_messages all_messages_body() @@ -195,8 +184,8 @@ all_messages_body() failed="" for msg in $(seq 0 344); do - name="$(printf 'msg_%03d.c' "${msg}")" - check_lint1 "${name}" \ + name="$(printf 'msg_%03d.c' "$msg")" + check_lint1 "$name" \ || failed="$failed${failed:+ }$name" done @@ -208,7 +197,7 @@ all_messages_body() atf_init_test_cases() { - for name in ${Names}; do - atf_add_test_case ${name} + for name in $test_case_names; do + atf_add_test_case "$name" done }