From 73228a4192f155fbf6cccf362b80a5f7e73a81fe Mon Sep 17 00:00:00 2001 From: John Blum Date: Wed, 5 Dec 2018 02:00:03 -0800 Subject: [PATCH] Polish. --- ...ationsUsingClientLocalRegionIntegrationTests.java | 4 +++- ...ationsUsingClientProxyRegionIntegrationTests.java | 12 +++++++++--- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientLocalRegionIntegrationTests.java b/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientLocalRegionIntegrationTests.java index e898eb5..525d95e 100644 --- a/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientLocalRegionIntegrationTests.java +++ b/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientLocalRegionIntegrationTests.java @@ -45,6 +45,8 @@ import org.springframework.test.context.junit4.SpringRunner; public class ConcurrentSessionOperationsUsingClientLocalRegionIntegrationTests extends AbstractConcurrentSessionOperationsIntegrationTests { + private static final String GEMFIRE_LOG_LEVEL = "error"; + @Test public void concurrentLocalSessionAccessIsCorrect() throws Throwable { TestFramework.runOnce(new ConcurrentLocalSessionAccessTestCase(this)); @@ -109,7 +111,7 @@ public class ConcurrentSessionOperationsUsingClientLocalRegionIntegrationTests } } - @ClientCacheApplication(logLevel = "error") + @ClientCacheApplication(logLevel = GEMFIRE_LOG_LEVEL) @EnableGemFireHttpSession( clientRegionShortcut = ClientRegionShortcut.LOCAL, poolName = "DEFAULT", diff --git a/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientProxyRegionIntegrationTests.java b/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientProxyRegionIntegrationTests.java index 6e3d0b6..20add1b 100644 --- a/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientProxyRegionIntegrationTests.java +++ b/spring-session-data-geode/src/integration-test/java/org/springframework/session/data/gemfire/ConcurrentSessionOperationsUsingClientProxyRegionIntegrationTests.java @@ -44,18 +44,24 @@ import org.springframework.test.context.junit4.SpringRunner; public class ConcurrentSessionOperationsUsingClientProxyRegionIntegrationTests extends AbstractConcurrentSessionOperationsIntegrationTests { + private static final String GEMFIRE_LOG_LEVEL = "error"; + @BeforeClass public static void startGemFireServer() throws IOException { startGemFireServer(GemFireServerConfiguration.class); } - @ClientCacheApplication(logLevel = "error", subscriptionEnabled = true) - @EnableGemFireHttpSession(clientRegionShortcut = ClientRegionShortcut.PROXY, poolName = "DEFAULT") + @ClientCacheApplication(logLevel = GEMFIRE_LOG_LEVEL, subscriptionEnabled = true) + @EnableGemFireHttpSession( + clientRegionShortcut = ClientRegionShortcut.PROXY, + poolName = "DEFAULT", + sessionSerializerBeanName = GemFireHttpSessionConfiguration.SESSION_DATA_SERIALIZER_BEAN_NAME + ) static class GemFireClientConfiguration { } @CacheServerApplication( name = "ConcurrentSessionOperationsUsingClientProxyRegionIntegrationTests", - logLevel = "error" + logLevel = GEMFIRE_LOG_LEVEL ) @EnableGemFireHttpSession( sessionSerializerBeanName = GemFireHttpSessionConfiguration.SESSION_DATA_SERIALIZER_BEAN_NAME