*"ndrpdr-weekly"* )
readarray -t test_tag_array <<< $(sed 's/ //g' \
${tfd}/mlr_weekly/${DUT}-${NODENESS}-${FLAVOR}.md |
- eval ${sed_nics_sub_cmd}) || die
+ eval ${sed_nics_sub_cmd} || echo "perftest") || die
;;
*"mrr-daily"* )
readarray -t test_tag_array <<< $(sed 's/ //g' \
${tfd}/mrr_daily/${DUT}-${NODENESS}-${FLAVOR}.md |
- eval ${sed_nics_sub_cmd}) || die
+ eval ${sed_nics_sub_cmd} || echo "perftest") || die
;;
*"mrr-weekly"* )
readarray -t test_tag_array <<< $(sed 's/ //g' \
${tfd}/mrr_weekly/${DUT}-${NODENESS}-${FLAVOR}.md |
- eval ${sed_nics_sub_cmd}) || die
+ eval ${sed_nics_sub_cmd} || echo "perftest") || die
;;
*"report-iterative"* )
test_sets=(${TEST_TAG_STRING//:/ })
report_file=${test_sets[0]}.md
readarray -t test_tag_array <<< $(sed 's/ //g' \
${tfd}/report_iterative/${NODENESS}-${FLAVOR}/${report_file} |
- eval ${sed_nics_sub_cmd}) || die
+ eval ${sed_nics_sub_cmd} || echo "perftest") || die
;;
*"report-coverage"* )
test_sets=(${TEST_TAG_STRING//:/ })
report_file=${test_sets[0]}.md
readarray -t test_tag_array <<< $(sed 's/ //g' \
${tfd}/report_coverage/${NODENESS}-${FLAVOR}/${report_file} |
- eval ${sed_nics_sub_cmd}) || die
+ eval ${sed_nics_sub_cmd} || echo "perftest") || die
;;
* )
if [[ -z "${TEST_TAG_STRING-}" ]]; then
warn "The following tag expression hints at bad trigger: ${tag}"
warn "Possible cause: Multiple triggers in a single comment."
die "Aborting to avoid triggering too many tests."
+ elif [[ "${tag}" == *"OR"* ]]; then
+ # If OR had higher precedence than AND, it would be useful here.
+ # Some people think it does, thus triggering way too much tests.
+ set -x
+ warn "The following tag expression hints at bad trigger: ${tag}"
+ warn "Operator OR has lower precedence than AND. Use space instead."
+ die "Aborting to avoid triggering too many tests."
elif [[ "${tag}" != "" && "${tag}" != "#"* ]]; then
# Empty and comment lines are skipped.
# Other lines are normal tags, they are to be prefixed.