Polishing

spring-context/src/test/java/org/springframework/context/event/EventPublicationInterceptorTests.java

@@ -26,7 +26,6 @@ import org.springframework.beans.factory.FactoryBean;
 import org.springframework.beans.testfixture.beans.ITestBean;
 import org.springframework.beans.testfixture.beans.TestBean;
 import org.springframework.context.ApplicationEvent;
-import org.springframework.context.ApplicationEventPublisher;
 import org.springframework.context.event.test.TestEvent;
 import org.springframework.context.support.StaticApplicationContext;
 import org.springframework.context.testfixture.beans.TestApplicationListener;
@@ -36,6 +35,8 @@ import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException
 import static org.mockito.Mockito.mock;
 
 /**
+ * Tests for {@link EventPublicationInterceptor}.
+ *
  * @author Dmitriy Kopylenko
  * @author Juergen Hoeller
  * @author Rick Evans
@@ -47,46 +48,44 @@ class EventPublicationInterceptorTests {
 
 	@BeforeEach
 	void setup() {
-		ApplicationEventPublisher publisher = mock();
+		this.interceptor.setApplicationEventPublisher(mock());
-		this.interceptor.setApplicationEventPublisher(publisher);
 	}
 
 
 	@Test
 	void withNoApplicationEventPublisherSupplied() {
 		this.interceptor.setApplicationEventPublisher(null);
-		assertThatIllegalArgumentException().isThrownBy(interceptor::afterPropertiesSet);
+		assertThatIllegalArgumentException()
+				.isThrownBy(interceptor::afterPropertiesSet)
+				.withMessage("Property 'applicationEventPublisher' is required");
 	}
 
-	@SuppressWarnings("unchecked")
+	@SuppressWarnings({ "unchecked", "rawtypes" })
 	@Test
 	void withNonApplicationEventClassSupplied() {
-		assertThatIllegalArgumentException().isThrownBy(() -> {
+		assertThatIllegalArgumentException()
-				interceptor.setApplicationEventClass((Class) getClass());
+				.isThrownBy(() -> interceptor.setApplicationEventClass((Class) getClass()))
-				interceptor.afterPropertiesSet();
+				.withMessage("'applicationEventClass' needs to extend ApplicationEvent");
-		});
 	}
 
 	@Test
 	void withAbstractStraightApplicationEventClassSupplied() {
-		assertThatIllegalArgumentException().isThrownBy(() -> {
+		assertThatIllegalArgumentException()
-				interceptor.setApplicationEventClass(ApplicationEvent.class);
+				.isThrownBy(() -> interceptor.setApplicationEventClass(ApplicationEvent.class))
-				interceptor.afterPropertiesSet();
+				.withMessage("'applicationEventClass' needs to extend ApplicationEvent");
-		});
 	}
 
 	@Test
 	void withApplicationEventClassThatDoesntExposeAValidCtor() {
-		assertThatIllegalArgumentException().isThrownBy(() -> {
+		assertThatIllegalArgumentException()
-				interceptor.setApplicationEventClass(TestEventWithNoValidOneArgObjectCtor.class);
+			.isThrownBy(() -> interceptor.setApplicationEventClass(TestEventWithNoValidOneArgObjectCtor.class))
-				interceptor.afterPropertiesSet();
+			.withMessageContaining("does not have the required Object constructor");
-		});
 	}
 
 	@Test
 	void expectedBehavior() {
 		TestBean target = new TestBean();
-		final TestApplicationListener listener = new TestApplicationListener();
+		TestApplicationListener listener = new TestApplicationListener();
 
 		class TestContext extends StaticApplicationContext {
 			@Override
@@ -103,8 +102,7 @@ class EventPublicationInterceptorTests {
 		ctx.registerSingleton("otherListener", FactoryBeanTestListener.class);
 		ctx.refresh();
 
-		EventPublicationInterceptor interceptor =
+		EventPublicationInterceptor interceptor = ctx.getBean(EventPublicationInterceptor.class);
-				(EventPublicationInterceptor) ctx.getBean("publisher");
 		ProxyFactory factory = new ProxyFactory(target);
 		factory.addAdvice(0, interceptor);
 
@@ -115,14 +113,14 @@ class EventPublicationInterceptorTests {
 
 		// two events: ContextRefreshedEvent and TestEvent
 		assertThat(listener.getEventCount()).as("Interceptor must have published 2 events").isEqualTo(2);
-		TestApplicationListener otherListener = (TestApplicationListener) ctx.getBean("&otherListener");
+		TestApplicationListener otherListener = ctx.getBean("&otherListener", TestApplicationListener.class);
 		assertThat(otherListener.getEventCount()).as("Interceptor must have published 2 events").isEqualTo(2);
 		ctx.close();
 	}
 
 
 	@SuppressWarnings("serial")
-	static final class TestEventWithNoValidOneArgObjectCtor extends ApplicationEvent {
+	static class TestEventWithNoValidOneArgObjectCtor extends ApplicationEvent {
 
 		public TestEventWithNoValidOneArgObjectCtor() {
 			super("");
@@ -130,10 +128,10 @@ class EventPublicationInterceptorTests {
 	}
 
 
-	static class FactoryBeanTestListener extends TestApplicationListener implements FactoryBean<Object> {
+	static class FactoryBeanTestListener extends TestApplicationListener implements FactoryBean<String> {
 
 		@Override
-		public Object getObject() {
+		public String getObject() {
 			return "test";
 		}
 

spring-tx/src/test/java/org/springframework/transaction/annotation/EnableTransactionManagementTests.java

@@ -373,8 +373,6 @@ class EnableTransactionManagementTests {
 		TestServiceWithRollback bean = ctx.getBean("testBean", TestServiceWithRollback.class);
 		CallCountingTransactionManager txManager = ctx.getBean(CallCountingTransactionManager.class);
 
-		Method method1 = TestServiceWithRollback.class.getMethod("methodOne");
-		Method method2 = TestServiceWithRollback.class.getMethod("methodTwo");
 		assertThatException().isThrownBy(bean::methodOne);
 		assertThatException().isThrownBy(bean::methodTwo);
 		assertThat(txManager.begun).isEqualTo(2);