HHH-6662, HHH-7445 - Fix and test

This commit is contained in:
Lukasz Antoniak 2012-07-25 23:23:17 +02:00
parent 183c914f57
commit 414e6111a8
9 changed files with 255 additions and 5 deletions

View File

@ -285,7 +285,7 @@ public abstract class AbstractPropertyHolder implements PropertyHolder {
Map<String, JoinTable> currentJoinTableOverride = buildJoinTableOverride( current, getPath() );
currentOverride.putAll( columnOverride ); //subclasses have precedence over superclasses
currentJoinOverride.putAll( joinColumnOverride ); //subclasses have precedence over superclasses
currentJoinOverride.putAll( joinColumnOverride ); //subclasses have precedence over superclasses
currentJoinTableOverride.putAll( joinTableOverride ); //subclasses have precedence over superclasses
columnOverride = currentOverride;
joinColumnOverride = currentJoinOverride;
joinTableOverride = currentJoinTableOverride;

View File

@ -33,9 +33,11 @@ public class HibernateSequenceTest extends BaseCoreFunctionalTestCase {
protected void configure(Configuration cfg) {
super.configure( cfg );
cfg.addResource( "org/hibernate/test/annotations/id/sequences/orm.xml" );
cfg.setProperty(
Environment.URL, cfg.getProperty( Environment.URL ) + ";INIT=CREATE SCHEMA IF NOT EXISTS " + SCHEMA_NAME
);
}
@Override
protected String createSecondSchema() {
return SCHEMA_NAME;
}
@Test

View File

@ -0,0 +1,58 @@
package org.hibernate.test.annotations.override;
import java.util.Iterator;
import org.junit.Assert;
import org.junit.Test;
import org.hibernate.dialect.H2Dialect;
import org.hibernate.mapping.Table;
import org.hibernate.test.util.SchemaUtil;
import org.hibernate.testing.RequiresDialect;
import org.hibernate.testing.TestForIssue;
import org.hibernate.testing.junit4.BaseCoreFunctionalTestCase;
/**
* @author Lukasz Antoniak (lukasz dot antoniak at gmail dot com)
*/
@RequiresDialect({ H2Dialect.class })
@TestForIssue(jiraKey = "HHH-6662")
public class AssociationOverrideSchemaTest extends BaseCoreFunctionalTestCase {
public static final String SCHEMA_NAME = "OTHER_SCHEMA";
public static final String TABLE_NAME = "BLOG_TAGS";
public static final String ID_COLUMN_NAME = "BLOG_ID";
public static final String VALUE_COLUMN_NAME = "BLOG_TAG";
@Override
protected Class<?>[] getAnnotatedClasses() {
return new Class[] { Entry.class, BlogEntry.class };
}
@Override
protected String createSecondSchema() {
return SCHEMA_NAME;
}
@Test
public void testSchemaName() {
Iterator<Table> tableIterator = configuration().getTableMappings();
while ( tableIterator.hasNext() ) {
Table table = tableIterator.next();
if ( TABLE_NAME.equals( table.getName() ) ) {
Assert.assertEquals( SCHEMA_NAME, table.getSchema() );
return;
}
}
Assert.fail();
}
@Test
public void testJoinColumnName() {
Assert.assertTrue( SchemaUtil.isColumnPresent( TABLE_NAME, ID_COLUMN_NAME, configuration() ) );
}
@Test
public void testColumnName() {
Assert.assertTrue( SchemaUtil.isColumnPresent( TABLE_NAME, VALUE_COLUMN_NAME, configuration() ) );
}
}

View File

@ -0,0 +1,56 @@
package org.hibernate.test.annotations.override;
import javax.persistence.AssociationOverride;
import javax.persistence.AttributeOverride;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.JoinColumn;
import javax.persistence.JoinTable;
import javax.persistence.Table;
/**
* @author Lukasz Antoniak (lukasz dot antoniak at gmail dot com)
*/
@Entity
@Table(schema = AssociationOverrideSchemaTest.SCHEMA_NAME)
@AssociationOverride(name = "tags",
joinTable = @JoinTable(name = AssociationOverrideSchemaTest.TABLE_NAME,
joinColumns = @JoinColumn(name = AssociationOverrideSchemaTest.ID_COLUMN_NAME),
schema = AssociationOverrideSchemaTest.SCHEMA_NAME))
@AttributeOverride(name = "tags", column = @Column(name = AssociationOverrideSchemaTest.VALUE_COLUMN_NAME))
public class BlogEntry extends Entry {
private String text;
@Override
public boolean equals(Object o) {
if ( this == o ) return true;
if ( !( o instanceof BlogEntry ) ) return false;
if ( !super.equals( o ) ) return false;
BlogEntry blogEntry = (BlogEntry) o;
if ( text != null ? !text.equals( blogEntry.text ) : blogEntry.text != null ) return false;
return true;
}
@Override
public int hashCode() {
int result = super.hashCode();
result = 31 * result + ( text != null ? text.hashCode() : 0 );
return result;
}
@Override
public String toString() {
return "BlogEntry(" + super.toString() + ", text = " + text + ")";
}
public String getText() {
return text;
}
public void setText(String text) {
this.text = text;
}
}

View File

@ -0,0 +1,73 @@
package org.hibernate.test.annotations.override;
import java.io.Serializable;
import java.util.HashSet;
import java.util.Set;
import javax.persistence.Column;
import javax.persistence.ElementCollection;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.JoinTable;
import javax.persistence.MappedSuperclass;
import org.hibernate.annotations.Fetch;
import org.hibernate.annotations.FetchMode;
/**
* @author Lukasz Antoniak (lukasz dot antoniak at gmail dot com)
*/
@MappedSuperclass
public abstract class Entry implements Serializable {
@Id
@GeneratedValue
private Long id;
@ElementCollection(fetch = FetchType.EAGER)
@JoinTable(name = "TAGS", joinColumns = @JoinColumn(name = "ID"))
@Column(name = "KEYWORD")
@Fetch(FetchMode.JOIN)
private Set<String> tags = new HashSet<String>();
@Override
public boolean equals(Object o) {
if ( this == o ) return true;
if ( !( o instanceof Entry ) ) return false;
Entry entry = (Entry) o;
if ( id != null ? !id.equals( entry.id ) : entry.id != null ) return false;
if ( tags != null ? !tags.equals( entry.tags ) : entry.tags != null ) return false;
return true;
}
@Override
public int hashCode() {
int result = id != null ? id.hashCode() : 0;
result = 31 * result + ( tags != null ? tags.hashCode() : 0 );
return result;
}
@Override
public String toString() {
return "Entry(id = " + id + ", tags = " + tags + ")";
}
public Long getId() {
return id;
}
public void setId(Long id) {
this.id = id;
}
public Set<String> getTags() {
return tags;
}
public void setTags(Set<String> tags) {
this.tags = tags;
}
}

View File

@ -35,6 +35,7 @@ import org.jboss.logging.Logger;
import org.hibernate.cfg.Configuration;
import org.hibernate.dialect.Dialect;
import org.hibernate.dialect.H2Dialect;
import org.hibernate.jpa.test.PersistenceUnitDescriptorAdapter;
import org.hibernate.engine.transaction.internal.jta.JtaStatusHelper;
import org.hibernate.envers.AuditReader;
@ -53,6 +54,7 @@ import org.junit.After;
import org.hibernate.testing.AfterClassOnce;
import org.hibernate.testing.BeforeClassOnce;
import org.hibernate.testing.jta.TestingJtaPlatformImpl;
import org.hibernate.testing.junit4.Helper;
/**
* @author Strong Liu (stliu@hibernate.org)
@ -114,6 +116,13 @@ public abstract class BaseEnversJPAFunctionalTestCase extends AbstractEnversTest
if ( createSchema() ) {
settings.put( org.hibernate.cfg.AvailableSettings.HBM2DDL_AUTO, "create-drop" );
final String secondSchemaName = createSecondSchema();
if ( StringHelper.isNotEmpty( secondSchemaName ) ) {
if ( !( getDialect() instanceof H2Dialect ) ) {
throw new UnsupportedOperationException( "Only H2 dialect supports creation of second schema." );
}
Helper.createH2Schema( secondSchemaName, settings );
}
}
if ( StringHelper.isNotEmpty( getAuditStrategy() ) ) {
@ -196,6 +205,15 @@ public abstract class BaseEnversJPAFunctionalTestCase extends AbstractEnversTest
protected boolean createSchema() {
return true;
}
/**
* Feature supported only by H2 dialect.
* @return Provide not empty name to create second schema.
*/
protected String createSecondSchema() {
return null;
}
protected boolean isAudit() {
return true;
}

View File

@ -29,10 +29,14 @@ public class DifferentDBSchemaTest extends BaseEnversJPAFunctionalTestCase {
super.addConfigOptions(options);
// Creates new schema after establishing connection
options.putAll(Environment.getProperties());
options.put(Environment.URL, options.get(Environment.URL) + ";INIT=CREATE SCHEMA IF NOT EXISTS " + SCHEMA_NAME);
options.put("org.hibernate.envers.default_schema", SCHEMA_NAME);
}
@Override
protected String createSecondSchema() {
return SCHEMA_NAME;
}
@Override
protected Class<?>[] getAnnotatedClasses() {
return new Class[] { StrTestEntity.class };

View File

@ -45,8 +45,10 @@ import org.hibernate.cfg.Configuration;
import org.hibernate.cfg.Environment;
import org.hibernate.cfg.Mappings;
import org.hibernate.dialect.Dialect;
import org.hibernate.dialect.H2Dialect;
import org.hibernate.engine.spi.SessionFactoryImplementor;
import org.hibernate.engine.spi.SessionImplementor;
import org.hibernate.internal.util.StringHelper;
import org.hibernate.internal.util.config.ConfigurationHelper;
import org.hibernate.jdbc.AbstractReturningWork;
import org.hibernate.jdbc.Work;
@ -178,6 +180,13 @@ public abstract class BaseCoreFunctionalTestCase extends BaseUnitTestCase {
configuration.setProperty( AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS, "true" );
if ( createSchema() ) {
configuration.setProperty( Environment.HBM2DDL_AUTO, "create-drop" );
final String secondSchemaName = createSecondSchema();
if ( StringHelper.isNotEmpty( secondSchemaName ) ) {
if ( !( getDialect() instanceof H2Dialect ) ) {
throw new UnsupportedOperationException( "Only H2 dialect supports creation of second schema." );
}
Helper.createH2Schema( secondSchemaName, configuration );
}
}
configuration.setProperty( Environment.DIALECT, getDialect().getClass().getName() );
return configuration;
@ -350,6 +359,14 @@ public abstract class BaseCoreFunctionalTestCase extends BaseUnitTestCase {
return true;
}
/**
* Feature supported only by H2 dialect.
* @return Provide not empty name to create second schema.
*/
protected String createSecondSchema() {
return null;
}
protected boolean rebuildSessionFactoryOnError() {
return true;
}

View File

@ -25,10 +25,13 @@ package org.hibernate.testing.junit4;
import java.lang.annotation.Annotation;
import java.lang.reflect.Method;
import java.util.Map;
import org.junit.runners.model.FrameworkMethod;
import org.junit.runners.model.TestClass;
import org.hibernate.cfg.Configuration;
import org.hibernate.cfg.Environment;
import org.hibernate.testing.FailureExpected;
/**
@ -98,4 +101,23 @@ public class Helper {
.toString();
}
/**
* @see #createH2Schema(String, Map)
*/
public static void createH2Schema(String schemaName, Configuration cfg) {
createH2Schema( schemaName, cfg.getProperties() );
}
/**
* Create additional H2 schema.
*
* @param schemaName New schema name.
* @param settings Current settings.
*/
public static void createH2Schema(String schemaName, Map settings) {
settings.put(
Environment.URL,
settings.get( Environment.URL ) + ";INIT=CREATE SCHEMA IF NOT EXISTS " + schemaName
);
}
}