From 25735b453bf003bb647e9a9b462a08d17a8eb941 Mon Sep 17 00:00:00 2001
From: Andrea Boriero <andrea@hibernate.org>
Date: Wed, 31 Jan 2018 10:28:47 +0000
Subject: [PATCH] HHH-1268 - Add test

---
 ...nidirectionalOneToManyIndexColumnTest.java | 110 ++++++++++++++++++
 1 file changed, 110 insertions(+)
 create mode 100644 hibernate-core/src/test/java/org/hibernate/jpa/test/mapping/UnidirectionalOneToManyIndexColumnTest.java

diff --git a/hibernate-core/src/test/java/org/hibernate/jpa/test/mapping/UnidirectionalOneToManyIndexColumnTest.java b/hibernate-core/src/test/java/org/hibernate/jpa/test/mapping/UnidirectionalOneToManyIndexColumnTest.java
new file mode 100644
index 0000000000..e24e6559dd
--- /dev/null
+++ b/hibernate-core/src/test/java/org/hibernate/jpa/test/mapping/UnidirectionalOneToManyIndexColumnTest.java
@@ -0,0 +1,110 @@
+/*
+ * Hibernate, Relational Persistence for Idiomatic Java
+ *
+ * License: GNU Lesser General Public License (LGPL), version 2.1 or later.
+ * See the lgpl.txt file in the root directory or <http://www.gnu.org/licenses/lgpl-2.1.html>.
+ */
+package org.hibernate.jpa.test.mapping;
+
+import java.util.ArrayList;
+import java.util.List;
+import javax.persistence.CascadeType;
+import javax.persistence.Entity;
+import javax.persistence.GeneratedValue;
+import javax.persistence.Id;
+import javax.persistence.OneToMany;
+import javax.persistence.Table;
+
+import org.hibernate.annotations.IndexColumn;
+
+import org.hibernate.testing.TestForIssue;
+import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
+import org.hibernate.testing.transaction.TransactionUtil;
+import org.junit.Test;
+
+import static org.hamcrest.core.Is.is;
+import static org.junit.Assert.assertThat;
+
+/**
+ * @author Andrea Boriero
+ */
+@TestForIssue(jiraKey = "HHH-1268")
+public class UnidirectionalOneToManyIndexColumnTest extends BaseCoreFunctionalTestCase {
+
+	@Override
+	protected Class<?>[] getAnnotatedClasses() {
+		return new Class[] { Parent.class, Child.class };
+	}
+
+	@Test
+	public void testRemovingAChild() {
+		int parentId = TransactionUtil.doInHibernate( this::sessionFactory, session -> {
+			Parent parent = new Parent();
+			parent.getChildren().add( new Child() );
+			parent.getChildren().add( new Child() );
+			parent.getChildren().add( new Child() );
+			session.persist( parent );
+			return parent.getId();
+		} );
+
+		TransactionUtil.doInHibernate( this::sessionFactory, session -> {
+			Parent parent = session.find( Parent.class, parentId );
+			List<Child> children = parent.getChildren();
+			assertThat( children.size(), is( 3 ) );
+			children.remove( 0 );
+			session.persist( parent );
+		} );
+
+		TransactionUtil.doInHibernate( this::sessionFactory, session -> {
+			Parent parent = session.find( Parent.class, parentId );
+			List<Child> children = parent.getChildren();
+			assertThat( children.size(), is( 2 ) );
+		} );
+	}
+
+
+	@Entity
+	@Table(name = "PARENT")
+	public static class Parent {
+
+		@Id
+		@GeneratedValue
+		private int id;
+
+		@OneToMany(targetEntity = Child.class, cascade = CascadeType.ALL)
+		@IndexColumn(name = "position")
+		private List<Child> children = new ArrayList<>();
+
+		public int getId() {
+			return id;
+		}
+
+		public void setId(int id) {
+			this.id = id;
+		}
+
+		public List<Child> getChildren() {
+			return children;
+		}
+
+		public void setChildren(List<Child> children) {
+			this.children = children;
+		}
+	}
+
+	@Entity
+	@Table(name = "CHILD")
+	public static class Child {
+		@Id
+		@GeneratedValue
+		private int id;
+
+		public int getId() {
+			return id;
+		}
+
+		public void setId(int id) {
+			this.id = id;
+		}
+	}
+}