Arvados

Ready for review in 8f6f8d5d727c9b0ae4600c7b546db248e012d203 on branch 16451-run-tests-interactive-run-test-many-times

Shifting the vars around this way leaves testargs as " " instead of empty when there are no opts provided. AFAICS the only side effect is that the "go test" recipe won't use coverflags, which doesn't seem too important. Maybe worth doing something like this instead?

if [[ "${nextcmd}" =~ ^[0-9] ]]; then
  read count nextcmd <<<"${nextcmd}" 
fi

Other than that nit, LGTM.

Tom Clegg wrote:

Shifting the vars around this way leaves testargs as " " instead of empty when there are no opts provided. AFAICS the only side effect is that the "go test" recipe won't use coverflags, which doesn't seem too important. Maybe worth doing something like this instead?

[...]

Other than that nit, LGTM.

Yes! That's also a nice simplification. Merged with that fix, thanks.