Do not (incorrectly) set the value of @GeneratedValue IDs in tests

It causes failures on ORM 6.6, because that's forbidden. Previous behavior is unclear but at the very least wasn't clearly defined. See https://hibernate.zulipchat.com/#narrow/stream/132094-hibernate-orm-dev/topic/EntityManager.2Emerge/near/447248421
quarkusio · Jun 28, 2024 · c98c96e · c98c96e
1 parent 444d794
commit c98c96e
Show file tree

Hide file tree

Showing 8 changed files with 14 additions and 23 deletions.
diff --git a/...rnate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/quoting_strategies/Group.java b/...rnate-orm/deployment/src/test/java/io/quarkus/hibernate/orm/quoting_strategies/Group.java
@@ -2,8 +2,6 @@
 
 import jakarta.persistence.Column;
 import jakarta.persistence.Entity;
-import jakarta.persistence.GeneratedValue;
-import jakarta.persistence.GenerationType;
 import jakarta.persistence.Id;
 import jakarta.persistence.Table;
 
@@ -24,7 +22,6 @@ public class Group {
     private String value;
 
     @Id
-    @GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "groupSeq")
     public Long getId() {
         return id;
     }

diff --git a/...ain/java/io/quarkus/hibernate/reactive/panache/common/runtime/CommonPanacheQueryImpl.java b/...ain/java/io/quarkus/hibernate/reactive/panache/common/runtime/CommonPanacheQueryImpl.java
@@ -307,7 +307,7 @@ public <T extends Entity> Multi<T> stream() {
         return (Multi<T>) results.toMulti().flatMap(list -> {
             return Multi.createFrom().iterable(list);
         });
-    }git s
+    }
 
     @SuppressWarnings("unchecked")
     public <T extends Entity> Uni<T> firstResult() {

diff --git a/...a-rest/deployment/src/test/java/io/quarkus/spring/data/rest/CrudAndPagedResourceTest.java b/...a-rest/deployment/src/test/java/io/quarkus/spring/data/rest/CrudAndPagedResourceTest.java
@@ -342,8 +342,8 @@ void shouldCreateHal() {
     void shouldCreateAndUpdate() {
         Response createResponse = given().accept("application/json")
                 .and().contentType("application/json")
-                .and().body("{\"id\": \"101\", \"name\": \"test-update-create\"}")
-                .when().put("/crud-and-paged-records/101")
+                .and().body("{\"name\": \"test-update-create\"}")
+                .when().post("/crud-and-paged-records/")
                 .thenReturn();
         assertThat(createResponse.statusCode()).isEqualTo(201);
 
@@ -370,8 +370,8 @@ void shouldCreateAndUpdate() {
     void shouldCreateAndUpdateHal() {
         Response createResponse = given().accept("application/hal+json")
                 .and().contentType("application/json")
-                .and().body("{\"id\": \"102\", \"name\": \"test-update-create-hal\"}")
-                .when().put("/crud-and-paged-records/102")
+                .and().body("{\"name\": \"test-update-create-hal\"}")
+                .when().post("/crud-and-paged-records/")
                 .thenReturn();
         assertThat(createResponse.statusCode()).isEqualTo(201);
 

diff --git a/...pring-data-rest/deployment/src/test/java/io/quarkus/spring/data/rest/JpaResourceTest.java b/...pring-data-rest/deployment/src/test/java/io/quarkus/spring/data/rest/JpaResourceTest.java
@@ -342,8 +342,8 @@ void shouldCreateHal() {
     void shouldCreateAndUpdate() {
         Response createResponse = given().accept("application/json")
                 .and().contentType("application/json")
-                .and().body("{\"id\": \"101\", \"name\": \"test-update-create\"}")
-                .when().put("/jpa-records/101")
+                .and().body("{\"name\": \"test-update-create\"}")
+                .when().post("/jpa-records/")
                 .thenReturn();
         assertThat(createResponse.statusCode()).isEqualTo(201);
 
@@ -370,8 +370,8 @@ void shouldCreateAndUpdate() {
     void shouldCreateAndUpdateHal() {
         Response createResponse = given().accept("application/hal+json")
                 .and().contentType("application/json")
-                .and().body("{\"id\": \"102\", \"name\": \"test-update-create-hal\"}")
-                .when().put("/jpa-records/102")
+                .and().body("{\"name\": \"test-update-create-hal\"}")
+                .when().post("/jpa-records/")
                 .thenReturn();
         assertThat(createResponse.statusCode()).isEqualTo(201);
 

diff --git a/...st/deployment/src/test/java/io/quarkus/spring/data/rest/crud/DefaultCrudResourceTest.java b/...st/deployment/src/test/java/io/quarkus/spring/data/rest/crud/DefaultCrudResourceTest.java
@@ -149,8 +149,8 @@ void shouldCreateHal() {
     void shouldCreateAndUpdate() {
         Response createResponse = given().accept("application/json")
                 .and().contentType("application/json")
-                .and().body("{\"id\": \"101\", \"name\": \"test-update-create\"}")
-                .when().put("/default-records/101")
+                .and().body("{\"name\": \"test-update-create\"}")
+                .when().post("/default-records/")
                 .thenReturn();
         assertThat(createResponse.statusCode()).isEqualTo(201);
 
@@ -177,8 +177,8 @@ void shouldCreateAndUpdate() {
     void shouldCreateAndUpdateHal() {
         Response createResponse = given().accept("application/hal+json")
                 .and().contentType("application/json")
-                .and().body("{\"id\": \"102\", \"name\": \"test-update-create-hal\"}")
-                .when().put("/default-records/102")
+                .and().body("{\"name\": \"test-update-create-hal\"}")
+                .when().post("/default-records/")
                 .thenReturn();
         assertThat(createResponse.statusCode()).isEqualTo(201);
 

diff --git a/...t/java/io/quarkus/it/hibernate/orm/rest/data/panache/HibernateOrmRestDataPanacheTest.java b/...t/java/io/quarkus/it/hibernate/orm/rest/data/panache/HibernateOrmRestDataPanacheTest.java
@@ -268,7 +268,7 @@ void shouldCreateUpdateAndDeleteBook() {
         Response response = given().accept("application/json")
                 .and().contentType("application/json")
                 .and().body(book.toString())
-                .when().put("/books/100")
+                .when().post("/books/")
                 .thenReturn();
         assertThat(response.statusCode()).isEqualTo(201);
         assertThat(response.header("Location")).isNotEmpty();

diff --git a/...ion-tests/spring-data-jpa/src/main/java/io/quarkus/it/spring/data/jpa/complex/Parent.java b/...ion-tests/spring-data-jpa/src/main/java/io/quarkus/it/spring/data/jpa/complex/Parent.java
@@ -1,14 +1,11 @@
 package io.quarkus.it.spring.data.jpa.complex;
 
 import jakarta.persistence.Entity;
-import jakarta.persistence.GeneratedValue;
-import jakarta.persistence.GenerationType;
 import jakarta.persistence.Id;
 
 @Entity
 public class Parent extends ParentBase {
     @Id
-    @GeneratedValue(strategy = GenerationType.IDENTITY)
     private Long id;
 
     public Parent(Long id, String name, String detail, int age, float test, TestEnum testEnum) {

diff --git a/...on-tests/spring-data-jpa/src/main/java/io/quarkus/it/spring/data/jpa/complex/Parent2.java b/...on-tests/spring-data-jpa/src/main/java/io/quarkus/it/spring/data/jpa/complex/Parent2.java
@@ -1,15 +1,12 @@
 package io.quarkus.it.spring.data.jpa.complex;
 
 import jakarta.persistence.Entity;
-import jakarta.persistence.GeneratedValue;
-import jakarta.persistence.GenerationType;
 import jakarta.persistence.Id;
 
 @Entity
 public class Parent2 extends ParentBase {
 
     @Id
-    @GeneratedValue(strategy = GenerationType.IDENTITY)
     private Long id;
 
     public Parent2(String name, String detail, int age, float test, TestEnum testEnum, Long id) {