Fix multi-word test case names to use lowerCamelCase.
This is to follow the style guide recommendation in:
https://google.github.io/styleguide/javaguide.html#s5.2.3-method-names
and to fix lint warnings that have already gotten in the way of two
of my recent CLs.
This change was automatically generated by collecting the list of all
affected files in a "list" file and doing:
for f in $(cat /tmp/list); do
sed -itmp -e '/void test.*_[A-Z]/s/\(_[A-Z]\)/\L\1/g' "$f"
done
... and then skimming through the diff to undo a few "fixes" that were
incorrect because we have test names encoding literals like "BUILD".
PiperOrigin-RevId: 329363594
diff --git a/src/test/java/com/google/devtools/build/lib/actions/ActionCacheCheckerTest.java b/src/test/java/com/google/devtools/build/lib/actions/ActionCacheCheckerTest.java
index 91d2f9a..c5f434d 100644
--- a/src/test/java/com/google/devtools/build/lib/actions/ActionCacheCheckerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/actions/ActionCacheCheckerTest.java
@@ -313,22 +313,22 @@
}
@Test
- public void testMiddleman_NotCached() throws Exception {
+ public void testMiddleman_notCached() throws Exception {
doTestNotCached(new NullMiddlemanAction(), MissReason.DIFFERENT_DEPS);
}
@Test
- public void testMiddleman_Cached() throws Exception {
+ public void testMiddleman_cached() throws Exception {
doTestCached(new NullMiddlemanAction(), MissReason.DIFFERENT_DEPS);
}
@Test
- public void testMiddleman_CorruptedCacheEntry() throws Exception {
+ public void testMiddleman_corruptedCacheEntry() throws Exception {
doTestCorruptedCacheEntry(new NullMiddlemanAction());
}
@Test
- public void testMiddleman_DifferentFiles() throws Exception {
+ public void testMiddleman_differentFiles() throws Exception {
Action action =
new NullMiddlemanAction() {
@Override
diff --git a/src/test/java/com/google/devtools/build/lib/actions/ActionResultTest.java b/src/test/java/com/google/devtools/build/lib/actions/ActionResultTest.java
index 820261a..10324c7 100644
--- a/src/test/java/com/google/devtools/build/lib/actions/ActionResultTest.java
+++ b/src/test/java/com/google/devtools/build/lib/actions/ActionResultTest.java
@@ -27,7 +27,7 @@
public final class ActionResultTest {
@Test
- public void testCumulativeCommandExecutionTime_NoSpawnResults() {
+ public void testCumulativeCommandExecutionTime_noSpawnResults() {
List<SpawnResult> spawnResults = ImmutableList.of();
ActionResult actionResult = ActionResult.create(spawnResults);
assertThat(actionResult.cumulativeCommandExecutionWallTime()).isEmpty();
@@ -40,7 +40,7 @@
}
@Test
- public void testCumulativeCommandExecutionTime_OneSpawnResult() {
+ public void testCumulativeCommandExecutionTime_oneSpawnResult() {
SpawnResult spawnResult =
new SpawnResult.Builder()
.setWallTime(Duration.ofMillis(1984))
@@ -64,7 +64,7 @@
}
@Test
- public void testCumulativeCommandExecutionTime_ManySpawnResults() {
+ public void testCumulativeCommandExecutionTime_manySpawnResults() {
SpawnResult spawnResult1 =
new SpawnResult.Builder()
.setWallTime(Duration.ofMillis(1979))
@@ -110,7 +110,7 @@
}
@Test
- public void testCumulativeCommandExecutionTime_ManyEmptySpawnResults() {
+ public void testCumulativeCommandExecutionTime_manyEmptySpawnResults() {
SpawnResult spawnResult1 =
new SpawnResult.Builder()
.setStatus(SpawnResult.Status.SUCCESS)
@@ -138,7 +138,7 @@
}
@Test
- public void testCumulativeCommandExecutionTime_ManySpawnResults_ButOnlyUserTime() {
+ public void testCumulativeCommandExecutionTime_manySpawnResults_butOnlyUserTime() {
SpawnResult spawnResult1 =
new SpawnResult.Builder()
.setUserTime(Duration.ofMillis(2))
@@ -164,7 +164,7 @@
}
@Test
- public void testCumulativeCommandExecutionTime_ManySpawnResults_ButOnlySystemTime() {
+ public void testCumulativeCommandExecutionTime_manySpawnResults_butOnlySystemTime() {
SpawnResult spawnResult1 =
new SpawnResult.Builder()
.setSystemTime(Duration.ofMillis(33))
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/AspectDefinitionTest.java b/src/test/java/com/google/devtools/build/lib/analysis/AspectDefinitionTest.java
index adb6643..416141c 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/AspectDefinitionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/AspectDefinitionTest.java
@@ -85,7 +85,7 @@
public static final TestAspectClass TEST_ASPECT_CLASS = new TestAspectClass();
@Test
- public void testAspectWithImplicitOrLateboundAttribute_AddsToAttributeMap() throws Exception {
+ public void testAspectWithImplicitOrLateboundAttribute_addsToAttributeMap() throws Exception {
Attribute implicit = attr("$runtime", BuildType.LABEL)
.value(Label.parseAbsoluteUnchecked("//run:time"))
.build();
@@ -102,7 +102,7 @@
}
@Test
- public void testAspectWithDuplicateAttribute_FailsToAdd() throws Exception {
+ public void testAspectWithDuplicateAttribute_failsToAdd() throws Exception {
assertThrows(
IllegalArgumentException.class,
() ->
@@ -116,7 +116,7 @@
}
@Test
- public void testAspectWithUserVisibleAttribute_FailsToAdd() throws Exception {
+ public void testAspectWithUserVisibleAttribute_failsToAdd() throws Exception {
assertThrows(
IllegalArgumentException.class,
() ->
@@ -129,7 +129,7 @@
}
@Test
- public void testAttributeAspect_WrapsAndAddsToMap() throws Exception {
+ public void testAttributeAspect_wrapsAndAddsToMap() throws Exception {
AspectDefinition withAspects = new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.propagateAlongAttribute("srcs")
.propagateAlongAttribute("deps")
@@ -140,7 +140,7 @@
}
@Test
- public void testAttributeAspect_AllAttributes() throws Exception {
+ public void testAttributeAspect_allAttributes() throws Exception {
AspectDefinition withAspects = new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.propagateAlongAllAttributes()
.build();
@@ -150,7 +150,7 @@
}
@Test
- public void testRequireProvider_AddsToSetOfRequiredProvidersAndNames() throws Exception {
+ public void testRequireProvider_addsToSetOfRequiredProvidersAndNames() throws Exception {
AspectDefinition requiresProviders =
new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.requireProviders(P1.class, P2.class)
@@ -175,8 +175,8 @@
.isFalse();
}
- @Test
- public void testRequireProvider_AddsTwoSetsOfRequiredProvidersAndNames() throws Exception {
+ @Test
+ public void testRequireProvider_addsTwoSetsOfRequiredProvidersAndNames() throws Exception {
AspectDefinition requiresProviders =
new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.requireProviderSets(
@@ -203,7 +203,7 @@
}
@Test
- public void testRequireAspectClass_DefaultAcceptsNothing() {
+ public void testRequireAspectClass_defaultAcceptsNothing() {
AspectDefinition noAspects = new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.build();
@@ -221,14 +221,14 @@
}
@Test
- public void testNoConfigurationFragmentPolicySetup_HasNonNullPolicy() throws Exception {
+ public void testNoConfigurationFragmentPolicySetup_hasNonNullPolicy() throws Exception {
AspectDefinition noPolicy = new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.build();
assertThat(noPolicy.getConfigurationFragmentPolicy()).isNotNull();
}
@Test
- public void testMissingFragmentPolicy_PropagatedToConfigurationFragmentPolicy() throws Exception {
+ public void testMissingFragmentPolicy_propagatedToConfigurationFragmentPolicy() throws Exception {
AspectDefinition missingFragments = new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.setMissingFragmentPolicy(MissingFragmentPolicy.IGNORE)
.build();
@@ -238,7 +238,7 @@
}
@Test
- public void testRequiresConfigurationFragments_PropagatedToConfigurationFragmentPolicy()
+ public void testRequiresConfigurationFragments_propagatedToConfigurationFragmentPolicy()
throws Exception {
AspectDefinition requiresFragments = new AspectDefinition.Builder(TEST_ASPECT_CLASS)
.requiresConfigurationFragments(Integer.class, String.class)
@@ -254,7 +254,7 @@
private static class BarFragment extends Fragment {}
@Test
- public void testRequiresHostConfigurationFragments_PropagatedToConfigurationFragmentPolicy()
+ public void testRequiresHostConfigurationFragments_propagatedToConfigurationFragmentPolicy()
throws Exception {
AspectDefinition requiresFragments =
ConfigAwareAspectBuilder.of(new AspectDefinition.Builder(TEST_ASPECT_CLASS))
@@ -268,7 +268,7 @@
}
@Test
- public void testRequiresConfigurationFragmentNames_PropagatedToConfigurationFragmentPolicy()
+ public void testRequiresConfigurationFragmentNames_propagatedToConfigurationFragmentPolicy()
throws Exception {
AspectDefinition requiresFragments =
new AspectDefinition.Builder(TEST_ASPECT_CLASS)
@@ -282,7 +282,7 @@
}
@Test
- public void testRequiresHostConfigurationFragmentNames_PropagatedToConfigurationFragmentPolicy()
+ public void testRequiresHostConfigurationFragmentNames_propagatedToConfigurationFragmentPolicy()
throws Exception {
AspectDefinition requiresFragments =
ConfigAwareAspectBuilder.of(new AspectDefinition.Builder(TEST_ASPECT_CLASS))
@@ -298,7 +298,7 @@
}
@Test
- public void testEmptyStarlarkConfigurationFragmentPolicySetup_HasNonNullPolicy()
+ public void testEmptyStarlarkConfigurationFragmentPolicySetup_hasNonNullPolicy()
throws Exception {
AspectDefinition noPolicy =
ConfigAwareAspectBuilder.of(new AspectDefinition.Builder(TEST_ASPECT_CLASS))
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java b/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java
index 5d44a4d..9ebd111 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/BuildViewTest.java
@@ -846,12 +846,12 @@
}
@Test
- public void testDepOnGoodTargetInBadPkgAndTransitiveCycle_NotIncremental() throws Exception {
+ public void testDepOnGoodTargetInBadPkgAndTransitiveCycle_notIncremental() throws Exception {
runTestDepOnGoodTargetInBadPkgAndTransitiveCycle(/*incremental=*/false);
}
@Test
- public void testDepOnGoodTargetInBadPkgAndTransitiveCycle_Incremental() throws Exception {
+ public void testDepOnGoodTargetInBadPkgAndTransitiveCycle_incremental() throws Exception {
if (getInternalTestExecutionMode() != TestConstants.InternalTestExecutionMode.NORMAL) {
// TODO(b/67412276): handle cycles properly.
return;
@@ -864,7 +864,7 @@
* in error.
*/
@Test
- public void testCycleReporting_TargetCycleWhenPackageInError() throws Exception {
+ public void testCycleReporting_targetCycleWhenPackageInError() throws Exception {
if (getInternalTestExecutionMode() != TestConstants.InternalTestExecutionMode.NORMAL) {
// TODO(b/67412276): handle cycles properly.
return;
@@ -1253,7 +1253,7 @@
}
@Test
- public void testNonTopLevelErrorsPrintedExactlyOnce_KeepGoing() throws Exception {
+ public void testNonTopLevelErrorsPrintedExactlyOnce_keepGoing() throws Exception {
if (getInternalTestExecutionMode() != TestConstants.InternalTestExecutionMode.NORMAL) {
// TODO(b/67651960): fix or justify disabling.
return;
@@ -1270,7 +1270,7 @@
}
@Test
- public void testNonTopLevelErrorsPrintedExactlyOnce_ActionListener() throws Exception {
+ public void testNonTopLevelErrorsPrintedExactlyOnce_actionListener() throws Exception {
if (getInternalTestExecutionMode() != TestConstants.InternalTestExecutionMode.NORMAL) {
// TODO(b/67651960): fix or justify disabling.
return;
@@ -1290,7 +1290,7 @@
}
@Test
- public void testNonTopLevelErrorsPrintedExactlyOnce_ActionListener_KeepGoing() throws Exception {
+ public void testNonTopLevelErrorsPrintedExactlyOnce_actionListener_keepGoing() throws Exception {
if (getInternalTestExecutionMode() != TestConstants.InternalTestExecutionMode.NORMAL) {
// TODO(b/67651960): fix or justify disabling.
return;
@@ -1550,8 +1550,7 @@
@Override
@Test
- public void testCycleReporting_TargetCycleWhenPackageInError() {
- }
+ public void testCycleReporting_targetCycleWhenPackageInError() {}
@Override
@Test
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformUtilsTest.java b/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformUtilsTest.java
index f1048b9..52db06b 100644
--- a/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformUtilsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/analysis/platform/PlatformUtilsTest.java
@@ -80,7 +80,7 @@
}
@Test
- public void testParsePlatformSortsProperties_ExecProperties() throws Exception {
+ public void testParsePlatformSortsProperties_execProperties() throws Exception {
// execProperties are chosen even if there are remoteOptions
ImmutableMap<String, String> map = ImmutableMap.of("aa", "99", "zz", "66", "dd", "11");
Spawn s = new SpawnBuilder("dummy").withExecProperties(map).build();
diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/BuildResultTestCase.java b/src/test/java/com/google/devtools/build/lib/buildtool/BuildResultTestCase.java
index e3444c7..645c818 100644
--- a/src/test/java/com/google/devtools/build/lib/buildtool/BuildResultTestCase.java
+++ b/src/test/java/com/google/devtools/build/lib/buildtool/BuildResultTestCase.java
@@ -209,7 +209,7 @@
}
@Test
- public void testSymlinkOutputAtCwdUnderWorkspace_WithFlagOn() throws Exception {
+ public void testSymlinkOutputAtCwdUnderWorkspace_withFlagOn() throws Exception {
write("my_clib/BUILD", "cc_library(name='my_clib', srcs=['myclib.cc'])\n");
write("my_clib/myclib.cc", "void f() {}");
@@ -227,7 +227,7 @@
}
@Test
- public void testSymlinkOutputAtCwdUnderWorkspace_WithFlagOff() throws Exception {
+ public void testSymlinkOutputAtCwdUnderWorkspace_withFlagOff() throws Exception {
write("my_clib/BUILD", "cc_library(name='my_clib', srcs=['myclib.cc'])\n");
write("my_clib/myclib.cc", "void f() {}");
@@ -247,7 +247,7 @@
}
@Test
- public void testSymlinkOutputAtCwdEqualWorkspace_WithFlagOn() throws Exception {
+ public void testSymlinkOutputAtCwdEqualWorkspace_withFlagOn() throws Exception {
write("my_clib/BUILD", "cc_library(name='my_clib', srcs=['myclib.cc'])\n");
write("my_clib/myclib.cc", "void f() {}");
@@ -265,7 +265,7 @@
}
@Test
- public void testSymlinkPrefixAtCwdEqualWorkspace_WithFlagOff() throws Exception {
+ public void testSymlinkPrefixAtCwdEqualWorkspace_withFlagOff() throws Exception {
write("my_clib/BUILD", "cc_library(name='my_clib', srcs=['myclib.cc'])\n");
write("my_clib/myclib.cc", "void f() {}");
@@ -283,7 +283,7 @@
}
@Test
- public void testSeeTempAtCwdUnderWorkspace_WithFlagOn() throws Exception {
+ public void testSeeTempAtCwdUnderWorkspace_withFlagOn() throws Exception {
write("bad_clib/BUILD", "cc_library(name='bad_clib', srcs=['badlib.cc'])\n");
// trigger a warning to make the build fail:
// "control reaches end of non-void function [-Werror,-Wreturn-type]"
@@ -307,7 +307,7 @@
}
@Test
- public void testSeeTempAtCwdUnderWorkspace_WithFlagOff() throws Exception {
+ public void testSeeTempAtCwdUnderWorkspace_withFlagOff() throws Exception {
write("bad_clib/BUILD", "cc_library(name='bad_clib', srcs=['badlib.cc'])\n");
// trigger a warning to make the build fail:
// "control reaches end of non-void function [-Werror,-Wreturn-type]"
@@ -327,7 +327,7 @@
}
@Test
- public void testSeeTempAtCwdEqualWorkspace_WithFlagOn() throws Exception {
+ public void testSeeTempAtCwdEqualWorkspace_withFlagOn() throws Exception {
write("bad_clib/BUILD", "cc_library(name='bad_clib', srcs=['badlib.cc'])\n");
// trigger a warning to make the build fail:
// "control reaches end of non-void function [-Werror,-Wreturn-type]"
@@ -343,7 +343,7 @@
}
@Test
- public void testSeeTempAtCwdEqualWorkspace_WithFlagOff() throws Exception {
+ public void testSeeTempAtCwdEqualWorkspace_withFlagOff() throws Exception {
write("bad_clib/BUILD", "cc_library(name='bad_clib', srcs=['badlib.cc'])\n");
// trigger a warning to make the build fail:
// "control reaches end of non-void function [-Werror,-Wreturn-type]"
diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/CustomRealFilesystemBuildIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/buildtool/CustomRealFilesystemBuildIntegrationTest.java
index d90d137..fdbe97d 100644
--- a/src/test/java/com/google/devtools/build/lib/buildtool/CustomRealFilesystemBuildIntegrationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/buildtool/CustomRealFilesystemBuildIntegrationTest.java
@@ -91,7 +91,7 @@
* Tests that IOExceptions encountered while handling non-mandatory inputs are properly handled.
*/
@Test
- public void testIOException_NonMandatoryInputs() throws Exception {
+ public void testIOException_nonMandatoryInputs() throws Exception {
Path fooBuildFile =
write("foo/BUILD", "cc_library(name = 'foo', srcs = ['foo.cc'], hdrs_check = 'loose')");
write("foo/foo.cc", "#include \"foo/foo.h\"");
@@ -150,7 +150,7 @@
* handled.
*/
@Test
- public void testIOException_NonMandatoryGeneratedInputs() throws Exception {
+ public void testIOException_nonMandatoryGeneratedInputs() throws Exception {
write(
"bar/BUILD",
"cc_library(",
diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/GenQueryIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/buildtool/GenQueryIntegrationTest.java
index 943e2df..8d030fb 100644
--- a/src/test/java/com/google/devtools/build/lib/buildtool/GenQueryIntegrationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/buildtool/GenQueryIntegrationTest.java
@@ -267,7 +267,7 @@
}
@Test
- public void testGraphOutput_Factored() throws Exception {
+ public void testGraphOutput_factored() throws Exception {
write(
"fruits/BUILD",
"sh_library(name='melon', deps=[':papaya', ':coconut', ':mango'])",
@@ -286,7 +286,7 @@
}
@Test
- public void testGraphOutput_Unfactored() throws Exception {
+ public void testGraphOutput_unfactored() throws Exception {
write(
"fruits/BUILD",
"sh_library(name='melon', deps=[':papaya', ':coconut', ':mango'])",
@@ -431,17 +431,17 @@
}
@Test
- public void testNodepDeps_DefaultIsFalse() throws Exception {
+ public void testNodepDeps_defaultIsFalse() throws Exception {
runNodepDepsTest(/*optsStringValue=*/ "[]", /*expectVisibilityDep=*/ false);
}
@Test
- public void testNodepDeps_False() throws Exception {
+ public void testNodepDeps_false() throws Exception {
runNodepDepsTest(/*optsStringValue=*/ "['--nodep_deps=false']", /*expectVisibilityDep=*/ false);
}
@Test
- public void testNodepDeps_True() throws Exception {
+ public void testNodepDeps_true() throws Exception {
runNodepDepsTest(/*optsStringValue=*/ "['--nodep_deps=true']", /*expectVisibilityDep=*/ true);
}
diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/LabelCrossesPackageBoundaryTest.java b/src/test/java/com/google/devtools/build/lib/buildtool/LabelCrossesPackageBoundaryTest.java
index a7c7f47..2d4a809 100644
--- a/src/test/java/com/google/devtools/build/lib/buildtool/LabelCrossesPackageBoundaryTest.java
+++ b/src/test/java/com/google/devtools/build/lib/buildtool/LabelCrossesPackageBoundaryTest.java
@@ -31,7 +31,7 @@
public class LabelCrossesPackageBoundaryTest extends BuildIntegrationTestCase {
@Test
- public void testLabelCrossesPackageBoundary_Target() throws Exception {
+ public void testLabelCrossesPackageBoundary_target() throws Exception {
write("x/BUILD",
"genrule(name = 'x',",
" srcs = ['//x:y/z'],",
diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/OutputArtifactConflictTest.java b/src/test/java/com/google/devtools/build/lib/buildtool/OutputArtifactConflictTest.java
index dbc23be..e507d8d 100644
--- a/src/test/java/com/google/devtools/build/lib/buildtool/OutputArtifactConflictTest.java
+++ b/src/test/java/com/google/devtools/build/lib/buildtool/OutputArtifactConflictTest.java
@@ -72,22 +72,22 @@
}
@Test
- public void testArtifactPrefix_KeepGoing() throws Exception {
+ public void testArtifactPrefix_keepGoing() throws Exception {
runArtifactPrefix(true, false);
}
@Test
- public void testArtifactPrefix_NoKeepGoing() throws Exception {
+ public void testArtifactPrefix_noKeepGoing() throws Exception {
runArtifactPrefix(false, false);
}
@Test
- public void testArtifactPrefix_KeepGoing_ModifyBuildFile() throws Exception {
+ public void testArtifactPrefix_keepGoing_modifyBuildFile() throws Exception {
runArtifactPrefix(true, true);
}
@Test
- public void testArtifactPrefix_NoKeepGoing_ModifyBuildFile() throws Exception {
+ public void testArtifactPrefix_noKeepGoing_modifyBuildFile() throws Exception {
runArtifactPrefix(false, true);
}
diff --git a/src/test/java/com/google/devtools/build/lib/buildtool/SymlinkForestTest.java b/src/test/java/com/google/devtools/build/lib/buildtool/SymlinkForestTest.java
index f17baf3..40cb14a 100644
--- a/src/test/java/com/google/devtools/build/lib/buildtool/SymlinkForestTest.java
+++ b/src/test/java/com/google/devtools/build/lib/buildtool/SymlinkForestTest.java
@@ -434,7 +434,7 @@
}
@Test
- public void test_withSubdirRepoLayout_TestExternalDirInMainRepoIsIgnored1() throws Exception {
+ public void test_withSubdirRepoLayout_testExternalDirInMainRepoIsIgnored1() throws Exception {
// Test external/ is ignored even when packages like "//external/foo" is specified.
Root outputBase = Root.fromPath(fileSystem.getPath("/ob"));
Root mainRepo = Root.fromPath(fileSystem.getPath("/my_repo"));
@@ -476,7 +476,7 @@
}
@Test
- public void test_withSubDirRepoLayout_TestExternalDirInMainRepoIsIgnored2() throws Exception {
+ public void test_withSubDirRepoLayout_testExternalDirInMainRepoIsIgnored2() throws Exception {
// Test external/ is ignored when root package "//:" is specified.
Root outputBase = Root.fromPath(fileSystem.getPath("/ob"));
Root mainRepo = Root.fromPath(fileSystem.getPath("/my_repo"));
@@ -520,7 +520,7 @@
}
@Test
- public void test_withSiblingRepoLayout_TestExternalDirInMainRepoExists() throws Exception {
+ public void test_withSiblingRepoLayout_testExternalDirInMainRepoExists() throws Exception {
// Test external/ is ignored even when packages like "//external/foo" is specified.
Root outputBase = Root.fromPath(fileSystem.getPath("/ob"));
Root mainRepo = Root.fromPath(fileSystem.getPath("/my_repo"));
@@ -585,7 +585,7 @@
}
@Test
- public void test_withSiblingRepoLayoutAndRootPackageInRoots_TestExternalDirInMainRepoExists()
+ public void test_withSiblingRepoLayoutAndRootPackageInRoots_testExternalDirInMainRepoExists()
throws Exception {
// Test external/ is ignored when root package "//:" is specified.
Root outputBase = Root.fromPath(fileSystem.getPath("/ob"));
diff --git a/src/test/java/com/google/devtools/build/lib/concurrent/MultisetSemaphoreTest.java b/src/test/java/com/google/devtools/build/lib/concurrent/MultisetSemaphoreTest.java
index c30c5ce..6e8cb8d 100644
--- a/src/test/java/com/google/devtools/build/lib/concurrent/MultisetSemaphoreTest.java
+++ b/src/test/java/com/google/devtools/build/lib/concurrent/MultisetSemaphoreTest.java
@@ -43,7 +43,7 @@
public class MultisetSemaphoreTest {
@Test
- public void testSimple_Serial() throws Exception {
+ public void testSimple_serial() throws Exception {
// When we have a MultisetSemaphore
MultisetSemaphore<String> multisetSemaphore = MultisetSemaphore.newBuilder()
// with 3 max num unique values,
@@ -91,7 +91,7 @@
}
@Test
- public void testSimple_Concurrent() throws Exception {
+ public void testSimple_concurrent() throws Exception {
// When we have N and M, with M > N and M|N.
final int n = 10;
int m = n * 2;
@@ -232,7 +232,7 @@
}
@Test
- public void testConcurrentRace_AllPermuations() throws Exception {
+ public void testConcurrentRace_allPermuations() throws Exception {
// When we have N values
int n = 6;
ArrayList<String> vals = new ArrayList<>();
@@ -285,7 +285,7 @@
}
@Test
- public void testConcurrentRace_AllSameSizedCombinations() throws Exception {
+ public void testConcurrentRace_allSameSizedCombinations() throws Exception {
// When we have n values
int n = 10;
ImmutableSet.Builder<String> valsBuilder = ImmutableSet.builder();
diff --git a/src/test/java/com/google/devtools/build/lib/exec/AbstractSpawnStrategyTest.java b/src/test/java/com/google/devtools/build/lib/exec/AbstractSpawnStrategyTest.java
index c469fb8..43c2ed5 100644
--- a/src/test/java/com/google/devtools/build/lib/exec/AbstractSpawnStrategyTest.java
+++ b/src/test/java/com/google/devtools/build/lib/exec/AbstractSpawnStrategyTest.java
@@ -295,7 +295,7 @@
}
@Test
- public void testLogSpawn_DefaultPlatform_getsLogged() throws Exception {
+ public void testLogSpawn_defaultPlatform_getsLogged() throws Exception {
RemoteOptions remoteOptions = Options.getDefaults(RemoteOptions.class);
remoteOptions.remoteDefaultPlatformProperties =
String.join(
@@ -325,7 +325,7 @@
}
@Test
- public void testLogSpawn_SpecifiedPlatform_overridesDefault() throws Exception {
+ public void testLogSpawn_specifiedPlatform_overridesDefault() throws Exception {
RemoteOptions remoteOptions = Options.getDefaults(RemoteOptions.class);
remoteOptions.remoteDefaultPlatformProperties =
String.join(
diff --git a/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java b/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java
index fcb5615..cf5e6ac 100644
--- a/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java
+++ b/src/test/java/com/google/devtools/build/lib/outputfilter/OutputFilterTest.java
@@ -277,7 +277,7 @@
}
@Test
- public void testPackagesAOF_JavaTestsA() throws Exception {
+ public void testPackagesAOF_javaTestsA() throws Exception {
enableDeprecationWarnings();
addOptions("--auto_output_filter=packages");
CommandEnvironment env = runtimeWrapper.newCommand();
@@ -290,7 +290,7 @@
}
@Test
- public void testPackagesAOF_JavaTestsAB() throws Exception {
+ public void testPackagesAOF_javaTestsAB() throws Exception {
enableDeprecationWarnings();
addOptions("--auto_output_filter=packages");
CommandEnvironment env = runtimeWrapper.newCommand();
@@ -303,7 +303,7 @@
}
@Test
- public void testPackagesAOF_JavaTestsD() throws Exception {
+ public void testPackagesAOF_javaTestsD() throws Exception {
addOptions("--auto_output_filter=packages");
CommandEnvironment env = runtimeWrapper.newCommand();
env.getReporter().addHandler(stderr);
diff --git a/src/test/java/com/google/devtools/build/lib/packages/AttributeValueSourceTest.java b/src/test/java/com/google/devtools/build/lib/packages/AttributeValueSourceTest.java
index 2ddfb0e..b5ab6e2 100644
--- a/src/test/java/com/google/devtools/build/lib/packages/AttributeValueSourceTest.java
+++ b/src/test/java/com/google/devtools/build/lib/packages/AttributeValueSourceTest.java
@@ -41,14 +41,14 @@
}
@Test
- public void testValidateStarlarkName_EmptyName() throws Exception {
+ public void testValidateStarlarkName_emptyName() throws Exception {
for (AttributeValueSource source : AttributeValueSource.values()) {
assertNameIsNotValid(source, "", "Attribute name must not be empty.");
}
}
@Test
- public void testValidateStarlarkName_MissingPrefix() throws Exception {
+ public void testValidateStarlarkName_missingPrefix() throws Exception {
String msg =
"When an attribute value is a function, the attribute must be private "
+ "(i.e. start with '_'). Found 'my_name'";
@@ -77,7 +77,7 @@
}
@Test
- public void testConvertToNativeName_InvalidName() throws Exception {
+ public void testConvertToNativeName_invalidName() throws Exception {
assertTranslationFails(AttributeValueSource.COMPUTED_DEFAULT, "name");
assertTranslationFails(AttributeValueSource.LATE_BOUND, "name");
}
diff --git a/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java b/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java
index 272c5c3..ad89e6f 100644
--- a/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java
+++ b/src/test/java/com/google/devtools/build/lib/packages/ConfigurationFragmentPolicyTest.java
@@ -63,7 +63,7 @@
}
@Test
- public void testRequiresConfigurationFragments_AddsToRequiredSet() throws Exception {
+ public void testRequiresConfigurationFragments_addsToRequiredSet() throws Exception {
// Although these aren't configuration fragments, there are no requirements as to what the class
// has to be, so...
ConfigurationFragmentPolicy policy =
@@ -96,7 +96,7 @@
};
@Test
- public void testRequiresConfigurationFragments_RequiredAndLegalForSpecifiedConfiguration()
+ public void testRequiresConfigurationFragments_requiredAndLegalForSpecifiedConfiguration()
throws Exception {
ConfigurationFragmentPolicy policy =
new ConfigurationFragmentPolicy.Builder()
@@ -131,7 +131,7 @@
}
@Test
- public void testRequiresConfigurationFragments_MapSetsLegalityByStarlarkModuleName_NoRequires()
+ public void testRequiresConfigurationFragments_mapSetsLegalityByStarlarkModuleName_noRequires()
throws Exception {
ConfigurationFragmentPolicy policy =
new ConfigurationFragmentPolicy.Builder()
@@ -169,7 +169,7 @@
}
@Test
- public void testIncludeConfigurationFragmentsFrom_MergesWithExistingFragmentSet()
+ public void testIncludeConfigurationFragmentsFrom_mergesWithExistingFragmentSet()
throws Exception {
ConfigurationFragmentPolicy basePolicy =
new ConfigurationFragmentPolicy.Builder()
diff --git a/src/test/java/com/google/devtools/build/lib/packages/GlobCacheTest.java b/src/test/java/com/google/devtools/build/lib/packages/GlobCacheTest.java
index e2263af..944ae45 100644
--- a/src/test/java/com/google/devtools/build/lib/packages/GlobCacheTest.java
+++ b/src/test/java/com/google/devtools/build/lib/packages/GlobCacheTest.java
@@ -205,13 +205,13 @@
}
@Test
- public void testSingleFileExclude_Star() throws Exception {
+ public void testSingleFileExclude_star() throws Exception {
assertThat(cache.globUnsorted(list("*"), list("first.txt"), false, true))
.containsExactly("BUILD", "bar", "first.js", "foo", "second.js", "second.txt");
}
@Test
- public void testSingleFileExclude_StarStar() throws Exception {
+ public void testSingleFileExclude_starStar() throws Exception {
assertThat(cache.globUnsorted(list("**"), list("first.txt"), false, true))
.containsExactly(
"BUILD",
@@ -227,22 +227,22 @@
}
@Test
- public void testExcludeAll_Star() throws Exception {
+ public void testExcludeAll_star() throws Exception {
assertThat(cache.globUnsorted(list("*"), list("*"), false, true)).isEmpty();
}
@Test
- public void testExcludeAll_Star_NoMatchesAnyway() throws Exception {
+ public void testExcludeAll_star_noMatchesAnyway() throws Exception {
assertThat(cache.globUnsorted(list("nope"), list("*"), false, true)).isEmpty();
}
@Test
- public void testExcludeAll_StarStar() throws Exception {
+ public void testExcludeAll_starStar() throws Exception {
assertThat(cache.globUnsorted(list("**"), list("**"), false, true)).isEmpty();
}
@Test
- public void testExcludeAll_Manual() throws Exception {
+ public void testExcludeAll_manual() throws Exception {
assertThat(cache.globUnsorted(list("**"), list("*", "*/*", "*/*/*"), false, true)).isEmpty();
}
diff --git a/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java b/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java
index b771861..ffb6cb0 100644
--- a/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/pkgcache/LoadingPhaseRunnerTest.java
@@ -726,7 +726,7 @@
/** Regression test: handle symlink cycles gracefully. */
@Test
- public void testCycleReporting_SymlinkCycleDuringTargetParsing() throws Exception {
+ public void testCycleReporting_symlinkCycleDuringTargetParsing() throws Exception {
tester.addFile("hello/BUILD", "cc_library(name = 'a', srcs = glob(['*.cc']))");
Path buildFilePath = tester.getWorkspace().getRelative("hello/BUILD");
Path dirPath = buildFilePath.getParentDirectory();
@@ -774,7 +774,7 @@
}
@Test
- public void testTopLevelTargetErrorsPrintedExactlyOnce_NoKeepGoing() throws Exception {
+ public void testTopLevelTargetErrorsPrintedExactlyOnce_noKeepGoing() throws Exception {
tester.addFile("bad/BUILD", "sh_binary(name = 'bad', srcs = ['bad.sh'])", "fail('some error')");
assertThrows(TargetParsingException.class, () -> tester.load("//bad"));
tester.assertContainsEventWithFrequency("some error", 1);
@@ -783,7 +783,7 @@
}
@Test
- public void testTopLevelTargetErrorsPrintedExactlyOnce_KeepGoing() throws Exception {
+ public void testTopLevelTargetErrorsPrintedExactlyOnce_keepGoing() throws Exception {
tester.addFile("bad/BUILD", "sh_binary(name = 'bad', srcs = ['bad.sh'])", "fail('some error')");
TargetPatternPhaseValue result = tester.loadKeepGoing("//bad");
assertThat(result.hasError()).isTrue();
@@ -1037,43 +1037,43 @@
}
@Test
- public void testPackageLoadingError_KeepGoing_ExplicitTarget() throws Exception {
+ public void testPackageLoadingError_keepGoing_explicitTarget() throws Exception {
runTestPackageLoadingError(/*keepGoing=*/ true, "//bad:BUILD");
}
@Test
- public void testPackageLoadingError_NoKeepGoing_ExplicitTarget() throws Exception {
+ public void testPackageLoadingError_noKeepGoing_explicitTarget() throws Exception {
runTestPackageLoadingError(/*keepGoing=*/ false, "//bad:BUILD");
}
@Test
- public void testPackageLoadingError_KeepGoing_TargetsInPackage() throws Exception {
+ public void testPackageLoadingError_keepGoing_targetsInPackage() throws Exception {
runTestPackageLoadingError(/*keepGoing=*/ true, "//bad:all");
}
@Test
- public void testPackageLoadingError_NoKeepGoing_TargetsInPackage() throws Exception {
+ public void testPackageLoadingError_noKeepGoing_targetsInPackage() throws Exception {
runTestPackageLoadingError(/*keepGoing=*/ false, "//bad:all");
}
@Test
- public void testPackageLoadingError_KeepGoing_TargetsBeneathDirectory() throws Exception {
+ public void testPackageLoadingError_keepGoing_targetsBeneathDirectory() throws Exception {
runTestPackageLoadingError(/*keepGoing=*/ true, "//bad/...");
}
@Test
- public void testPackageLoadingError_NoKeepGoing_TargetsBeneathDirectory() throws Exception {
+ public void testPackageLoadingError_noKeepGoing_targetsBeneathDirectory() throws Exception {
runTestPackageLoadingError(/*keepGoing=*/ false, "//bad/...");
}
@Test
- public void testPackageLoadingError_KeepGoing_SomeGoodTargetsBeneathDirectory() throws Exception {
+ public void testPackageLoadingError_keepGoing_someGoodTargetsBeneathDirectory() throws Exception {
tester.addFile("good/BUILD", "sh_library(name = 't')\n");
runTestPackageLoadingError(/*keepGoing=*/ true, "//...");
}
@Test
- public void testPackageLoadingError_NoKeepGoing_SomeGoodTargetsBeneathDirectory()
+ public void testPackageLoadingError_noKeepGoing_someGoodTargetsBeneathDirectory()
throws Exception {
tester.addFile("good/BUILD", "sh_library(name = 't')\n");
runTestPackageLoadingError(/*keepGoing=*/ false, "//...");
@@ -1098,46 +1098,46 @@
}
@Test
- public void testPackageFileInconsistencyError_KeepGoing_ExplicitTarget() throws Exception {
+ public void testPackageFileInconsistencyError_keepGoing_explicitTarget() throws Exception {
runTestPackageFileInconsistencyError(true, "//bad:BUILD");
}
@Test
- public void testPackageFileInconsistencyError_NoKeepGoing_ExplicitTarget() throws Exception {
+ public void testPackageFileInconsistencyError_noKeepGoing_explicitTarget() throws Exception {
runTestPackageFileInconsistencyError(false, "//bad:BUILD");
}
@Test
- public void testPackageFileInconsistencyError_KeepGoing_TargetsInPackage() throws Exception {
+ public void testPackageFileInconsistencyError_keepGoing_targetsInPackage() throws Exception {
runTestPackageFileInconsistencyError(true, "//bad:all");
}
@Test
- public void testPackageFileInconsistencyError_NoKeepGoing_TargetsInPackage() throws Exception {
+ public void testPackageFileInconsistencyError_noKeepGoing_targetsInPackage() throws Exception {
runTestPackageFileInconsistencyError(false, "//bad:all");
}
@Test
- public void testPackageFileInconsistencyError_KeepGoing_argetsBeneathDirectory()
+ public void testPackageFileInconsistencyError_keepGoing_argetsBeneathDirectory()
throws Exception {
runTestPackageFileInconsistencyError(true, "//bad/...");
}
@Test
- public void testPackageFileInconsistencyError_NoKeepGoing_TargetsBeneathDirectory()
+ public void testPackageFileInconsistencyError_noKeepGoing_targetsBeneathDirectory()
throws Exception {
runTestPackageFileInconsistencyError(false, "//bad/...");
}
@Test
- public void testPackageFileInconsistencyError_KeepGoing_SomeGoodTargetsBeneathDirectory()
+ public void testPackageFileInconsistencyError_keepGoing_someGoodTargetsBeneathDirectory()
throws Exception {
tester.addFile("good/BUILD", "sh_library(name = 't')\n");
runTestPackageFileInconsistencyError(true, "//...");
}
@Test
- public void testPackageFileInconsistencyError_NoKeepGoing_SomeGoodTargetsBeneathDirectory()
+ public void testPackageFileInconsistencyError_noKeepGoing_someGoodTargetsBeneathDirectory()
throws Exception {
tester.addFile("good/BUILD", "sh_library(name = 't')\n");
runTestPackageFileInconsistencyError(false, "//...");
@@ -1165,44 +1165,44 @@
}
@Test
- public void testExtensionLoadingError_KeepGoing_ExplicitTarget() throws Exception {
+ public void testExtensionLoadingError_keepGoing_explicitTarget() throws Exception {
runTestExtensionLoadingError(/*keepGoing=*/ true, "//bad:BUILD");
}
@Test
- public void testExtensionLoadingError_NoKeepGoing_ExplicitTarget() throws Exception {
+ public void testExtensionLoadingError_noKeepGoing_explicitTarget() throws Exception {
runTestExtensionLoadingError(/*keepGoing=*/ false, "//bad:BUILD");
}
@Test
- public void testExtensionLoadingError_KeepGoing_TargetsInPackage() throws Exception {
+ public void testExtensionLoadingError_keepGoing_targetsInPackage() throws Exception {
runTestExtensionLoadingError(/*keepGoing=*/ true, "//bad:all");
}
@Test
- public void testExtensionLoadingError_NoKeepGoing_TargetsInPackage() throws Exception {
+ public void testExtensionLoadingError_noKeepGoing_targetsInPackage() throws Exception {
runTestExtensionLoadingError(/*keepGoing=*/ false, "//bad:all");
}
@Test
- public void testExtensionLoadingError_KeepGoing_TargetsBeneathDirectory() throws Exception {
+ public void testExtensionLoadingError_keepGoing_targetsBeneathDirectory() throws Exception {
runTestExtensionLoadingError(/*keepGoing=*/ true, "//bad/...");
}
@Test
- public void testExtensionLoadingError_NoKeepGoing_TargetsBeneathDirectory() throws Exception {
+ public void testExtensionLoadingError_noKeepGoing_targetsBeneathDirectory() throws Exception {
runTestExtensionLoadingError(/*keepGoing=*/ false, "//bad/...");
}
@Test
- public void testExtensionLoadingError_KeepGoing_SomeGoodTargetsBeneathDirectory()
+ public void testExtensionLoadingError_keepGoing_someGoodTargetsBeneathDirectory()
throws Exception {
tester.addFile("good/BUILD", "sh_library(name = 't')\n");
runTestExtensionLoadingError(/*keepGoing=*/ true, "//...");
}
@Test
- public void testExtensionLoadingError_NoKeepGoing_SomeGoodTargetsBeneathDirectory()
+ public void testExtensionLoadingError_noKeepGoing_someGoodTargetsBeneathDirectory()
throws Exception {
tester.addFile("good/BUILD", "sh_library(name = 't')\n");
runTestExtensionLoadingError(/*keepGoing=*/ false, "//...");
diff --git a/src/test/java/com/google/devtools/build/lib/pkgcache/TargetPatternEvaluatorTest.java b/src/test/java/com/google/devtools/build/lib/pkgcache/TargetPatternEvaluatorTest.java
index ec33040..29d0874 100644
--- a/src/test/java/com/google/devtools/build/lib/pkgcache/TargetPatternEvaluatorTest.java
+++ b/src/test/java/com/google/devtools/build/lib/pkgcache/TargetPatternEvaluatorTest.java
@@ -264,21 +264,21 @@
}
@Test
- public void testSequenceOfTargetPatterns_Union() throws Exception {
+ public void testSequenceOfTargetPatterns_union() throws Exception {
// No prefix negation operator => union. Order is not significant.
assertThat(parseList("foo/...", "foo/bar/...")).containsExactlyElementsIn(rulesBeneathFoo);
assertThat(parseList("foo/bar/...", "foo/...")).containsExactlyElementsIn(rulesBeneathFoo);
}
@Test
- public void testSequenceOfTargetPatterns_SetDifference() throws Exception {
+ public void testSequenceOfTargetPatterns_setDifference() throws Exception {
// Prefix negation operator => set difference. Order is significant.
assertThat(parseList("foo/...", "-foo/bar/...")).containsExactlyElementsIn(rulesInFoo);
assertThat(parseList("-foo/bar/...", "foo/...")).containsExactlyElementsIn(rulesBeneathFoo);
}
@Test
- public void testSequenceOfTargetPatterns_SetDifferenceRelative() throws Exception {
+ public void testSequenceOfTargetPatterns_setDifferenceRelative() throws Exception {
// Prefix negation operator => set difference. Order is significant.
assertThat(parseListRelative("...", "-bar/...")).containsExactlyElementsIn(rulesInFoo);
assertThat(parseListRelative("-bar/...", "...")).containsExactlyElementsIn(rulesBeneathFoo);
@@ -545,65 +545,65 @@
}
@Test
- public void testTopLevelPackage_Relative_BuildFile() throws Exception {
+ public void testTopLevelPackage_relative_buildFile() throws Exception {
Set<Label> result = parseList("BUILD");
assertThat(result).containsExactly(Label.parseAbsolute("//:BUILD", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Relative_DeclaredTarget() throws Exception {
+ public void testTopLevelPackage_relative_declaredTarget() throws Exception {
Set<Label> result = parseList("fg");
assertThat(result).containsExactly(Label.parseAbsolute("//:fg", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Relative_All() throws Exception {
+ public void testTopLevelPackage_relative_all() throws Exception {
expectError("no such target '//:all'", "all");
}
@Test
- public void testTopLevelPackage_Relative_ColonAll() throws Exception {
+ public void testTopLevelPackage_relative_colonAll() throws Exception {
Set<Label> result = parseList(":all");
assertThat(result).containsExactly(Label.parseAbsolute("//:fg", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Relative_InputFile() throws Exception {
+ public void testTopLevelPackage_relative_inputFile() throws Exception {
Set<Label> result = parseList("foo.cc");
assertThat(result).containsExactly(Label.parseAbsolute("//:foo.cc", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Relative_InputFile_NoSuchInputFile() throws Exception {
+ public void testTopLevelPackage_relative_inputFile_noSuchInputFile() throws Exception {
expectError("no such target '//:nope.cc'", "nope.cc");
}
@Test
- public void testTopLevelPackage_Absolute_BuildFile() throws Exception {
+ public void testTopLevelPackage_absolute_buildFile() throws Exception {
Set<Label> result = parseList("//:BUILD");
assertThat(result).containsExactly(Label.parseAbsolute("//:BUILD", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Absolute_DeclaredTarget() throws Exception {
+ public void testTopLevelPackage_absolute_declaredTarget() throws Exception {
Set<Label> result = parseList("//:fg");
assertThat(result).containsExactly(Label.parseAbsolute("//:fg", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Absolute_All() throws Exception {
+ public void testTopLevelPackage_absolute_all() throws Exception {
Set<Label> result = parseList("//:all");
assertThat(result).containsExactly(Label.parseAbsolute("//:fg", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Absolute_InputFile() throws Exception {
+ public void testTopLevelPackage_absolute_inputFile() throws Exception {
Set<Label> result = parseList("//:foo.cc");
assertThat(result).containsExactly(Label.parseAbsolute("//:foo.cc", ImmutableMap.of()));
}
@Test
- public void testTopLevelPackage_Absolute_InputFile_NoSuchInputFile() throws Exception {
+ public void testTopLevelPackage_absolute_inputFile_noSuchInputFile() throws Exception {
expectError("no such target '//:nope.cc'", "//:nope.cc");
}
}
diff --git a/src/test/java/com/google/devtools/build/lib/query2/aquery/ActionGraphProtoOutputFormatterCallbackTest.java b/src/test/java/com/google/devtools/build/lib/query2/aquery/ActionGraphProtoOutputFormatterCallbackTest.java
index 71cc5f8..ab9833d 100644
--- a/src/test/java/com/google/devtools/build/lib/query2/aquery/ActionGraphProtoOutputFormatterCallbackTest.java
+++ b/src/test/java/com/google/devtools/build/lib/query2/aquery/ActionGraphProtoOutputFormatterCallbackTest.java
@@ -522,7 +522,7 @@
}
@Test
- public void testIncludeAspects_AspectOnAspect() throws Exception {
+ public void testIncludeAspects_aspectOnAspect() throws Exception {
options.useAspects = true;
writeFile(
"test/rule.bzl",
@@ -638,7 +638,7 @@
}
@Test
- public void testIncludeAspects_SingleAspect() throws Exception {
+ public void testIncludeAspects_singleAspect() throws Exception {
options.useAspects = true;
writeFile(
"test/rule.bzl",
@@ -834,7 +834,7 @@
}
@Test
- public void testIncludeAspects_flagDisabled_NoAspect() throws Exception {
+ public void testIncludeAspects_flagDisabled_noAspect() throws Exception {
// The flag --include_aspects is set to false by default.
writeFile(
"test/rule.bzl",
diff --git a/src/test/java/com/google/devtools/build/lib/query2/cquery/ConfiguredTargetQuerySemanticsTest.java b/src/test/java/com/google/devtools/build/lib/query2/cquery/ConfiguredTargetQuerySemanticsTest.java
index 728d5cf..d245478 100644
--- a/src/test/java/com/google/devtools/build/lib/query2/cquery/ConfiguredTargetQuerySemanticsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/query2/cquery/ConfiguredTargetQuerySemanticsTest.java
@@ -483,7 +483,7 @@
}
@Test
- public void testSomePath_DepInCustomConfiguration() throws Exception {
+ public void testSomePath_depInCustomConfiguration() throws Exception {
createConfigTransitioningRuleClass();
writeFile(
"test/BUILD",
diff --git a/src/test/java/com/google/devtools/build/lib/query2/testutil/AbstractQueryTest.java b/src/test/java/com/google/devtools/build/lib/query2/testutil/AbstractQueryTest.java
index 978ea46..14691da 100644
--- a/src/test/java/com/google/devtools/build/lib/query2/testutil/AbstractQueryTest.java
+++ b/src/test/java/com/google/devtools/build/lib/query2/testutil/AbstractQueryTest.java
@@ -263,7 +263,7 @@
}
@Test
- public void testAlgebraicSetOperations_ManyOperands() throws Exception {
+ public void testAlgebraicSetOperations_manyOperands() throws Exception {
writeBuildFiles1();
assertThat(evalToString("//a:BUILD + //a:x + //a:y + //a:z + //a/b:BUILD + //a/b:p + //a/b:q"))
.isEqualTo(A_AB_FILES);
@@ -933,12 +933,12 @@
}
@Test
- public void testNodepDeps_DefaultIsTrue() throws Exception {
+ public void testNodepDeps_defaultIsTrue() throws Exception {
runNodepDepsTest(/*expectVisibilityDep=*/ true);
}
@Test
- public void testNodepDeps_False() throws Exception {
+ public void testNodepDeps_false() throws Exception {
runNodepDepsTest(/*expectVisibilityDep=*/ false, Setting.NO_NODEP_DEPS);
}
@@ -1138,7 +1138,7 @@
}
@Test
- public void testDefaultVisibilityReturnedInDeps_NonEmptyDependencyFilter() throws Exception {
+ public void testDefaultVisibilityReturnedInDeps_nonEmptyDependencyFilter() throws Exception {
writeFile(
"kiwi/BUILD", "package(default_visibility=['//mango:mango'])", "sh_library(name='kiwi')");
writeFile("mango/BUILD", "package_group(name='mango', packages=[])");
@@ -1772,7 +1772,7 @@
}
@Test
- public void testSiblings_Simple() throws Exception {
+ public void testSiblings_simple() throws Exception {
writeFile(
"foo/BUILD",
"sh_library(name = 'a')",
@@ -1784,7 +1784,7 @@
}
@Test
- public void testSiblings_DuplicatePackages() throws Exception {
+ public void testSiblings_duplicatePackages() throws Exception {
writeFile(
"foo/BUILD",
"sh_library(name = 'a')",
@@ -1796,7 +1796,7 @@
}
@Test
- public void testSiblings_SamePackageRdeps() throws Exception {
+ public void testSiblings_samePackageRdeps() throws Exception {
writeFile(
"foo/BUILD",
"sh_library(name = 'a', deps = [':b'])",
@@ -1817,7 +1817,7 @@
}
@Test
- public void testSiblings_MatchesTargetNamedAll() throws Exception {
+ public void testSiblings_matchesTargetNamedAll() throws Exception {
writeFile(
"foo/BUILD",
// NOTE: target named 'all' collides with, takes precedence over the ':all' wildcard
@@ -1836,7 +1836,7 @@
// thing blaze can do with the unfortunate implementation details of 'buildfiles' and 'loadfiles'
// (see FakeLoadTarget and other tests dealing with these functions).
@Test
- public void testSiblings_WithBuildfiles() throws Exception {
+ public void testSiblings_withBuildfiles() throws Exception {
writeFile("foo/BUILD", "load('//bar:bar.bzl', 'x')", "sh_library(name = 'foo')");
writeFile("bar/BUILD", "sh_library(name = 'bar')");
writeFile("bar/bar.bzl", "x = 42");
@@ -1925,7 +1925,7 @@
// Regression test for default visibility of output file targets being traversed even with
// --noimplicit_deps is set.
@Test
- public void testDefaultVisibilityOfOutputTarget_NoImplicitDeps() throws Exception {
+ public void testDefaultVisibilityOfOutputTarget_noImplicitDeps() throws Exception {
writeFile(
"foo/BUILD",
"package(default_visibility = [':pg'])",
diff --git a/src/test/java/com/google/devtools/build/lib/query2/testutil/PostAnalysisQueryTest.java b/src/test/java/com/google/devtools/build/lib/query2/testutil/PostAnalysisQueryTest.java
index fad2b66..3aec7e5 100644
--- a/src/test/java/com/google/devtools/build/lib/query2/testutil/PostAnalysisQueryTest.java
+++ b/src/test/java/com/google/devtools/build/lib/query2/testutil/PostAnalysisQueryTest.java
@@ -623,19 +623,19 @@
// siblings() operator.
@Override
- public void testSiblings_DuplicatePackages() {}
+ public void testSiblings_duplicatePackages() {}
@Override
- public void testSiblings_SamePackageRdeps() {}
+ public void testSiblings_samePackageRdeps() {}
@Override
- public void testSiblings_MatchesTargetNamedAll() {}
+ public void testSiblings_matchesTargetNamedAll() {}
@Override
- public void testSiblings_Simple() {}
+ public void testSiblings_simple() {}
@Override
- public void testSiblings_WithBuildfiles() {}
+ public void testSiblings_withBuildfiles() {}
// same_pkg_direct_rdeps() operator.
@@ -679,10 +679,10 @@
// We don't support --nodep_deps=false.
@Override
@Test
- public void testNodepDeps_False() throws Exception {}
+ public void testNodepDeps_false() throws Exception {}
// package_group instances have a null configuration and are filtered out by --host_deps=false.
@Override
@Test
- public void testDefaultVisibilityReturnedInDeps_NonEmptyDependencyFilter() throws Exception {}
+ public void testDefaultVisibilityReturnedInDeps_nonEmptyDependencyFilter() throws Exception {}
}
diff --git a/src/test/java/com/google/devtools/build/lib/remote/RemoteServerCapabilitiesTest.java b/src/test/java/com/google/devtools/build/lib/remote/RemoteServerCapabilitiesTest.java
index 989abae..62d567e5 100644
--- a/src/test/java/com/google/devtools/build/lib/remote/RemoteServerCapabilitiesTest.java
+++ b/src/test/java/com/google/devtools/build/lib/remote/RemoteServerCapabilitiesTest.java
@@ -204,7 +204,7 @@
}
@Test
- public void testCheckClientServerCompatibility_NoChecks() throws Exception {
+ public void testCheckClientServerCompatibility_noChecks() throws Exception {
RemoteServerCapabilities.ClientServerCompatibilityStatus st =
RemoteServerCapabilities.checkClientServerCompatibility(
ServerCapabilities.getDefaultInstance(),
@@ -214,7 +214,7 @@
}
@Test
- public void testCheckClientServerCompatibility_ApiVersionDeprecated() throws Exception {
+ public void testCheckClientServerCompatibility_apiVersionDeprecated() throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
.setDeprecatedApiVersion(ApiVersion.current.toSemVer())
@@ -238,7 +238,7 @@
}
@Test
- public void testCheckClientServerCompatibility_ApiVersionUnsupported() throws Exception {
+ public void testCheckClientServerCompatibility_apiVersionUnsupported() throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
.setLowApiVersion(new ApiVersion(100, 0, 0, "").toSemVer())
@@ -260,7 +260,7 @@
}
@Test
- public void testCheckClientServerCompatibility_RemoteCacheDoesNotSupportDigestFunction()
+ public void testCheckClientServerCompatibility_remoteCacheDoesNotSupportDigestFunction()
throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
@@ -283,7 +283,7 @@
}
@Test
- public void testCheckClientServerCompatibility_RemoteCacheDoesNotSupportUpdate()
+ public void testCheckClientServerCompatibility_remoteCacheDoesNotSupportUpdate()
throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
@@ -312,7 +312,7 @@
}
@Test
- public void testCheckClientServerCompatibility_RemoteExecutionIsDisabled() throws Exception {
+ public void testCheckClientServerCompatibility_remoteExecutionIsDisabled() throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
.setLowApiVersion(ApiVersion.current.toSemVer())
@@ -339,7 +339,7 @@
}
@Test
- public void testCheckClientServerCompatibility_RemoteExecutionDoesNotSupportDigestFunction()
+ public void testCheckClientServerCompatibility_remoteExecutionDoesNotSupportDigestFunction()
throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
@@ -367,7 +367,7 @@
}
@Test
- public void testCheckClientServerCompatibility_LocalFallbackNoRemoteCacheUpdate()
+ public void testCheckClientServerCompatibility_localFallbackNoRemoteCacheUpdate()
throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
@@ -410,7 +410,7 @@
}
@Test
- public void testCheckClientServerCompatibility_CachePriority() throws Exception {
+ public void testCheckClientServerCompatibility_cachePriority() throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
.setLowApiVersion(ApiVersion.current.toSemVer())
@@ -452,7 +452,7 @@
}
@Test
- public void testCheckClientServerCompatibility_ExecutionPriority() throws Exception {
+ public void testCheckClientServerCompatibility_executionPriority() throws Exception {
ServerCapabilities caps =
ServerCapabilities.newBuilder()
.setLowApiVersion(ApiVersion.current.toSemVer())
diff --git a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidBinaryTest.java b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidBinaryTest.java
index f398d60..9a164ce 100644
--- a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidBinaryTest.java
+++ b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidBinaryTest.java
@@ -601,7 +601,7 @@
// regression test for #3169095
@Test
- public void testXmbInSrcs_NotPermittedButDoesNotThrow() throws Exception {
+ public void testXmbInSrcs_notPermittedButDoesNotThrow() throws Exception {
reporter.removeHandler(failFastHandler);
scratchConfiguredTarget(
"java/xmb",
@@ -674,7 +674,7 @@
}
@Test
- public void testNativeLibrary_LinksLibrariesWhenCodeIsPresent() throws Exception {
+ public void testNativeLibrary_linksLibrariesWhenCodeIsPresent() throws Exception {
setupNativeLibrariesForLinking();
assertNativeLibraryLinked(
getConfiguredTarget("//java/android/app:auto"),
@@ -687,7 +687,7 @@
}
@Test
- public void testNativeLibrary_CopiesLibrariesDespiteExtraLayersOfIndirection() throws Exception {
+ public void testNativeLibrary_copiesLibrariesDespiteExtraLayersOfIndirection() throws Exception {
scratch.file(
"java/android/app/BUILD",
"cc_library(name = 'native_dep',",
@@ -709,7 +709,7 @@
}
@Test
- public void testNativeLibrary_CopiesLibrariesWrappedInCcLibraryWithSameName() throws Exception {
+ public void testNativeLibrary_copiesLibrariesWrappedInCcLibraryWithSameName() throws Exception {
scratch.file(
"java/android/app/BUILD",
"cc_library(name = 'native',",
@@ -724,7 +724,7 @@
}
@Test
- public void testNativeLibrary_LinksWhenPrebuiltArchiveIsSupplied() throws Exception {
+ public void testNativeLibrary_linksWhenPrebuiltArchiveIsSupplied() throws Exception {
scratch.file(
"java/android/app/BUILD",
"cc_library(name = 'native_dep',",
@@ -744,7 +744,7 @@
}
@Test
- public void testNativeLibrary_CopiesFullLibrariesInIfsoMode() throws Exception {
+ public void testNativeLibrary_copiesFullLibrariesInIfsoMode() throws Exception {
useConfiguration("--interface_shared_objects");
scratch.file(
"java/android/app/BUILD",
@@ -766,7 +766,7 @@
}
@Test
- public void testNativeLibrary_ProvidesLinkerScriptToLinkAction() throws Exception {
+ public void testNativeLibrary_providesLinkerScriptToLinkAction() throws Exception {
scratch.file(
"java/android/app/BUILD",
"cc_library(name = 'native',",
@@ -1316,7 +1316,7 @@
}
@Test
- public void testResourceShrinking_RequiresProguard() throws Exception {
+ public void testResourceShrinking_requiresProguard() throws Exception {
scratch.file(
"java/com/google/android/hello/BUILD",
"android_binary(name = 'hello',",
@@ -2582,7 +2582,7 @@
}
@Test
- public void testResourcesWithConfigurationQualifier_LocalResources() throws Exception {
+ public void testResourcesWithConfigurationQualifier_localResources() throws Exception {
scratch.file(
"java/android/resources/BUILD",
"android_binary(name = 'r',",
@@ -2603,7 +2603,7 @@
}
@Test
- public void testResourcesInOtherPackage_exported_LocalResources() throws Exception {
+ public void testResourcesInOtherPackage_exported_localResources() throws Exception {
scratch.file(
"java/android/resources/BUILD",
"android_binary(name = 'r',",
@@ -2619,7 +2619,7 @@
}
@Test
- public void testResourcesInOtherPackage_filegroup_LocalResources() throws Exception {
+ public void testResourcesInOtherPackage_filegroup_localResources() throws Exception {
scratch.file(
"java/android/resources/BUILD",
"android_binary(name = 'r',",
@@ -2639,7 +2639,7 @@
}
@Test
- public void testResourcesInOtherPackage_filegroupWithExternalSources_LocalResources()
+ public void testResourcesInOtherPackage_filegroupWithExternalSources_localResources()
throws Exception {
scratch.file(
"java/android/resources/BUILD",
@@ -2658,7 +2658,7 @@
}
@Test
- public void testMultipleDependentResourceDirectories_LocalResources() throws Exception {
+ public void testMultipleDependentResourceDirectories_localResources() throws Exception {
scratch.file(
"java/android/resources/d1/BUILD",
"android_library(name = 'd1',",
@@ -2690,7 +2690,7 @@
// Regression test for b/11924769
@Test
- public void testResourcesInOtherPackage_doubleFilegroup_LocalResources() throws Exception {
+ public void testResourcesInOtherPackage_doubleFilegroup_localResources() throws Exception {
scratch.file(
"java/android/resources/BUILD",
"android_binary(name = 'r',",
@@ -2712,7 +2712,7 @@
}
@Test
- public void testManifestMissingFails_LocalResources() throws Exception {
+ public void testManifestMissingFails_localResources() throws Exception {
checkError(
"java/android/resources",
"r",
@@ -2725,7 +2725,7 @@
}
@Test
- public void testResourcesDoesNotMatchDirectoryLayout_BadFile_LocalResources() throws Exception {
+ public void testResourcesDoesNotMatchDirectoryLayout_badFile_localResources() throws Exception {
checkError(
"java/android/resources",
"r",
@@ -2740,7 +2740,7 @@
}
@Test
- public void testResourcesDoesNotMatchDirectoryLayout_BadDirectory_LocalResources()
+ public void testResourcesDoesNotMatchDirectoryLayout_badDirectory_localResources()
throws Exception {
checkError(
"java/android/resources",
@@ -2756,7 +2756,7 @@
}
@Test
- public void testResourcesNotUnderCommonDirectoryFails_LocalResources() throws Exception {
+ public void testResourcesNotUnderCommonDirectoryFails_localResources() throws Exception {
checkError(
"java/android/resources",
"r",
@@ -2771,7 +2771,7 @@
}
@Test
- public void testAssetsAndNoAssetsDirFails_LocalResources() throws Exception {
+ public void testAssetsAndNoAssetsDirFails_localResources() throws Exception {
scratch.file(
"java/android/resources/assets/values/strings.xml",
"<resources><string name = 'hello'>Hello Android!</string></resources>");
@@ -2786,7 +2786,7 @@
}
@Test
- public void testAssetsDirAndNoAssetsFails_LocalResources() throws Exception {
+ public void testAssetsDirAndNoAssetsFails_localResources() throws Exception {
checkError(
"java/cpp/android",
"r",
@@ -2798,7 +2798,7 @@
}
@Test
- public void testAssetsNotUnderAssetsDirFails_LocalResources() throws Exception {
+ public void testAssetsNotUnderAssetsDirFails_localResources() throws Exception {
checkError(
"java/android/resources",
"r",
@@ -2812,7 +2812,7 @@
}
@Test
- public void testFileLocation_LocalResources() throws Exception {
+ public void testFileLocation_localResources() throws Exception {
scratch.file(
"java/android/resources/BUILD",
"android_binary(name = 'r',",
@@ -2826,7 +2826,7 @@
}
@Test
- public void testCustomPackage_LocalResources() throws Exception {
+ public void testCustomPackage_localResources() throws Exception {
scratch.file(
"a/r/BUILD",
"android_binary(name = 'r',",
@@ -2927,7 +2927,7 @@
}
@Test
- public void testNoApplicationId_LocalResources() throws Exception {
+ public void testNoApplicationId_localResources() throws Exception {
scratch.file(
"java/a/r/BUILD",
"android_binary(name = 'r',",
diff --git a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidDataConverterTest.java b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidDataConverterTest.java
index eb8f826..4ce8263 100644
--- a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidDataConverterTest.java
+++ b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidDataConverterTest.java
@@ -45,7 +45,7 @@
}
@Test
- public void testWithLabel_EscapingNotNeeded() throws LabelSyntaxException {
+ public void testWithLabel_escapingNotNeeded() throws LabelSyntaxException {
assertMap(
AndroidDataConverter.<String>builder(JoinerType.SEMICOLON_AMPERSAND)
.withLabel(getFunction(Label.create("foo/bar", "baz")))
diff --git a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java
index 855b27e..b1cc56c 100644
--- a/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java
+++ b/src/test/java/com/google/devtools/build/lib/rules/android/AndroidLibraryTest.java
@@ -1076,7 +1076,7 @@
}
@Test
- public void testResourcesDoesNotMatchDirectoryLayout_BadFile() throws Exception {
+ public void testResourcesDoesNotMatchDirectoryLayout_badFile() throws Exception {
checkError(
"java/android",
"r",
@@ -1095,7 +1095,7 @@
}
@Test
- public void testResourcesDoesNotMatchDirectoryLayout_BadDirectory() throws Exception {
+ public void testResourcesDoesNotMatchDirectoryLayout_badDirectory() throws Exception {
checkError(
"java/android",
"r",
@@ -1228,7 +1228,7 @@
}
@Test
- public void testNeverlinkResources_AndroidResourcesInfo() throws Exception {
+ public void testNeverlinkResources_androidResourcesInfo() throws Exception {
scratch.file(
"java/apps/android/BUILD",
"android_library(",
@@ -2087,7 +2087,7 @@
}
@Test
- public void testAndroidLibrary_SrcsLessDepsHostConfigurationNoOverride() throws Exception {
+ public void testAndroidLibrary_srcsLessDepsHostConfigurationNoOverride() throws Exception {
scratch.file(
"java/srclessdeps/BUILD",
"android_library(",
@@ -2300,7 +2300,7 @@
}
@Test
- public void testAarGeneration_LocalResources() throws Exception {
+ public void testAarGeneration_localResources() throws Exception {
scratch.file(
"java/android/aartest/BUILD",
"android_library(",
@@ -2334,7 +2334,7 @@
}
@Test
- public void testAarGeneration_NoResources() throws Exception {
+ public void testAarGeneration_noResources() throws Exception {
scratch.file(
"java/android/aartest/BUILD",
"android_library(",
diff --git a/src/test/java/com/google/devtools/build/lib/rules/cpp/CcToolchainFeaturesTest.java b/src/test/java/com/google/devtools/build/lib/rules/cpp/CcToolchainFeaturesTest.java
index fff4ad6..e8581ca 100644
--- a/src/test/java/com/google/devtools/build/lib/rules/cpp/CcToolchainFeaturesTest.java
+++ b/src/test/java/com/google/devtools/build/lib/rules/cpp/CcToolchainFeaturesTest.java
@@ -1255,7 +1255,7 @@
}
@Test
- public void testWithFeature_OneSetOneFeature() throws Exception {
+ public void testWithFeature_oneSetOneFeature() throws Exception {
CcToolchainFeatures features =
buildFeatures(
"feature {",
@@ -1282,7 +1282,7 @@
}
@Test
- public void testWithFeature_OneSetMultipleFeatures() throws Exception {
+ public void testWithFeature_oneSetMultipleFeatures() throws Exception {
CcToolchainFeatures features =
buildFeatures(
"feature {",
@@ -1315,7 +1315,7 @@
}
@Test
- public void testWithFeature_MulipleSetsMultipleFeatures() throws Exception {
+ public void testWithFeature_mulipleSetsMultipleFeatures() throws Exception {
CcToolchainFeatures features =
buildFeatures(
"feature {",
@@ -1351,7 +1351,7 @@
}
@Test
- public void testWithFeature_NotFeature() throws Exception {
+ public void testWithFeature_notFeature() throws Exception {
CcToolchainFeatures features =
buildFeatures(
"feature {",
diff --git a/src/test/java/com/google/devtools/build/lib/rules/proto/BazelProtoLibraryTest.java b/src/test/java/com/google/devtools/build/lib/rules/proto/BazelProtoLibraryTest.java
index cef10c9..6996171 100644
--- a/src/test/java/com/google/devtools/build/lib/rules/proto/BazelProtoLibraryTest.java
+++ b/src/test/java/com/google/devtools/build/lib/rules/proto/BazelProtoLibraryTest.java
@@ -391,7 +391,7 @@
}
@Test
- public void test_siblingRepoLayout_ExternalRepoWithGeneratedProto() throws Exception {
+ public void test_siblingRepoLayout_externalRepoWithGeneratedProto() throws Exception {
testExternalRepoWithGeneratedProto(/*siblingRepoLayout=*/ true);
}
diff --git a/src/test/java/com/google/devtools/build/lib/runtime/CommandLineEventTest.java b/src/test/java/com/google/devtools/build/lib/runtime/CommandLineEventTest.java
index 944aa36..61f7d7b 100644
--- a/src/test/java/com/google/devtools/build/lib/runtime/CommandLineEventTest.java
+++ b/src/test/java/com/google/devtools/build/lib/runtime/CommandLineEventTest.java
@@ -52,7 +52,7 @@
}
@Test
- public void testMostlyEmpty_OriginalCommandLine() {
+ public void testMostlyEmpty_originalCommandLine() {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
OptionsParser fakeCommandOptions =
@@ -79,7 +79,7 @@
}
@Test
- public void testMostlyEmpty_CanonicalCommandLine() {
+ public void testMostlyEmpty_canonicalCommandLine() {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
OptionsParser fakeCommandOptions =
@@ -105,7 +105,7 @@
}
@Test
- public void testActiveBazelrcs_OriginalCommandLine() throws OptionsParsingException {
+ public void testActiveBazelrcs_originalCommandLine() throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder()
.optionsClasses(BlazeServerStartupOptions.class, Options.class)
@@ -144,7 +144,7 @@
}
@Test
- public void testPassedInBazelrcs_OriginalCommandLine() throws OptionsParsingException {
+ public void testPassedInBazelrcs_originalCommandLine() throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder()
.optionsClasses(BlazeServerStartupOptions.class, Options.class)
@@ -188,7 +188,7 @@
}
@Test
- public void testBazelrcs_CanonicalCommandLine() throws OptionsParsingException {
+ public void testBazelrcs_canonicalCommandLine() throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder()
.optionsClasses(BlazeServerStartupOptions.class, Options.class)
@@ -220,7 +220,7 @@
}
@Test
- public void testOptionsAtVariousPriorities_OriginalCommandLine() throws OptionsParsingException {
+ public void testOptionsAtVariousPriorities_originalCommandLine() throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
OptionsParser fakeCommandOptions =
@@ -264,7 +264,7 @@
}
@Test
- public void testOptionsAtVariousPriorities_CanonicalCommandLine() throws OptionsParsingException {
+ public void testOptionsAtVariousPriorities_canonicalCommandLine() throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
OptionsParser fakeCommandOptions =
@@ -307,7 +307,7 @@
}
@Test
- public void testExpansionOption_OriginalCommandLine() throws OptionsParsingException {
+ public void testExpansionOption_originalCommandLine() throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
OptionsParser fakeCommandOptions =
@@ -340,7 +340,7 @@
}
@Test
- public void testExpansionOption_CanonicalCommandLine() throws OptionsParsingException {
+ public void testExpansionOption_canonicalCommandLine() throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
OptionsParser fakeCommandOptions =
@@ -375,7 +375,7 @@
}
@Test
- public void testOptionWithImplicitRequirement_OriginalCommandLine()
+ public void testOptionWithImplicitRequirement_originalCommandLine()
throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
@@ -410,7 +410,7 @@
}
@Test
- public void testOptionWithImplicitRequirement_CanonicalCommandLine()
+ public void testOptionWithImplicitRequirement_canonicalCommandLine()
throws OptionsParsingException {
OptionsParser fakeStartupOptions =
OptionsParser.builder().optionsClasses(BlazeServerStartupOptions.class).build();
diff --git a/src/test/java/com/google/devtools/build/lib/runtime/ProcessWrapperTest.java b/src/test/java/com/google/devtools/build/lib/runtime/ProcessWrapperTest.java
index b509cbb..4f6b1a8 100644
--- a/src/test/java/com/google/devtools/build/lib/runtime/ProcessWrapperTest.java
+++ b/src/test/java/com/google/devtools/build/lib/runtime/ProcessWrapperTest.java
@@ -47,7 +47,7 @@
}
@Test
- public void testProcessWrapperCommandLineBuilder_BuildsWithoutOptionalArguments()
+ public void testProcessWrapperCommandLineBuilder_buildsWithoutOptionalArguments()
throws IOException {
ImmutableList<String> commandArguments = ImmutableList.of("echo", "hello, world");
@@ -63,7 +63,7 @@
}
@Test
- public void testProcessWrapperCommandLineBuilder_BuildsWithOptionalArguments()
+ public void testProcessWrapperCommandLineBuilder_buildsWithOptionalArguments()
throws IOException {
ImmutableList<String> extraFlags = ImmutableList.of("--debug");
ImmutableList<String> commandArguments = ImmutableList.of("echo", "hello, world");
diff --git a/src/test/java/com/google/devtools/build/lib/sandbox/AbstractContainerizingSandboxedSpawnTest.java b/src/test/java/com/google/devtools/build/lib/sandbox/AbstractContainerizingSandboxedSpawnTest.java
index bad79b7..3634968 100644
--- a/src/test/java/com/google/devtools/build/lib/sandbox/AbstractContainerizingSandboxedSpawnTest.java
+++ b/src/test/java/com/google/devtools/build/lib/sandbox/AbstractContainerizingSandboxedSpawnTest.java
@@ -113,7 +113,7 @@
}
@Test
- public void testMoveOutputs_WarnOnceIfCopyHappened() throws Exception {
+ public void testMoveOutputs_warnOnceIfCopyHappened() throws Exception {
class MultipleDeviceFS extends InMemoryFileSystem {
@Override
public void renameTo(Path source, Path target) throws IOException {
diff --git a/src/test/java/com/google/devtools/build/lib/sandbox/BaseSandboxfsProcessIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/sandbox/BaseSandboxfsProcessIntegrationTest.java
index e823128..275001b 100644
--- a/src/test/java/com/google/devtools/build/lib/sandbox/BaseSandboxfsProcessIntegrationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/sandbox/BaseSandboxfsProcessIntegrationTest.java
@@ -57,7 +57,7 @@
}
@Test
- public void testMount_MissingDirectory() throws IOException {
+ public void testMount_missingDirectory() throws IOException {
IOException expected = assertThrows(
IOException.class, () -> mount(tmpDir.getRelative("missing")));
assertThat(expected).hasMessageThat().matches(".*(/missing.*does not exist|failed to start).*");
diff --git a/src/test/java/com/google/devtools/build/lib/sandbox/FakeSandboxfsProcessIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/sandbox/FakeSandboxfsProcessIntegrationTest.java
index 586febf..2b0bd63 100644
--- a/src/test/java/com/google/devtools/build/lib/sandbox/FakeSandboxfsProcessIntegrationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/sandbox/FakeSandboxfsProcessIntegrationTest.java
@@ -44,7 +44,7 @@
}
@Test
- public void testMount_NotADirectory() throws IOException {
+ public void testMount_notADirectory() throws IOException {
FileSystemUtils.createEmptyFile(tmpDir.getRelative("file"));
IOException expected = assertThrows(
IOException.class, () -> mount(tmpDir.getRelative("file")));
@@ -52,7 +52,7 @@
}
@Test
- public void testSandboxCreate_EnforcesNonRepeatedNames() throws IOException {
+ public void testSandboxCreate_enforcesNonRepeatedNames() throws IOException {
Path mountPoint = tmpDir.getRelative("mnt");
mountPoint.createDirectory();
SandboxfsProcess process = mount(mountPoint);
@@ -64,7 +64,7 @@
}
@Test
- public void testSandboxDestroy_EnforcesExistingName() throws IOException {
+ public void testSandboxDestroy_enforcesExistingName() throws IOException {
Path mountPoint = tmpDir.getRelative("mnt");
mountPoint.createDirectory();
SandboxfsProcess process = mount(mountPoint);
@@ -75,7 +75,7 @@
}
@Test
- public void testCreateAndDestroySandbox_ReuseName() throws IOException {
+ public void testCreateAndDestroySandbox_reuseName() throws IOException {
Path mountPoint = tmpDir.getRelative("mnt");
mountPoint.createDirectory();
SandboxfsProcess process = mount(mountPoint);
diff --git a/src/test/java/com/google/devtools/build/lib/sandbox/LinuxSandboxUtilTest.java b/src/test/java/com/google/devtools/build/lib/sandbox/LinuxSandboxUtilTest.java
index 744d53e..e5eafd0 100644
--- a/src/test/java/com/google/devtools/build/lib/sandbox/LinuxSandboxUtilTest.java
+++ b/src/test/java/com/google/devtools/build/lib/sandbox/LinuxSandboxUtilTest.java
@@ -57,7 +57,7 @@
}
@Test
- public void testLinuxSandboxCommandLineBuilder_BuildsWithoutOptionalArguments() {
+ public void testLinuxSandboxCommandLineBuilder_buildsWithoutOptionalArguments() {
Path linuxSandboxPath = testFS.getPath("/linux-sandbox");
ImmutableList<String> commandArguments = ImmutableList.of("echo", "hello, max");
@@ -76,7 +76,7 @@
}
@Test
- public void testLinuxSandboxCommandLineBuilder_BuildsWithOptionalArguments() {
+ public void testLinuxSandboxCommandLineBuilder_buildsWithOptionalArguments() {
Path linuxSandboxPath = testFS.getPath("/linux-sandbox");
ImmutableList<String> commandArguments = ImmutableList.of("echo", "hello, tom");
diff --git a/src/test/java/com/google/devtools/build/lib/sandbox/SandboxOptionsTest.java b/src/test/java/com/google/devtools/build/lib/sandbox/SandboxOptionsTest.java
index e18d875..14f7c8b 100644
--- a/src/test/java/com/google/devtools/build/lib/sandbox/SandboxOptionsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/sandbox/SandboxOptionsTest.java
@@ -30,7 +30,7 @@
private ImmutableMap.Entry<String, String> pathPair;
@Test
- public void testParsingAdditionalMounts_SinglePathWithoutColonSucess() throws Exception {
+ public void testParsingAdditionalMounts_singlePathWithoutColonSucess() throws Exception {
String source = "/a/bc/def/gh";
String target = source;
String input = source;
@@ -39,7 +39,7 @@
}
@Test
- public void testParsingAdditionalMounts_SinglePathWithColonSucess() throws Exception {
+ public void testParsingAdditionalMounts_singlePathWithColonSucess() throws Exception {
String source = "/a/b:c/def/gh";
String target = source;
String input = "/a/b\\:c/def/gh";
@@ -48,7 +48,7 @@
}
@Test
- public void testParsingAdditionalMounts_PathPairWithoutColonSucess() throws Exception {
+ public void testParsingAdditionalMounts_pathPairWithoutColonSucess() throws Exception {
String source = "/a/bc/def/gh";
String target = "/1/2/3/4/5";
String input = source + ":" + target;
@@ -57,7 +57,7 @@
}
@Test
- public void testParsingAdditionalMounts_PathPairWithColonSucess() throws Exception {
+ public void testParsingAdditionalMounts_pathPairWithColonSucess() throws Exception {
String source = "/a:/bc:/d:ef/gh";
String target = ":/1/2/3/4/5";
String input = "/a\\:/bc\\:/d\\:ef/gh:\\:/1/2/3/4/5";
@@ -66,7 +66,7 @@
}
@Test
- public void testParsingAdditionalMounts_TooManyPaths() throws Exception {
+ public void testParsingAdditionalMounts_tooManyPaths() throws Exception {
String input = "a/bc/def/gh:/1/2/3:x/y/z";
OptionsParsingException e =
assertThrows(
@@ -80,7 +80,7 @@
}
@Test
- public void testParsingAdditionalMounts_EmptyInput() throws Exception {
+ public void testParsingAdditionalMounts_emptyInput() throws Exception {
String input = "";
OptionsParsingException e =
assertThrows(
diff --git a/src/test/java/com/google/devtools/build/lib/sandbox/SandboxfsSandboxedSpawnTest.java b/src/test/java/com/google/devtools/build/lib/sandbox/SandboxfsSandboxedSpawnTest.java
index 7320863..c3d37b2 100644
--- a/src/test/java/com/google/devtools/build/lib/sandbox/SandboxfsSandboxedSpawnTest.java
+++ b/src/test/java/com/google/devtools/build/lib/sandbox/SandboxfsSandboxedSpawnTest.java
@@ -274,12 +274,12 @@
}
@Test
- public void testSymlinks_TargetsMappedIfRequested() throws Exception {
+ public void testSymlinks_targetsMappedIfRequested() throws Exception {
testSymlinks(true);
}
@Test
- public void testSymlinks_TargetsNotMappedIfNotRequested() throws Exception {
+ public void testSymlinks_targetsNotMappedIfNotRequested() throws Exception {
testSymlinks(false);
}
}
diff --git a/src/test/java/com/google/devtools/build/lib/shell/CommandResultTest.java b/src/test/java/com/google/devtools/build/lib/shell/CommandResultTest.java
index 5c5e4fd..471538e 100644
--- a/src/test/java/com/google/devtools/build/lib/shell/CommandResultTest.java
+++ b/src/test/java/com/google/devtools/build/lib/shell/CommandResultTest.java
@@ -28,7 +28,7 @@
public final class CommandResultTest {
@Test
- public void testBuilder_WithNoStderr() {
+ public void testBuilder_withNoStderr() {
Exception e =
assertThrows(
IllegalStateException.class,
@@ -41,7 +41,7 @@
}
@Test
- public void testBuilder_WithNoStdout() {
+ public void testBuilder_withNoStdout() {
Exception e =
assertThrows(
IllegalStateException.class,
@@ -54,7 +54,7 @@
}
@Test
- public void testBuilder_WithNoTerminationStatus() {
+ public void testBuilder_withNoTerminationStatus() {
Exception e =
assertThrows(
IllegalStateException.class,
@@ -67,7 +67,7 @@
}
@Test
- public void testBuilder_WithNoExecutionTime() {
+ public void testBuilder_withNoExecutionTime() {
CommandResult commandResult =
CommandResult.builder()
.setStdoutStream(CommandResult.EMPTY_OUTPUT)
@@ -80,7 +80,7 @@
}
@Test
- public void testBuilder_WithExecutionTime() {
+ public void testBuilder_withExecutionTime() {
CommandResult commandResult =
CommandResult.builder()
.setStdoutStream(CommandResult.EMPTY_OUTPUT)
diff --git a/src/test/java/com/google/devtools/build/lib/shell/CommandUsingLinuxSandboxTest.java b/src/test/java/com/google/devtools/build/lib/shell/CommandUsingLinuxSandboxTest.java
index 2da5669..2c8d2e4 100644
--- a/src/test/java/com/google/devtools/build/lib/shell/CommandUsingLinuxSandboxTest.java
+++ b/src/test/java/com/google/devtools/build/lib/shell/CommandUsingLinuxSandboxTest.java
@@ -56,7 +56,7 @@
}
@Test
- public void testCommand_Echo() throws Exception {
+ public void testCommand_echo() throws Exception {
ImmutableList<String> commandArguments = ImmutableList.of("echo", "colorless green ideas");
Command command = new Command(commandArguments.toArray(new String[0]));
@@ -67,7 +67,7 @@
}
@Test
- public void testLinuxSandboxedCommand_Echo() throws Exception {
+ public void testLinuxSandboxedCommand_echo() throws Exception {
// TODO(b/62588075) Currently no linux-sandbox tool support in Windows.
assumeTrue(OS.getCurrent() != OS.WINDOWS);
// TODO(b/62588075) Currently no linux-sandbox tool support in MacOS.
@@ -106,7 +106,7 @@
}
@Test
- public void testLinuxSandboxedCommand_WithStatistics_SpendUserTime()
+ public void testLinuxSandboxedCommand_withStatistics_spendUserTime()
throws CommandException, IOException, InterruptedException {
// TODO(b/62588075) Currently no linux-sandbox tool support in Windows.
assumeTrue(OS.getCurrent() != OS.WINDOWS);
@@ -120,7 +120,7 @@
}
@Test
- public void testLinuxSandboxedCommand_WithStatistics_SpendSystemTime()
+ public void testLinuxSandboxedCommand_withStatistics_spendSystemTime()
throws CommandException, IOException, InterruptedException {
// TODO(b/62588075) Currently no linux-sandbox tool support in Windows.
assumeTrue(OS.getCurrent() != OS.WINDOWS);
@@ -134,7 +134,7 @@
}
@Test
- public void testLinuxSandboxedCommand_WithStatistics_SpendUserAndSystemTime()
+ public void testLinuxSandboxedCommand_withStatistics_spendUserAndSystemTime()
throws CommandException, IOException, InterruptedException {
// TODO(b/62588075) Currently no linux-sandbox tool support in Windows.
assumeTrue(OS.getCurrent() != OS.WINDOWS);
diff --git a/src/test/java/com/google/devtools/build/lib/shell/CommandUsingProcessWrapperTest.java b/src/test/java/com/google/devtools/build/lib/shell/CommandUsingProcessWrapperTest.java
index eb259ca..31c44ee 100644
--- a/src/test/java/com/google/devtools/build/lib/shell/CommandUsingProcessWrapperTest.java
+++ b/src/test/java/com/google/devtools/build/lib/shell/CommandUsingProcessWrapperTest.java
@@ -57,7 +57,7 @@
}
@Test
- public void testCommand_Echo() throws Exception {
+ public void testCommand_echo() throws Exception {
ImmutableList<String> commandArguments = ImmutableList.of("echo", "worker bees can leave");
Command command = new Command(commandArguments.toArray(new String[0]));
@@ -68,7 +68,7 @@
}
@Test
- public void testProcessWrappedCommand_Echo() throws Exception {
+ public void testProcessWrappedCommand_echo() throws Exception {
ImmutableList<String> commandArguments = ImmutableList.of("echo", "even drones can fly away");
List<String> fullCommandLine = getProcessWrapper().commandLineBuilder(commandArguments).build();
@@ -102,7 +102,7 @@
}
@Test
- public void testProcessWrappedCommand_WithStatistics_SpendUserTime()
+ public void testProcessWrappedCommand_withStatistics_spendUserTime()
throws CommandException, IOException, InterruptedException {
Duration userTimeToSpend = Duration.ofSeconds(10);
Duration systemTimeToSpend = Duration.ZERO;
@@ -111,7 +111,7 @@
}
@Test
- public void testProcessWrappedCommand_WithStatistics_SpendSystemTime()
+ public void testProcessWrappedCommand_withStatistics_spendSystemTime()
throws CommandException, IOException, InterruptedException {
Duration userTimeToSpend = Duration.ZERO;
Duration systemTimeToSpend = Duration.ofSeconds(10);
@@ -120,7 +120,7 @@
}
@Test
- public void testProcessWrappedCommand_WithStatistics_SpendUserAndSystemTime()
+ public void testProcessWrappedCommand_withStatistics_spendUserAndSystemTime()
throws CommandException, IOException, InterruptedException {
Duration userTimeToSpend = Duration.ofSeconds(10);
Duration systemTimeToSpend = Duration.ofSeconds(10);
diff --git a/src/test/java/com/google/devtools/build/lib/shell/TerminationStatusTest.java b/src/test/java/com/google/devtools/build/lib/shell/TerminationStatusTest.java
index 7eb77eb..ed68c1c 100644
--- a/src/test/java/com/google/devtools/build/lib/shell/TerminationStatusTest.java
+++ b/src/test/java/com/google/devtools/build/lib/shell/TerminationStatusTest.java
@@ -49,19 +49,19 @@
}
@Test
- public void testBuilder_WithNoWaitResponse() {
+ public void testBuilder_withNoWaitResponse() {
assertThrows(
IllegalStateException.class, () -> TerminationStatus.builder().setTimedOut(false).build());
}
@Test
- public void testBuilder_WithNoTimedOut() {
+ public void testBuilder_withNoTimedOut() {
assertThrows(
IllegalStateException.class, () -> TerminationStatus.builder().setWaitResponse(0).build());
}
@Test
- public void testBuilder_WithNoExecutionTime() {
+ public void testBuilder_withNoExecutionTime() {
TerminationStatus terminationStatus =
TerminationStatus.builder().setWaitResponse(0).setTimedOut(false).build();
assertThat(terminationStatus.getWallExecutionTime()).isEmpty();
@@ -70,7 +70,7 @@
}
@Test
- public void testBuilder_WithExecutionTime() {
+ public void testBuilder_withExecutionTime() {
TerminationStatus terminationStatus =
TerminationStatus.builder()
.setWaitResponse(0)
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java
index 8789086..3f9eb35 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/ArtifactFunctionTest.java
@@ -167,11 +167,11 @@
}
/**
- * Tests that ArtifactFunction rethrows transitive {@link IOException}s as
- * {@link MissingInputFileException}s.
+ * Tests that ArtifactFunction rethrows transitive {@link IOException}s as {@link
+ * MissingInputFileException}s.
*/
@Test
- public void testIOException_EndToEnd() throws Throwable {
+ public void testIOException_endToEnd() throws Throwable {
final IOException exception = new IOException("beep");
setupRoot(
new CustomInMemoryFs() {
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/FileFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/FileFunctionTest.java
index 524565e..ed89669 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/FileFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/FileFunctionTest.java
@@ -902,7 +902,7 @@
}
@Test
- public void testFilesystemInconsistencies_GetFastDigest() throws Exception {
+ public void testFilesystemInconsistencies_getFastDigest() throws Exception {
CustomInMemoryFs fs = (CustomInMemoryFs) this.fs;
file("a");
// Our custom filesystem says "a/b" exists but "a" does not exist.
@@ -919,7 +919,7 @@
}
@Test
- public void testFilesystemInconsistencies_GetFastDigestAndIsReadableFailure() throws Exception {
+ public void testFilesystemInconsistencies_getFastDigestAndIsReadableFailure() throws Exception {
createFsAndRoot(
new CustomInMemoryFs(manualClock) {
@Override
@@ -1033,22 +1033,22 @@
}
@Test
- public void testSymlinkCycle_AncestorCycle_StartInCycle() throws Exception {
+ public void testSymlinkCycle_ancestorCycle_startInCycle() throws Exception {
runTestSymlinkCycle(/*ancestorCycle=*/ true, /*startInCycle=*/ true);
}
@Test
- public void testSymlinkCycle_AncestorCycle_StartOutOfCycle() throws Exception {
+ public void testSymlinkCycle_ancestorCycle_startOutOfCycle() throws Exception {
runTestSymlinkCycle(/*ancestorCycle=*/ true, /*startInCycle=*/ false);
}
@Test
- public void testSymlinkCycle_RegularCycle_StartInCycle() throws Exception {
+ public void testSymlinkCycle_regularCycle_startInCycle() throws Exception {
runTestSymlinkCycle(/*ancestorCycle=*/ false, /*startInCycle=*/ true);
}
@Test
- public void testSymlinkCycle_RegularCycle_StartOutOfCycle() throws Exception {
+ public void testSymlinkCycle_regularCycle_startOutOfCycle() throws Exception {
runTestSymlinkCycle(/*ancestorCycle=*/ false, /*startInCycle=*/ false);
}
@@ -1181,30 +1181,30 @@
}
@Test
- public void testInfiniteSymlinkExpansion_AbsoluteSymlinkToDescendant() throws Exception {
+ public void testInfiniteSymlinkExpansion_absoluteSymlinkToDescendant() throws Exception {
runTestSimpleInfiniteSymlinkExpansion(
/* symlinkToAncestor= */ false, /*absoluteSymlink=*/ true);
}
@Test
- public void testInfiniteSymlinkExpansion_RelativeSymlinkToDescendant() throws Exception {
+ public void testInfiniteSymlinkExpansion_relativeSymlinkToDescendant() throws Exception {
runTestSimpleInfiniteSymlinkExpansion(
/* symlinkToAncestor= */ false, /*absoluteSymlink=*/ false);
}
@Test
- public void testInfiniteSymlinkExpansion_AbsoluteSymlinkToAncestor() throws Exception {
+ public void testInfiniteSymlinkExpansion_absoluteSymlinkToAncestor() throws Exception {
runTestSimpleInfiniteSymlinkExpansion(/* symlinkToAncestor= */ true, /*absoluteSymlink=*/ true);
}
@Test
- public void testInfiniteSymlinkExpansion_RelativeSymlinkToAncestor() throws Exception {
+ public void testInfiniteSymlinkExpansion_relativeSymlinkToAncestor() throws Exception {
runTestSimpleInfiniteSymlinkExpansion(
/* symlinkToAncestor= */ true, /*absoluteSymlink=*/ false);
}
@Test
- public void testInfiniteSymlinkExpansion_SymlinkToReferrerToAncestor() throws Exception {
+ public void testInfiniteSymlinkExpansion_symlinkToReferrerToAncestor() throws Exception {
symlink("d", "a");
Path abPath = directory("a/b");
Path abcPath = abPath.getChild("c");
@@ -1246,7 +1246,7 @@
}
@Test
- public void testInfiniteSymlinkExpansion_SymlinkToReferrerToAncestor_LevelsOfDirectorySymlinks()
+ public void testInfiniteSymlinkExpansion_symlinkToReferrerToAncestor_levelsOfDirectorySymlinks()
throws Exception {
symlink("dir1/a", "../dir2");
symlink("dir2/b", "../dir1");
@@ -1331,7 +1331,7 @@
}
@Test
- public void testMultipleLevelsOfDirectorySymlinks_Clean() throws Exception {
+ public void testMultipleLevelsOfDirectorySymlinks_clean() throws Exception {
symlink("a/b/c", "../c");
Path abcd = path("a/b/c/d");
symlink("a/c/d", "../d");
@@ -1339,7 +1339,7 @@
}
@Test
- public void testMultipleLevelsOfDirectorySymlinks_Incremental() throws Exception {
+ public void testMultipleLevelsOfDirectorySymlinks_incremental() throws Exception {
SequentialBuildDriver driver = makeDriver();
symlink("a/b/c", "../c");
@@ -1377,7 +1377,7 @@
}
@Test
- public void testLogicalChainDuringResolution_Directory_SimpleSymlink() throws Exception {
+ public void testLogicalChainDuringResolution_directory_simpleSymlink() throws Exception {
symlink("a", "b");
symlink("b", "c");
directory("c");
@@ -1390,7 +1390,7 @@
}
@Test
- public void testLogicalChainDuringResolution_Directory_SimpleAncestorSymlink() throws Exception {
+ public void testLogicalChainDuringResolution_directory_simpleAncestorSymlink() throws Exception {
symlink("a", "b");
symlink("b", "c");
directory("c/d");
@@ -1403,7 +1403,7 @@
}
@Test
- public void testLogicalChainDuringResolution_File_SimpleSymlink() throws Exception {
+ public void testLogicalChainDuringResolution_file_simpleSymlink() throws Exception {
symlink("a", "b");
symlink("b", "c");
file("c");
@@ -1414,7 +1414,7 @@
}
@Test
- public void testLogicalChainDuringResolution_File_SimpleAncestorSymlink() throws Exception {
+ public void testLogicalChainDuringResolution_file_simpleAncestorSymlink() throws Exception {
symlink("a", "b");
symlink("b", "c");
file("c/d");
@@ -1425,7 +1425,7 @@
}
@Test
- public void testLogicalChainDuringResolution_Complicated() throws Exception {
+ public void testLogicalChainDuringResolution_complicated() throws Exception {
symlink("a", "b");
symlink("b", "c");
directory("c");
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/GlobFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/GlobFunctionTest.java
index c3ce6cf..decb897 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/GlobFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/GlobFunctionTest.java
@@ -676,7 +676,7 @@
/** Regression test for b/13319874: Directory listing crash. */
@Test
- public void testResilienceToFilesystemInconsistencies_DirectoryExistence() throws Exception {
+ public void testResilienceToFilesystemInconsistencies_directoryExistence() throws Exception {
// Our custom filesystem says "pkgPath/BUILD" exists but "pkgPath" does not exist.
fs.stubStat(pkgPath, null);
RootedPath pkgRootedPath = RootedPath.toRootedPath(Root.fromPath(root), pkgPath);
@@ -701,7 +701,7 @@
}
@Test
- public void testResilienceToFilesystemInconsistencies_SubdirectoryExistence() throws Exception {
+ public void testResilienceToFilesystemInconsistencies_subdirectoryExistence() throws Exception {
// Our custom filesystem says directory "pkgPath/foo/bar" contains a subdirectory "wiz" but a
// direct stat on "pkgPath/foo/bar/wiz" says it does not exist.
Path fooBarDir = pkgPath.getRelative("foo/bar");
@@ -725,7 +725,7 @@
}
@Test
- public void testResilienceToFilesystemInconsistencies_SymlinkType() throws Exception {
+ public void testResilienceToFilesystemInconsistencies_symlinkType() throws Exception {
RootedPath wizRootedPath =
RootedPath.toRootedPath(Root.fromPath(root), pkgPath.getRelative("foo/bar/wiz"));
RootedPath fileRootedPath =
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/LocalRepositoryLookupFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/LocalRepositoryLookupFunctionTest.java
index 9edbdac..f87dbda 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/LocalRepositoryLookupFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/LocalRepositoryLookupFunctionTest.java
@@ -275,7 +275,7 @@
}
@Test
- public void testLocalRepository_LocalWorkspace_SymlinkCycle() throws Exception {
+ public void testLocalRepository_localWorkspace_symlinkCycle() throws Exception {
scratch.overwriteFile("WORKSPACE", "local_repository(name='local', path='local/repo')");
Path localRepoWorkspace = scratch.resolve("local/repo/WORKSPACE");
Path localRepoWorkspaceLink = scratch.resolve("local/repo/WORKSPACE.link");
@@ -301,7 +301,7 @@
}
@Test
- public void testLocalRepository_MainWorkspace_NotFound() throws Exception {
+ public void testLocalRepository_mainWorkspace_notFound() throws Exception {
// Do not add a local_repository to WORKSPACE.
scratch.overwriteFile("WORKSPACE", "");
scratch.deleteFile("WORKSPACE");
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
index e79f5d2..05157c5 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/PackageFunctionTest.java
@@ -318,7 +318,7 @@
}
@Test
- public void testPropagatesFilesystemInconsistencies_Globbing() throws Exception {
+ public void testPropagatesFilesystemInconsistencies_globbing() throws Exception {
getSkyframeExecutor().turnOffSyscallCacheForTesting();
reporter.removeHandler(failFastHandler);
RecordingDifferencer differencer = getSkyframeExecutor().getDifferencerForTesting();
@@ -937,7 +937,7 @@
}
@Test
- public void testGlobAllowEmpty_ParamValueMustBeBoolean() throws Exception {
+ public void testGlobAllowEmpty_paramValueMustBeBoolean() throws Exception {
reporter.removeHandler(failFastHandler);
scratch.file("pkg/BUILD", "x = " + "glob(['*.foo'], allow_empty = 5)");
@@ -950,7 +950,7 @@
}
@Test
- public void testGlobAllowEmpty_FunctionParam() throws Exception {
+ public void testGlobAllowEmpty_functionParam() throws Exception {
scratch.file("pkg/BUILD", "x = " + "glob(['*.foo'], allow_empty=True)");
invalidatePackages();
@@ -961,7 +961,7 @@
}
@Test
- public void testGlobAllowEmpty_StarlarkOption() throws Exception {
+ public void testGlobAllowEmpty_starlarkOption() throws Exception {
preparePackageLoadingWithCustomStarklarkSemanticsOptions(
Options.parse(StarlarkSemanticsOptions.class, "--incompatible_disallow_empty_glob=false")
.getOptions(),
@@ -977,7 +977,7 @@
}
@Test
- public void testGlobDisallowEmpty_FunctionParam_WasNonEmptyAndBecomesEmpty() throws Exception {
+ public void testGlobDisallowEmpty_functionParam_wasNonEmptyAndBecomesEmpty() throws Exception {
scratch.file("pkg/BUILD", "x = " + "glob(['*.foo'], allow_empty=False)");
scratch.file("pkg/blah.foo");
invalidatePackages();
@@ -1004,7 +1004,7 @@
}
@Test
- public void testGlobDisallowEmpty_StarlarkOption_WasNonEmptyAndBecomesEmpty() throws Exception {
+ public void testGlobDisallowEmpty_starlarkOption_wasNonEmptyAndBecomesEmpty() throws Exception {
preparePackageLoadingWithCustomStarklarkSemanticsOptions(
Options.parse(StarlarkSemanticsOptions.class, "--incompatible_disallow_empty_glob=true")
.getOptions(),
@@ -1036,7 +1036,7 @@
}
@Test
- public void testGlobDisallowEmpty_FunctionParam_WasEmptyAndStaysEmpty() throws Exception {
+ public void testGlobDisallowEmpty_functionParam_wasEmptyAndStaysEmpty() throws Exception {
scratch.file("pkg/BUILD", "x = " + "glob(['*.foo'], allow_empty=False)");
invalidatePackages();
@@ -1063,7 +1063,7 @@
}
@Test
- public void testGlobDisallowEmpty_StarlarkOption_WasEmptyAndStaysEmpty() throws Exception {
+ public void testGlobDisallowEmpty_starlarkOption_wasEmptyAndStaysEmpty() throws Exception {
preparePackageLoadingWithCustomStarklarkSemanticsOptions(
Options.parse(StarlarkSemanticsOptions.class, "--incompatible_disallow_empty_glob=true")
.getOptions(),
@@ -1095,7 +1095,7 @@
}
@Test
- public void testGlobDisallowEmpty_FunctionParam_WasEmptyDueToExcludeAndStaysEmpty()
+ public void testGlobDisallowEmpty_functionParam_wasEmptyDueToExcludeAndStaysEmpty()
throws Exception {
scratch.file("pkg/BUILD", "x = glob(include=['*.foo'], exclude=['blah.*'], allow_empty=False)");
scratch.file("pkg/blah.foo");
@@ -1126,7 +1126,7 @@
}
@Test
- public void testGlobDisallowEmpty_StarlarkOption_WasEmptyDueToExcludeAndStaysEmpty()
+ public void testGlobDisallowEmpty_starlarkOption_wasEmptyDueToExcludeAndStaysEmpty()
throws Exception {
preparePackageLoadingWithCustomStarklarkSemanticsOptions(
Options.parse(StarlarkSemanticsOptions.class, "--incompatible_disallow_empty_glob=true")
@@ -1160,7 +1160,7 @@
}
@Test
- public void testGlobDisallowEmpty_FunctionParam_WasEmptyAndBecomesNonEmpty() throws Exception {
+ public void testGlobDisallowEmpty_functionParam_wasEmptyAndBecomesNonEmpty() throws Exception {
scratch.file("pkg/BUILD", "x = " + "glob(['*.foo'], allow_empty=False)");
invalidatePackages();
@@ -1188,7 +1188,7 @@
}
@Test
- public void testGlobDisallowEmpty_StarlarkOption_WasEmptyAndBecomesNonEmpty() throws Exception {
+ public void testGlobDisallowEmpty_starlarkOption_wasEmptyAndBecomesNonEmpty() throws Exception {
preparePackageLoadingWithCustomStarklarkSemanticsOptions(
Options.parse(StarlarkSemanticsOptions.class, "--incompatible_disallow_empty_glob=true")
.getOptions(),
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/TimestampBuilderMediumTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/TimestampBuilderMediumTest.java
index 2892f83..eaf87f1 100644
--- a/src/test/java/com/google/devtools/build/lib/skyframe/TimestampBuilderMediumTest.java
+++ b/src/test/java/com/google/devtools/build/lib/skyframe/TimestampBuilderMediumTest.java
@@ -123,7 +123,7 @@
}
@Test
- public void testPersistentCache_ModifyingInputCausesActionReexecution() throws Exception {
+ public void testPersistentCache_modifyingInputCausesActionReexecution() throws Exception {
// /hello -> [action] -> /goodbye
Artifact hello = createSourceArtifact("hello");
BlazeTestUtils.makeEmptyFile(hello.getPath());
@@ -316,7 +316,7 @@
}
@Test
- public void testPersistentCache_ModifyingOutputCausesActionReexecution() throws Exception {
+ public void testPersistentCache_modifyingOutputCausesActionReexecution() throws Exception {
// [action] -> /hello
Artifact hello = createDerivedArtifact("hello");
Button button = createActionButton(emptyNestedSet, ImmutableSet.of(hello));
diff --git a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkIntegrationTest.java
index 4bb1f2b..24333d8 100644
--- a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkIntegrationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkIntegrationTest.java
@@ -3314,7 +3314,7 @@
}
@Test
- public void testLegacyProvider_AddCanonicalLegacyKeyAndModernKey() throws Exception {
+ public void testLegacyProvider_addCanonicalLegacyKeyAndModernKey() throws Exception {
setStarlarkSemanticsOptions("--incompatible_disallow_struct_provider_syntax=false");
scratch.file(
"test/starlark/extension.bzl",
@@ -3342,7 +3342,7 @@
}
@Test
- public void testLegacyProvider_DontAutomaticallyAddKeysAlreadyPresent() throws Exception {
+ public void testLegacyProvider_dontAutomaticallyAddKeysAlreadyPresent() throws Exception {
setStarlarkSemanticsOptions("--incompatible_disallow_struct_provider_syntax=false");
scratch.file(
"test/starlark/extension.bzl",
@@ -3371,7 +3371,7 @@
}
@Test
- public void testLegacyProvider_FirstNoncanonicalKeyBecomesCanonical() throws Exception {
+ public void testLegacyProvider_firstNoncanonicalKeyBecomesCanonical() throws Exception {
setStarlarkSemanticsOptions("--incompatible_disallow_struct_provider_syntax=false");
scratch.file(
"test/starlark/extension.bzl",
diff --git a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleContextTest.java b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleContextTest.java
index 8405999..eebb9ee 100644
--- a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleContextTest.java
+++ b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleContextTest.java
@@ -310,7 +310,7 @@
* errors already happen when loading the file. Consequently, all tests would fail at the same
* statement. */
@Test
- public void testPackageBoundaryError_NativeRule() throws Exception {
+ public void testPackageBoundaryError_nativeRule() throws Exception {
scratch.file("test/BUILD", "cc_library(name = 'cclib',", " srcs = ['sub/my_sub_lib.h'])");
scratch.file("test/sub/BUILD", "cc_library(name = 'my_sub_lib', srcs = ['my_sub_lib.h'])");
reporter.removeHandler(failFastHandler);
@@ -322,7 +322,7 @@
}
@Test
- public void testPackageBoundaryError_StarlarkRule() throws Exception {
+ public void testPackageBoundaryError_starlarkRule() throws Exception {
scratch.file(
"test/BUILD",
"load('//test:macros.bzl', 'starlark_rule')",
@@ -348,7 +348,7 @@
}
@Test
- public void testPackageBoundaryError_StarlarkMacro() throws Exception {
+ public void testPackageBoundaryError_starlarkMacro() throws Exception {
scratch.file(
"test/BUILD",
"load('//test:macros.bzl', 'macro_starlark_rule')",
@@ -377,7 +377,7 @@
/* The error message for this case used to be wrong. */
@Test
- public void testPackageBoundaryError_ExternalRepository_Boundary() throws Exception {
+ public void testPackageBoundaryError_externalRepository_boundary() throws Exception {
scratch.file("r/WORKSPACE");
scratch.file("r/BUILD");
scratch.overwriteFile(
@@ -398,7 +398,7 @@
/* The error message for this case used to be wrong. */
@Test
- public void testPackageBoundaryError_ExternalRepository_EntirelyInside() throws Exception {
+ public void testPackageBoundaryError_externalRepository_entirelyInside() throws Exception {
scratch.file("/r/WORKSPACE");
scratch.file("/r/BUILD", "cc_library(name = 'cclib',", " srcs = ['sub/my_sub_lib.h'])");
scratch.file("/r/sub/BUILD", "cc_library(name = 'my_sub_lib', srcs = ['my_sub_lib.h'])");
@@ -428,7 +428,7 @@
* with it.
*/
@Test
- public void testPackageBoundaryError_StarlarkMacroWithErrorInBzlFile() throws Exception {
+ public void testPackageBoundaryError_starlarkMacroWithErrorInBzlFile() throws Exception {
scratch.file(
"test/BUILD",
"load('//test:macros.bzl', 'macro_starlark_rule')",
@@ -454,7 +454,7 @@
}
@Test
- public void testPackageBoundaryError_NativeMacro() throws Exception {
+ public void testPackageBoundaryError_nativeMacro() throws Exception {
scratch.file(
"test/BUILD",
"load('//test:macros.bzl', 'macro_native_rule')",
diff --git a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleImplementationFunctionsTest.java b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleImplementationFunctionsTest.java
index 6110ebe..f2c1688 100644
--- a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleImplementationFunctionsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkRuleImplementationFunctionsTest.java
@@ -2478,7 +2478,7 @@
}
@Test
- public void testConfigurationField_StarlarkSplitTransitionProhibited() throws Exception {
+ public void testConfigurationField_starlarkSplitTransitionProhibited() throws Exception {
scratch.file(
"tools/allowlists/function_transition_allowlist/BUILD",
"package_group(",
@@ -2516,7 +2516,7 @@
}
@Test
- public void testConfigurationField_NativeSplitTransitionProviderProhibited() throws Exception {
+ public void testConfigurationField_nativeSplitTransitionProviderProhibited() throws Exception {
scratch.file(
"test/rule.bzl",
"def _foo_impl(ctx):",
@@ -2537,7 +2537,7 @@
}
@Test
- public void testConfigurationField_NativeSplitTransitionProhibited() throws Exception {
+ public void testConfigurationField_nativeSplitTransitionProhibited() throws Exception {
scratch.file(
"test/rule.bzl",
"def _foo_impl(ctx):",
diff --git a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkStringRepresentationsTest.java b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkStringRepresentationsTest.java
index b1f2764..c85a0d5 100644
--- a/src/test/java/com/google/devtools/build/lib/starlark/StarlarkStringRepresentationsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/starlark/StarlarkStringRepresentationsTest.java
@@ -223,7 +223,7 @@
}
@Test
- public void testStringRepresentations_Strings() throws Exception {
+ public void testStringRepresentations_strings() throws Exception {
assertThat(starlarkLoadingEval("str('foo')")).isEqualTo("foo");
assertThat(starlarkLoadingEval("'%s' % 'foo'")).isEqualTo("foo");
assertThat(starlarkLoadingEval("'{}'.format('foo')")).isEqualTo("foo");
@@ -232,7 +232,7 @@
}
@Test
- public void testStringRepresentations_Labels() throws Exception {
+ public void testStringRepresentations_labels() throws Exception {
assertThat(starlarkLoadingEval("str(Label('//foo:bar'))")).isEqualTo("//foo:bar");
assertThat(starlarkLoadingEval("'%s' % Label('//foo:bar')")).isEqualTo("//foo:bar");
assertThat(starlarkLoadingEval("'{}'.format(Label('//foo:bar'))")).isEqualTo("//foo:bar");
@@ -244,7 +244,7 @@
}
@Test
- public void testStringRepresentations_Primitives() throws Exception {
+ public void testStringRepresentations_primitives() throws Exception {
// Strings are tested in a separate test case as they have different str and repr values.
assertStringRepresentation("1543", "1543");
assertStringRepresentation("True", "True");
@@ -252,7 +252,7 @@
}
@Test
- public void testStringRepresentations_Containers() throws Exception {
+ public void testStringRepresentations_containers() throws Exception {
assertStringRepresentation("['a', 'b']", "[\"a\", \"b\"]");
assertStringRepresentation("('a', 'b')", "(\"a\", \"b\")");
assertStringRepresentation("{'a': 'b', 'c': 'd'}", "{\"a\": \"b\", \"c\": \"d\"}");
@@ -260,38 +260,38 @@
}
@Test
- public void testStringRepresentations_Functions() throws Exception {
+ public void testStringRepresentations_functions() throws Exception {
assertStringRepresentation("all", "<built-in function all>");
assertStringRepresentation("def f(): pass", "f", "<function f from //eval:eval.bzl>");
}
@Test
- public void testStringRepresentations_Rules() throws Exception {
+ public void testStringRepresentations_rules() throws Exception {
assertStringRepresentation("native.cc_library", "<built-in rule cc_library>");
assertStringRepresentation("def f(): pass", "rule(implementation=f)", "<rule>");
}
@Test
- public void testStringRepresentations_Aspects() throws Exception {
+ public void testStringRepresentations_aspects() throws Exception {
assertStringRepresentation("def f(): pass", "aspect(implementation=f)", "<aspect>");
}
@Test
- public void testStringRepresentations_Providers() throws Exception {
+ public void testStringRepresentations_providers() throws Exception {
assertStringRepresentation("provider()", "<provider>");
assertStringRepresentation(
"p = provider()", "p(b = 'foo', a = 1)", "struct(a = 1, b = \"foo\")");
}
@Test
- public void testStringRepresentations_Select() throws Exception {
+ public void testStringRepresentations_select() throws Exception {
assertStringRepresentation(
"select({'//foo': ['//bar']}) + select({'//foo2': ['//bar2']})",
"select({\"//foo\": [\"//bar\"]}) + select({\"//foo2\": [\"//bar2\"]})");
}
@Test
- public void testStringRepresentations_RuleContext() throws Exception {
+ public void testStringRepresentations_ruleContext() throws Exception {
generateFilesToTestStrings();
ConfiguredTarget target = getConfiguredTarget("//test/starlark:check");
@@ -306,7 +306,7 @@
}
@Test
- public void testStringRepresentations_Files() throws Exception {
+ public void testStringRepresentations_files() throws Exception {
generateFilesToTestStrings();
ConfiguredTarget target = getConfiguredTarget("//test/starlark:check");
@@ -319,7 +319,7 @@
}
@Test
- public void testStringRepresentations_Root() throws Exception {
+ public void testStringRepresentations_root() throws Exception {
generateFilesToTestStrings();
ConfiguredTarget target = getConfiguredTarget("//test/starlark:check");
@@ -330,7 +330,7 @@
}
@Test
- public void testStringRepresentations_Glob() throws Exception {
+ public void testStringRepresentations_glob() throws Exception {
scratch.file("eval/one.txt");
scratch.file("eval/two.txt");
scratch.file("eval/three.txt");
@@ -341,7 +341,7 @@
}
@Test
- public void testStringRepresentations_Attr() throws Exception {
+ public void testStringRepresentations_attr() throws Exception {
assertStringRepresentation("attr", "<attr>");
assertStringRepresentation("attr.int()", "<attr.int>");
assertStringRepresentation("attr.string()", "<attr.string>");
@@ -358,7 +358,7 @@
}
@Test
- public void testStringRepresentations_Targets() throws Exception {
+ public void testStringRepresentations_targets() throws Exception {
generateFilesToTestStrings();
ConfiguredTarget target = getConfiguredTarget("//test/starlark:check");
diff --git a/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java b/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java
index 68f7bec..138f1ab 100644
--- a/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/syntax/EvaluationTest.java
@@ -636,7 +636,7 @@
}
@Test
- public void testDictComprehension_ManyClauses() throws Exception {
+ public void testDictComprehension_manyClauses() throws Exception {
ev.new Scenario()
.testExpression(
"{x : x * y for x in range(1, 10) if x % 2 == 0 for y in range(1, 10) if y == x}",
@@ -644,7 +644,7 @@
}
@Test
- public void testDictComprehensions_MultipleKey() throws Exception {
+ public void testDictComprehensions_multipleKey() throws Exception {
ev.new Scenario()
.testExpression("{x : x for x in [1, 2, 1]}", ImmutableMap.of(1, 1, 2, 2))
.testExpression(
diff --git a/src/test/java/com/google/devtools/build/lib/syntax/StarlarkEvaluationTest.java b/src/test/java/com/google/devtools/build/lib/syntax/StarlarkEvaluationTest.java
index 0b253a0..4a81193 100644
--- a/src/test/java/com/google/devtools/build/lib/syntax/StarlarkEvaluationTest.java
+++ b/src/test/java/com/google/devtools/build/lib/syntax/StarlarkEvaluationTest.java
@@ -564,17 +564,17 @@
}
@Test
- public void testIfElifElse_IfExecutes() throws Exception {
+ public void testIfElifElse_ifExecutes() throws Exception {
execIfElifElse(1, 0, 1);
}
@Test
- public void testIfElifElse_ElifExecutes() throws Exception {
+ public void testIfElifElse_elifExecutes() throws Exception {
execIfElifElse(0, 1, 2);
}
@Test
- public void testIfElifElse_ElseExecutes() throws Exception {
+ public void testIfElifElse_elseExecutes() throws Exception {
execIfElifElse(0, 0, 3);
}
@@ -1512,7 +1512,7 @@
}
@Test
- public void testInvalidAugmentedAssignment_ListExpression() throws Exception {
+ public void testInvalidAugmentedAssignment_listExpression() throws Exception {
assertResolutionError(
"cannot perform augmented assignment on a list or tuple expression",
//
@@ -1521,9 +1521,8 @@
"f(1, 2)");
}
-
@Test
- public void testInvalidAugmentedAssignment_NotAnLValue() throws Exception {
+ public void testInvalidAugmentedAssignment_notAnLValue() throws Exception {
assertResolutionError(
"cannot assign to 'x + 1'",
//
@@ -1553,7 +1552,7 @@
}
@Test
- public void testDictComprehensions_IterationOrder() throws Exception {
+ public void testDictComprehensions_iterationOrder() throws Exception {
ev.new Scenario()
.setUp(
"def foo():",
diff --git a/src/test/java/com/google/devtools/build/lib/unix/NativePosixFilesTest.java b/src/test/java/com/google/devtools/build/lib/unix/NativePosixFilesTest.java
index f14a9bc..7fc0c6a 100644
--- a/src/test/java/com/google/devtools/build/lib/unix/NativePosixFilesTest.java
+++ b/src/test/java/com/google/devtools/build/lib/unix/NativePosixFilesTest.java
@@ -80,7 +80,7 @@
}
@Test
- public void testGetxattr_AttributeFound() throws Exception {
+ public void testGetxattr_attributeFound() throws Exception {
assumeXattrsSupported();
String myfile = Files.createTempFile("getxattrtest", null).toString();
@@ -92,7 +92,7 @@
}
@Test
- public void testGetxattr_AttributeNotFoundReturnsNull() throws Exception {
+ public void testGetxattr_attributeNotFoundReturnsNull() throws Exception {
assumeXattrsSupported();
String myfile = Files.createTempFile("getxattrtest", null).toString();
@@ -102,7 +102,7 @@
}
@Test
- public void testGetxattr_FileNotFound() throws Exception {
+ public void testGetxattr_fileNotFound() throws Exception {
String nonexistentFile = workingDir.getChild("nonexistent").toString();
assertThrows(
diff --git a/src/test/java/com/google/devtools/build/lib/versioning/GnuVersionParserTest.java b/src/test/java/com/google/devtools/build/lib/versioning/GnuVersionParserTest.java
index c76e368..abc5ba5 100644
--- a/src/test/java/com/google/devtools/build/lib/versioning/GnuVersionParserTest.java
+++ b/src/test/java/com/google/devtools/build/lib/versioning/GnuVersionParserTest.java
@@ -43,7 +43,7 @@
};
@Test
- public void testParse_Ok() throws Exception {
+ public void testParse_ok() throws Exception {
GnuVersionParser<SemVer> parser = new GnuVersionParser<>("sandboxfs", SemVer::parse);
assertThat(parser.parse("sandboxfs 0.1.3")).isEqualTo(SemVer.from(0, 1, 3));
}
@@ -54,7 +54,7 @@
}
@Test
- public void testFromInputStream_Ok() throws Exception {
+ public void testFromInputStream_ok() throws Exception {
GnuVersionParser<SemVer> parser = new GnuVersionParser<>("GNU Emacs", SemVer::parse);
for (String stdout :
new String[] {
@@ -68,7 +68,7 @@
}
@Test
- public void testFromInputStream_ErrorIfEmpty() {
+ public void testFromInputStream_errorIfEmpty() {
GnuVersionParser<SemVer> parser = new GnuVersionParser<>("GNU Emacs", SemVer::parse);
InputStream input = newInputStream("");
Exception e = assertThrows(ParseException.class, () -> parser.fromInputStream(input));
@@ -76,7 +76,7 @@
}
@Test
- public void testFromInputStream_ErrorOnBadVersion() {
+ public void testFromInputStream_errorOnBadVersion() {
GnuVersionParser<SemVer> parser = new GnuVersionParser<>("GNU Emacs", SemVer::parse);
InputStream input = newInputStream("GNU Emacs 0.abc");
Exception e = assertThrows(ParseException.class, () -> parser.fromInputStream(input));
@@ -84,7 +84,7 @@
}
@Test
- public void testFromInputStream_DrainsInput() throws Exception {
+ public void testFromInputStream_drainsInput() throws Exception {
GnuVersionParser<SemVer> parser = new GnuVersionParser<>("foo", SemVer::parse);
InputStream input = newInputStream("foo 1\nsome extra text");
parser.fromInputStream(input);
@@ -106,14 +106,14 @@
}
@Test
- public void testFromProgram_Ok() throws Exception {
+ public void testFromProgram_ok() throws Exception {
Path helper = createHelper("echo test 9.8; echo some more text that is ignored");
GnuVersionParser<SemVer> parser = new GnuVersionParser<>("test", SemVer::parse);
assertThat(parser.fromProgram(helper.asFragment())).isEqualTo(SemVer.from(9, 8));
}
@Test
- public void testFromProgram_BadPackageName() throws Exception {
+ public void testFromProgram_badPackageName() throws Exception {
Path helper = createHelper("echo foo 9.8; echo some more text that is ignored");
GnuVersionParser<?> parser = new GnuVersionParser<>("test", UNUSED_PARSER);
Exception e = assertThrows(ParseException.class, () -> parser.fromProgram(helper.asFragment()));
@@ -121,7 +121,7 @@
}
@Test
- public void testFromProgram_BadVersion() throws Exception {
+ public void testFromProgram_badVersion() throws Exception {
Path helper = createHelper("echo test 8.3a");
GnuVersionParser<?> parser = new GnuVersionParser<>("test", SemVer::parse);
Exception e = assertThrows(ParseException.class, () -> parser.fromProgram(helper.asFragment()));
@@ -129,7 +129,7 @@
}
@Test
- public void testFromProgram_BadExitCode() throws Exception {
+ public void testFromProgram_badExitCode() throws Exception {
Path helper = createHelper("echo test 9.1; return 1");
GnuVersionParser<?> parser = new GnuVersionParser<>("test", SemVer::parse);
Exception e = assertThrows(IOException.class, () -> parser.fromProgram(helper.asFragment()));
@@ -137,7 +137,7 @@
}
@Test
- public void testFromProgram_ExecError() {
+ public void testFromProgram_execError() {
GnuVersionParser<?> parser = new GnuVersionParser<>("test", UNUSED_PARSER);
Exception e =
assertThrows(
diff --git a/src/test/java/com/google/devtools/build/lib/versioning/SemVerTest.java b/src/test/java/com/google/devtools/build/lib/versioning/SemVerTest.java
index 8820f4d..ba25f4c 100644
--- a/src/test/java/com/google/devtools/build/lib/versioning/SemVerTest.java
+++ b/src/test/java/com/google/devtools/build/lib/versioning/SemVerTest.java
@@ -26,13 +26,13 @@
public class SemVerTest {
@Test
- public void testFrom_UnspecifiedComponentsAreZero() {
+ public void testFrom_unspecifiedComponentsAreZero() {
assertThat(SemVer.from(8)).isEqualTo(SemVer.from(8, 0, 0));
assertThat(SemVer.from(9, 15)).isEqualTo(SemVer.from(9, 15, 0));
}
@Test
- public void testParse_Ok() throws Exception {
+ public void testParse_ok() throws Exception {
assertThat(SemVer.parse("1")).isEqualTo(SemVer.from(1));
assertThat(SemVer.parse("2.3")).isEqualTo(SemVer.from(2, 3));
assertThat(SemVer.parse("3.5.1")).isEqualTo(SemVer.from(3, 5, 1));
@@ -40,7 +40,7 @@
}
@Test
- public void testParse_Errors() {
+ public void testParse_errors() {
for (String s : new String[] {"", "foo", "1..2", "1.2.", "1.-1.2"}) {
ParseException e = assertThrows(ParseException.class, () -> SemVer.parse(s));
assertThat(e).hasMessageThat().contains("Invalid semver");
@@ -54,7 +54,7 @@
}
@Test
- public void testComparison_OneComponent() {
+ public void testComparison_oneComponent() {
assertThat(SemVer.from(1, 0, 0)).isLessThan(SemVer.from(2, 0, 0));
assertThat(SemVer.from(1, 0, 0)).isEqualTo(SemVer.from(1, 0, 0));
assertThat(SemVer.from(2, 0, 0)).isGreaterThan(SemVer.from(1, 0, 0));
@@ -69,7 +69,7 @@
}
@Test
- public void testComparison_MultipleComponents() {
+ public void testComparison_multipleComponents() {
assertThat(SemVer.from(1, 0, 0)).isLessThan(SemVer.from(1, 1, 0));
assertThat(SemVer.from(1, 1, 0)).isGreaterThan(SemVer.from(1, 0, 0));
@@ -78,7 +78,7 @@
}
@Test
- public void testToString_KeepsAllComponents() {
+ public void testToString_keepsAllComponents() {
assertThat(SemVer.from(0, 0, 0).toString()).isEqualTo("0.0.0");
assertThat(SemVer.from(7, 1, 8).toString()).isEqualTo("7.1.8");
}
diff --git a/src/test/java/com/google/devtools/build/lib/vfs/FileSystemTest.java b/src/test/java/com/google/devtools/build/lib/vfs/FileSystemTest.java
index 2e0c1a5..64a2e0f 100644
--- a/src/test/java/com/google/devtools/build/lib/vfs/FileSystemTest.java
+++ b/src/test/java/com/google/devtools/build/lib/vfs/FileSystemTest.java
@@ -1657,7 +1657,7 @@
}
@Test
- public void testCreateHardLink_Success() throws Exception {
+ public void testCreateHardLink_success() throws Exception {
if (!testFS.supportsHardLinksNatively(xFile)) {
return;
}
@@ -1670,7 +1670,7 @@
}
@Test
- public void testCreateHardLink_NeitherOriginalNorLinkExists() throws Exception {
+ public void testCreateHardLink_neitherOriginalNorLinkExists() throws Exception {
if (!testFS.supportsHardLinksNatively(xFile)) {
return;
}
@@ -1687,7 +1687,7 @@
}
@Test
- public void testCreateHardLink_OriginalDoesNotExistAndLinkExists() throws Exception {
+ public void testCreateHardLink_originalDoesNotExistAndLinkExists() throws Exception {
if (!testFS.supportsHardLinksNatively(xFile)) {
return;
@@ -1707,7 +1707,7 @@
}
@Test
- public void testCreateHardLink_BothOriginalAndLinkExist() throws Exception {
+ public void testCreateHardLink_bothOriginalAndLinkExist() throws Exception {
if (!testFS.supportsHardLinksNatively(xFile)) {
return;
diff --git a/src/test/java/com/google/devtools/build/lib/vfs/FileSystemUtilsTest.java b/src/test/java/com/google/devtools/build/lib/vfs/FileSystemUtilsTest.java
index 780a49c..765f63f 100644
--- a/src/test/java/com/google/devtools/build/lib/vfs/FileSystemUtilsTest.java
+++ b/src/test/java/com/google/devtools/build/lib/vfs/FileSystemUtilsTest.java
@@ -194,7 +194,7 @@
}
@Test
- public void testRemoveExtension_Strings() throws Exception {
+ public void testRemoveExtension_strings() throws Exception {
assertThat(removeExtension("foo.c")).isEqualTo("foo");
assertThat(removeExtension("a/foo.c")).isEqualTo("a/foo");
assertThat(removeExtension("a.b/foo")).isEqualTo("a.b/foo");
@@ -203,7 +203,7 @@
}
@Test
- public void testRemoveExtension_Paths() throws Exception {
+ public void testRemoveExtension_paths() throws Exception {
assertPath("/foo", removeExtension(fileSystem.getPath("/foo.c")));
assertPath("/a/foo", removeExtension(fileSystem.getPath("/a/foo.c")));
assertPath("/a.b/foo", removeExtension(fileSystem.getPath("/a.b/foo")));
@@ -220,7 +220,7 @@
}
@Test
- public void testReplaceExtension_Path() throws Exception {
+ public void testReplaceExtension_path() throws Exception {
assertPath("/foo/bar.baz",
FileSystemUtils.replaceExtension(fileSystem.getPath("/foo/bar"), ".baz"));
assertPath("/foo/bar.baz",
@@ -235,7 +235,7 @@
}
@Test
- public void testReplaceExtension_PathFragment() throws Exception {
+ public void testReplaceExtension_pathFragment() throws Exception {
assertPath("foo/bar.baz",
FileSystemUtils.replaceExtension(PathFragment.create("foo/bar"), ".baz"));
assertPath("foo/bar.baz",
@@ -773,7 +773,7 @@
}
@Test
- public void testCreateHardLinkForFile_Success() throws Exception {
+ public void testCreateHardLinkForFile_success() throws Exception {
/* Original file exists and link file does not exist */
Path originalPath = workingDir.getRelative("original");
@@ -787,7 +787,7 @@
}
@Test
- public void testCreateHardLinkForEmptyDirectory_Success() throws Exception {
+ public void testCreateHardLinkForEmptyDirectory_success() throws Exception {
Path originalDir = workingDir.getRelative("originalDir");
Path linkPath = workingDir.getRelative("link");
@@ -800,7 +800,7 @@
}
@Test
- public void testCreateHardLinkForNonEmptyDirectory_Success() throws Exception {
+ public void testCreateHardLinkForNonEmptyDirectory_success() throws Exception {
/* Test when original path is a directory */
Path originalDir = workingDir.getRelative("originalDir");
diff --git a/src/test/java/com/google/devtools/build/lib/vfs/RootTest.java b/src/test/java/com/google/devtools/build/lib/vfs/RootTest.java
index d12cdb6..b06215e 100644
--- a/src/test/java/com/google/devtools/build/lib/vfs/RootTest.java
+++ b/src/test/java/com/google/devtools/build/lib/vfs/RootTest.java
@@ -108,7 +108,7 @@
}
@Test
- public void testSerialization_Simple() throws Exception {
+ public void testSerialization_simple() throws Exception {
Root fooPathRoot = Root.fromPath(fs.getPath("/foo"));
Root barPathRoot = Root.fromPath(fs.getPath("/bar"));
new SerializationTester(Root.absoluteRoot(fs), fooPathRoot, barPathRoot)
@@ -120,7 +120,7 @@
}
@Test
- public void testSerialization_LikelyPopularRootIsCanonicalized() throws Exception {
+ public void testSerialization_likelyPopularRootIsCanonicalized() throws Exception {
Root fooPathRoot = Root.fromPath(fs.getPath("/foo"));
Root otherFooPathRoot = Root.fromPath(fs.getPath("/foo"));
Root barPathRoot = Root.fromPath(fs.getPath("/bar"));