HHH-10593 - Avoid NullPointerException with auto_evict_collection_cache and many-to-many

Cherry pick from Janoiro PR

hibernate-core/src/test/java/org/hibernate/test/cache/ManyToManyCollectionCacheEvictionTest.java

@@ -0,0 +1,113 @@
+package org.hibernate.test.cache;
+
+import java.util.ArrayList;
+import java.util.List;
+import javax.persistence.Cacheable;
+import javax.persistence.Entity;
+import javax.persistence.GeneratedValue;
+import javax.persistence.Id;
+import javax.persistence.ManyToMany;
+import javax.persistence.Table;
+
+import org.hibernate.Session;
+import org.hibernate.annotations.Cache;
+import org.hibernate.annotations.CacheConcurrencyStrategy;
+import org.hibernate.cache.internal.CollectionCacheInvalidator;
+import org.hibernate.cfg.Configuration;
+import org.hibernate.cfg.Environment;
+
+import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+
+import static org.junit.Assert.assertEquals;
+
+/**
+ * @author Janario Oliveira
+ */
+public class ManyToManyCollectionCacheEvictionTest extends BaseCoreFunctionalTestCase {
+	@Override
+	protected Class<?>[] getAnnotatedClasses() {
+		return new Class[] {Customer.class, Application.class};
+	}
+
+	@Before
+	public void before() {
+		CollectionCacheInvalidator.PROPAGATE_EXCEPTION = true;
+	}
+
+	@After
+	public void after() {
+		CollectionCacheInvalidator.PROPAGATE_EXCEPTION = false;
+	}
+
+	@Override
+	protected void configure(Configuration cfg) {
+		super.configure( cfg );
+		cfg.setProperty( Environment.AUTO_EVICT_COLLECTION_CACHE, "true" );
+		cfg.setProperty( Environment.USE_SECOND_LEVEL_CACHE, "true" );
+		cfg.setProperty( Environment.USE_QUERY_CACHE, "true" );
+		cfg.setProperty( Environment.CACHE_PROVIDER_CONFIG, "true" );
+	}
+
+	@Test
+	public void testManyToManyPersist() {
+		//if an error happen, it will propagate the exception failing the test case
+		Session s = openSession();
+		s.beginTransaction();
+
+		Application application = new Application();
+		s.save( application );
+
+		Customer customer = new Customer();
+		customer.applications.add( application );
+		s.save( customer );
+
+		s.getTransaction().commit();
+		s.close();
+
+
+		s = openSession();
+
+		assertEquals( 1, s.get( Application.class, application.id ).customers.size() );
+		assertEquals( 1, s.get( Customer.class, customer.id ).applications.size() );
+
+		s.close();
+
+		s = openSession();
+		s.beginTransaction();
+
+		Customer customer2 = new Customer();
+		customer2.applications.add( application );
+		s.save( customer2 );
+
+		s.getTransaction().commit();
+		s.close();
+	}
+
+	@Entity
+	@Table(name = "customer")
+	@Cacheable
+	public static class Customer {
+		@Id
+		@GeneratedValue
+		private Integer id;
+		@ManyToMany
+		@Cache(usage = CacheConcurrencyStrategy.READ_WRITE)
+		private List<Application> applications = new ArrayList<Application>();
+	}
+
+	@Entity
+	@Table(name = "application")
+	@Cacheable
+	public static class Application {
+		@Id
+		@GeneratedValue
+		private Integer id;
+
+		@ManyToMany(mappedBy = "applications")
+		@Cache(usage = CacheConcurrencyStrategy.READ_WRITE)
+		private List<Customer> customers = new ArrayList<Customer>();
+	}
+}