]> source.dussan.org Git - pf4j.git/commitdiff
Improve code quality (sonar hints)
authorDecebal Suiu <decebal.suiu@gmail.com>
Sun, 23 Jan 2022 22:24:23 +0000 (00:24 +0200)
committerDecebal Suiu <decebal.suiu@gmail.com>
Sun, 23 Jan 2022 22:24:23 +0000 (00:24 +0200)
29 files changed:
pf4j/src/main/java/org/pf4j/DefaultPluginRepository.java
pf4j/src/main/java/org/pf4j/util/io/CompoundPathFilter.java
pf4j/src/test/java/org/pf4j/AbstractExtensionFinderTest.java
pf4j/src/test/java/org/pf4j/AbstractPluginManagerTest.java
pf4j/src/test/java/org/pf4j/CompoundPluginDescriptorFinderTest.java
pf4j/src/test/java/org/pf4j/DefaultExtensionFactoryTest.java
pf4j/src/test/java/org/pf4j/DefaultPluginFactoryTest.java
pf4j/src/test/java/org/pf4j/DefaultPluginManagerTest.java
pf4j/src/test/java/org/pf4j/DefaultPluginRepositoryTest.java
pf4j/src/test/java/org/pf4j/DefaultPluginStatusProviderTest.java
pf4j/src/test/java/org/pf4j/DefaultVersionManagerTest.java
pf4j/src/test/java/org/pf4j/DependencyResolverTest.java
pf4j/src/test/java/org/pf4j/DevelopmentPluginRepositoryTest.java
pf4j/src/test/java/org/pf4j/ExtensionAnnotationProcessorTest.java
pf4j/src/test/java/org/pf4j/JarPluginManagerTest.java
pf4j/src/test/java/org/pf4j/JarPluginRepositoryTest.java
pf4j/src/test/java/org/pf4j/LegacyExtensionFinderTest.java
pf4j/src/test/java/org/pf4j/LegacyExtensionStorageTest.java
pf4j/src/test/java/org/pf4j/LoadPluginsFromMultipleRootsTest.java
pf4j/src/test/java/org/pf4j/LoadPluginsTest.java
pf4j/src/test/java/org/pf4j/ManifestPluginDescriptorFinderTest.java
pf4j/src/test/java/org/pf4j/PluginClassLoaderTest.java
pf4j/src/test/java/org/pf4j/PluginDependencyTest.java
pf4j/src/test/java/org/pf4j/PropertiesPluginDescriptorFinderTest.java
pf4j/src/test/java/org/pf4j/SecurePluginManagerWrapperTest.java
pf4j/src/test/java/org/pf4j/SingletonExtensionFactoryTest.java
pf4j/src/test/java/org/pf4j/processor/ServiceProviderExtensionStorageTest.java
pf4j/src/test/java/org/pf4j/util/DirectedGraphTest.java
pf4j/src/test/java/org/pf4j/util/FileUtilsTest.java

index 02c8c79ad24f1670599fd1224a9a0d26ca0f5776..9f668f0b26a765da01d52e52c951008d09ffdcd4 100644 (file)
@@ -23,8 +23,6 @@ import org.pf4j.util.io.NotPathFilter;
 import org.pf4j.util.io.OrPathFilter;
 import org.pf4j.util.io.PathFilter;
 import org.pf4j.util.io.ZipPathFilter;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import java.io.IOException;
 import java.io.UncheckedIOException;
@@ -37,8 +35,6 @@ import java.util.List;
  */
 public class DefaultPluginRepository extends BasePluginRepository {
 
-    private static final Logger log = LoggerFactory.getLogger(DefaultPluginRepository.class);
-
     public DefaultPluginRepository(Path... pluginsRoots) {
         this(Arrays.asList(pluginsRoots));
     }
index c7dfbbdadedc4d959d58f3269e1b2eaf7ae36293..d3466ddbdf3379c8028323669c37196159e20ef8 100644 (file)
@@ -13,15 +13,15 @@ public abstract class CompoundPathFilter implements PathFilter {
     /** The list of path filters. */
     protected List<PathFilter> pathFilters;
 
-    public CompoundPathFilter() {
+    protected CompoundPathFilter() {
         this(new ArrayList<>());
     }
 
-    public CompoundPathFilter(PathFilter... pathFilters) {
+    protected CompoundPathFilter(PathFilter... pathFilters) {
         this(Arrays.asList(pathFilters));
     }
 
-    public CompoundPathFilter(List<PathFilter> pathFilters) {
+    protected CompoundPathFilter(List<PathFilter> pathFilters) {
         this.pathFilters = new ArrayList<>(pathFilters);
     }
 
index eff6ebef2ca7f3492ea3912b84d628af6bcb8891..c1e842221d5d46cc5d6d1f2e0bba715fb58e260c 100644 (file)
@@ -79,7 +79,7 @@ public class AbstractExtensionFinderTest {
      * Test of {@link AbstractExtensionFinder#find(Class)}.
      */
     @Test
-    public void testFindFailType() {
+    void testFindFailType() {
         ExtensionFinder instance = new AbstractExtensionFinder(pluginManager) {
 
             @Override
@@ -101,7 +101,7 @@ public class AbstractExtensionFinderTest {
      * Test of {@link AbstractExtensionFinder#find(Class)}.
      */
     @Test
-    public void testFindFromClasspath() {
+    void testFindFromClasspath() {
         ExtensionFinder instance = new AbstractExtensionFinder(pluginManager) {
 
             @Override
@@ -131,7 +131,7 @@ public class AbstractExtensionFinderTest {
      * Test of {@link AbstractExtensionFinder#find(Class, String)}.
      */
     @Test
-    public void testFindFromPlugin() {
+    void testFindFromPlugin() {
         ExtensionFinder instance = new AbstractExtensionFinder(pluginManager) {
 
             @Override
@@ -170,7 +170,7 @@ public class AbstractExtensionFinderTest {
      * Test of {@link AbstractExtensionFinder#findClassNames(String)}.
      */
     @Test
-    public void testFindClassNames() {
+    void testFindClassNames() {
         ExtensionFinder instance = new AbstractExtensionFinder(pluginManager) {
 
             @Override
@@ -209,7 +209,7 @@ public class AbstractExtensionFinderTest {
      * Test of {@link org.pf4j.AbstractExtensionFinder#find(java.lang.String)}.
      */
     @Test
-    public void testFindExtensionWrappersFromPluginId() {
+    void testFindExtensionWrappersFromPluginId() {
         ExtensionFinder instance = new AbstractExtensionFinder(pluginManager) {
 
             @Override
@@ -245,7 +245,7 @@ public class AbstractExtensionFinderTest {
     }
 
     @Test
-    public void findExtensionAnnotation() throws Exception {
+    void findExtensionAnnotation() throws Exception {
         Compilation compilation = javac().compile(ExtensionAnnotationProcessorTest.Greeting,
             ExtensionAnnotationProcessorTest.WhazzupGreeting);
         assertThat(compilation).succeededWithoutWarnings();
@@ -260,7 +260,7 @@ public class AbstractExtensionFinderTest {
     }
 
     @Test
-    public void findExtensionAnnotationThatMissing() throws Exception {
+    void findExtensionAnnotationThatMissing() throws Exception {
         Compilation compilation = javac().compile(ExtensionAnnotationProcessorTest.Greeting,
             ExtensionAnnotationProcessorTest.SpinnakerExtension_NoExtension,
             ExtensionAnnotationProcessorTest.WhazzupGreeting_SpinnakerExtension);
index c43bc635b9bd982d694b990669c7612b8dd6e5d0..9bc19b99aa467ae23562c8482745780877d64180 100644 (file)
@@ -34,7 +34,7 @@ import static org.mockito.Mockito.when;
 public class AbstractPluginManagerTest {
 
     @Test
-    public void getExtensionsByType() {
+    void getExtensionsByType() {
         AbstractPluginManager pluginManager = mock(AbstractPluginManager.class, CALLS_REAL_METHODS);
 
         ExtensionFinder extensionFinder = mock(ExtensionFinder.class);
@@ -46,9 +46,9 @@ public class AbstractPluginManagerTest {
         List<TestExtensionPoint> extensions = pluginManager.getExtensions(TestExtensionPoint.class);
         assertEquals(1, extensions.size());
     }
-    
+
     @Test
-    public void getVersion() {
+    void getVersion() {
         AbstractPluginManager pluginManager = mock(AbstractPluginManager.class, CALLS_REAL_METHODS);
         assertNotEquals("0.0.0", pluginManager.getVersion());
     }
index 3f9d0073bd1076ab414ce9b271a3451d66d6ff7e..9aa1af304804733ad3e55bcbebbe68caeaa4dc20 100644 (file)
@@ -43,7 +43,7 @@ public class CompoundPluginDescriptorFinderTest {
     Path pluginsPath;
 
     @Test
-    public void add() {
+    void add() {
         CompoundPluginDescriptorFinder descriptorFinder = new CompoundPluginDescriptorFinder();
         assertEquals(0, descriptorFinder.size());
 
@@ -52,7 +52,7 @@ public class CompoundPluginDescriptorFinderTest {
     }
 
     @Test
-    public void find() throws Exception {
+    void find() throws Exception {
         Path pluginPath = Files.createDirectories(pluginsPath.resolve("test-plugin-1"));
         storePropertiesToPath(getPlugin1Properties(), pluginPath);
 
@@ -66,7 +66,7 @@ public class CompoundPluginDescriptorFinderTest {
     }
 
     @Test
-    public void findInJar() throws Exception {
+    void findInJar() throws Exception {
         PluginDescriptorFinder descriptorFinder = new CompoundPluginDescriptorFinder()
             .add(new ManifestPluginDescriptorFinder());
 
@@ -83,13 +83,13 @@ public class CompoundPluginDescriptorFinderTest {
     }
 
     @Test
-    public void testNotFound() {
+    void testNotFound() {
         PluginDescriptorFinder descriptorFinder = new CompoundPluginDescriptorFinder();
         assertThrows(PluginRuntimeException.class, () -> descriptorFinder.find(pluginsPath.resolve("test-plugin-3")));
     }
 
     @Test
-    public void testSpaceCharacterInFileName() throws Exception {
+    void testSpaceCharacterInFileName() throws Exception {
         PluginDescriptorFinder descriptorFinder = new CompoundPluginDescriptorFinder()
             .add(new ManifestPluginDescriptorFinder());
 
index 9d73b5e42ce1bc71ea9ab1bf728d7791d4a9fb77..c71becb0fee2d4571ff2edf52d60593302865c49 100644 (file)
@@ -45,7 +45,7 @@ public class DefaultExtensionFactoryTest {
      * Test of create method, of class DefaultExtensionFactory.
      */
     @Test
-    public void testCreate() {
+    void testCreate() {
         assertNotNull(extensionFactory.create(TestExtension.class));
     }
 
@@ -53,7 +53,7 @@ public class DefaultExtensionFactoryTest {
      * Test of create method, of class DefaultExtensionFactory.
      */
     @Test
-    public void testCreateFailConstructor() {
+    void testCreateFailConstructor() {
         assertThrows(PluginRuntimeException.class, () -> extensionFactory.create(FailTestExtension.class));
     }
 
index 31070839821f2d036fa63ca30c2ab01ba2b4661c..02f7be8d8bd8201073a69713b146a2b37cc2fbc2 100644 (file)
@@ -30,10 +30,10 @@ import static org.mockito.Mockito.when;
 /**
  * @author Mario Franco
  */
-public class DefaultPluginFactoryTest {
+class DefaultPluginFactoryTest {
 
     @Test
-    public void testCreate() {
+    void testCreate() {
         PluginDescriptor pluginDescriptor = mock(PluginDescriptor.class);
         when(pluginDescriptor.getPluginClass()).thenReturn(TestPlugin.class.getName());
 
@@ -49,7 +49,7 @@ public class DefaultPluginFactoryTest {
     }
 
     @Test
-    public void testCreateFail() {
+    void testCreateFail() {
         PluginDescriptor pluginDescriptor = mock(PluginDescriptor.class);
         when(pluginDescriptor.getPluginClass()).thenReturn(FailTestPlugin.class.getName());
 
@@ -64,7 +64,7 @@ public class DefaultPluginFactoryTest {
     }
 
     @Test
-    public void testCreateFailNotFound() {
+    void testCreateFailNotFound() {
         PluginDescriptor pluginDescriptor = mock(PluginDescriptor.class);
         when(pluginDescriptor.getPluginClass()).thenReturn("org.pf4j.plugin.NotFoundTestPlugin");
 
@@ -79,7 +79,7 @@ public class DefaultPluginFactoryTest {
     }
 
     @Test
-    public void testCreateFailConstructor() {
+    void testCreateFailConstructor() {
         PluginDescriptor pluginDescriptor = mock(PluginDescriptor.class);
         when(pluginDescriptor.getPluginClass()).thenReturn(AnotherFailTestPlugin.class.getName());
 
index b9a11f99158f35427c0c9a6082924eb5eb85ecdd..23ee3286d5eac19746ccdec2ea45c000340f1797 100644 (file)
@@ -34,7 +34,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
-public class DefaultPluginManagerTest {
+class DefaultPluginManagerTest {
 
     private DefaultPluginManager pluginManager;
     private DefaultPluginDescriptor pluginDescriptor;
@@ -44,7 +44,7 @@ public class DefaultPluginManagerTest {
     Path pluginsPath;
 
     @BeforeEach
-    public void setUp() throws IOException {
+    void setUp() throws IOException {
         pluginManager = new DefaultPluginManager(pluginsPath);
 
         pluginDescriptor = new DefaultPluginDescriptor();
@@ -59,37 +59,37 @@ public class DefaultPluginManagerTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         pluginManager = null;
         pluginDescriptor = null;
         pluginWrapper = null;
     }
 
     @Test
-    public void validateOK() {
+    void validateOK() {
         pluginManager.validatePluginDescriptor(pluginDescriptor);
     }
 
     @Test
-    public void validateFailsOnId() {
+    void validateFailsOnId() {
         pluginDescriptor.setPluginId("");
         assertThrows(PluginRuntimeException.class, () -> pluginManager.validatePluginDescriptor(pluginDescriptor));
     }
 
     @Test
-    public void validateFailsOnVersion() {
+    void validateFailsOnVersion() {
         pluginDescriptor.setPluginVersion(null);
         assertThrows(PluginRuntimeException.class, () -> pluginManager.validatePluginDescriptor(pluginDescriptor));
     }
 
     @Test
-    public void validateNoPluginClass() {
+    void validateNoPluginClass() {
         pluginManager.validatePluginDescriptor(pluginDescriptor);
         assertEquals(Plugin.class.getName(), pluginDescriptor.getPluginClass());
     }
 
     @Test
-    public void isPluginValid() {
+    void isPluginValid() {
         // By default accept all since system version not given
         assertTrue(pluginManager.isPluginValid(pluginWrapper));
 
@@ -104,7 +104,7 @@ public class DefaultPluginManagerTest {
     }
 
     @Test
-    public void isPluginValidAllowExact() {
+    void isPluginValidAllowExact() {
         pluginManager.setExactVersionAllowed(true);
 
         // By default accept all since system version not given
@@ -121,7 +121,7 @@ public class DefaultPluginManagerTest {
     }
 
     @Test
-    public void testDefaultExactVersionAllowed() {
+    void testDefaultExactVersionAllowed() {
         assertFalse(pluginManager.isExactVersionAllowed());
     }
 
@@ -130,7 +130,7 @@ public class DefaultPluginManagerTest {
      * See https://github.com/pf4j/pf4j/issues/223.
      */
     @Test
-    public void testPluginDisabledNoStart() throws IOException {
+    void testPluginDisabledNoStart() throws IOException {
         new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
@@ -157,7 +157,7 @@ public class DefaultPluginManagerTest {
     }
 
     @Test
-    public void deleteZipPlugin() throws Exception {
+    void deleteZipPlugin() throws Exception {
         PluginZip pluginZip = new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
@@ -174,7 +174,7 @@ public class DefaultPluginManagerTest {
     }
 
     @Test
-    public void deleteJarPlugin() throws Exception {
+    void deleteJarPlugin() throws Exception {
         PluginJar pluginJar = new PluginJar.Builder(pluginsPath.resolve("my-plugin-1.2.3.jar"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
index c11938d44b5c46327784f9708be581aa7478b69b..7f9de313cff3e1967c6a49d5cb0cf72ed864d321 100644 (file)
@@ -34,13 +34,13 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
  * @author Mario Franco
  * @author Decebal Suiu
  */
-public class DefaultPluginRepositoryTest {
+class DefaultPluginRepositoryTest {
 
     Path pluginsPath1;
     Path pluginsPath2;
 
     @BeforeEach
-    public void setUp() throws IOException {
+    void setUp() throws IOException {
         pluginsPath1 = Files.createTempDirectory("junit-pf4j-");
         pluginsPath2 = Files.createTempDirectory("junit-pf4j-");
         Path plugin1Path = Files.createDirectory(pluginsPath1.resolve("plugin-1"));
@@ -53,7 +53,7 @@ public class DefaultPluginRepositoryTest {
     }
 
     @AfterEach
-    public void tearDown() throws IOException {
+    void tearDown() throws IOException {
         FileUtils.delete(pluginsPath1);
         FileUtils.delete(pluginsPath2);
     }
@@ -62,7 +62,7 @@ public class DefaultPluginRepositoryTest {
      * Test of {@link DefaultPluginRepository#getPluginPaths()} method.
      */
     @Test
-    public void testGetPluginArchivesFromSinglePath() {
+    void testGetPluginArchivesFromSinglePath() {
         PluginRepository repository = new DefaultPluginRepository(pluginsPath2);
 
         List<Path> pluginPaths = repository.getPluginPaths();
@@ -76,7 +76,7 @@ public class DefaultPluginRepositoryTest {
      * Test of {@link DefaultPluginRepository#getPluginPaths()} method.
      */
     @Test
-    public void testGetPluginArchives() {
+    void testGetPluginArchives() {
         PluginRepository repository = new DefaultPluginRepository(pluginsPath1, pluginsPath2);
 
         List<Path> pluginPaths = repository.getPluginPaths();
@@ -91,7 +91,7 @@ public class DefaultPluginRepositoryTest {
      * Test of {@link DefaultPluginRepository#deletePluginPath(Path)} method.
      */
     @Test
-    public void testDeletePluginPath() {
+    void testDeletePluginPath() {
         PluginRepository repository = new DefaultPluginRepository(pluginsPath1, pluginsPath2);
 
         assertTrue(Files.exists(pluginsPath1.resolve("plugin-1.zip")));
index b0a0e8e03c23edf70db29ded5fc0d5e2841fc4e2..1d23e641dcb94d0523e9a372f1be049e87dbf3b2 100644 (file)
@@ -32,13 +32,13 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
  * @author Mario Franco
  * @author Decebal Suiu
  */
-public class DefaultPluginStatusProviderTest {
+class DefaultPluginStatusProviderTest {
 
     @TempDir
     Path pluginsPath;
 
     @Test
-    public void testIsPluginDisabled() throws IOException {
+    void testIsPluginDisabled() throws IOException {
         createEnabledFile();
         createDisabledFile();
 
@@ -50,7 +50,7 @@ public class DefaultPluginStatusProviderTest {
     }
 
     @Test
-    public void testIsPluginDisabledWithEnableEmpty() throws IOException {
+    void testIsPluginDisabledWithEnableEmpty() throws IOException {
         createDisabledFile();
 
         PluginStatusProvider statusProvider = new DefaultPluginStatusProvider(pluginsPath);
@@ -61,7 +61,7 @@ public class DefaultPluginStatusProviderTest {
     }
 
     @Test
-    public void testDisablePlugin() throws Exception {
+    void testDisablePlugin() throws Exception {
         createEnabledFile();
         createDisabledFile();
 
@@ -74,7 +74,7 @@ public class DefaultPluginStatusProviderTest {
     }
 
     @Test
-    public void testDisablePluginWithEnableEmpty() throws Exception {
+    void testDisablePluginWithEnableEmpty() throws Exception {
         // scenario with "disabled.txt"
         createDisabledFile();
 
@@ -107,7 +107,7 @@ public class DefaultPluginStatusProviderTest {
     }
 
     @Test
-    public void testEnablePlugin() throws Exception {
+    void testEnablePlugin() throws Exception {
         // scenario with "enabled.txt"
         createEnabledFile();
 
@@ -140,7 +140,7 @@ public class DefaultPluginStatusProviderTest {
     }
 
     @Test
-    public void testEnablePluginWithEnableEmpty() {
+    void testEnablePluginWithEnableEmpty() {
         PluginStatusProvider statusProvider = new DefaultPluginStatusProvider(pluginsPath);
         statusProvider.enablePlugin("plugin-2");
 
@@ -150,7 +150,7 @@ public class DefaultPluginStatusProviderTest {
     }
 
     @Test
-    public void testDisablePluginWithoutDisabledFile() {
+    void testDisablePluginWithoutDisabledFile() {
         PluginStatusProvider statusProvider = new DefaultPluginStatusProvider(pluginsPath);
 
         assertFalse(statusProvider.isPluginDisabled("plugin-1"));
index a7a007e2fd95e34a00d66595bac70726de1731b4..b4f398bc2387cd4a192ccb42ce3c3f949c979595 100644 (file)
@@ -26,34 +26,34 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 /**
  * @author Decebal Suiu
  */
-public class DefaultVersionManagerTest {
+class DefaultVersionManagerTest {
 
     private VersionManager versionManager;
 
     @BeforeEach
-    public void init() {
+    void init() {
         versionManager = new DefaultVersionManager();
     }
 
     @Test
-    public void checkVersionConstraint() {
+    void checkVersionConstraint() {
         assertFalse(versionManager.checkVersionConstraint("1.4.3", ">2.0.0")); // simple
         assertTrue(versionManager.checkVersionConstraint("1.4.3", ">=1.4.0 & <1.6.0")); // range
         assertTrue(versionManager.checkVersionConstraint("undefined", "*"));
     }
 
     @Test
-    public void nullOrEmptyVersion() {
+    void nullOrEmptyVersion() {
         assertThrows(IllegalArgumentException.class, () -> versionManager.checkVersionConstraint(null, ">2.0.0"));
     }
 
     @Test
-    public void invalidVersion() {
+    void invalidVersion() {
         assertThrows(ParseException.class, () -> versionManager.checkVersionConstraint("1.0", ">2.0.0"));
     }
 
     @Test
-    public void compareVersions() {
+    void compareVersions() {
         assertTrue(versionManager.compareVersions("1.1.0", "1.0.0") > 0);
     }
 
index 6e35780b607f038baafe1e89bf089d24af9b7508..296bd35bd77bb8efc5b9fccfd6437a693e1e530d 100644 (file)
@@ -29,18 +29,18 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 /**
  * @author Decebal Suiu
  */
-public class DependencyResolverTest {
+class DependencyResolverTest {
 
     private DependencyResolver resolver;
 
     @BeforeEach
-    public void init() {
+    void init() {
         VersionManager versionManager = new DefaultVersionManager();
         resolver = new DependencyResolver(versionManager);
     }
 
     @Test
-    public void sortedPlugins() {
+    void sortedPlugins() {
         // create incomplete plugin descriptor (ignore some attributes)
         PluginDescriptor pd1 = new DefaultPluginDescriptor()
             .setPluginId("p1")
@@ -61,7 +61,7 @@ public class DependencyResolverTest {
     }
 
     @Test
-    public void notFoundDependencies() {
+    void notFoundDependencies() {
         PluginDescriptor pd1 = new DefaultPluginDescriptor()
             .setPluginId("p1")
             .setDependencies("p2, p3");
@@ -76,7 +76,7 @@ public class DependencyResolverTest {
     }
 
     @Test
-    public void cyclicDependencies() {
+    void cyclicDependencies() {
         PluginDescriptor pd1 = new DefaultPluginDescriptor()
             .setPluginId("p1")
             .setPluginVersion("0.0.0")
@@ -103,7 +103,7 @@ public class DependencyResolverTest {
     }
 
     @Test
-    public void wrongDependencyVersion() {
+    void wrongDependencyVersion() {
         PluginDescriptor pd1 = new DefaultPluginDescriptor()
             .setPluginId("p1")
 //            .setDependencies("p2@2.0.0"); // simple version
@@ -123,7 +123,7 @@ public class DependencyResolverTest {
     }
 
     @Test
-    public void goodDependencyVersion() {
+    void goodDependencyVersion() {
         PluginDescriptor pd1 = new DefaultPluginDescriptor()
             .setPluginId("p1")
             .setDependencies("p2@2.0.0");
index 20d912937d7c3e2f60d812d6607c2e8521007746..051b2f3c1968e0c904ab6fdc4028aab6601f1b3d 100644 (file)
@@ -30,20 +30,20 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
 /**
  * @author Decebal Suiu
  */
-public class DevelopmentPluginRepositoryTest {
+class DevelopmentPluginRepositoryTest {
 
     @TempDir
     Path pluginsPath;
 
     @BeforeEach
-    public void setUp() throws IOException {
+    void setUp() throws IOException {
         // standard maven/gradle bin folder - these should be skipped in development mode because the cause errors
         Files.createDirectory(pluginsPath.resolve(DevelopmentPluginRepository.MAVEN_BUILD_DIR));
         Files.createDirectory(pluginsPath.resolve(DevelopmentPluginRepository.GRADLE_BUILD_DIR));
     }
 
     @Test
-    public void testGetPluginArchivesInDevelopmentMode() {
+    void testGetPluginArchivesInDevelopmentMode() {
         PluginRepository repository = new DevelopmentPluginRepository(pluginsPath);
 
         List<Path> pluginPaths = repository.getPluginPaths();
index f966523fa4f782137bdd213759c2c2f09c07f9af..542e56e822ad0736beb8e37f22e5611209472110 100644 (file)
@@ -36,9 +36,9 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
  * @author Mario Franco
  * @author Decebal Suiu
  */
-public class ExtensionAnnotationProcessorTest {
+class ExtensionAnnotationProcessorTest {
 
-    public static final JavaFileObject Greeting = JavaFileObjects.forSourceLines(
+    static final JavaFileObject Greeting = JavaFileObjects.forSourceLines(
         "Greeting",
         "package test;",
         "import org.pf4j.ExtensionPoint;",
@@ -47,7 +47,7 @@ public class ExtensionAnnotationProcessorTest {
         "   String getGreeting();",
         "}");
 
-    public static final JavaFileObject WhazzupGreeting = JavaFileObjects.forSourceLines(
+    static final JavaFileObject WhazzupGreeting = JavaFileObjects.forSourceLines(
         "WhazzupGreeting",
         "package test;",
         "import org.pf4j.Extension;",
@@ -60,7 +60,7 @@ public class ExtensionAnnotationProcessorTest {
         "    }",
         "}");
 
-    public static final JavaFileObject WhazzupGreeting_NoExtensionPoint = JavaFileObjects.forSourceLines(
+    static final JavaFileObject WhazzupGreeting_NoExtensionPoint = JavaFileObjects.forSourceLines(
         "WhazzupGreeting",
         "package test;",
         "import org.pf4j.Extension;",
@@ -73,7 +73,7 @@ public class ExtensionAnnotationProcessorTest {
         "    }",
         "}");
 
-    public static final JavaFileObject SpinnakerExtension = JavaFileObjects.forSourceLines(
+    static final JavaFileObject SpinnakerExtension = JavaFileObjects.forSourceLines(
         "SpinnakerExtension",
         "package test;",
         "",
@@ -91,7 +91,7 @@ public class ExtensionAnnotationProcessorTest {
         "public @interface SpinnakerExtension {",
         "}");
 
-    public static final JavaFileObject WhazzupGreeting_SpinnakerExtension = JavaFileObjects.forSourceLines(
+    static final JavaFileObject WhazzupGreeting_SpinnakerExtension = JavaFileObjects.forSourceLines(
         "WhazzupGreeting",
         "package test;",
         "",
@@ -106,7 +106,7 @@ public class ExtensionAnnotationProcessorTest {
     /**
      * The same like {@link #SpinnakerExtension} but without {@code Extension} annotation.
      */
-    public static final JavaFileObject SpinnakerExtension_NoExtension = JavaFileObjects.forSourceLines(
+    static final JavaFileObject SpinnakerExtension_NoExtension = JavaFileObjects.forSourceLines(
         "SpinnakerExtension",
         "package test;",
         "",
@@ -125,7 +125,7 @@ public class ExtensionAnnotationProcessorTest {
         "}");
 
     @Test
-    public void getSupportedAnnotationTypes() {
+    void getSupportedAnnotationTypes() {
         ExtensionAnnotationProcessor instance = new ExtensionAnnotationProcessor();
         Set<String> result = instance.getSupportedAnnotationTypes();
         assertEquals(1, result.size());
@@ -133,14 +133,14 @@ public class ExtensionAnnotationProcessorTest {
     }
 
     @Test
-    public void getSupportedOptions() {
+    void getSupportedOptions() {
         ExtensionAnnotationProcessor instance = new ExtensionAnnotationProcessor();
         Set<String> result = instance.getSupportedOptions();
         assertEquals(2, result.size());
     }
 
     @Test
-    public void options() {
+    void options() {
         ExtensionAnnotationProcessor processor = new ExtensionAnnotationProcessor();
         Compilation compilation = javac().withProcessors(processor).withOptions("-Ab=2", "-Ac=3")
             .compile(Greeting, WhazzupGreeting);
@@ -152,7 +152,7 @@ public class ExtensionAnnotationProcessorTest {
     }
 
     @Test
-    public void storage() {
+    void storage() {
         ExtensionAnnotationProcessor processor = new ExtensionAnnotationProcessor();
         Compilation compilation = javac().withProcessors(processor).compile(Greeting, WhazzupGreeting);
         assertEquals(compilation.status(), Compilation.Status.SUCCESS);
@@ -160,14 +160,14 @@ public class ExtensionAnnotationProcessorTest {
     }
 
     @Test
-    public void compileWithoutError() {
+    void compileWithoutError() {
         ExtensionAnnotationProcessor processor = new ExtensionAnnotationProcessor();
          Compilation compilation = javac().withProcessors(processor).compile(Greeting, WhazzupGreeting);
          assertThat(compilation).succeededWithoutWarnings();
     }
 
     @Test
-    public void compileWithError() {
+    void compileWithError() {
         ExtensionAnnotationProcessor processor = new ExtensionAnnotationProcessor();
         Compilation compilation = javac().withProcessors(processor).compile(Greeting, WhazzupGreeting_NoExtensionPoint);
         assertThat(compilation).failed();
@@ -178,7 +178,7 @@ public class ExtensionAnnotationProcessorTest {
     }
 
     @Test
-    public void getExtensions() {
+    void getExtensions() {
         ExtensionAnnotationProcessor processor = new ExtensionAnnotationProcessor();
         Compilation compilation = javac().withProcessors(processor).compile(Greeting, WhazzupGreeting);
         assertThat(compilation).succeededWithoutWarnings();
@@ -188,7 +188,7 @@ public class ExtensionAnnotationProcessorTest {
     }
 
     @Test
-    public void compileNestedExtensionAnnotation() {
+    void compileNestedExtensionAnnotation() {
         ExtensionAnnotationProcessor processor = new ExtensionAnnotationProcessor();
         Compilation compilation = javac().withProcessors(processor).compile(Greeting, SpinnakerExtension, WhazzupGreeting_SpinnakerExtension);
         assertThat(compilation).succeededWithoutWarnings();
index 589b0eaa460a3324bca31389504a641799355832..42e4cde2ee06977a0f9df9ed2e832ddf53a902b0 100644 (file)
@@ -42,7 +42,7 @@ public class JarPluginManagerTest {
     Path pluginsPath;
 
     @BeforeEach
-    public void setUp() throws IOException {
+    void setUp() throws IOException {
         pluginJar = new PluginJar.Builder(pluginsPath.resolve("test-plugin.jar"), "test-plugin")
             .pluginClass(TestPlugin.class.getName())
             .pluginVersion("1.2.3")
@@ -53,7 +53,7 @@ public class JarPluginManagerTest {
     }
 
     @AfterEach
-    public void tearDown() {
+    void tearDown() {
         pluginManager.unloadPlugins();
 
         pluginJar = null;
@@ -61,7 +61,7 @@ public class JarPluginManagerTest {
     }
 
     @Test
-    public void getExtensions() {
+    void getExtensions() {
         pluginManager.loadPlugins();
         pluginManager.startPlugins();
 
@@ -73,7 +73,7 @@ public class JarPluginManagerTest {
     }
 
     @Test
-    public void unloadPlugin() throws Exception {
+    void unloadPlugin() throws Exception {
         pluginManager.loadPlugins();
 
         assertEquals(1, pluginManager.getPlugins().size());
@@ -85,7 +85,7 @@ public class JarPluginManagerTest {
     }
 
     @Test
-    public void deletePlugin() {
+    void deletePlugin() {
         pluginManager.loadPlugins();
 
         assertEquals(1, pluginManager.getPlugins().size());
index 4501223ddfef7a65e1ccf1737d883d96d208a4f4..a9e3e57855851097e9a20c27a8f194b1307c804d 100644 (file)
@@ -29,7 +29,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 /**
  * @author Decebal Suiu
  */
-public class JarPluginRepositoryTest {
+class JarPluginRepositoryTest {
 
     @TempDir
     Path pluginsPath;
@@ -38,7 +38,7 @@ public class JarPluginRepositoryTest {
      * Test of {@link JarPluginRepository#deletePluginPath(Path)} method.
      */
     @Test
-    public void testDeletePluginPath() throws Exception {
+    void testDeletePluginPath() throws Exception {
         PluginRepository repository = new JarPluginRepository(pluginsPath);
 
         Path plugin1Path = Files.createDirectory(pluginsPath.resolve("plugin-1"));
index f48a3a717e79d1e9e170171555642bbb18e9c3ce..2d59911dbdaf50adcb9872123af1f0501ab9eab4 100644 (file)
@@ -35,14 +35,14 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.condition.OS.WINDOWS;
 
-public class LegacyExtensionFinderTest {
+class LegacyExtensionFinderTest {
 
     @TempDir
     Path pluginsPath;
 
     @Test
     @EnabledOnOs(WINDOWS)
-    public void shouldUnlockFileAfterReadingExtensionsFromPlugin() throws Exception {
+    void shouldUnlockFileAfterReadingExtensionsFromPlugin() throws Exception {
         PluginJar pluginJar = new PluginJar.Builder(pluginsPath.resolve("test-plugin.jar"), "test-plugin")
                 .pluginClass(TestPlugin.class.getName())
                 .pluginVersion("1.2.3")
index 1acc567ea1ff9430b0e2c5b66a9da67201e78f52..130cbdbfa1789a961e54721fbd11bb64c1e3a179 100644 (file)
@@ -29,13 +29,13 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
 /**
  * @author Decebal Suiu
  */
-public class LegacyExtensionStorageTest {
+class LegacyExtensionStorageTest {
 
     /**
      * Test of {@link LegacyExtensionStorage#read(Reader, Set)}.
      */
     @Test
-    public void testRead() throws IOException {
+    void read() throws IOException {
         Reader reader = new StringReader(
             "# comment\n"
                 + "org.pf4j.demo.hello.HelloPlugin$HelloGreeting\n"
index 9396f15bd20bcee4fbaf5c844b32908614533114..80d7ca8681b7015f323bdd2cc1b4c8f5d6146cb0 100644 (file)
@@ -28,7 +28,7 @@ import java.nio.file.Path;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-public class LoadPluginsFromMultipleRootsTest {
+class LoadPluginsFromMultipleRootsTest {
 
     private DefaultPluginManager pluginManager;
 
@@ -36,20 +36,20 @@ public class LoadPluginsFromMultipleRootsTest {
     Path pluginsPath2;
 
     @BeforeEach
-    public void setUp() throws IOException {
+    void setUp() throws IOException {
         pluginsPath1 = Files.createTempDirectory("junit-pf4j-");
         pluginsPath2 = Files.createTempDirectory("junit-pf4j-");
         pluginManager = new DefaultPluginManager(pluginsPath1, pluginsPath2);
     }
 
     @AfterEach
-    public void tearDown() throws IOException {
+    void tearDown() throws IOException {
         FileUtils.delete(pluginsPath1);
         FileUtils.delete(pluginsPath2);
     }
 
     @Test
-    public void load() throws Exception {
+    void load() throws Exception {
         PluginZip pluginZip1 = new PluginZip.Builder(pluginsPath1.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
index ad966354ff2238e4381d9fc61d2b0c46c610d139..363acc368197d0832d08211b69d92b722a704965 100644 (file)
@@ -37,7 +37,7 @@ import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assertions.fail;
 
-public class LoadPluginsTest {
+class LoadPluginsTest {
 
     private DefaultPluginManager pluginManager;
 
@@ -45,12 +45,12 @@ public class LoadPluginsTest {
     Path pluginsPath;
 
     @BeforeEach
-    public void setUp() {
+    void setUp() {
         pluginManager = new DefaultPluginManager(pluginsPath);
     }
 
     @Test
-    public void load() throws Exception {
+    void load() throws Exception {
         PluginZip pluginZip = new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
@@ -66,12 +66,12 @@ public class LoadPluginsTest {
     }
 
     @Test
-    public void loadNonExisting() {
+    void loadNonExisting() {
         assertThrows(IllegalArgumentException.class, () -> pluginManager.loadPlugin(Paths.get("nonexisting")));
     }
 
     @Test
-    public void loadTwiceFails() throws Exception {
+    void loadTwiceFails() throws Exception {
         PluginZip pluginZip = new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
@@ -82,7 +82,7 @@ public class LoadPluginsTest {
     }
 
     @Test
-    public void loadPluginWithSameIdDifferentPathFails() throws Exception {
+    void loadPluginWithSameIdDifferentPathFails() throws Exception {
         String pluginId = "myPlugin";
         String pluginVersion = "1.2.3";
         Path plugin1Path = pluginsPath.resolve("my-plugin-1.2.3.zip");
@@ -121,7 +121,7 @@ public class LoadPluginsTest {
      * uses pluginId as the unique identifier of the loaded plugin.
      */
     @Test
-    public void loadPluginWithSameIdDifferentVersionsFails() throws Exception {
+    void loadPluginWithSameIdDifferentVersionsFails() throws Exception {
         String pluginId = "myPlugin";
         String plugin1Version = "1.2.3";
         Path plugin1Path = pluginsPath.resolve("my-plugin-1.2.3.zip");
@@ -151,7 +151,7 @@ public class LoadPluginsTest {
     }
 
     @Test
-    public void loadUnloadLoad() throws Exception {
+    void loadUnloadLoad() throws Exception {
         PluginZip pluginZip = new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
@@ -168,7 +168,7 @@ public class LoadPluginsTest {
     }
 
     @Test
-    public void upgrade() throws Exception {
+    void upgrade() throws Exception {
         String pluginId = "myPlugin";
 
         new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), pluginId)
@@ -197,17 +197,17 @@ public class LoadPluginsTest {
     }
 
     @Test
-    public void getRoot() {
+    void getRoot() {
         assertEquals(pluginsPath, pluginManager.getPluginsRoot());
     }
 
     @Test
-    public void getRoots() {
+    void getRoots() {
         assertEquals(Collections.singletonList(pluginsPath), pluginManager.getPluginsRoots());
     }
 
     @Test
-    public void notAPlugin() {
+    void notAPlugin() {
         pluginsPath.resolve("not-a-zip");
 
         pluginManager.loadPlugins();
@@ -216,7 +216,7 @@ public class LoadPluginsTest {
     }
 
     @Test
-    public void deletePlugin() throws Exception {
+    void deletePlugin() throws Exception {
         PluginZip pluginZip1 = new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();
index 0d19b55257476f3932b90e2266488e1e66e39dd7..73919f710da740423864a6900f0e72e4ecf588fa 100644 (file)
@@ -20,7 +20,6 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
 import org.pf4j.test.PluginJar;
 
-import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.OutputStream;
 import java.nio.file.Files;
@@ -37,7 +36,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
  * @author Mario Franco
  * @author Decebal Suiu
  */
-public class ManifestPluginDescriptorFinderTest {
+class ManifestPluginDescriptorFinderTest {
 
     private VersionManager versionManager;
 
@@ -45,7 +44,7 @@ public class ManifestPluginDescriptorFinderTest {
     Path pluginsPath;
 
     @BeforeEach
-    public void setUp() throws IOException {
+    void setUp() throws IOException {
         Path pluginPath = Files.createDirectories(pluginsPath.resolve("test-plugin-1"));
         storeManifestToPath(getPlugin1Manifest(), pluginPath);
 
@@ -74,7 +73,7 @@ public class ManifestPluginDescriptorFinderTest {
      * Test of {@link ManifestPluginDescriptorFinder#find(Path)} method.
      */
     @Test
-    public void testFind() throws Exception {
+    void find() throws Exception {
         PluginDescriptorFinder descriptorFinder = new ManifestPluginDescriptorFinder();
 
         PluginDescriptor plugin1 = descriptorFinder.find(pluginsPath.resolve("test-plugin-1"));
@@ -105,7 +104,7 @@ public class ManifestPluginDescriptorFinderTest {
      * Test of {@link ManifestPluginDescriptorFinder#find(Path)} method.
      */
     @Test
-    public void testFindNotFound() {
+    void findNotFound() {
         PluginDescriptorFinder descriptorFinder = new ManifestPluginDescriptorFinder();
         assertThrows(PluginRuntimeException.class, () -> descriptorFinder.find(pluginsPath.resolve("test-plugin-3")));
     }
index 229b0ce05a7f25d946bf1c9f4c95cc53f6dbaac3..f97da55d52a5c171af2fef61016393b6caf76ade 100644 (file)
@@ -43,7 +43,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 /**
  * @author Sebastian Lövdahl
  */
-public class PluginClassLoaderTest {
+class PluginClassLoaderTest {
 
     private TestPluginManager pluginManager;
     private TestPluginManager pluginManagerParentFirst;
index 662c28733c993dbfd2997d4832f7e5cdfd552983..75a9adf31cd075ea06b7edd4202df6b7db1c732b 100644 (file)
@@ -29,7 +29,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 /**
  * @author Mario Franco
  */
-public class PluginDependencyTest {
+class PluginDependencyTest {
 
     private DefaultPluginManager pluginManager;
 
@@ -37,15 +37,15 @@ public class PluginDependencyTest {
     Path pluginsPath;
 
     @BeforeEach
-    public void setUp() {
+    void setUp() {
         pluginManager = new DefaultPluginManager(pluginsPath);
     }
 
     /**
-     * Test of getPluginId method, of class PluginDependency.
+     * Test of {@code getPluginId} method, of class {@link PluginDependency}.
      */
     @Test
-    public void testPluginDependecy() {
+    void pluginDependency() {
         PluginDependency instance = new PluginDependency("test");
         PluginDependency instance2 = new PluginDependency("test");
         assertEquals(instance, instance2);
@@ -92,7 +92,7 @@ public class PluginDependencyTest {
     }
 
     @Test
-    public void dependentStop() throws Exception {
+    void dependentStop() throws Exception {
         // B depends on A
         PluginZip pluginA = new PluginZip.Builder(pluginsPath.resolve("A-plugin-1.2.3.zip"), "plugin.a")
             .pluginVersion("1.2.3").build();
@@ -121,7 +121,7 @@ public class PluginDependencyTest {
     }
 
     @Test
-    public void dependentUnload() throws Exception {
+    void dependentUnload() throws Exception {
         // B depends on A
         PluginZip pluginA = new PluginZip.Builder(pluginsPath.resolve("A-plugin-1.2.3.zip"), "plugin.a")
             .pluginVersion("1.2.3").build();
index 0862904e5f60a49783c71a4d9e52dcb4c7bcfcea..4513b04af9a8d96bc598226b12d8d683eb8325f4 100644 (file)
@@ -35,7 +35,7 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-public class PropertiesPluginDescriptorFinderTest {
+class PropertiesPluginDescriptorFinderTest {
 
     private VersionManager versionManager;
 
@@ -43,7 +43,7 @@ public class PropertiesPluginDescriptorFinderTest {
     Path pluginsPath;
 
     @BeforeEach
-    public void setUp() throws IOException {
+    void setUp() throws IOException {
         Path pluginPath = Files.createDirectory(pluginsPath.resolve("test-plugin-1"));
         storePropertiesToPath(getPlugin1Properties(), pluginPath);
 
@@ -69,7 +69,7 @@ public class PropertiesPluginDescriptorFinderTest {
     }
 
     @Test
-    public void testFind() throws Exception {
+    void find() throws Exception {
         PluginDescriptorFinder descriptorFinder = new PropertiesPluginDescriptorFinder();
 
         PluginDescriptor plugin1 = descriptorFinder.find(pluginsPath.resolve("test-plugin-1"));
@@ -100,7 +100,7 @@ public class PropertiesPluginDescriptorFinderTest {
     }
 
     @Test
-    public void testNotFound() {
+    void notFound() {
         PluginDescriptorFinder descriptorFinder = new PropertiesPluginDescriptorFinder();
         assertThrows(PluginRuntimeException.class, () -> descriptorFinder.find(pluginsPath.resolve("test-plugin-3")));
     }
index f42d83e3ba1c9c986821b565c104dd4d90327238..eca280a2fdc408e0c241441071a9b0944458edea 100644 (file)
-package org.pf4j;\r
-\r
-import static org.junit.jupiter.api.Assertions.assertTrue;\r
-import static org.junit.Assert.assertNotNull;\r
-import static org.junit.jupiter.api.Assertions.assertEquals;\r
-import static org.junit.jupiter.api.Assertions.assertThrows;\r
-\r
-import java.io.IOException;\r
-import java.nio.file.Path;\r
-import java.util.List;\r
-\r
-import org.junit.jupiter.api.AfterEach;\r
-import org.junit.jupiter.api.BeforeEach;\r
-import org.junit.jupiter.api.Test;\r
-import org.junit.jupiter.api.io.TempDir;\r
-import org.pf4j.test.PluginJar;\r
-import org.pf4j.test.TestExtension;\r
-import org.pf4j.test.TestExtensionPoint;\r
-import org.pf4j.test.TestPlugin;\r
-\r
-public class SecurePluginManagerWrapperTest {\r
-\r
-    private static final String OTHER_PLUGIN_ID = "test-plugin-2";\r
-    private static final String THIS_PLUGIN_ID = "test-plugin-1";\r
-    private PluginJar thisPlugin;\r
-    private PluginJar otherPlugin;\r
-    private PluginManager pluginManager;\r
-    private PluginManager wrappedPluginManager;\r
-    private int pluginManagerEvents = 0;\r
-    private int wrappedPluginManagerEvents = 0;\r
-\r
-    @TempDir\r
-    Path pluginsPath;\r
-\r
-    @BeforeEach\r
-    public void setUp() throws IOException {\r
-        pluginManagerEvents = 0;\r
-        wrappedPluginManagerEvents = 0;\r
-        thisPlugin = new PluginJar.Builder(pluginsPath.resolve("test-plugin1.jar"), THIS_PLUGIN_ID).pluginClass(TestPlugin.class.getName()).pluginVersion("1.2.3").extension(TestExtension.class.getName()).build();\r
-        otherPlugin = new PluginJar.Builder(pluginsPath.resolve("test-plugin2.jar"), OTHER_PLUGIN_ID).pluginClass(TestPlugin.class.getName()).pluginVersion("1.2.3").extension(TestExtension.class.getName()).build();\r
-\r
-        pluginManager = new JarPluginManager(pluginsPath);\r
-        wrappedPluginManager = new SecurePluginManagerWrapper(pluginManager, THIS_PLUGIN_ID);\r
-    }\r
-\r
-    @AfterEach\r
-    public void tearDown() {\r
-        pluginManager.unloadPlugins();\r
-\r
-        thisPlugin = null;\r
-        otherPlugin = null;\r
-        pluginManager = null;\r
-    }\r
-\r
-    @Test\r
-    public void pluginStateListeners() {\r
-        pluginManager.addPluginStateListener(new PluginStateListener() {\r
-            @Override\r
-            public void pluginStateChanged(PluginStateEvent event) {\r
-                pluginManagerEvents++;\r
-            }\r
-        });\r
-        wrappedPluginManager.addPluginStateListener(new PluginStateListener() {\r
-            @Override\r
-            public void pluginStateChanged(PluginStateEvent event) {\r
-                wrappedPluginManagerEvents++;\r
-            }\r
-        });\r
-        pluginManager.loadPlugins();\r
-        pluginManager.startPlugins();\r
-        assertEquals(4, pluginManagerEvents);\r
-        assertEquals(2, wrappedPluginManagerEvents);\r
-    }\r
-\r
-    @Test\r
-    public void deletePlugin() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.deletePlugin(OTHER_PLUGIN_ID));\r
-        assertTrue(wrappedPluginManager.deletePlugin(THIS_PLUGIN_ID));\r
-    }\r
-\r
-    @Test\r
-    public void disablePlugin() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.disablePlugin(OTHER_PLUGIN_ID));\r
-        assertTrue(wrappedPluginManager.disablePlugin(THIS_PLUGIN_ID));\r
-    }\r
-\r
-    @Test\r
-    public void enablePlugin() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.enablePlugin(OTHER_PLUGIN_ID));\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.enablePlugin(THIS_PLUGIN_ID));\r
-    }\r
-\r
-    @Test\r
-    public void getExtensionClasses() {\r
-        pluginManager.loadPlugins();\r
-        pluginManager.startPlugins();\r
-        assertEquals(1, wrappedPluginManager.getExtensionClasses(TestExtensionPoint.class).size());\r
-\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensionClasses(TestExtensionPoint.class, OTHER_PLUGIN_ID));\r
-        assertEquals(1, wrappedPluginManager.getExtensionClasses(TestExtensionPoint.class, THIS_PLUGIN_ID).size());\r
-\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensionClasses(OTHER_PLUGIN_ID));\r
-        assertEquals(1, wrappedPluginManager.getExtensionClasses(THIS_PLUGIN_ID).size());\r
-    }\r
-\r
-    @Test\r
-    public void getExtensionClassNames() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensionClassNames(OTHER_PLUGIN_ID));\r
-        assertEquals(1, wrappedPluginManager.getExtensionClassNames(THIS_PLUGIN_ID).size());\r
-    }\r
-\r
-    @Test\r
-    public void getExtensionFactory() {\r
-        pluginManager.loadPlugins();\r
-        assertEquals(pluginManager.getExtensionFactory(), wrappedPluginManager.getExtensionFactory());\r
-    }\r
-\r
-    @Test\r
-    public void getExtensions() {\r
-        pluginManager.loadPlugins();\r
-        pluginManager.startPlugins();\r
-        assertEquals(1, wrappedPluginManager.getExtensions(TestExtensionPoint.class).size());\r
-\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensions(TestExtensionPoint.class, OTHER_PLUGIN_ID));\r
-        assertEquals(1, wrappedPluginManager.getExtensions(TestExtensionPoint.class, THIS_PLUGIN_ID).size());\r
-\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensions(OTHER_PLUGIN_ID));\r
-        assertEquals(1, wrappedPluginManager.getExtensions(THIS_PLUGIN_ID).size());\r
-    }\r
-\r
-    @Test\r
-    public void getPlugin() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPlugin(OTHER_PLUGIN_ID));\r
-        assertEquals(THIS_PLUGIN_ID, wrappedPluginManager.getPlugin(THIS_PLUGIN_ID).getPluginId());\r
-    }\r
-\r
-    @Test\r
-    public void getPluginClassLoader() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPluginClassLoader(OTHER_PLUGIN_ID));\r
-        assertNotNull(wrappedPluginManager.getPluginClassLoader(THIS_PLUGIN_ID));\r
-    }\r
-\r
-    @Test\r
-    public void getPlugins() {\r
-        pluginManager.loadPlugins();\r
-        assertEquals(2, pluginManager.getPlugins().size());\r
-        assertEquals(1, wrappedPluginManager.getPlugins().size());\r
-    }\r
-\r
-    @Test\r
-    public void getPluginsRoot() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPluginsRoot());\r
-    }\r
-\r
-    @Test\r
-    public void getPluginsRoots() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPluginsRoots());\r
-    }\r
-\r
-    @Test\r
-    public void getResolvedPlugins() {\r
-        pluginManager.loadPlugins();\r
-        assertEquals(2, pluginManager.getResolvedPlugins().size());\r
-        assertEquals(1, wrappedPluginManager.getResolvedPlugins().size());\r
-    }\r
-\r
-    @Test\r
-    public void getRuntimeMode() {\r
-        assertEquals(pluginManager.getRuntimeMode(), wrappedPluginManager.getRuntimeMode());\r
-    }\r
-\r
-    @Test\r
-    public void getStartedPlugins() {\r
-        pluginManager.loadPlugins();\r
-        pluginManager.startPlugins();\r
-        assertEquals(2, pluginManager.getStartedPlugins().size());\r
-        assertEquals(1, wrappedPluginManager.getStartedPlugins().size());\r
-    }\r
-\r
-    @Test\r
-    public void getSystemVersion() {\r
-        assertEquals(pluginManager.getSystemVersion(), wrappedPluginManager.getSystemVersion());\r
-    }\r
-\r
-    @Test\r
-    public void getUnresolvedPlugins() {\r
-        assertNotNull(wrappedPluginManager);\r
-        assertNotNull(wrappedPluginManager.getUnresolvedPlugins());\r
-        assertTrue(wrappedPluginManager.getUnresolvedPlugins().isEmpty());\r
-    }\r
-\r
-    @Test\r
-    public void getVersionManager() {\r
-        assertEquals(pluginManager.getVersionManager(), wrappedPluginManager.getVersionManager());\r
-    }\r
-\r
-    @Test\r
-    public void isDevelopment() {\r
-        assertEquals(pluginManager.isDevelopment(), wrappedPluginManager.isDevelopment());\r
-    }\r
-\r
-    @Test\r
-    public void isNotDevelopment() {\r
-        assertEquals(pluginManager.isNotDevelopment(), wrappedPluginManager.isNotDevelopment());\r
-    }\r
-\r
-    @Test\r
-    public void loadPlugin() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.loadPlugin(thisPlugin.path()));\r
-    }\r
-\r
-    @Test\r
-    public void loadPlugins() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.loadPlugins());\r
-    }\r
-\r
-    @Test\r
-    public void setSystemVersion() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.setSystemVersion("1.0.0"));\r
-    }\r
-\r
-    @Test\r
-    public void startPlugin() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.startPlugin(OTHER_PLUGIN_ID));\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.startPlugin(THIS_PLUGIN_ID));\r
-    }\r
-\r
-    @Test\r
-    public void startPlugins() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.startPlugins());\r
-    }\r
-\r
-    @Test\r
-    public void stopPlugin() {\r
-        pluginManager.loadPlugins();\r
-        pluginManager.startPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.stopPlugin(OTHER_PLUGIN_ID));\r
-        assertEquals(PluginState.STOPPED, wrappedPluginManager.stopPlugin(THIS_PLUGIN_ID));\r
-    }\r
-\r
-    @Test\r
-    public void stopPlugins() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.stopPlugins());\r
-    }\r
-\r
-    @Test\r
-    public void unloadPlugin() {\r
-        pluginManager.loadPlugins();\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.unloadPlugin(OTHER_PLUGIN_ID));\r
-        assertTrue(wrappedPluginManager.unloadPlugin(THIS_PLUGIN_ID));\r
-    }\r
-\r
-    @Test\r
-    public void unloadPlugins() {\r
-        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.unloadPlugins());\r
-    }\r
-\r
-    @Test\r
-    public void whichPlugin() {\r
-        pluginManager.loadPlugins();\r
-        pluginManager.startPlugins();\r
-        assertEquals(null, wrappedPluginManager.whichPlugin(pluginManager.getExtensionClasses(OTHER_PLUGIN_ID).get(0)));\r
-        assertEquals(THIS_PLUGIN_ID, wrappedPluginManager.whichPlugin(pluginManager.getExtensionClasses(THIS_PLUGIN_ID).get(0)).getPluginId());\r
-    }\r
-}\r
+package org.pf4j;
+
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.io.TempDir;
+import org.pf4j.test.PluginJar;
+import org.pf4j.test.TestExtension;
+import org.pf4j.test.TestExtensionPoint;
+import org.pf4j.test.TestPlugin;
+
+import java.io.IOException;
+import java.nio.file.Path;
+
+import static org.junit.Assert.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+class SecurePluginManagerWrapperTest {
+
+    private static final String OTHER_PLUGIN_ID = "test-plugin-2";
+    private static final String THIS_PLUGIN_ID = "test-plugin-1";
+
+    private PluginJar thisPlugin;
+    private PluginJar otherPlugin;
+    private PluginManager pluginManager;
+    private PluginManager wrappedPluginManager;
+    private int pluginManagerEvents = 0;
+    private int wrappedPluginManagerEvents = 0;
+
+    @TempDir
+    Path pluginsPath;
+
+    @BeforeEach
+    void setUp() throws IOException {
+        pluginManagerEvents = 0;
+        wrappedPluginManagerEvents = 0;
+        thisPlugin = new PluginJar.Builder(pluginsPath.resolve("test-plugin1.jar"), THIS_PLUGIN_ID).pluginClass(TestPlugin.class.getName()).pluginVersion("1.2.3").extension(TestExtension.class.getName()).build();
+        otherPlugin = new PluginJar.Builder(pluginsPath.resolve("test-plugin2.jar"), OTHER_PLUGIN_ID).pluginClass(TestPlugin.class.getName()).pluginVersion("1.2.3").extension(TestExtension.class.getName()).build();
+
+        pluginManager = new JarPluginManager(pluginsPath);
+        wrappedPluginManager = new SecurePluginManagerWrapper(pluginManager, THIS_PLUGIN_ID);
+    }
+
+    @AfterEach
+    void tearDown() {
+        pluginManager.unloadPlugins();
+
+        thisPlugin = null;
+        otherPlugin = null;
+        pluginManager = null;
+    }
+
+    @Test
+    void pluginStateListeners() {
+        pluginManager.addPluginStateListener(event -> pluginManagerEvents++);
+        wrappedPluginManager.addPluginStateListener(event -> wrappedPluginManagerEvents++);
+        pluginManager.loadPlugins();
+        pluginManager.startPlugins();
+        assertEquals(4, pluginManagerEvents);
+        assertEquals(2, wrappedPluginManagerEvents);
+    }
+
+    @Test
+    void deletePlugin() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.deletePlugin(OTHER_PLUGIN_ID));
+        assertTrue(wrappedPluginManager.deletePlugin(THIS_PLUGIN_ID));
+    }
+
+    @Test
+    void disablePlugin() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.disablePlugin(OTHER_PLUGIN_ID));
+        assertTrue(wrappedPluginManager.disablePlugin(THIS_PLUGIN_ID));
+    }
+
+    @Test
+    void enablePlugin() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.enablePlugin(OTHER_PLUGIN_ID));
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.enablePlugin(THIS_PLUGIN_ID));
+    }
+
+    @Test
+    void getExtensionClasses() {
+        pluginManager.loadPlugins();
+        pluginManager.startPlugins();
+        assertEquals(1, wrappedPluginManager.getExtensionClasses(TestExtensionPoint.class).size());
+
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensionClasses(TestExtensionPoint.class, OTHER_PLUGIN_ID));
+        assertEquals(1, wrappedPluginManager.getExtensionClasses(TestExtensionPoint.class, THIS_PLUGIN_ID).size());
+
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensionClasses(OTHER_PLUGIN_ID));
+        assertEquals(1, wrappedPluginManager.getExtensionClasses(THIS_PLUGIN_ID).size());
+    }
+
+    @Test
+    void getExtensionClassNames() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensionClassNames(OTHER_PLUGIN_ID));
+        assertEquals(1, wrappedPluginManager.getExtensionClassNames(THIS_PLUGIN_ID).size());
+    }
+
+    @Test
+    void getExtensionFactory() {
+        pluginManager.loadPlugins();
+        assertEquals(pluginManager.getExtensionFactory(), wrappedPluginManager.getExtensionFactory());
+    }
+
+    @Test
+    void getExtensions() {
+        pluginManager.loadPlugins();
+        pluginManager.startPlugins();
+        assertEquals(1, wrappedPluginManager.getExtensions(TestExtensionPoint.class).size());
+
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensions(TestExtensionPoint.class, OTHER_PLUGIN_ID));
+        assertEquals(1, wrappedPluginManager.getExtensions(TestExtensionPoint.class, THIS_PLUGIN_ID).size());
+
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getExtensions(OTHER_PLUGIN_ID));
+        assertEquals(1, wrappedPluginManager.getExtensions(THIS_PLUGIN_ID).size());
+    }
+
+    @Test
+    void getPlugin() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPlugin(OTHER_PLUGIN_ID));
+        assertEquals(THIS_PLUGIN_ID, wrappedPluginManager.getPlugin(THIS_PLUGIN_ID).getPluginId());
+    }
+
+    @Test
+    void getPluginClassLoader() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPluginClassLoader(OTHER_PLUGIN_ID));
+        assertNotNull(wrappedPluginManager.getPluginClassLoader(THIS_PLUGIN_ID));
+    }
+
+    @Test
+    void getPlugins() {
+        pluginManager.loadPlugins();
+        assertEquals(2, pluginManager.getPlugins().size());
+        assertEquals(1, wrappedPluginManager.getPlugins().size());
+    }
+
+    @Test
+    void getPluginsRoot() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPluginsRoot());
+    }
+
+    @Test
+    void getPluginsRoots() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.getPluginsRoots());
+    }
+
+    @Test
+    void getResolvedPlugins() {
+        pluginManager.loadPlugins();
+        assertEquals(2, pluginManager.getResolvedPlugins().size());
+        assertEquals(1, wrappedPluginManager.getResolvedPlugins().size());
+    }
+
+    @Test
+    void getRuntimeMode() {
+        assertEquals(pluginManager.getRuntimeMode(), wrappedPluginManager.getRuntimeMode());
+    }
+
+    @Test
+    void getStartedPlugins() {
+        pluginManager.loadPlugins();
+        pluginManager.startPlugins();
+        assertEquals(2, pluginManager.getStartedPlugins().size());
+        assertEquals(1, wrappedPluginManager.getStartedPlugins().size());
+    }
+
+    @Test
+    void getSystemVersion() {
+        assertEquals(pluginManager.getSystemVersion(), wrappedPluginManager.getSystemVersion());
+    }
+
+    @Test
+    void getUnresolvedPlugins() {
+        assertNotNull(wrappedPluginManager);
+        assertNotNull(wrappedPluginManager.getUnresolvedPlugins());
+        assertTrue(wrappedPluginManager.getUnresolvedPlugins().isEmpty());
+    }
+
+    @Test
+    void getVersionManager() {
+        assertEquals(pluginManager.getVersionManager(), wrappedPluginManager.getVersionManager());
+    }
+
+    @Test
+    void isDevelopment() {
+        assertEquals(pluginManager.isDevelopment(), wrappedPluginManager.isDevelopment());
+    }
+
+    @Test
+    void isNotDevelopment() {
+        assertEquals(pluginManager.isNotDevelopment(), wrappedPluginManager.isNotDevelopment());
+    }
+
+    @Test
+    void loadPlugin() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.loadPlugin(thisPlugin.path()));
+    }
+
+    @Test
+    void loadPlugins() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.loadPlugins());
+    }
+
+    @Test
+    void setSystemVersion() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.setSystemVersion("1.0.0"));
+    }
+
+    @Test
+    void startPlugin() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.startPlugin(OTHER_PLUGIN_ID));
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.startPlugin(THIS_PLUGIN_ID));
+    }
+
+    @Test
+    void startPlugins() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.startPlugins());
+    }
+
+    @Test
+    void stopPlugin() {
+        pluginManager.loadPlugins();
+        pluginManager.startPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.stopPlugin(OTHER_PLUGIN_ID));
+        assertEquals(PluginState.STOPPED, wrappedPluginManager.stopPlugin(THIS_PLUGIN_ID));
+    }
+
+    @Test
+    void stopPlugins() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.stopPlugins());
+    }
+
+    @Test
+    void unloadPlugin() {
+        pluginManager.loadPlugins();
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.unloadPlugin(OTHER_PLUGIN_ID));
+        assertTrue(wrappedPluginManager.unloadPlugin(THIS_PLUGIN_ID));
+    }
+
+    @Test
+    void unloadPlugins() {
+        assertThrows(IllegalAccessError.class, () -> wrappedPluginManager.unloadPlugins());
+    }
+
+    @Test
+    void whichPlugin() {
+        pluginManager.loadPlugins();
+        pluginManager.startPlugins();
+        assertEquals(null, wrappedPluginManager.whichPlugin(pluginManager.getExtensionClasses(OTHER_PLUGIN_ID).get(0)));
+        assertEquals(THIS_PLUGIN_ID, wrappedPluginManager.whichPlugin(pluginManager.getExtensionClasses(THIS_PLUGIN_ID).get(0)).getPluginId());
+    }
+
+}
index 739a76f08d8a3fdb569010df2326fd84d0500e91..bd8789582f1621a837d7819981062c815e0e02ff 100644 (file)
@@ -31,10 +31,10 @@ import static org.junit.jupiter.api.Assertions.assertSame;
  * @author Decebal Suiu
  * @author Ajith Kumar
  */
-public class SingletonExtensionFactoryTest {
+class SingletonExtensionFactoryTest {
 
     @Test
-    public void create() {
+    void create() {
         ExtensionFactory extensionFactory = new SingletonExtensionFactory();
         Object extensionOne = extensionFactory.create(TestExtension.class);
         Object extensionTwo = extensionFactory.create(TestExtension.class);
@@ -42,7 +42,7 @@ public class SingletonExtensionFactoryTest {
     }
 
     @Test
-    public void createNewEachTime() {
+    void createNewEachTime() {
         ExtensionFactory extensionFactory = new SingletonExtensionFactory(FailTestExtension.class.getName());
         Object extensionOne = extensionFactory.create(TestExtension.class);
         Object extensionTwo = extensionFactory.create(TestExtension.class);
@@ -50,8 +50,7 @@ public class SingletonExtensionFactoryTest {
     }
 
     @Test
-    @SuppressWarnings("unchecked")
-    public void createNewEachTimeFromDifferentClassLoaders() throws Exception {
+    void createNewEachTimeFromDifferentClassLoaders() throws Exception {
         ExtensionFactory extensionFactory = new SingletonExtensionFactory();
 
         // Get classpath locations
@@ -59,9 +58,9 @@ public class SingletonExtensionFactoryTest {
 
         // Create different classloaders for the classpath references and load classes respectively
         ClassLoader klassLoaderOne = new URLClassLoader(classpathReferences, null);
-        Class klassOne = klassLoaderOne.loadClass(TestExtension.class.getName());
+        Class<?> klassOne = klassLoaderOne.loadClass(TestExtension.class.getName());
         ClassLoader klassLoaderTwo = new URLClassLoader(classpathReferences, null);
-        Class klassTwo = klassLoaderTwo.loadClass(TestExtension.class.getName());
+        Class<?> klassTwo = klassLoaderTwo.loadClass(TestExtension.class.getName());
 
         // create instances
         Object instanceOne = extensionFactory.create(klassOne);
@@ -80,6 +79,7 @@ public class SingletonExtensionFactoryTest {
         for (int index = 0; index < classpaths.length; index++) {
             uris[index] = new File(classpaths[index]).toURI().toURL();
         }
+
         return uris;
     }
 
index ac16b2311aad5745da20e3236f646bd8843c9663..63a942ed12f88ba2c3ada56ca51c22bdc0dc83ad 100644 (file)
@@ -38,10 +38,10 @@ import static org.mockito.Mockito.mock;
 /**
  * @author Josiah Haswell
  */
-public class ServiceProviderExtensionStorageTest {
+class ServiceProviderExtensionStorageTest {
 
     @Test
-    public void ensureServiceProviderExtensionStorageReadWorks() throws IOException {
+    void ensureServiceProviderExtensionStorageReadWorks() throws IOException {
         final StringReader file = new StringReader("#hello\n    World");
         final Set<String> entries = new HashSet<>();
         ServiceProviderExtensionStorage.read(file, entries);
@@ -51,7 +51,7 @@ public class ServiceProviderExtensionStorageTest {
     }
 
     @Test
-    public void ensureReadingExtensionsProducesCorrectListOfExtensions() {
+    void ensureReadingExtensionsProducesCorrectListOfExtensions() {
         final StringReader file = new StringReader("#hello\n    World");
         final ExtensionAnnotationProcessor processor = mock(ExtensionAnnotationProcessor.class);
         final Map<String, Set<String>> extensions = new HashMap<>();
index 44b8c5415a1b5ae5197d9b0ddf82b88044b67b3f..ab977a152e042cf0dd4f8a8dea81139481f70798 100644 (file)
@@ -28,7 +28,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
 /**
  * @author Decebal Suiu
  */
-public class DirectedGraphTest {
+class DirectedGraphTest {
 
     private static DirectedGraph<Character> graph;
 
@@ -54,21 +54,21 @@ public class DirectedGraphTest {
     }
 
     @Test
-    public void reverseTopologicalSort() {
+    void reverseTopologicalSort() {
         List<Character> result = graph.reverseTopologicalSort();
         List<Character> expected = Arrays.asList('C', 'G', 'F', 'B', 'A', 'E', 'D');
         assertEquals(expected, result);
     }
 
     @Test
-    public void topologicalSort() {
+    void topologicalSort() {
         List<Character> result = graph.topologicalSort();
         List<Character> expected = Arrays.asList('D', 'E', 'A', 'B', 'F', 'G', 'C');
         assertEquals(expected, result);
     }
 
     @Test
-    public void inDegree() {
+    void inDegree() {
         Map<Character, Integer> result = graph.inDegree();
         Map<Character, Integer> expected = new HashMap<>(7);
         expected.put('A', 0);
@@ -82,7 +82,7 @@ public class DirectedGraphTest {
     }
 
     @Test
-    public void outDegree() {
+    void outDegree() {
         Map<Character, Integer> result = graph.outDegree();
         Map<Character, Integer> expected = new HashMap<>(7);
         expected.put('A', 1);
index 8e41af17015c17ed9bc14d42069351e76f90ab93..078e9465d4423c820fe088e40d956f7393aae893 100644 (file)
@@ -30,13 +30,13 @@ import java.util.stream.Collectors;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-public class FileUtilsTest {
+class FileUtilsTest {
 
     @TempDir
     Path pluginsPath;
 
     @Test
-    public void expandIfZipForZipWithOnlyModuleDescriptor() throws IOException {
+    void expandIfZipForZipWithOnlyModuleDescriptor() throws IOException {
         PluginZip pluginZip = new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
                 .pluginVersion("1.2.3")
                 .build();
@@ -47,7 +47,7 @@ public class FileUtilsTest {
     }
 
     @Test
-    public void expandIfZipForZipWithResourceFile() throws IOException {
+    void expandIfZipForZipWithResourceFile() throws IOException {
         PluginZip pluginZip = new PluginZip.Builder(pluginsPath.resolve("my-second-plugin-1.2.3.zip"), "myPlugin")
                 .pluginVersion("1.2.3")
                 .addFile(Paths.get("classes/META-INF/plugin-file"), "plugin")
@@ -59,7 +59,7 @@ public class FileUtilsTest {
     }
 
     @Test
-    public void expandIfZipNonZipFiles() throws IOException {
+    void expandIfZipNonZipFiles() throws IOException {
         // File without .suffix
         Path extra = pluginsPath.resolve("extra");
         assertEquals(extra, FileUtils.expandIfZip(extra));
@@ -70,7 +70,7 @@ public class FileUtilsTest {
     }
 
     @Test
-    public void findPaths() throws IOException {
+    void findPaths() throws IOException {
         PluginZip pluginZip = new PluginZip.Builder(pluginsPath.resolve("my-plugin-1.2.3.zip"), "myPlugin")
             .pluginVersion("1.2.3")
             .build();