METAGEN-81 Formatting and moving test into its own method annotated w…

…ith @testForissue
hibernate · Nov 11, 2013 · 0ff88cf · 0ff88cf
1 parent 7bb932f
commit 0ff88cf
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 13 deletions.
diff --git a/...del-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/AccessTypeTest.java b/...del-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/AccessTypeTest.java
@@ -23,6 +23,7 @@
 import org.testng.annotations.Test;
 
 import org.hibernate.jpamodelgen.test.util.CompilationTest;
+import org.hibernate.jpamodelgen.test.util.TestForIssue;
 import org.hibernate.jpamodelgen.test.util.TestUtil;
 
 import static org.hibernate.jpamodelgen.test.util.TestUtil.assertAbsenceOfFieldInMetamodelFor;
@@ -67,12 +68,13 @@ public void testInheritedAccessTypeForEmbeddable() {
 		assertAbsenceOfFieldInMetamodelFor(
 				Pet.class, "nonPersistent", "Collection of embeddable not taken care of"
 		);
-
-		// see METAGEN-81
-		// access type should be inherited from the position of the @Id field annotation from the root entity
-		// via the @Embedded annotation
+	}
+
+	@Test
+	@TestForIssue( jiraKey = " METAGEN-81")
+	public void testAccessTypeForEmbeddableDeterminedByIdAnnotationInRootEntity() {
 		assertPresenceOfFieldInMetamodelFor( Hotel.class, "webmaster",
-				"Access type should be inherited from the hierarchy" );
+				"Access type should be inherited position of the @Id field annotation in the root entity" );
 	}
 
 	@Test

diff --git a/...ng/metamodel-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/Hotel.java b/...ng/metamodel-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/Hotel.java
@@ -16,7 +16,6 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
  * MA  02110-1301, USA.
  */
-
 package org.hibernate.jpamodelgen.test.accesstype;
 
 import javax.persistence.Embeddable;
@@ -27,7 +26,7 @@
  */
 @Embeddable
 public class Hotel {
-	
+
 	@OneToOne
 	private User webmaster;
 
@@ -38,5 +37,4 @@ public User getWebmaster() {
 	public void setWebmaster(User webmaster) {
 		this.webmaster = webmaster;
 	}
-
 }
diff --git a/...etamodel-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/HotelRoom.java b/...etamodel-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/HotelRoom.java
@@ -16,7 +16,6 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
  * MA  02110-1301, USA.
  */
-
 package org.hibernate.jpamodelgen.test.accesstype;
 
 import javax.persistence.Embedded;
@@ -38,5 +37,4 @@ public Hotel getHotel() {
 	public void setHotel(Hotel hotel) {
 		this.hotel = hotel;
 	}
-
 }
diff --git a/...ing/metamodel-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/Room.java b/...ing/metamodel-generator/src/test/java/org/hibernate/jpamodelgen/test/accesstype/Room.java
@@ -16,7 +16,6 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
  * MA  02110-1301, USA.
  */
-
 package org.hibernate.jpamodelgen.test.accesstype;
 
 import javax.persistence.Entity;
@@ -27,7 +26,6 @@
  */
 @Entity
 public class Room {
-
 	@Id
 	private Integer id;
 
@@ -38,5 +36,4 @@ public Integer getId() {
 	public void setId(Integer id) {
 		this.id = id;
 	}
-
 }