Feature #16451

[testing] give run-tests interactive a mode to run a test many times

Added by Ward Vandewege over 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Story points:
-
Release relationship:
Auto

Associated revisions

Revision 9007cff9
Added by Ward Vandewege over 1 year ago

Merge branch '16451-run-tests-interactive-run-test-many-times'

closes #16451

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

History

#1 Updated by Ward Vandewege over 1 year ago

  • Status changed from New to In Progress

#2 Updated by Ward Vandewege over 1 year ago

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

#3 Updated by Tom Clegg over 1 year ago

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.

#4 Updated by Ward Vandewege over 1 year ago

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.

#5 Updated by Anonymous over 1 year ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Resolved

#6 Updated by Ward Vandewege about 1 year ago

  • Release set to 25

Also available in: Atom PDF