jakartaee/persistence#457 - add union/intersect/except to JPQL and cr…

…iteria Fixed failing tests, code cleanup. Signed-off-by: Tomáš Kraus <[email protected]>
Tomas-Kraus · Nov 15, 2023 · 990b473 · 990b473
1 parent 487c745
commit 990b473
Show file tree

Hide file tree

Showing 16 changed files with 179 additions and 64 deletions.
diff --git a/...ork/src/main/java/org/eclipse/persistence/testing/framework/TogglingFastTableCreator.java b/...ork/src/main/java/org/eclipse/persistence/testing/framework/TogglingFastTableCreator.java
@@ -187,7 +187,7 @@ protected static FieldDefinition createNumericPk(
         field.setSize(size);
         field.setShouldAllowNull(false);
         field.setIsPrimaryKey(true);
-        field.setUnique(true);
+        field.setUnique(false);
         field.setIsIdentity(true);
         return field;
     }
@@ -210,7 +210,6 @@ protected static FieldDefinition createNumericPk(final String name) {
      * @param fkName Foreign key name (e.g. {@code "MY_TABLE.ID"}.
      * @param allowNull Allow {@code null} values for column.
      * @return Initialized {@link FieldDefinition} instance.
-     * @noinspection SameParameterValue
      */
     protected static FieldDefinition createNumericFk(
             final String name, final int size, final String fkName, final boolean allowNull) {

diff --git a/...e.core/src/main/java/org/eclipse/persistence/internal/queries/JoinedAttributeManager.java b/...e.core/src/main/java/org/eclipse/persistence/internal/queries/JoinedAttributeManager.java
@@ -845,6 +845,13 @@ protected Expression prepareJoinExpression(Expression expression, AbstractSessio
         }
         objectExpression.getBuilder().setSession(session.getRootSession(null));
 
+        // Sometimes current expression builder does not support expression class mapping.
+        // Try ExpressionBuilder for the descriptor's class
+        if (objectExpression.getMapping() == null && objectExpression.getBuilder().getQueryClass() != descriptor.getJavaClass()) {
+            objectExpression = (QueryKeyExpression)objectExpression.rebuildOn(new ExpressionBuilder(descriptor.getJavaClass()));
+            objectExpression.getBuilder().setSession(session.getRootSession(null));
+        }
+
         // Can only join relationships.
         if ((objectExpression.getMapping() == null) || (!objectExpression.getMapping().isJoiningSupported())) {
             throw QueryException.mappingForExpressionDoesNotSupportJoining(objectExpression);

diff --git a/...a/org/eclipse/persistence/testing/models/jpa/persistence32/Persistence32TableCreator.java b/...a/org/eclipse/persistence/testing/models/jpa/persistence32/Persistence32TableCreator.java
@@ -9,7 +9,6 @@
  *
  * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
  */
-
 package org.eclipse.persistence.testing.models.jpa.persistence32;
 
 import org.eclipse.persistence.testing.framework.TogglingFastTableCreator;
@@ -19,18 +18,28 @@ public class Persistence32TableCreator extends TogglingFastTableCreator {
 
     public Persistence32TableCreator() {
         setName("Persistence32Project");
+        addTableDefinition(buildTeamTable());
         addTableDefinition(buildTrainerTable());
         addTableDefinition(buildTypeTable());
         addTableDefinition(buildPokemonTable());
         addTableDefinition(buildPokemonTypeTable());
         addTableDefinition(buildSyntaxEntityTable());
     }
 
+    public static TableDefinition buildTeamTable() {
+        TableDefinition table = new TableDefinition();
+        table.setName("PERSISTENCE32_TEAM");
+        table.addField(createNumericPk("ID"));
+        table.addField(createStringColumn("NAME", 64, false));
+        return table;
+    }
+
     public static TableDefinition buildTrainerTable() {
         TableDefinition table = new TableDefinition();
         table.setName("PERSISTENCE32_TRAINER");
         table.addField(createNumericPk("ID"));
         table.addField(createStringColumn("NAME", 64, false));
+        table.addField(createNumericFk("TEAM_ID", 15,"PERSISTENCE32_TEAM.ID", false));
         return table;
     }
 

diff --git a/...nce32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Pokemon.java b/...nce32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Pokemon.java
@@ -9,7 +9,6 @@
  *
  * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
  */
-
 package org.eclipse.persistence.testing.models.jpa.persistence32;
 
 import java.util.Collection;

diff --git a/.../src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/SyntaxEntity.java b/.../src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/SyntaxEntity.java
@@ -1,3 +1,14 @@
+/*
+ * Copyright (c) 2023 Oracle and/or its affiliates. All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v. 2.0 which is available at
+ * http://www.eclipse.org/legal/epl-2.0,
+ * or the Eclipse Distribution License v. 1.0 which is available at
+ * http://www.eclipse.org/org/documents/edl-v10.php.
+ *
+ * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
+ */
 package org.eclipse.persistence.testing.models.jpa.persistence32;
 
 import java.time.LocalDate;

diff --git a/...stence32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Team.java b/...stence32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Team.java
@@ -0,0 +1,72 @@
+/*
+ * Copyright (c) 2023 Oracle and/or its affiliates. All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v. 2.0 which is available at
+ * http://www.eclipse.org/legal/epl-2.0,
+ * or the Eclipse Distribution License v. 1.0 which is available at
+ * http://www.eclipse.org/org/documents/edl-v10.php.
+ *
+ * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
+ */
+package org.eclipse.persistence.testing.models.jpa.persistence32;
+
+import java.util.Objects;
+
+import jakarta.persistence.Entity;
+import jakarta.persistence.Id;
+import jakarta.persistence.NamedNativeQuery;
+import jakarta.persistence.Table;
+
+@Entity
+@Table(name="PERSISTENCE32_TEAM")
+@NamedNativeQuery(name="Team.deleteAll", query="DELETE FROM PERSISTENCE32_TEAM")
+public class Team {
+
+    // ID is assigned in tests to avoid collisions
+    @Id
+    private int id;
+
+    private String name;
+
+    public Team() {
+    }
+
+    public Team(int id, String name) {
+        this.id = id;
+        this.name = name;
+    }
+
+    public int getId() {
+        return id;
+    }
+
+    public void setId(int id) {
+        this.id = id;
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    @Override
+    public boolean equals(Object obj) {
+        if (obj == null || obj.getClass() != this.getClass()) {
+            return false;
+        }
+        return id == ((Team) obj).id
+                && Objects.equals(name, ((Team) obj).name);
+    }
+
+    @Override
+    public int hashCode() {
+        int result = Objects.hash(id, name);
+        return result;
+    }
+
+
+}
diff --git a/...nce32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Trainer.java b/...nce32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Trainer.java
@@ -1,12 +1,32 @@
+/*
+ * Copyright (c) 2023 Oracle and/or its affiliates. All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v. 2.0 which is available at
+ * http://www.eclipse.org/legal/epl-2.0,
+ * or the Eclipse Distribution License v. 1.0 which is available at
+ * http://www.eclipse.org/org/documents/edl-v10.php.
+ *
+ * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
+ */
 package org.eclipse.persistence.testing.models.jpa.persistence32;
 
+import java.util.List;
 import java.util.Objects;
 
 import jakarta.persistence.Entity;
 import jakarta.persistence.Id;
+import jakarta.persistence.JoinColumn;
+import jakarta.persistence.ManyToOne;
 import jakarta.persistence.NamedNativeQuery;
 import jakarta.persistence.NamedQuery;
+import jakarta.persistence.OneToMany;
 import jakarta.persistence.Table;
+import org.eclipse.persistence.annotations.JoinFetch;
+import org.eclipse.persistence.annotations.JoinFetchType;
+import org.eclipse.persistence.annotations.Property;
+
+import static jakarta.persistence.FetchType.EAGER;
 
 @Entity
 @Table(name="PERSISTENCE32_TRAINER")
@@ -20,12 +40,19 @@ public class Trainer {
 
     private String name;
 
+    @ManyToOne
+    private Team team;
+
+    @OneToMany(mappedBy = "trainer")
+    private List<Pokemon> pokemons;
+
     public Trainer() {
     }
 
-    public Trainer(int id, String name) {
+    public Trainer(int id, String name, Team team) {
         this.id = id;
         this.name = name;
+        this.team = team;
     }
 
     public int getId() {
@@ -44,6 +71,22 @@ public void setName(String name) {
         this.name = name;
     }
 
+    public Team getTeam() {
+        return team;
+    }
+
+    public void setTeam(Team team) {
+        this.team = team;
+    }
+
+    public List<Pokemon> getPokemons() {
+        return pokemons;
+    }
+
+    public void setPokemons(List<Pokemon> pokemons) {
+        this.pokemons = pokemons;
+    }
+
     @Override
     public boolean equals(Object obj) {
         if (obj == null || obj.getClass() != this.getClass()) {

diff --git a/...stence32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Type.java b/...stence32/src/main/java/org/eclipse/persistence/testing/models/jpa/persistence32/Type.java
@@ -9,7 +9,6 @@
  *
  * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
  */
-
 package org.eclipse.persistence.testing.models.jpa.persistence32;
 
 import java.util.Objects;

diff --git a/...lipselink.jpa.testapps/jpa.test.persistence32/src/main/resources/META-INF/persistence.xml b/...lipselink.jpa.testapps/jpa.test.persistence32/src/main/resources/META-INF/persistence.xml
@@ -11,7 +11,6 @@
     SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
 
 -->
-
 <persistence xmlns="http://xmlns.jcp.org/xml/ns/persistence"
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://xmlns.jcp.org/xml/ns/persistence
@@ -21,6 +20,7 @@
     <!-- This is the default persistence unit -->
     <persistence-unit name="persistence32" transaction-type="RESOURCE_LOCAL">
         <provider>org.eclipse.persistence.jpa.PersistenceProvider</provider>
+        <class>org.eclipse.persistence.testing.models.jpa.persistence32.Team</class>
         <class>org.eclipse.persistence.testing.models.jpa.persistence32.Trainer</class>
         <class>org.eclipse.persistence.testing.models.jpa.persistence32.Pokemon</class>
         <class>org.eclipse.persistence.testing.models.jpa.persistence32.Type</class>

diff --git a/...rc/test/java/org/eclipse/persistence/testing/tests/jpa/persistence32/AbstractPokemon.java b/...rc/test/java/org/eclipse/persistence/testing/tests/jpa/persistence32/AbstractPokemon.java
@@ -9,7 +9,6 @@
  *
  * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
  */
-
 package org.eclipse.persistence.testing.tests.jpa.persistence32;
 
 import java.util.HashMap;
@@ -21,6 +20,7 @@
 import org.eclipse.persistence.jpa.JpaEntityManagerFactory;
 import org.eclipse.persistence.testing.framework.jpa.junit.JUnitTestCase;
 import org.eclipse.persistence.testing.models.jpa.persistence32.Persistence32TableCreator;
+import org.eclipse.persistence.testing.models.jpa.persistence32.Team;
 import org.eclipse.persistence.testing.models.jpa.persistence32.Trainer;
 import org.eclipse.persistence.testing.models.jpa.persistence32.Type;
 
@@ -29,12 +29,18 @@
  */
 public abstract class AbstractPokemon extends JUnitTestCase {
 
+    // Trainer's teams
+    static final Team[] TEAMS = new Team[] {
+            null, // Skip array index 0
+            new Team(1, "Kanto")
+    };
+
     // Pokemon trainers. Array index is ID value.
     // Value of ID = 0 does not exist so it's array instance is set to null.
     static final Trainer[] TRAINERS = new Trainer[] {
             null, // Skip array index 0
-            new Trainer(1, "Ash"),
-            new Trainer(2, "Brock")
+            new Trainer(1, "Ash", TEAMS[1]),
+            new Trainer(2, "Brock", TEAMS[1])
     };
 
     // Pokemon types. Array index is ID value.
@@ -125,6 +131,9 @@ public void testSetup() {
         new Persistence32TableCreator().replaceTables(JUnitTestCase.getServerSession(getPersistenceUnitName()));
         clearCache();
         emf.runInTransaction(em -> {
+            for (int i = 1; i < TEAMS.length; i++) {
+                em.persist(TEAMS[i]);
+            }
             for (int i = 1; i < TRAINERS.length; i++) {
                 em.persist(TRAINERS[i]);
             }
@@ -144,6 +153,7 @@ public void testCleanup() {
             em.createNamedQuery("Pokemon.deleteAll").executeUpdate();
             em.createNamedQuery("Type.deleteAll").executeUpdate();
             em.createNamedQuery("Trainer.deleteAll").executeUpdate();
+            em.createNamedQuery("Team.deleteAll").executeUpdate();
         });
     }
 

diff --git a/...est/java/org/eclipse/persistence/testing/tests/jpa/persistence32/CriteriaBuilderTest.java b/...est/java/org/eclipse/persistence/testing/tests/jpa/persistence32/CriteriaBuilderTest.java
@@ -9,7 +9,6 @@
  *
  * SPDX-License-Identifier: EPL-2.0 OR BSD-3-Clause
  */
-
 package org.eclipse.persistence.testing.tests.jpa.persistence32;
 
 import java.time.LocalDate;