Merge JunitTestUtils into MoreAsserts; update all callers.
--
MOS_MIGRATED_REVID=98914195
diff --git a/src/test/java/com/google/devtools/build/skyframe/ParallelEvaluatorTest.java b/src/test/java/com/google/devtools/build/skyframe/ParallelEvaluatorTest.java
index dfd14d4..af2c4e0 100644
--- a/src/test/java/com/google/devtools/build/skyframe/ParallelEvaluatorTest.java
+++ b/src/test/java/com/google/devtools/build/skyframe/ParallelEvaluatorTest.java
@@ -16,6 +16,11 @@
import static com.google.common.collect.Iterables.getOnlyElement;
import static com.google.common.truth.Truth.assertThat;
import static com.google.common.truth.Truth.assertWithMessage;
+import static com.google.devtools.build.lib.testutil.MoreAsserts.assertContainsEvent;
+import static com.google.devtools.build.lib.testutil.MoreAsserts.assertEventCount;
+import static com.google.devtools.build.lib.testutil.MoreAsserts.assertGreaterThan;
+import static com.google.devtools.build.lib.testutil.MoreAsserts.assertLessThan;
+import static com.google.devtools.build.lib.testutil.MoreAsserts.assertNoEvents;
import static com.google.devtools.build.skyframe.GraphTester.CONCATENATE;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
@@ -39,8 +44,6 @@
import com.google.devtools.build.lib.events.EventKind;
import com.google.devtools.build.lib.events.OutputFilter.RegexOutputFilter;
import com.google.devtools.build.lib.events.Reporter;
-import com.google.devtools.build.lib.testutil.JunitTestUtils;
-import com.google.devtools.build.lib.testutil.MoreAsserts;
import com.google.devtools.build.lib.testutil.TestThread;
import com.google.devtools.build.lib.testutil.TestUtils;
import com.google.devtools.build.skyframe.GraphTester.StringValue;
@@ -139,7 +142,7 @@
tester.getOrCreate("ab").addDependency("a").addDependency("b").setComputedValue(CONCATENATE);
StringValue value = (StringValue) eval(false, GraphTester.toSkyKey("ab"));
assertEquals("ab", value.getValue());
- JunitTestUtils.assertNoEvents(eventCollector);
+ assertNoEvents(eventCollector);
}
/**
@@ -403,8 +406,8 @@
set("a", "a").setWarning("warning on 'a'");
StringValue value = (StringValue) eval(false, GraphTester.toSkyKey("a"));
assertEquals("a", value.getValue());
- JunitTestUtils.assertContainsEvent(eventCollector, "warning on 'a'");
- JunitTestUtils.assertEventCount(1, eventCollector);
+ assertContainsEvent(eventCollector, "warning on 'a'");
+ assertEventCount(1, eventCollector);
}
@Test
@@ -416,8 +419,8 @@
tester.getOrCreate(a).setTag("a");
StringValue value = (StringValue) eval(false, a);
assertEquals("a value", value.getValue());
- JunitTestUtils.assertContainsEvent(eventCollector, "warning message");
- JunitTestUtils.assertEventCount(1, eventCollector);
+ assertContainsEvent(eventCollector, "warning message");
+ assertEventCount(1, eventCollector);
}
@Test
@@ -429,7 +432,7 @@
tester.getOrCreate(a).setTag("b");
StringValue value = (StringValue) eval(false, a);
assertEquals("a value", value.getValue());
- JunitTestUtils.assertEventCount(0, eventCollector); }
+ assertEventCount(0, eventCollector); }
@Test
public void warningDoesNotMatchRegex() throws Exception {
@@ -440,7 +443,7 @@
tester.getOrCreate(a).setTag("a");
StringValue value = (StringValue) eval(false, a);
assertEquals("a", value.getValue());
- JunitTestUtils.assertEventCount(0, eventCollector);
+ assertEventCount(0, eventCollector);
}
/** Regression test: events from already-done value not replayed. */
@@ -453,15 +456,15 @@
tester.getOrCreate(top).addDependency(a).setComputedValue(CONCATENATE);
// Build a so that it is already in the graph.
eval(false, a);
- JunitTestUtils.assertEventCount(1, eventCollector);
+ assertEventCount(1, eventCollector);
eventCollector.clear();
// Build top. The warning from a should be reprinted.
eval(false, top);
- JunitTestUtils.assertEventCount(1, eventCollector);
+ assertEventCount(1, eventCollector);
eventCollector.clear();
// Build top again. The warning should have been stored in the value.
eval(false, top);
- JunitTestUtils.assertEventCount(1, eventCollector);
+ assertEventCount(1, eventCollector);
}
@Test
@@ -495,9 +498,9 @@
});
evaluator.eval(ImmutableList.of(a));
assertTrue(evaluated.get());
- JunitTestUtils.assertEventCount(2, eventCollector);
- JunitTestUtils.assertContainsEvent(eventCollector, "boop");
- JunitTestUtils.assertContainsEvent(eventCollector, "beep");
+ assertEventCount(2, eventCollector);
+ assertContainsEvent(eventCollector, "boop");
+ assertContainsEvent(eventCollector, "beep");
eventCollector.clear();
evaluator = makeEvaluator(graph,
ImmutableMap.of(GraphTester.NODE_TYPE, tester.createDelegatingFunction()),
@@ -505,8 +508,8 @@
evaluated.set(false);
evaluator.eval(ImmutableList.of(a));
assertFalse(evaluated.get());
- JunitTestUtils.assertEventCount(1, eventCollector);
- JunitTestUtils.assertContainsEvent(eventCollector, "boop");
+ assertEventCount(1, eventCollector);
+ assertContainsEvent(eventCollector, "boop");
}
@Test
@@ -1150,8 +1153,8 @@
* topKey, if {@code selfEdge} is true.
*/
private static void assertManyCycles(ErrorInfo errorInfo, SkyKey topKey, boolean selfEdge) {
- MoreAsserts.assertGreaterThan(1, Iterables.size(errorInfo.getCycleInfo()));
- MoreAsserts.assertLessThan(50, Iterables.size(errorInfo.getCycleInfo()));
+ assertGreaterThan(1, Iterables.size(errorInfo.getCycleInfo()));
+ assertLessThan(50, Iterables.size(errorInfo.getCycleInfo()));
boolean foundSelfEdge = false;
for (CycleInfo cycle : errorInfo.getCycleInfo()) {
assertEquals(1, cycle.getCycle().size()); // Self-edge.
