diff options
author | 2015-11-23 18:51:55 +0000 | |
---|---|---|
committer | 2015-11-24 14:41:06 +0000 | |
commit | 4f487f4a4d27504965c2ecdc965d17e7ba846db5 (patch) | |
tree | 24a8a2ca8a66ff2a5879e09e1b6ce8aec9fb4851 /src/test/java/com/google | |
parent | 18212f10f2bbb6a29c5c1f112e0a43b92071b11c (diff) |
Extract ReverseDepsUtil interface so that InMemoryNodeEntry can be partially isolated from implementation details.
--
MOS_MIGRATED_REVID=108523104
Diffstat (limited to 'src/test/java/com/google')
-rw-r--r-- | src/test/java/com/google/devtools/build/skyframe/ReverseDepsUtilImplTest.java (renamed from src/test/java/com/google/devtools/build/skyframe/ReverseDepsUtilTest.java) | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/src/test/java/com/google/devtools/build/skyframe/ReverseDepsUtilTest.java b/src/test/java/com/google/devtools/build/skyframe/ReverseDepsUtilImplTest.java index 9d743f5f65..4c2d9c6f62 100644 --- a/src/test/java/com/google/devtools/build/skyframe/ReverseDepsUtilTest.java +++ b/src/test/java/com/google/devtools/build/skyframe/ReverseDepsUtilImplTest.java @@ -28,10 +28,10 @@ import java.util.Collections; import java.util.List; /** - * Test for {@code ReverseDepsUtil}. + * Test for {@code ReverseDepsUtilImpl}. */ @RunWith(Parameterized.class) -public class ReverseDepsUtilTest { +public class ReverseDepsUtilImplTest { private static final SkyFunctionName NODE_TYPE = SkyFunctionName.create("Type"); private final int numElements; @@ -40,17 +40,17 @@ public class ReverseDepsUtilTest { public static List<Object[]> paramenters() { List<Object[]> params = new ArrayList<>(); for (int i = 0; i < 20; i++) { - params.add(new Object[]{i}); + params.add(new Object[] {i}); } return params; } - public ReverseDepsUtilTest(int numElements) { + public ReverseDepsUtilImplTest(int numElements) { this.numElements = numElements; } private static final ReverseDepsUtil<Example> REVERSE_DEPS_UTIL = - new ReverseDepsUtil<Example>() { + new ReverseDepsUtilImpl<Example>() { @Override void setReverseDepsObject(Example container, Object object) { container.reverseDeps = object; @@ -142,7 +142,8 @@ public class ReverseDepsUtilTest { try { REVERSE_DEPS_UTIL.getReverseDeps(example); assertThat(numElements).isEqualTo(0); - } catch (Exception expected) { } + } catch (Exception expected) { + } } @Test @@ -199,7 +200,9 @@ public class ReverseDepsUtilTest { try { REVERSE_DEPS_UTIL.maybeCheckReverseDepNotPresent(example, new SkyKey(NODE_TYPE, 0)); // Should only fail if empty or above the checking threshold. - assertThat(numElements == 0 || numElements >= ReverseDepsUtil.MAYBE_CHECK_THRESHOLD).isTrue(); - } catch (Exception expected) { } + assertThat(numElements == 0 || numElements >= ReverseDepsUtilImpl.MAYBE_CHECK_THRESHOLD) + .isTrue(); + } catch (Exception expected) { + } } } |