Allow 'runs_per_test' to be overridden.

runs_per_test allows multiple argument instances, because there's an
option of using a per-label regex format.

Unlike most such flags, earlier instances of the flag were dominating
later ones.

PiperOrigin-RevId: 175530675

1 files changed, 2 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/test/TestConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/test/TestConfiguration.java
index 5eebe89a4c..12b3d51b32 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/test/TestConfiguration.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/test/TestConfiguration.java
@@ -16,6 +16,7 @@ package com.google.devtools.build.lib.analysis.test;
 
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Iterables;
+import com.google.common.collect.Lists;
 import com.google.devtools.build.lib.analysis.config.BuildConfiguration.Fragment;
 import com.google.devtools.build.lib.analysis.config.BuildConfiguration.RunsPerTestConverter;
 import com.google.devtools.build.lib.analysis.config.BuildOptions;
@@ -194,7 +195,7 @@ public class TestConfiguration extends Fragment {
    *     filters, runs it once.
    */
   public int getRunsPerTestForLabel(Label label) {
-    for (PerLabelOptions perLabelRuns : options.runsPerTest) {
+    for (PerLabelOptions perLabelRuns : Lists.reverse(options.runsPerTest)) {
       if (perLabelRuns.isIncluded(label)) {
         return Integer.parseInt(Iterables.getOnlyElement(perLabelRuns.getOptions()));
       }