commit 39ff208b0e6ee18f4d684db458e8599f60e08dd4
author ruperts <ruperts@google.com> Fri Dec 15 12:58:52 2017 -0800
committer Copybara-Service <copybara-piper@google.com> Fri Dec 15 13:00:37 2017 -0800
tree 2b73dd577c0200829d44232007409091da96507e
parent 70c2be189c9c0021f7c424c7399f8bdecc579b4e

In LocalSpawnRunnerTest, be more generous with the expected wall time upper bound (because of interference).

RELNOTES: None.
PiperOrigin-RevId: 179229423

src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java b/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java
index e99899d..31b46c2 100644
--- a/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/exec/local/LocalSpawnRunnerTest.java
@@ -790,9 +790,12 @@
     options.collectLocalExecutionStatistics = true;
 
     Duration minimumWallTimeToSpend = Duration.ofSeconds(10);
-    Duration maximumWallTimeToSpend = minimumWallTimeToSpend.plus(minimumWallTimeToSpend); // double
+    // Because of e.g. interference, wall time taken may be much larger than CPU time used.
+    Duration maximumWallTimeToSpend = Duration.ofSeconds(40);
+
     Duration minimumUserTimeToSpend = minimumWallTimeToSpend;
     Duration maximumUserTimeToSpend = minimumUserTimeToSpend.plus(Duration.ofSeconds(2));
+
     Duration minimumSystemTimeToSpend = Duration.ZERO;
     Duration maximumSystemTimeToSpend = minimumSystemTimeToSpend.plus(Duration.ofSeconds(2));