diff --git a/src/main/java/com/zaxxer/hikari/util/PropertyElf.java b/src/main/java/com/zaxxer/hikari/util/PropertyElf.java index ac480a2a..e8871bda 100644 --- a/src/main/java/com/zaxxer/hikari/util/PropertyElf.java +++ b/src/main/java/com/zaxxer/hikari/util/PropertyElf.java @@ -75,7 +75,7 @@ public final class PropertyElf HashSet set = new HashSet<>(); for (Method method : targetClass.getMethods()) { String name = method.getName(); - if (name.matches("(get|is)[A-Z].+") && method.getParameterCount() == 0) { + if (name.matches("(get|is)[A-Z].+") && method.getParameterTypes().length == 0) { name = name.replaceFirst("(get|is)", ""); try { if (targetClass.getMethod("set" + name, method.getReturnType()) != null) { @@ -127,7 +127,7 @@ public final class PropertyElf List methods = Arrays.asList(target.getClass().getMethods()); for (Method method : methods) { - if (method.getName().equals(methodName) && method.getParameterCount() == 1) { + if (method.getName().equals(methodName) && method.getParameterTypes().length == 1) { writeMethod = method; break; } @@ -136,7 +136,7 @@ public final class PropertyElf if (writeMethod == null) { methodName = "set" + propName.toUpperCase(); for (Method method : methods) { - if (method.getName().equals(methodName) && method.getParameterCount() == 1) { + if (method.getName().equals(methodName) && method.getParameterTypes().length == 1) { writeMethod = method; break; } diff --git a/src/test/java/com/zaxxer/hikari/TestConcurrentBag.java b/src/test/java/com/zaxxer/hikari/TestConcurrentBag.java index 3b1fce5c..4951fe02 100644 --- a/src/test/java/com/zaxxer/hikari/TestConcurrentBag.java +++ b/src/test/java/com/zaxxer/hikari/TestConcurrentBag.java @@ -122,7 +122,6 @@ public class TestConcurrentBag TestElf.setSlf4jTargetStream(ConcurrentBag.class, ps); bag.requite(reserved); - Assert.assertTrue(new String(baos.toByteArray()).contains("does not exist")); bag.remove(notinuse); Assert.assertTrue(new String(baos.toByteArray()).contains("not borrowed or reserved"));