Skip to content

Commit 9b52a52

Browse files
committed
HHH-19916 - More drop JUnit 4 usage work
1 parent 109f8aa commit 9b52a52

File tree

4 files changed

+15
-20
lines changed

4 files changed

+15
-20
lines changed

hibernate-integrationtest-java-modules/hibernate-integrationtest-java-modules.gradle

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,7 @@ dependencies {
2424
api project( ':hibernate-envers' )
2525
implementation project( ':hibernate-scan-jandex' )
2626

27-
// todo : to get rid of these
28-
testImplementation testLibs.junit4
29-
testImplementation testLibs.junit4Engine
27+
testImplementation testLibs.junitJupiterApi
3028

3129
//Provide the jakarta.cdi module, as it's required by module jakarta.transaction
3230
//but not provided as transitive dependency of Narayana.

hibernate-integrationtest-java-modules/src/test/java/module-info.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,6 @@
2222
* Test-only configuration.
2323
*/
2424

25-
opens org.hibernate.orm.integrationtest.java.module.test to junit;
26-
requires junit;
25+
opens org.hibernate.orm.integrationtest.java.module.test to org.junit.platform.commons;
26+
requires org.junit.jupiter.api;
2727
}

hibernate-integrationtest-java-modules/src/test/java/org/hibernate/orm/integrationtest/java/module/test/JavaModulePathIT.java

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,15 @@
44
*/
55
package org.hibernate.orm.integrationtest.java.module.test;
66

7-
8-
97
import org.hibernate.Session;
108
import org.hibernate.envers.boot.internal.EnversIntegrator;
119
import org.hibernate.orm.integrationtest.java.module.test.service.AuthorService;
12-
13-
import org.junit.Assert;
14-
import org.junit.Test;
10+
import org.junit.jupiter.api.Test;
1511

1612
import java.util.Arrays;
1713

18-
import static org.junit.Assert.assertEquals;
14+
import static org.junit.jupiter.api.Assertions.assertEquals;
15+
import static org.junit.jupiter.api.Assertions.assertTrue;
1916

2017
public class JavaModulePathIT {
2118

@@ -67,9 +64,8 @@ public void integrator() {
6764
}
6865

6966
private void checkIsInModulePath(Class<?> clazz) {
70-
Assert.assertTrue(
71-
clazz + " should be part of a named module - there is a problem in test setup",
72-
clazz.getModule().isNamed()
67+
assertTrue( clazz.getModule().isNamed(),
68+
clazz + " should be part of a named module - there is a problem in test setup"
7369
);
7470
}
7571
}

hibernate-integrationtest-java-modules/src/test/java/org/hibernate/orm/integrationtest/java/module/test/ScannerTest.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,10 @@
1414
import org.hibernate.boot.archive.scan.spi.ClassDescriptor;
1515
import org.hibernate.boot.archive.scan.spi.ScanResult;
1616
import org.hibernate.orm.integrationtest.java.module.test.entity.Author;
17+
import org.junit.jupiter.api.Test;
1718

18-
import org.junit.Assert;
19-
import org.junit.Test;
19+
import static org.junit.jupiter.api.Assertions.assertEquals;
20+
import static org.junit.jupiter.api.Assertions.assertNotNull;
2021

2122
/**
2223
* We need to test that the scanner works, including when there is a module-info.class
@@ -34,11 +35,11 @@ public void verifyModuleInfoScanner() {
3435
StandardScanParameters.INSTANCE
3536
);
3637
Set<ClassDescriptor> locatedClasses = scan.getLocatedClasses();
37-
Assert.assertEquals( 1, locatedClasses.size() );
38+
assertEquals( 1, locatedClasses.size() );
3839
ClassDescriptor classDescriptor = locatedClasses.iterator().next();
39-
Assert.assertNotNull( classDescriptor );
40-
Assert.assertEquals( Author.class.getName(), classDescriptor.getName() );
41-
Assert.assertEquals( ClassDescriptor.Categorization.MODEL, classDescriptor.getCategorization() );
40+
assertNotNull( classDescriptor );
41+
assertEquals( Author.class.getName(), classDescriptor.getName() );
42+
assertEquals( ClassDescriptor.Categorization.MODEL, classDescriptor.getCategorization() );
4243
}
4344

4445
}

0 commit comments

Comments
 (0)