You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ji...@apache.org on 2016/12/14 18:00:53 UTC
geode git commit: GEODE-2196: spotless
Repository: geode
Updated Branches:
refs/heads/develop bbac13690 -> a225060f4
GEODE-2196: spotless
Project: http://git-wip-us.apache.org/repos/asf/geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/a225060f
Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/a225060f
Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/a225060f
Branch: refs/heads/develop
Commit: a225060f46e4349a5840a6d8bbbbb8dd22ce0240
Parents: bbac136
Author: Jinmei Liao <ji...@pivotal.io>
Authored: Wed Dec 14 10:00:30 2016 -0800
Committer: Jinmei Liao <ji...@pivotal.io>
Committed: Wed Dec 14 10:00:30 2016 -0800
----------------------------------------------------------------------
.../configuration/ClusterConfigDUnitTest.java | 22 +++++++++++---------
1 file changed, 12 insertions(+), 10 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/geode/blob/a225060f/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDUnitTest.java b/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDUnitTest.java
index a0445d4..5655e48 100644
--- a/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDUnitTest.java
+++ b/geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDUnitTest.java
@@ -200,7 +200,7 @@ public class ClusterConfigDUnitTest extends JUnit4DistributedTestCase {
Member locator = lsRule.startLocatorVM(0, locatorProps);
// server1 in no group
Member server1 = lsRule.startServerVM(1, serverProps, locator.getPort());
- //server2 in group1
+ // server2 in group1
serverProps.setProperty(GROUPS, "group1");
Member server2 = lsRule.startServerVM(2, serverProps, locator.getPort());
// server3 in group1 and group2
@@ -418,7 +418,7 @@ public class ClusterConfigDUnitTest extends JUnit4DistributedTestCase {
server.invoke(() -> verifyServerConfigInMemory(expectedConfig));
}
- public static void verifyLocatorConfig(ExpectedConfig expectedConfig, Member locator){
+ public static void verifyLocatorConfig(ExpectedConfig expectedConfig, Member locator) {
// verify info exists in memeory
locator.invoke(() -> {
InternalLocator internalLocator = LocatorServerStartupRule.locatorStarter.locator;
@@ -428,13 +428,14 @@ public class ClusterConfigDUnitTest extends JUnit4DistributedTestCase {
});
// verify files exists on disc
- for(String jar : expectedConfig.jars) {
- assertThat(new File(locator.getWorkingDir(), "/cluster_config/"+expectedConfig.name+"/"+jar))
- .exists();
+ for (String jar : expectedConfig.jars) {
+ assertThat(
+ new File(locator.getWorkingDir(), "/cluster_config/" + expectedConfig.name + "/" + jar))
+ .exists();
}
}
- public static void verifyLocatorConfigNotExist(String configName, Member locator){
+ public static void verifyLocatorConfigNotExist(String configName, Member locator) {
// verify info not in memeory
locator.invoke(() -> {
InternalLocator internalLocator = LocatorServerStartupRule.locatorStarter.locator;
@@ -444,8 +445,7 @@ public class ClusterConfigDUnitTest extends JUnit4DistributedTestCase {
});
// verify files does not
- assertThat(new File(locator.getWorkingDir(), "/cluster_config/"+configName))
- .doesNotExist();
+ assertThat(new File(locator.getWorkingDir(), "/cluster_config/" + configName)).doesNotExist();
}
private static void verifyServerConfigInMemory(ExpectedConfig expectedConfig)
@@ -473,7 +473,9 @@ public class ClusterConfigDUnitTest extends JUnit4DistributedTestCase {
for (String jarName : jarNames) {
expectedJarNames.add(getServerJarName(jarName));
}
- Set<String> actualJarNames = Arrays.stream(workingDir.list((dir, filename) -> filename.contains(".jar"))).collect(Collectors.toSet());
+ Set<String> actualJarNames =
+ Arrays.stream(workingDir.list((dir, filename) -> filename.contains(".jar")))
+ .collect(Collectors.toSet());
assertThat(actualJarNames).isEqualTo(expectedJarNames);
}
@@ -533,7 +535,7 @@ public class ClusterConfigDUnitTest extends JUnit4DistributedTestCase {
return this;
}
- public ExpectedConfig name(String name){
+ public ExpectedConfig name(String name) {
this.name = name;
return this;
}