Skip to content

Commit e411109

Browse files
committed
some minor cleanups to tests
1 parent c783603 commit e411109

File tree

6 files changed

+12
-20
lines changed

6 files changed

+12
-20
lines changed

hibernate-core/src/test/java/org/hibernate/orm/test/boot/BootLoggingTests.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
*
3434
* @author Steve Ebersole
3535
*/
36-
@SuppressWarnings("JUnitMalformedDeclaration")
3736
@MessageKeyInspection(
3837
messageKey = "HHH10001005",
3938
logger = @Logger( loggerName = ConnectionInfoLogger.LOGGER_NAME )

hibernate-core/src/test/java/org/hibernate/orm/test/boot/SessionFactoryNamingTests.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
/**
2424
* @author Steve Ebersole
2525
*/
26-
@SuppressWarnings("JUnitMalformedDeclaration")
2726
@MessageKeyInspection(
2827
messageKey = "HHH020277",
2928
logger = @Logger( loggerName = SessionFactoryRegistryMessageLogger.LOGGER_NAME )
@@ -41,10 +40,6 @@ void testExplicitJndiName(SessionFactoryScope scope, MessageKeyWatcher logWatche
4140
assertThat( logWatcher.wasTriggered() ).isTrue();
4241
}
4342

44-
45-
46-
47-
4843
@Test
4944
@DomainModel
5045
@ServiceRegistry( settings = @Setting( name = AvailableSettings.SESSION_FACTORY_NAME_IS_JNDI, value = "true" ) )

hibernate-core/src/test/java/org/hibernate/orm/test/jdbc/env/LobCreationCheckSkipTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313

1414
import org.junit.jupiter.api.Test;
1515

16-
import static org.junit.Assert.assertFalse;
16+
import static org.junit.jupiter.api.Assertions.assertFalse;
1717

1818
/**
1919
* @author Vlad Mihalcea

hibernate-core/src/test/java/org/hibernate/orm/test/jpa/schemagen/SchemaCreateDropExtraWarningsTest.java

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,10 @@
44
*/
55
package org.hibernate.orm.test.jpa.schemagen;
66

7-
import static org.assertj.core.api.Assertions.assertThat;
8-
7+
import jakarta.persistence.EntityManagerFactory;
98
import org.hibernate.SessionFactory;
109
import org.hibernate.dialect.PostgreSQLDialect;
11-
import org.hibernate.engine.jdbc.spi.SqlExceptionHelper;
12-
10+
import org.hibernate.engine.jdbc.spi.SQLExceptionLogging;
1311
import org.hibernate.testing.orm.junit.EntityManagerFactoryBasedFunctionalTest;
1412
import org.hibernate.testing.orm.junit.JiraKey;
1513
import org.hibernate.testing.orm.junit.Logger;
@@ -18,11 +16,13 @@
1816
import org.hibernate.testing.orm.junit.RequiresDialect;
1917
import org.junit.jupiter.api.Test;
2018

21-
import jakarta.persistence.EntityManagerFactory;
19+
import static org.junit.jupiter.api.Assertions.assertFalse;
2220

2321
@JiraKey("HHH-18296")
24-
@MessageKeyInspection(messageKey = "SQL Warning", logger = @Logger(loggerNameClass = SqlExceptionHelper.class))
25-
@RequiresDialect(value = PostgreSQLDialect.class, comment = "Only the PostgreSQL driver is known to generate 'success' warnings on table drop")
22+
@MessageKeyInspection(messageKey = "HHH",
23+
logger = @Logger(loggerName = SQLExceptionLogging.WARN_NAME))
24+
@RequiresDialect(value = PostgreSQLDialect.class,
25+
comment = "Only the PostgreSQL driver is known to generate 'success' warnings on table drop")
2626
public class SchemaCreateDropExtraWarningsTest extends EntityManagerFactoryBasedFunctionalTest {
2727

2828
@Override
@@ -35,15 +35,13 @@ public void testNoWarning(MessageKeyWatcher watcher) {
3535
// Create the emf in test, so that we can capture logs
3636
try ( EntityManagerFactory emf = produceEntityManagerFactory() ) {
3737
// No warnings on startup
38-
assertThat( watcher.getTriggeredMessages() )
39-
.isEmpty();
38+
assertFalse( watcher.wasTriggered() );
4039

4140
emf.unwrap( SessionFactory.class )
4241
.getSchemaManager()
4342
.dropMappedObjects( false );
4443
// No warnings on explicit drop
45-
assertThat( watcher.getTriggeredMessages() )
46-
.isEmpty();
44+
assertFalse( watcher.wasTriggered() );
4745
}
4846
}
4947
}

hibernate-core/src/test/java/org/hibernate/orm/test/mapping/converted/enums/OrdinalEnumTypeTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
import org.junit.jupiter.api.BeforeEach;
2424
import org.junit.jupiter.api.Test;
2525

26-
import static org.junit.Assert.assertTrue;
26+
import static org.junit.jupiter.api.Assertions.assertTrue;
2727

2828
/**
2929
* @author Vlad Mihacea

hibernate-core/src/test/java/org/hibernate/orm/test/mapping/converted/enums/VarcharEnumTypeTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
import jakarta.persistence.GeneratedValue;
2424
import jakarta.persistence.Id;
2525

26-
import static org.junit.Assert.assertTrue;
26+
import static org.junit.jupiter.api.Assertions.assertTrue;
2727

2828
@MessageKeyInspection(
2929
logger = @Logger( loggerName = JdbcBindingLogging.NAME ),

0 commit comments

Comments
 (0)