OGM-1269 Throw exception when cache configuration is missing

Author: DavideD

infinispan/src/main/java/org/hibernate/ogm/datastore/infinispan/logging/impl/Log.java

@@ -41,4 +41,7 @@ public interface Log extends org.hibernate.ogm.util.impl.Log {
 	@Message(id = 1105, value = "Infinispan Externalizer mistmatch: id [%1$d] was registered but taken " +
 			"by implementation '%2$s'. Expected externalizer: '%3$s' ")
 	HibernateException externalizerIdNotMatchingType(Integer externalizerId, AdvancedExternalizer<?> registeredExternalizer, AdvancedExternalizer expectedExternalizer);
+
+	@Message(id = 1106, value = "Missing configuration for cache '%1$s' and a default configuration was not provided")
+	Exception missingCacheConfiguration(String cacheName);
 }

infinispan/src/main/java/org/hibernate/ogm/datastore/infinispan/persistencestrategy/impl/LocalCacheManager.java

@@ -17,6 +17,8 @@
 import org.hibernate.HibernateException;
 import org.hibernate.engine.transaction.jta.platform.spi.JtaPlatform;
 import org.hibernate.ogm.datastore.infinispan.impl.TransactionManagerLookupDelegator;
+import org.hibernate.ogm.datastore.infinispan.logging.impl.Log;
+import org.hibernate.ogm.datastore.infinispan.logging.impl.LoggerFactory;
 import org.hibernate.ogm.model.key.spi.AssociationKeyMetadata;
 import org.hibernate.ogm.model.key.spi.EntityKeyMetadata;
 import org.hibernate.ogm.model.key.spi.IdSourceKeyMetadata;
@@ -51,6 +53,8 @@
  */
 public abstract class LocalCacheManager<EK, AK, ISK> {
 
+	private static final Log LOG = LoggerFactory.getLogger();
+
 	private final EmbeddedCacheManager cacheManager;
 	private final boolean isProvidedCacheManager;
 
@@ -94,6 +98,9 @@ private static EmbeddedCacheManager createCustomCacheManager(URL configUrl, JtaP
 					if ( originalCfg == null ) {
 						originalCfg = tmpCacheManager.getDefaultCacheConfiguration();
 					}
+					if ( originalCfg == null ) {
+						throw LOG.missingCacheConfiguration( cacheName );
+					}
 					Configuration newCfg;
 					if ( originalCfg.transaction().transactionMode() == TransactionMode.TRANSACTIONAL ) {
 						//Inject our TransactionManager lookup delegate for transactional caches ONLY!