HHH-19846 - Remove JUnit4: org.hibernate.bytecode.internal.bytebuddy

Signed-off-by: Jan Schatteman <[email protected]>

Author: jrenaat

hibernate-core/src/test/java/org/hibernate/bytecode/internal/bytebuddy/ByteBuddyBasicProxyFactoryTest.java

@@ -4,14 +4,14 @@
  */
 package org.hibernate.bytecode.internal.bytebuddy;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assertions.assertNotEquals;
 
 import org.hibernate.testing.orm.junit.JiraKey;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 public class ByteBuddyBasicProxyFactoryTest {
 

hibernate-core/src/test/java/org/hibernate/bytecode/internal/bytebuddy/EnhanceByteCodeNotInProvidedClassLoaderTest.java

@@ -14,8 +14,8 @@
 import org.hibernate.bytecode.enhance.spi.Enhancer;
 
 import org.hibernate.testing.orm.junit.JiraKey;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
 
 /**
  * Tests that bytecode can be enhanced when the original class cannot be loaded from
@@ -31,9 +31,9 @@ public void test() {
 		// Now use a fake class name so it won't be found in the ClassLoader
 		// provided by DefaultEnhancementContext
 		byte[] enhanced = enhancer.enhance( SimpleEntity.class.getName() + "Fake", buffer );
-		Assert.assertNotNull( "This is null when there have been swallowed exceptions during enhancement. Check Logs!", enhanced );
+		Assertions.assertNotNull( enhanced, "This is null when there have been swallowed exceptions during enhancement. Check Logs!" );
 		// Make sure enhanced bytecode is different from original bytecode.
-		Assert.assertFalse( Arrays.equals( buffer, enhanced ) );
+		Assertions.assertFalse( Arrays.equals( buffer, enhanced ) );
 	}
 
 	private byte[] readResource(Class<?> clazz) {
@@ -52,7 +52,7 @@ private byte[] readResource(Class<?> clazz) {
 			os.close();
 		}
 		catch (IOException ex) {
-			Assert.fail( "Should not have an IOException here" );
+			Assertions.fail( "Should not have an IOException here" );
 		}
 		return os.toByteArray();
 	}

hibernate-core/src/test/java/org/hibernate/bytecode/internal/bytebuddy/EnhancerWildFlyNamesTest.java

@@ -13,8 +13,8 @@
 import org.hibernate.bytecode.enhance.spi.Enhancer;
 
 import org.hibernate.testing.orm.junit.JiraKey;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
 
 /**
  * WildFly will use class names in "internal JVM format" when invoking the enhancer,
@@ -34,10 +34,10 @@ public void test() {
 			buffer = readResource( resourceName );
 		}
 		catch (IOException e) {
-			Assert.fail( "Should not have an IOException here" );
+			Assertions.fail( "Should not have an IOException here" );
 		}
 		byte[] enhanced = enhancer.enhance( internalName, buffer );
-		Assert.assertNotNull( "This is null when there have been swallowed exceptions during enhancement. Check Logs!", enhanced );
+		Assertions.assertNotNull( enhanced, "This is null when there have been swallowed exceptions during enhancement. Check Logs!" );
 	}
 
 	private byte[] readResource(String resourceName) throws IOException {

hibernate-core/src/test/java/org/hibernate/bytecode/internal/bytebuddy/GenerateProxiesTest.java

@@ -4,12 +4,12 @@
  */
 package org.hibernate.bytecode.internal.bytebuddy;
 
-import static org.junit.Assert.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 
 import java.lang.reflect.InvocationTargetException;
 
 import org.hibernate.proxy.pojo.bytebuddy.ByteBuddyProxyHelper;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 public class GenerateProxiesTest {
 

hibernate-core/src/test/java/org/hibernate/bytecode/internal/bytebuddy/ReflectionOptimizerTests.java

@@ -6,10 +6,10 @@
 
 import org.hibernate.bytecode.spi.ReflectionOptimizer;
 import org.hibernate.testing.orm.junit.JiraKey;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.junit.Assert.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 
 /**
  * @author Steve Ebersole

hibernate-core/src/test/java/org/hibernate/bytecode/internal/bytebuddy/SimpleEnhancerTests.java

@@ -9,7 +9,7 @@
 import org.hibernate.bytecode.enhance.spi.EnhancementException;
 import org.hibernate.bytecode.enhance.spi.Enhancer;
 import org.hibernate.bytecode.spi.ByteCodeHelper;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 
 import java.io.IOException;
 

hibernate-core/src/test/java/org/hibernate/bytecode/internal/bytebuddy/SuperTypesEnhancementTest.java

@@ -17,12 +17,16 @@
 import org.hibernate.proxy.pojo.bytebuddy.ByteBuddyProxyHelper;
 
 import org.hibernate.testing.orm.junit.JiraKey;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
 /**
  * Verifies that proxies being generated by ByteBuddyProxyHelper
  * do not break the contract with PrimeAmongSecondarySupertypes.
@@ -58,36 +62,36 @@ private static Stream<Arguments> interfaces() {
 	public void testNamingConventions(Method m) {
 		final Class<?> returnType = m.getReturnType();
 		final String expectedMethodName = "as" + returnType.getSimpleName();
-		Assert.assertEquals( expectedMethodName, m.getName() );
-		Assert.assertNotNull( m.isDefault() );
+		assertEquals( expectedMethodName, m.getName() );
+		assertNotNull( m.isDefault() );
 	}
 
 	@ParameterizedTest
 	@MethodSource("superTypeMethods")
 	public void testAllsubInterfacesExtendTheSingleparent(Method m) {
 		final Class<?> returnType = m.getReturnType();
-		Assert.assertTrue( PrimeAmongSecondarySupertypes.class.isAssignableFrom( returnType ) );
+		assertTrue( PrimeAmongSecondarySupertypes.class.isAssignableFrom( returnType ) );
 	}
 
 	@ParameterizedTest
 	@MethodSource("superTypeMethods")
 	public void testSubInterfaceOverrides(Method m) throws NoSuchMethodException {
 		final Class<?> returnType = m.getReturnType();
 		final Method subMethod = returnType.getMethod( m.getName(), m.getParameterTypes() );
-		Assert.assertNotNull( subMethod );
-		Assert.assertNotNull( subMethod.isDefault() );
+		assertNotNull( subMethod );
+		assertNotNull( subMethod.isDefault() );
 	}
 
 	@Test
 	public void testHibernateProxyGeneration() {
 		ProxyFactory enhancer = createProxyFactory( SampleClass.class, HibernateProxy.class );
 		final Object proxy = enhancer.getProxy( Integer.valueOf( 1 ), null );
-		Assert.assertTrue( HibernateProxy.class.isAssignableFrom( proxy.getClass() ) );
-		Assert.assertTrue( proxy instanceof HibernateProxy );
+		assertTrue( HibernateProxy.class.isAssignableFrom( proxy.getClass() ) );
+		assertTrue( proxy instanceof HibernateProxy );
 		PrimeAmongSecondarySupertypes casted = (PrimeAmongSecondarySupertypes) proxy;
 		final HibernateProxy extracted = casted.asHibernateProxy();
-		Assert.assertNotNull( extracted );
-		Assert.assertSame( proxy, extracted );
+		assertNotNull( extracted );
+		Assertions.assertSame( proxy, extracted );
 		testForLIE( (SampleClass) proxy );
 	}
 
@@ -96,8 +100,8 @@ public void testHibernateProxyGeneration() {
 	 */
 	private void testForLIE(SampleClass sampleProxy) {
 		SampleClass other = new SampleClass();
-		Assert.assertEquals( 7, other.additionMethod( 3,4 ) );
-		Assert.assertThrows( LazyInitializationException.class, () -> sampleProxy.additionMethod( 3, 4 ) );
+		assertEquals( 7, other.additionMethod( 3,4 ) );
+		Assertions.assertThrows( LazyInitializationException.class, () -> sampleProxy.additionMethod( 3, 4 ) );
 	}
 
 	private ProxyFactory createProxyFactory(Class<?> persistentClass, Class<?>... interfaces) {