From 64d5e904e8056f51a52c546c81edf058eaa8b333 Mon Sep 17 00:00:00 2001 From: Yanming Zhou Date: Mon, 27 Nov 2023 16:26:59 +0800 Subject: [PATCH] Use AssertJ isInstanceOf where feasible See gh-31694 --- .../support/ConversionServiceFactoryBeanTests.java | 8 ++++---- .../socket/config/HandlersBeanDefinitionParserTests.java | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/spring-context/src/test/java/org/springframework/context/support/ConversionServiceFactoryBeanTests.java b/spring-context/src/test/java/org/springframework/context/support/ConversionServiceFactoryBeanTests.java index 697e4fc834..46e2ea2881 100644 --- a/spring-context/src/test/java/org/springframework/context/support/ConversionServiceFactoryBeanTests.java +++ b/spring-context/src/test/java/org/springframework/context/support/ConversionServiceFactoryBeanTests.java @@ -117,14 +117,14 @@ class ConversionServiceFactoryBeanTests { private void doTestConversionServiceInApplicationContext(String fileName, Class resourceClass) { ConfigurableApplicationContext ctx = new ClassPathXmlApplicationContext(fileName, getClass()); ResourceTestBean tb = ctx.getBean("resourceTestBean", ResourceTestBean.class); - assertThat(resourceClass.isInstance(tb.getResource())).isTrue(); + assertThat(tb.getResource()).isInstanceOf(resourceClass); assertThat(tb.getResourceArray()).hasSize(1); - assertThat(resourceClass.isInstance(tb.getResourceArray()[0])).isTrue(); + assertThat(tb.getResourceArray()[0]).isInstanceOf(resourceClass); assertThat(tb.getResourceMap()).hasSize(1); - assertThat(resourceClass.isInstance(tb.getResourceMap().get("key1"))).isTrue(); + assertThat(tb.getResourceMap().get("key1")).isInstanceOf(resourceClass); assertThat(tb.getResourceArrayMap()).hasSize(1); assertThat(tb.getResourceArrayMap().get("key1")).isNotEmpty(); - assertThat(resourceClass.isInstance(tb.getResourceArrayMap().get("key1")[0])).isTrue(); + assertThat(tb.getResourceArrayMap().get("key1")[0]).isInstanceOf(resourceClass); ctx.close(); } diff --git a/spring-websocket/src/test/java/org/springframework/web/socket/config/HandlersBeanDefinitionParserTests.java b/spring-websocket/src/test/java/org/springframework/web/socket/config/HandlersBeanDefinitionParserTests.java index d146858d97..5dd4402aa0 100644 --- a/spring-websocket/src/test/java/org/springframework/web/socket/config/HandlersBeanDefinitionParserTests.java +++ b/spring-websocket/src/test/java/org/springframework/web/socket/config/HandlersBeanDefinitionParserTests.java @@ -243,7 +243,7 @@ public class HandlersBeanDefinitionParserTests { if (handler instanceof WebSocketHandlerDecorator) { handler = ((WebSocketHandlerDecorator) handler).getLastHandler(); } - assertThat(handlerClass.isInstance(handler)).isTrue(); + assertThat(handler).isInstanceOf(handlerClass); } }