diff --git a/pom.xml b/pom.xml index c7961681..39400c12 100644 --- a/pom.xml +++ b/pom.xml @@ -222,6 +222,14 @@ limitations under the License. 1.5 + + org.apache.maven.plugins + maven-surefire-plugin + + always + + + @@ -319,6 +327,12 @@ limitations under the License. org.apache.maven.plugins maven-pmd-plugin + 2.5 + + 1.5 + true + + diff --git a/src/main/java/org/springframework/data/gemfire/GemfireBeanFactoryLocator.java b/src/main/java/org/springframework/data/gemfire/GemfireBeanFactoryLocator.java index 12ba6ab8..318d8215 100644 --- a/src/main/java/org/springframework/data/gemfire/GemfireBeanFactoryLocator.java +++ b/src/main/java/org/springframework/data/gemfire/GemfireBeanFactoryLocator.java @@ -136,7 +136,8 @@ public class GemfireBeanFactoryLocator implements BeanFactoryLocator, BeanFactor if (!StringUtils.hasText(factoryKey)) { if (!canUseDefaultBeanFactory) throw new IllegalArgumentException( - "a non-null factoryKey needs to be specified as there are more then one factoryKeys available "); + "a non-null factoryKey needs to be specified as there are more then one factoryKeys available; " + + beanFactories.keySet()); factory = defaultFactory; } else { diff --git a/src/test/java/org/springframework/data/gemfire/CacheIntegrationTest.java b/src/test/java/org/springframework/data/gemfire/CacheIntegrationTest.java index 4b58e265..f49e6043 100644 --- a/src/test/java/org/springframework/data/gemfire/CacheIntegrationTest.java +++ b/src/test/java/org/springframework/data/gemfire/CacheIntegrationTest.java @@ -17,6 +17,8 @@ package org.springframework.data.gemfire; +import junit.framework.Assert; + import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.beans.factory.annotation.Autowired; @@ -50,7 +52,7 @@ public class CacheIntegrationTest { public void testCacheWithProps() throws Exception { Cache cache = ctx.getBean("cache-with-props", Cache.class); // the name property seems to be ignored - // Assert.assertEquals("cache-with-props", cache.getDistributedSystem().getName()); + Assert.assertEquals("cache-with-props", cache.getDistributedSystem().getName()); } @Test @@ -61,6 +63,5 @@ public class CacheIntegrationTest { @Test public void testCacheWithXml() throws Exception { Cache cache = ctx.getBean("cache-with-xml", Cache.class); - //Assert.assertEquals("cache-with-props", cache.getDistributedSystem().getName()); } }