diff options
author | Klaus Aehlig <aehlig@google.com> | 2016-05-27 12:23:39 +0000 |
---|---|---|
committer | Dmitry Lomov <dslomov@google.com> | 2016-05-30 09:17:37 +0000 |
commit | 521510ac5a55dc84244d27c639d86a710d05148f (patch) | |
tree | c81f1748883ed26944d5ae02e1da9e208f70fbec /src/test/java/com | |
parent | c6fd6bb3caa78518fbb9148d81efecdee540c29a (diff) |
Remove now dead class LoadingProgressReceiver
With the change of reporting progress during loading and analysis
by a custom package tracking class, the LoadingProgressReceiver
is not used anywhere anymore. Threfore, remove it.
--
Change-Id: Ia88d2d5608dea1bb395cd5b3c3e12c16200ecb41
Reviewed-on: https://bazel-review.googlesource.com/#/c/3701
MOS_MIGRATED_REVID=123410961
Diffstat (limited to 'src/test/java/com')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/skyframe/LoadingProgressReceiverTest.java | 77 |
1 files changed, 0 insertions, 77 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/LoadingProgressReceiverTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/LoadingProgressReceiverTest.java deleted file mode 100644 index 21ab30718f..0000000000 --- a/src/test/java/com/google/devtools/build/lib/skyframe/LoadingProgressReceiverTest.java +++ /dev/null @@ -1,77 +0,0 @@ -// Copyright 2016 The Bazel Authors. All rights reserved. -// -// Licensed under the Apache License, Version 2.0 (the "License"); -// you may not use this file except in compliance with the License. -// You may obtain a copy of the License at -// -// http://www.apache.org/licenses/LICENSE-2.0 -// -// Unless required by applicable law or agreed to in writing, software -// distributed under the License is distributed on an "AS IS" BASIS, -// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -// See the License for the specific language governing permissions and -// limitations under the License. -package com.google.devtools.build.lib.skyframe; - -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertTrue; - -import com.google.common.base.Supplier; -import com.google.devtools.build.skyframe.EvaluationProgressReceiver; -import com.google.devtools.build.skyframe.SkyKey; -import com.google.devtools.build.skyframe.SkyValue; - -import org.junit.Test; -import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; -import org.mockito.Mockito; - -/** - * Tests {@link LoadingProgressReceiver} - */ -@RunWith(JUnit4.class) -public class LoadingProgressReceiverTest { - - @Test - public void testPackageVisible() { - // If there is only one package being loaded, it should be visible - // somewhere in the progress string. - LoadingProgressReceiver progressReceiver = new LoadingProgressReceiver(); - - String packageName = "//some/fancy/package/name"; - progressReceiver.enqueueing(SkyKey.create(SkyFunctions.PACKAGE, packageName)); - String state = progressReceiver.progressState(); - - assertTrue( - "Package name '" + packageName + "' should occour in state '" + state + "'", - state.contains(packageName)); - } - - @Test - public void testFinishedPackage() { - // Consider the situation where two packages are loaded, and the one that - // started loading second has already finished loading. ... - LoadingProgressReceiver progressReceiver = new LoadingProgressReceiver(); - - String packageName = "//some/fancy/package/name"; - String otherPackageName = "//some/other/unrelated/package"; - SkyKey otherKey = SkyKey.create(SkyFunctions.PACKAGE, otherPackageName); - Supplier<SkyValue> valueSupplier = Mockito.mock(Supplier.class); - progressReceiver.enqueueing(SkyKey.create(SkyFunctions.PACKAGE, packageName)); - progressReceiver.enqueueing(otherKey); - progressReceiver.evaluated( - otherKey, valueSupplier, EvaluationProgressReceiver.EvaluationState.BUILT); - String state = progressReceiver.progressState(); - - // ... Then, the finished package is not mentioned in the state, - assertFalse( - "Package name '" + otherPackageName + "' should not occur in state '" + state + "'", - state.contains(otherPackageName)); - // the only running package is mentioned in the state, and - assertTrue( - "Package name '" + packageName + "' should occour in state '" + state + "'", - state.contains(packageName)); - // the progress count is correct. - assertTrue("Progress count should occur in state", state.contains("1 / 2")); - } -} |