DATAJPA-1822 - skip optional detection with EclipseLink

Author: pblanchardie

src/test/java/org/springframework/data/jpa/repository/query/EclipseLinkQueryUtilsIntegrationTests.java

@@ -27,4 +27,8 @@ int getNumberOfJoinsAfterCreatingAPath() {
 		return 1;
 	}
 
+	boolean isSkipOptionalDetection() {
+		return true;
+	}
+
 }

src/test/java/org/springframework/data/jpa/repository/query/QueryUtilsIntegrationTests.java

@@ -118,6 +118,9 @@ void createsJoinForOptionalOneToOneInReverseDirection() {
 	@Test // DATAJPA-1822
 	void createsLeftJoinForOptionalToOneWithNestedNonOptional() {
 
+		if(isSkipOptionalDetection())
+			return;
+
 		CriteriaBuilder builder = em.getCriteriaBuilder();
 		CriteriaQuery<Invoice> query = builder.createQuery(Invoice.class);
 		Root<Invoice> root = query.from(Invoice.class);
@@ -134,6 +137,9 @@ void createsLeftJoinForOptionalToOneWithNestedNonOptional() {
 	@Test // DATAJPA-1822
 	void createsLeftJoinForNonOptionalToOneWithNestedOptional() {
 
+		if(isSkipOptionalDetection())
+			return;
+
 		CriteriaBuilder builder = em.getCriteriaBuilder();
 		CriteriaQuery<InvoiceItem> query = builder.createQuery(InvoiceItem.class);
 		Root<InvoiceItem> root = query.from(InvoiceItem.class);
@@ -153,6 +159,9 @@ void createsLeftJoinForNonOptionalToOneWithNestedOptional() {
 	@Test // DATAJPA-1822
 	void reusesLeftJoinForNonOptionalToOneWithNestedOptional() {
 
+		if(isSkipOptionalDetection())
+			return;
+
 		CriteriaBuilder builder = em.getCriteriaBuilder();
 		CriteriaQuery<InvoiceItem> query = builder.createQuery(InvoiceItem.class);
 		Root<InvoiceItem> root = query.from(InvoiceItem.class);
@@ -177,6 +186,9 @@ void reusesLeftJoinForNonOptionalToOneWithNestedOptional() {
 	@Test // DATAJPA-1822
 	void reusesInnerJoinForNonOptionalToOneWithNestedOptional() {
 
+		if(isSkipOptionalDetection())
+			return;
+
 		CriteriaBuilder builder = em.getCriteriaBuilder();
 		CriteriaQuery<InvoiceItem> query = builder.createQuery(InvoiceItem.class);
 		Root<InvoiceItem> root = query.from(InvoiceItem.class);
@@ -332,6 +344,10 @@ int getNumberOfJoinsAfterCreatingAPath() {
 		return 0;
 	}
 
+	boolean isSkipOptionalDetection() {
+		return false;
+	}
+
 	private Set<Join<?, ?>> getNonInnerJoins(Root<?> root) {
 
 		return getNonInnerJoins((From<?, ?>) root);