Change default for new_generator_mappings to true

Closes gh-7612
pull/5720/merge
Madhura Bhave 8 years ago
parent aff1f18fe2
commit 735dbc4583

@ -202,7 +202,7 @@ public class JpaProperties {
this.useNewIdGeneratorMappings.toString()); this.useNewIdGeneratorMappings.toString());
} }
else if (!result.containsKey(USE_NEW_ID_GENERATOR_MAPPINGS)) { else if (!result.containsKey(USE_NEW_ID_GENERATOR_MAPPINGS)) {
result.put(USE_NEW_ID_GENERATOR_MAPPINGS, "false"); result.put(USE_NEW_ID_GENERATOR_MAPPINGS, "true");
} }
} }

@ -108,17 +108,17 @@ public class JpaPropertiesTests {
Map<String, String> hibernateProperties = properties Map<String, String> hibernateProperties = properties
.getHibernateProperties(mockStandaloneDataSource()); .getHibernateProperties(mockStandaloneDataSource());
assertThat(hibernateProperties) assertThat(hibernateProperties)
.containsEntry(AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS, "false"); .containsEntry(AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS, "true");
} }
@Test @Test
public void useNewIdGeneratorMappingsTrue() throws Exception { public void useNewIdGeneratorMappingsFalse() throws Exception {
JpaProperties properties = load( JpaProperties properties = load(
"spring.jpa.hibernate.use-new-id-generator-mappings:true"); "spring.jpa.hibernate.use-new-id-generator-mappings:false");
Map<String, String> hibernateProperties = properties Map<String, String> hibernateProperties = properties
.getHibernateProperties(mockStandaloneDataSource()); .getHibernateProperties(mockStandaloneDataSource());
assertThat(hibernateProperties) assertThat(hibernateProperties)
.containsEntry(AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS, "true"); .containsEntry(AvailableSettings.USE_NEW_ID_GENERATOR_MAPPINGS, "false");
} }
@Test @Test

Loading…
Cancel
Save