print-failed-test-output: minor improvements

Drop the subshell used to create the string of dashes (and rename the
variable to "sep" at the same time).  Replace $(cat file) with the
equivalent but faster $(< file).
This commit is contained in:
Todd Zullinger 2019-02-01 11:33:19 -05:00
parent bb15dc829e
commit a245ca5676

View File

@ -3,11 +3,11 @@
shopt -s failglob shopt -s failglob
# Print output from failing tests # Print output from failing tests
dashes=$(printf "%80s" '' | tr ' ' '-') printf -v sep "%0.s-" {1..80}
for exit_file in t/test-results/*.exit; do for exit_file in t/test-results/*.exit; do
[ "$(cat "$exit_file")" -eq 0 ] && continue [ "$(< "$exit_file")" -eq 0 ] && continue
out_file="${exit_file%exit}out" out_file="${exit_file%exit}out"
printf '\n%s\n%s\n%s\n' "$dashes" "$out_file" "$dashes" printf '\n%s\n%s\n%s\n' "$sep" "$out_file" "$sep"
cat "$out_file" cat "$out_file"
done done
exit 1 exit 1