From 70963ea174976051d586f0bb2829596de1c507b7 Mon Sep 17 00:00:00 2001
From: Decebal Suiu <decebal.suiu@gmail.com>
Date: Mon, 15 Apr 2019 20:51:02 +0300
Subject: [PATCH] Resolve #311

---
 .../java/org/pf4j/BasePluginRepository.java   |  8 ++-
 .../org/pf4j/JarPluginRepositoryTest.java     | 59 +++++++++++++++++++
 .../test/java/org/pf4j/LoadPluginsTest.java   | 14 +----
 3 files changed, 66 insertions(+), 15 deletions(-)
 create mode 100644 pf4j/src/test/java/org/pf4j/JarPluginRepositoryTest.java

diff --git a/pf4j/src/main/java/org/pf4j/BasePluginRepository.java b/pf4j/src/main/java/org/pf4j/BasePluginRepository.java
index 0bf6e6c..aa152be 100644
--- a/pf4j/src/main/java/org/pf4j/BasePluginRepository.java
+++ b/pf4j/src/main/java/org/pf4j/BasePluginRepository.java
@@ -87,11 +87,15 @@ public class BasePluginRepository implements PluginRepository {
 
     @Override
     public boolean deletePluginPath(Path pluginPath) {
+        if (!filter.accept(pluginPath.toFile())) {
+            return false;
+        }
+
         try {
             FileUtils.delete(pluginPath);
             return true;
-        } catch (NoSuchFileException nsf) {
-            return false; // Return false on not found to be compatible with previous API
+        } catch (NoSuchFileException e) {
+            return false; // Return false on not found to be compatible with previous API (#135)
         } catch (IOException e) {
             throw new RuntimeException(e);
         }
diff --git a/pf4j/src/test/java/org/pf4j/JarPluginRepositoryTest.java b/pf4j/src/test/java/org/pf4j/JarPluginRepositoryTest.java
new file mode 100644
index 0000000..2260680
--- /dev/null
+++ b/pf4j/src/test/java/org/pf4j/JarPluginRepositoryTest.java
@@ -0,0 +1,59 @@
+/*
+ * Copyright (C) 2012-present the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.pf4j;
+
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.io.TempDir;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.util.List;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+/**
+ * @author Decebal Suiu
+ */
+public class JarPluginRepositoryTest {
+
+    @TempDir
+    Path pluginsPath;
+
+    /**
+     * Test of {@link JarPluginRepository#deletePluginPath(Path)} method.
+     */
+    @Test
+    public void testDeletePluginPath() throws IOException {
+        PluginRepository repository = new JarPluginRepository(pluginsPath);
+
+        Path plugin1Path = Files.createDirectory(pluginsPath.resolve("plugin-1"));
+        Path plugin1JarPath = Files.createFile(pluginsPath.resolve("plugin-1.jar"));
+
+        assertFalse(repository.deletePluginPath(plugin1Path));
+
+        List<Path> pluginsPaths = repository.getPluginsPaths();
+        assertEquals(1, pluginsPaths.size());
+
+        assertTrue(repository.deletePluginPath(plugin1JarPath));
+
+        pluginsPaths = repository.getPluginsPaths();
+        assertEquals(0, pluginsPaths.size());
+    }
+
+}
diff --git a/pf4j/src/test/java/org/pf4j/LoadPluginsTest.java b/pf4j/src/test/java/org/pf4j/LoadPluginsTest.java
index f86cee4..d2a7e25 100644
--- a/pf4j/src/test/java/org/pf4j/LoadPluginsTest.java
+++ b/pf4j/src/test/java/org/pf4j/LoadPluginsTest.java
@@ -45,19 +45,7 @@ public class LoadPluginsTest {
 
     @BeforeEach
     public void setUp() {
-        pluginManager = new DefaultPluginManager(pluginsPath) {
-
-            @Override
-            protected PluginRepository createPluginRepository() {
-                return new DefaultPluginRepository(getPluginsRoot(), isDevelopment());
-            }
-
-            @Override
-            protected PluginLoader createPluginLoader() {
-                return new DefaultPluginLoader(this, pluginClasspath);
-            }
-
-        };
+        pluginManager = new DefaultPluginManager(pluginsPath);
     }
 
     @Test
-- 
2.39.5