test/*.sh: add explicit error checks/returns
authorJames Zern <jzern@google.com>
Wed, 29 Apr 2020 01:51:01 +0000 (18:51 -0700)
committerJames Zern <jzern@google.com>
Wed, 29 Apr 2020 01:51:01 +0000 (18:51 -0700)
commitb120ba5781a520c015b356a7c77c89545ccdee32
treef52513afcd3d455867edc633f410edde32f2ee27
parent3d28ff98039134325cf689d8d08996fc8dabb225
test/*.sh: add explicit error checks/returns

there was an assumption that function calls would terminate early with
an error given 'set -e' was being used. this is true, but only when the
function is part of a simple command otherwise it won't inherit the
behavior. many of the call sites use 'func || return 1' syntax meaning
the function would continue to completion return with the status of the
last command executed. this hid errors with e.g., eval statements. inner
calls within the functions are now explicitly tested for failure.

BUG=aomedia:2669

Change-Id: Ie33a5ac4023dcc800bd302cb8cc54c6c3f2282f5
14 files changed:
test/cx_set_ref.sh
test/decode_to_md5.sh
test/decode_with_drops.sh
test/postproc.sh
test/resize_util.sh
test/set_maps.sh
test/simple_decoder.sh
test/simple_encoder.sh
test/stress.sh
test/twopass_encoder.sh
test/vp8_multi_resolution_encoder.sh
test/vpx_temporal_svc_encoder.sh
test/vpxdec.sh
test/vpxenc.sh