From 4e3d2fa1422cbbaf212f01fb171130fb082b4d95 Mon Sep 17 00:00:00 2001 From: ahaha-vip <48004552+ahaha-vip@users.noreply.github.com> Date: Wed, 24 Mar 2021 16:46:39 +0800 Subject: Add a unit test in AbstractExtensionFinderTest to reproduce #428 (#430) --- pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'pf4j/src/main/java') diff --git a/pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java b/pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java index a156fab..42d59ae 100644 --- a/pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java +++ b/pf4j/src/main/java/org/pf4j/AbstractExtensionFinder.java @@ -83,7 +83,7 @@ public abstract class AbstractExtensionFinder implements ExtensionFinder, Plugin // classpath's extensions <=> pluginId = null Set classNames = findClassNames(pluginId); - if (classNames == null || classNames.isEmpty()) { + if (classNames.isEmpty()) { return result; } @@ -218,7 +218,12 @@ public abstract class AbstractExtensionFinder implements ExtensionFinder, Plugin @Override public Set findClassNames(String pluginId) { - return getEntries().get(pluginId); + Set classNames = getEntries().get(pluginId); + if (classNames == null) { + return Collections.emptySet(); + } + + return classNames; } @Override -- cgit v1.2.3