From 2b3180cc348cdc382fe58670a6c3aae1af2b132f Mon Sep 17 00:00:00 2001 From: Sylwester Lachiewicz Date: Mon, 3 Nov 2025 22:52:20 +0000 Subject: [PATCH] JUnit Jupiter best practices Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.java.testing.junit.JupiterBestPractices?organizationId=NzQ1YmJlODUtZjNkMy00OTNkLThhNDAtZWJmZDg4N2U1ZjU1 Co-authored-by: Moderne --- .../AbstractResourceAttributesTCK.java | 12 +++---- .../io/attributes/AttributeUtilsTest.java | 11 +++---- .../io/attributes/FileAttributesTest.java | 7 ++--- .../PlexusIoResourceAttributeUtilsTest.java | 31 +++++++++---------- .../io/attributes/SymlinkUtilsTest.java | 6 ++-- .../io/filemappers/FileMapperTest.java | 18 +++++------ .../io/filemappers/FileSelectorTest.java | 8 ++--- .../io/filemappers/ResourcesTest.java | 8 ++--- ...bstractPlexusIoResourceCollectionTest.java | 4 +-- .../PlexusIoFileResourceDeferredTest.java | 9 +++--- ...sIoFileSupplierResourceCollectionTest.java | 4 +-- .../PlexusIoPlexusIoFileResourceTest.java | 13 ++++---- .../PlexusIoProxyResourceCollectionTest.java | 7 ++--- .../io/resources/proxy/ProxyFactoryTest.java | 18 +++++------ 14 files changed, 74 insertions(+), 82 deletions(-) diff --git a/src/test/java/org/codehaus/plexus/components/io/attributes/AbstractResourceAttributesTCK.java b/src/test/java/org/codehaus/plexus/components/io/attributes/AbstractResourceAttributesTCK.java index 50daec3..af823a9 100644 --- a/src/test/java/org/codehaus/plexus/components/io/attributes/AbstractResourceAttributesTCK.java +++ b/src/test/java/org/codehaus/plexus/components/io/attributes/AbstractResourceAttributesTCK.java @@ -29,7 +29,7 @@ protected AbstractResourceAttributesTCK() {} protected abstract PlexusIoResourceAttributes newAttributes(String mode); @Test - final void testSetOctalModeString_OwnerModes() { + final void setOctalModeStringOwnerModes() { verifyStringOctalModeSet("700", new boolean[] {true, true, true, false, false, false, false, false, false}); verifyStringOctalModeSet("600", new boolean[] {true, true, false, false, false, false, false, false, false}); verifyStringOctalModeSet("400", new boolean[] {true, false, false, false, false, false, false, false, false}); @@ -37,7 +37,7 @@ final void testSetOctalModeString_OwnerModes() { } @Test - final void testSetOctalModeString_GroupModes() { + final void setOctalModeStringGroupModes() { verifyStringOctalModeSet("070", new boolean[] {false, false, false, true, true, true, false, false, false}); verifyStringOctalModeSet("060", new boolean[] {false, false, false, true, true, false, false, false, false}); verifyStringOctalModeSet("040", new boolean[] {false, false, false, true, false, false, false, false, false}); @@ -45,7 +45,7 @@ final void testSetOctalModeString_GroupModes() { } @Test - final void testSetOctalModeString_WorldModes() { + final void setOctalModeStringWorldModes() { verifyStringOctalModeSet("007", new boolean[] {false, false, false, false, false, false, true, true, true}); verifyStringOctalModeSet("006", new boolean[] {false, false, false, false, false, false, true, true, false}); verifyStringOctalModeSet("004", new boolean[] {false, false, false, false, false, false, true, false, false}); @@ -53,7 +53,7 @@ final void testSetOctalModeString_WorldModes() { } @Test - final void testSetOctalMode_OwnerModes() { + final void setOctalModeOwnerModes() { verifyOctalModeSet("700", new boolean[] {true, true, true, false, false, false, false, false, false}); verifyOctalModeSet("600", new boolean[] {true, true, false, false, false, false, false, false, false}); verifyOctalModeSet("400", new boolean[] {true, false, false, false, false, false, false, false, false}); @@ -61,7 +61,7 @@ final void testSetOctalMode_OwnerModes() { } @Test - final void testSetOctalMode_GroupModes() { + final void setOctalModeGroupModes() { verifyOctalModeSet("070", new boolean[] {false, false, false, true, true, true, false, false, false}); verifyOctalModeSet("060", new boolean[] {false, false, false, true, true, false, false, false, false}); verifyOctalModeSet("040", new boolean[] {false, false, false, true, false, false, false, false, false}); @@ -69,7 +69,7 @@ final void testSetOctalMode_GroupModes() { } @Test - final void testSetOctalMode_WorldModes() { + final void setOctalModeWorldModes() { verifyOctalModeSet("007", new boolean[] {false, false, false, false, false, false, true, true, true}); verifyOctalModeSet("006", new boolean[] {false, false, false, false, false, false, true, true, false}); verifyOctalModeSet("004", new boolean[] {false, false, false, false, false, false, true, false, false}); diff --git a/src/test/java/org/codehaus/plexus/components/io/attributes/AttributeUtilsTest.java b/src/test/java/org/codehaus/plexus/components/io/attributes/AttributeUtilsTest.java index 8a56959..49cee7f 100644 --- a/src/test/java/org/codehaus/plexus/components/io/attributes/AttributeUtilsTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/attributes/AttributeUtilsTest.java @@ -1,7 +1,6 @@ package org.codehaus.plexus.components.io.attributes; import java.io.File; -import java.io.IOException; import java.nio.file.attribute.PosixFilePermission; import java.util.Set; @@ -17,9 +16,9 @@ * @author Kristian Rosenvold */ @SuppressWarnings("OctalInteger") -public class AttributeUtilsTest { +class AttributeUtilsTest { @Test - void testMiscPatterns() { + void miscPatterns() { final Set permissions = AttributeUtils.getPermissions(0124); assertTrue(permissions.contains(PosixFilePermission.OWNER_EXECUTE)); assertTrue(permissions.contains(PosixFilePermission.GROUP_WRITE)); @@ -27,7 +26,7 @@ void testMiscPatterns() { } @Test - void testMorePatterns() { + void morePatterns() { final Set permissions = AttributeUtils.getPermissions(0241); assertTrue(permissions.contains(PosixFilePermission.OWNER_WRITE)); assertTrue(permissions.contains(PosixFilePermission.GROUP_READ)); @@ -35,7 +34,7 @@ void testMorePatterns() { } @Test - void testEvenMorePatterns() { + void evenMorePatterns() { final Set permissions = AttributeUtils.getPermissions(0412); assertTrue(permissions.contains(PosixFilePermission.OWNER_READ)); assertTrue(permissions.contains(PosixFilePermission.GROUP_EXECUTE)); @@ -50,7 +49,7 @@ void test777() { @Test @DisabledOnOs(OS.WINDOWS) - void testChmodBackAndForth() throws IOException { + void chmodBackAndForth() throws Exception { final File bxx = File.createTempFile("bxx", "ff"); AttributeUtils.chmod(bxx, 0422); PlexusIoResourceAttributes firstAttrs = new FileAttributes(bxx); diff --git a/src/test/java/org/codehaus/plexus/components/io/attributes/FileAttributesTest.java b/src/test/java/org/codehaus/plexus/components/io/attributes/FileAttributesTest.java index c77e09c..65e917d 100644 --- a/src/test/java/org/codehaus/plexus/components/io/attributes/FileAttributesTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/attributes/FileAttributesTest.java @@ -17,7 +17,6 @@ */ import java.io.File; -import java.io.IOException; import java.nio.file.Files; import org.junit.jupiter.api.Test; @@ -29,17 +28,17 @@ /** * @author Kristian Rosenvold */ -public class FileAttributesTest { +class FileAttributesTest { @Test @DisabledOnOs(OS.WINDOWS) - void testGetPosixFileAttributes() throws Exception { + void getPosixFileAttributes() throws Exception { File file = new File("."); PlexusIoResourceAttributes fa = new FileAttributes(file); assertNotNull(fa); } @Test - void testFileAttributesHandlesIOException() throws IOException { + void fileAttributesHandlesIOException() throws Exception { // Test that FileAttributes can be constructed for a regular file // even if ownership information is not available (e.g., WSL2 mapped network drives) File tempFile = Files.createTempFile("plexus-io-test", ".tmp").toFile(); diff --git a/src/test/java/org/codehaus/plexus/components/io/attributes/PlexusIoResourceAttributeUtilsTest.java b/src/test/java/org/codehaus/plexus/components/io/attributes/PlexusIoResourceAttributeUtilsTest.java index 4712e82..73c5460 100644 --- a/src/test/java/org/codehaus/plexus/components/io/attributes/PlexusIoResourceAttributeUtilsTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/attributes/PlexusIoResourceAttributeUtilsTest.java @@ -17,13 +17,11 @@ */ import java.io.File; -import java.io.IOException; import java.net.URL; import java.nio.file.NoSuchFileException; import java.util.Map; import org.codehaus.plexus.util.StringUtils; -import org.codehaus.plexus.util.cli.CommandLineException; import org.codehaus.plexus.util.cli.CommandLineUtils; import org.codehaus.plexus.util.cli.Commandline; import org.junit.jupiter.api.Test; @@ -34,11 +32,11 @@ import static org.junit.jupiter.api.Assertions.*; @SuppressWarnings("OctalInteger") -public class PlexusIoResourceAttributeUtilsTest { +class PlexusIoResourceAttributeUtilsTest { @Test @DisabledOnOs(OS.WINDOWS) - void testGetAttributesForThisTestClass() throws IOException { + void getAttributesForThisTestClass() throws Exception { URL resource = Thread.currentThread() .getContextClassLoader() .getResource(getClass().getName().replace('.', '/') + ".class"); @@ -62,7 +60,7 @@ void testGetAttributesForThisTestClass() throws IOException { @Test @DisabledOnOs(OS.WINDOWS) - void testDirectory() throws IOException, CommandLineException { + void directory() throws Exception { URL resource = Thread.currentThread() .getContextClassLoader() .getResource(getClass().getName().replace('.', '/') + ".class"); @@ -100,7 +98,7 @@ void testDirectory() throws IOException, CommandLineException { @Test @DisabledOnOs(OS.WINDOWS) - void testSrcResource() throws IOException { + void srcResource() throws Exception { File dir = new File("src/test/resources/symlinks"); final Map fileAttributesByPathScreenScrape = PlexusIoResourceAttributeUtils.getFileAttributesByPath(dir, true); @@ -115,15 +113,14 @@ void testSrcResource() throws IOException { } @Test - void testNonExistingDirectory() { - assertThrows(NoSuchFileException.class, () -> { - File dir = new File("src/test/noSuchDirectory"); - PlexusIoResourceAttributeUtils.getFileAttributesByPath(dir, true); - }); + void nonExistingDirectory() { + File dir = new File("src/test/noSuchDirectory"); + assertThrows(NoSuchFileException.class, () -> + PlexusIoResourceAttributeUtils.getFileAttributesByPath(dir, true)); } @Test - void testMergeAttributesWithNullBase() { + void mergeAttributesWithNullBase() { PlexusIoResourceAttributes override = new SimpleResourceAttributes(1001, "myUser", 1001, "test", 0); PlexusIoResourceAttributes defaults = new SimpleResourceAttributes(1000, "defaultUser", 1000, "defaultTest", 0); @@ -135,7 +132,7 @@ void testMergeAttributesWithNullBase() { } @Test - void testMergeAttributesWithNullOverrideGroup() { + void mergeAttributesWithNullOverrideGroup() { final PlexusIoResourceAttributes override = new SimpleResourceAttributes(1001, "myUser", -1, null, 0); final PlexusIoResourceAttributes defaults = new SimpleResourceAttributes(1000, "defaultUser", 1000, "defaultGroup", 0); @@ -148,7 +145,7 @@ void testMergeAttributesWithNullOverrideGroup() { } @Test - void testMergeAttributesOverride() { + void mergeAttributesOverride() { final PlexusIoResourceAttributes blank = new SimpleResourceAttributes(); final PlexusIoResourceAttributes invalid = new SimpleResourceAttributes(-1, null, -1, null, -1); final PlexusIoResourceAttributes override = @@ -250,7 +247,7 @@ void testMergeAttributesOverride() { } @Test - void testFileAttributesGeneric() throws IOException { + void fileAttributesGeneric() throws Exception { PlexusIoResourceAttributes attrs = getFileAttributes(new File("src/test/resources/symlinks/src/fileW.txt")); assertFalse(attrs.isSymbolicLink()); assertTrue(StringUtils.isNotEmpty(attrs.getUserName())); @@ -258,7 +255,7 @@ void testFileAttributesGeneric() throws IOException { @Test @DisabledOnOs(OS.WINDOWS) - void testFileAttributes() throws IOException { + void fileAttributes() throws Exception { PlexusIoResourceAttributes attrs = getFileAttributes(new File("src/test/resources/symlinks/src/fileW.txt")); assertFalse(attrs.isSymbolicLink()); assertTrue(StringUtils.isNotEmpty(attrs.getUserName())); @@ -268,7 +265,7 @@ void testFileAttributes() throws IOException { } @Test - void testMergeAttributesDefault() { + void mergeAttributesDefault() { final PlexusIoResourceAttributes blank = new SimpleResourceAttributes(null, null, null, null, 0); final PlexusIoResourceAttributes invalid = new SimpleResourceAttributes(-1, null, -1, null, -1); final PlexusIoResourceAttributes defaults = diff --git a/src/test/java/org/codehaus/plexus/components/io/attributes/SymlinkUtilsTest.java b/src/test/java/org/codehaus/plexus/components/io/attributes/SymlinkUtilsTest.java index 7b216bb..4977a02 100644 --- a/src/test/java/org/codehaus/plexus/components/io/attributes/SymlinkUtilsTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/attributes/SymlinkUtilsTest.java @@ -25,15 +25,15 @@ import org.junit.jupiter.api.Test; import static java.nio.charset.StandardCharsets.UTF_8; -import static org.junit.jupiter.api.Assertions.*; +import static org.junit.jupiter.api.Assertions.assertEquals; -public class SymlinkUtilsTest { +class SymlinkUtilsTest { final File target = new File("target/symlinkCapabilities"); final String expected = "This is a filed that we'll be symlinking to\n"; @BeforeEach - public void setup() throws IOException { + void setup() throws IOException { FileUtils.deleteDirectory(target); Files.createDirectories(target.toPath()); } diff --git a/src/test/java/org/codehaus/plexus/components/io/filemappers/FileMapperTest.java b/src/test/java/org/codehaus/plexus/components/io/filemappers/FileMapperTest.java index fbd7d94..4848796 100644 --- a/src/test/java/org/codehaus/plexus/components/io/filemappers/FileMapperTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/filemappers/FileMapperTest.java @@ -31,7 +31,7 @@ * Test case for the various file mappers. */ @PlexusTest -public class FileMapperTest { +class FileMapperTest { @Inject PlexusContainer container; @@ -85,7 +85,7 @@ protected void testFileMapper(FileMapper pMapper, String[] pInput, String[] pOut }; @Test - void testIdentityMapper() throws Exception { + void identityMapper() throws Exception { final String[] results = getIdentityResults(); testFileMapper(new IdentityMapper(), SAMPLES, results); } @@ -98,7 +98,7 @@ private String[] getIdentityResults() { } @Test - void testDefaultMapper() throws Exception { + void defaultMapper() throws Exception { final String[] results = getIdentityResults(); testFileMapper(container.lookup(FileMapper.class), SAMPLES, results); testFileMapper(container.lookup(FileMapper.class, IdentityMapper.ROLE_HINT), SAMPLES, results); @@ -106,7 +106,7 @@ void testDefaultMapper() throws Exception { } @Test - void testFileExtensionMapper() throws Exception { + void fileExtensionMapper() throws Exception { final String[] results = getIdentityResults(); for (int i = 2; i <= 10; i += 2) { results[i] += ".png"; @@ -127,7 +127,7 @@ private void testFileExtensionMapper(final String[] results, final FileExtension } @Test - void testFlattenMapper() throws Exception { + void flattenMapper() throws Exception { final String[] results = getIdentityResults(); results[4] = results[6] = results[8] = results[10] = results[2]; results[5] = results[7] = results[9] = results[11] = results[3]; @@ -141,7 +141,7 @@ private void testMergeMapper(String pTargetName, String[] pResults, MergeFileMap } @Test - void testMergeMapper() throws Exception { + void mergeMapper() throws Exception { final String[] results = getIdentityResults(); final String targetName = "zgh"; for (int i = 2; i < results.length; i++) { @@ -153,7 +153,7 @@ void testMergeMapper() throws Exception { } @Test - void testPrefixMapper() throws Exception { + void prefixMapper() throws Exception { final String prefix = "x7Rtf"; final String[] results = getIdentityResults(); testFileMapper(new PrefixFileMapper(), SAMPLES, results); @@ -172,7 +172,7 @@ void testPrefixMapper() throws Exception { } @Test - void testSuffixMapper() throws Exception { + void suffixMapper() throws Exception { final String suffix = "suffix"; String[] samples = Arrays.copyOf(SAMPLES, SAMPLES.length + 2); samples[samples.length - 2] = "archive.tar.gz"; @@ -208,7 +208,7 @@ private RegExpFileMapper configure(RegExpFileMapper pMapper, String pPattern, St } @Test - void testRegExpFileMapper() throws Exception { + void regExpFileMapper() throws Exception { final String[] results = getIdentityResults(); results[3] = "xyz.jpg"; results[5] = "b/xyz.jpg"; diff --git a/src/test/java/org/codehaus/plexus/components/io/filemappers/FileSelectorTest.java b/src/test/java/org/codehaus/plexus/components/io/filemappers/FileSelectorTest.java index 6879c7f..1034302 100644 --- a/src/test/java/org/codehaus/plexus/components/io/filemappers/FileSelectorTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/filemappers/FileSelectorTest.java @@ -39,7 +39,7 @@ * Test case for implementations of {@link FileSelector}. */ @PlexusTest -public class FileSelectorTest { +class FileSelectorTest { @Inject PlexusContainer container; @@ -80,7 +80,7 @@ private boolean[] getAllTrues() { } @Test - void testAllFilesFileSelector() throws Exception { + void allFilesFileSelector() throws Exception { testFileSelector(new AllFilesFileSelector()); testFileSelector((AllFilesFileSelector) container.lookup(FileSelector.class)); testFileSelector((AllFilesFileSelector) container.lookup(FileSelector.class, AllFilesFileSelector.ROLE_HINT)); @@ -114,14 +114,14 @@ protected void testFileSelector(IncludeExcludeFileSelector pSelector) throws Exc } @Test - void testIncludeExcludeFileSelector() throws Exception { + void includeExcludeFileSelector() throws Exception { testFileSelector(new IncludeExcludeFileSelector()); testFileSelector((IncludeExcludeFileSelector) container.lookup(FileSelector.class, IncludeExcludeFileSelector.ROLE_HINT)); } @Test - void testIncludeExcludeFileSelector_SetExcludes() { + void includeExcludeFileSelectorSetExcludes() { IncludeExcludeFileSelector selector = new IncludeExcludeFileSelector(); // Test that the setExcludes method does not modify the excludes. diff --git a/src/test/java/org/codehaus/plexus/components/io/filemappers/ResourcesTest.java b/src/test/java/org/codehaus/plexus/components/io/filemappers/ResourcesTest.java index 540da59..ff5c4f7 100644 --- a/src/test/java/org/codehaus/plexus/components/io/filemappers/ResourcesTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/filemappers/ResourcesTest.java @@ -46,7 +46,7 @@ * Test case for resource collections. */ @PlexusTest -public class ResourcesTest { +class ResourcesTest { @Inject PlexusContainer container; @@ -132,11 +132,11 @@ private void compare(InputStream in, File file) throws IOException { } private void compare(PlexusIoResource res, File file) throws IOException { - assertTrue(res.getLastModified() != PlexusIoResource.UNKNOWN_MODIFICATION_DATE); + assertNotSame(res.getLastModified(), PlexusIoResource.UNKNOWN_MODIFICATION_DATE); if (res instanceof PlexusIoFileResource) { assertEquals(res.getLastModified() / 1000, file.lastModified() / 1000); } - assertTrue(res.getSize() != PlexusIoResource.UNKNOWN_RESOURCE_SIZE); + assertNotSame(res.getSize(), PlexusIoResource.UNKNOWN_RESOURCE_SIZE); assertEquals(res.getSize(), file.length()); InputStream in = res.getContents(); compare(in, file); @@ -200,7 +200,7 @@ private void testFileResourceCollection(PlexusIoFileResourceCollection resourceC } @Test - void testFileCollection() throws Exception { + void fileCollection() throws Exception { createFiles(); testFileResourceCollection((PlexusIoFileResourceCollection) container.lookup(PlexusIoResourceCollection.class)); testFileResourceCollection((PlexusIoFileResourceCollection) diff --git a/src/test/java/org/codehaus/plexus/components/io/resources/AbstractPlexusIoResourceCollectionTest.java b/src/test/java/org/codehaus/plexus/components/io/resources/AbstractPlexusIoResourceCollectionTest.java index 4e068ba..12e7f29 100644 --- a/src/test/java/org/codehaus/plexus/components/io/resources/AbstractPlexusIoResourceCollectionTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/resources/AbstractPlexusIoResourceCollectionTest.java @@ -17,10 +17,10 @@ /** * @author Kristian Rosenvold */ -public class AbstractPlexusIoResourceCollectionTest { +class AbstractPlexusIoResourceCollectionTest { @SuppressWarnings("ResultOfMethodCallIgnored") @Test - void testGetIncludes() throws Exception { + void getIncludes() throws Exception { AbstractPlexusIoResourceCollection sut = new AbstractPlexusIoResourceCollection() { public Iterator getResources() { return Arrays.asList(getResource("r1"), getResource("r2")).iterator(); diff --git a/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileResourceDeferredTest.java b/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileResourceDeferredTest.java index 04ffd54..3a5e6c8 100644 --- a/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileResourceDeferredTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileResourceDeferredTest.java @@ -1,7 +1,6 @@ package org.codehaus.plexus.components.io.resources; import java.io.File; -import java.io.IOException; import java.io.InputStream; import java.io.OutputStream; import java.nio.charset.StandardCharsets; @@ -20,13 +19,13 @@ * This specifically tests the fix for issue #118 where temp files were deleted * before they could be accessed, causing FileNotFoundException. */ -public class PlexusIoFileResourceDeferredTest { +class PlexusIoFileResourceDeferredTest { @TempDir File tempDir; @Test - void testFileResourceWithTransformerCanReadContentsMultipleTimes() throws IOException { + void fileResourceWithTransformerCanReadContentsMultipleTimes() throws Exception { // Create a test file with content larger than typical buffer size File testFile = new File(tempDir, "test-file.txt"); byte[] largeContent = new byte[10000]; // 10KB @@ -60,7 +59,7 @@ void testFileResourceWithTransformerCanReadContentsMultipleTimes() throws IOExce } @Test - void testFileResourceWithTransformerLargeFile() throws IOException { + void fileResourceWithTransformerLargeFile() throws Exception { // Create a large file that exceeds the DeferredFileOutputStream threshold (5MB) File testFile = new File(tempDir, "large-test-file.bin"); byte[] chunk = new byte[1024 * 1024]; // 1MB chunks @@ -94,7 +93,7 @@ void testFileResourceWithTransformerLargeFile() throws IOException { } @Test - void testFileResourceWithTransformerSmallFile() throws IOException { + void fileResourceWithTransformerSmallFile() throws Exception { // Test with a small file that stays in memory (below 5MB threshold) File testFile = new File(tempDir, "small-test-file.txt"); String content = "Hello, World!"; diff --git a/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileSupplierResourceCollectionTest.java b/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileSupplierResourceCollectionTest.java index dfcce72..a5ac14a 100755 --- a/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileSupplierResourceCollectionTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoFileSupplierResourceCollectionTest.java @@ -10,9 +10,9 @@ /** * @author Kristian Rosenvold */ -public class PlexusIoFileSupplierResourceCollectionTest { +class PlexusIoFileSupplierResourceCollectionTest { @Test - void testGetName() throws Exception { + void getName() throws Exception { PlexusIoFileResourceCollection coll = new PlexusIoFileResourceCollection(); char nonSeparator = File.separatorChar == '/' ? '\\' : '/'; coll.setPrefix("fud" + nonSeparator); diff --git a/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoPlexusIoFileResourceTest.java b/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoPlexusIoFileResourceTest.java index d61cf54..64f37b2 100644 --- a/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoPlexusIoFileResourceTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/resources/PlexusIoPlexusIoFileResourceTest.java @@ -1,7 +1,6 @@ package org.codehaus.plexus.components.io.resources; import java.io.File; -import java.io.IOException; import org.codehaus.plexus.components.io.attributes.FileAttributes; import org.codehaus.plexus.components.io.attributes.PlexusIoResourceAttributes; @@ -13,11 +12,11 @@ import static org.junit.jupiter.api.Assertions.*; -public class PlexusIoPlexusIoFileResourceTest { +class PlexusIoPlexusIoFileResourceTest { @Test @DisabledOnOs(OS.WINDOWS) - void testRealSymlink() throws IOException { + void realSymlink() throws Exception { final File file = new File("src/test/resources/symlinks/src/symDir"); PlexusIoResourceAttributes attrs = FileAttributes.uncached(file); assertTrue(attrs.isSymbolicLink()); @@ -26,13 +25,13 @@ void testRealSymlink() throws IOException { assertTrue(r.isDirectory()); final File target = SymlinkUtils.readSymbolicLink(file); assertTrue(target.getName().endsWith("targetDir")); - assertTrue(r instanceof SymlinkDestinationSupplier); + assertInstanceOf(SymlinkDestinationSupplier.class, r); assertEquals("targetDir/", ((SymlinkDestinationSupplier) r).getSymlinkDestination()); } @Test @DisabledOnOs(OS.WINDOWS) - void testSymSymlinkFile() throws IOException { + void symSymlinkFile() throws Exception { final File file = new File("src/test/resources/symlinks/src/symSymR"); PlexusIoResource r = ResourceFactory.createResource(file); assertTrue(r.isSymbolicLink()); @@ -40,14 +39,14 @@ void testSymSymlinkFile() throws IOException { PlexusIoResource rL = ((PlexusIoSymlinkResource) r).getLink(); assertFalse(rL instanceof PlexusIoSymlinkResource); PlexusIoResource rT = ((PlexusIoSymlinkResource) r).getTarget(); - assertTrue(rT instanceof PlexusIoSymlinkResource); + assertInstanceOf(PlexusIoSymlinkResource.class, rT); PlexusIoResource rTT = ((PlexusIoSymlinkResource) rT).getTarget(); assertFalse(rTT instanceof PlexusIoSymlinkResource); } @Test @DisabledOnOs(OS.WINDOWS) - void testSymlinkFile() throws IOException { + void symlinkFile() throws Exception { final File file = new File("src/test/resources/symlinks/src/symR"); PlexusIoResource r = ResourceFactory.createResource(file); assertTrue(r.isSymbolicLink()); diff --git a/src/test/java/org/codehaus/plexus/components/io/resources/proxy/PlexusIoProxyResourceCollectionTest.java b/src/test/java/org/codehaus/plexus/components/io/resources/proxy/PlexusIoProxyResourceCollectionTest.java index 1ce7f92..e27301a 100644 --- a/src/test/java/org/codehaus/plexus/components/io/resources/proxy/PlexusIoProxyResourceCollectionTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/resources/proxy/PlexusIoProxyResourceCollectionTest.java @@ -19,7 +19,6 @@ import javax.annotation.Nonnull; import java.io.Closeable; -import java.io.IOException; import java.io.InputStream; import java.net.URL; import java.util.Iterator; @@ -36,13 +35,13 @@ /** * Test case for {@link PlexusIoProxyResourceCollection}. */ -public class PlexusIoProxyResourceCollectionTest { +class PlexusIoProxyResourceCollectionTest { private final String[] SAMPLE_INCLUDES = {"junk.*", "test/**", "dir*/file.xml"}; private final String[] SAMPLE_EXCLUDES = {"*.junk", "somwhere/**"}; @Test - void testGetDefaultFileSelector() { + void getDefaultFileSelector() { PlexusIoProxyResourceCollection resCol = new PlexusIoProxyResourceCollection(null); // This will throw an exception if there is a bug @@ -91,7 +90,7 @@ public void remove() { } @Test - void testClosing() throws IOException { + void closing() throws Exception { final CloseableIterator closeableIterator = new CloseableIterator(); PlexusIoProxyResourceCollection resCol = new PlexusIoProxyResourceCollection(new AbstractPlexusIoResourceCollection() { diff --git a/src/test/java/org/codehaus/plexus/components/io/resources/proxy/ProxyFactoryTest.java b/src/test/java/org/codehaus/plexus/components/io/resources/proxy/ProxyFactoryTest.java index 152b633..0b072a2 100644 --- a/src/test/java/org/codehaus/plexus/components/io/resources/proxy/ProxyFactoryTest.java +++ b/src/test/java/org/codehaus/plexus/components/io/resources/proxy/ProxyFactoryTest.java @@ -22,25 +22,25 @@ import static org.junit.jupiter.api.Assertions.*; -public class ProxyFactoryTest { +class ProxyFactoryTest { @Test - void testCreateProxy() throws Exception { + void createProxy() throws Exception { final PlexusIoResource proxy = ProxyFactory.createProxy(getPomResource(), null); - assertTrue(proxy instanceof ResourceAttributeSupplier); - assertTrue(proxy instanceof FileSupplier); + assertInstanceOf(ResourceAttributeSupplier.class, proxy); + assertInstanceOf(FileSupplier.class, proxy); assertFalse(proxy instanceof SymlinkDestinationSupplier); } @Test - void testCreateProxyWithNameOverride() throws Exception { + void createProxyWithNameOverride() throws Exception { NameSupplier ns = () -> "fred"; final PlexusIoResource proxy = ProxyFactory.createProxy(getPomResource(), ns); assertEquals("fred", proxy.getName()); } @Test - void testCreateProxyWithResourceAttributeOverride() throws Exception { + void createProxyWithResourceAttributeOverride() throws Exception { final PlexusIoResourceAttributes s = SimpleResourceAttributes.lastResortDummyAttributesForBrokenOS(); ResourceAttributeSupplier ns = () -> s; final PlexusIoResource proxy = ProxyFactory.createProxy(getPomResource(), ns); @@ -48,7 +48,7 @@ void testCreateProxyWithResourceAttributeOverride() throws Exception { } @Test - void testCreateProxyWithSizeSupplierOverride() throws Exception { + void createProxyWithSizeSupplierOverride() throws Exception { final PlexusIoResourceAttributes s = SimpleResourceAttributes.lastResortDummyAttributesForBrokenOS(); SizeSupplier ns = () -> 42; final PlexusIoResource proxy = ProxyFactory.createProxy(getPomResource(), ns); @@ -56,7 +56,7 @@ void testCreateProxyWithSizeSupplierOverride() throws Exception { } @Test - void testCreateProxyWithContentSupplierOverride() throws Exception { + void createProxyWithContentSupplierOverride() throws Exception { final InputStream s = new ByteArrayInputStream(new byte[10]); ContentSupplier ns = () -> s; final PlexusIoResource proxy = ProxyFactory.createProxy(getPomResource(), ns); @@ -64,7 +64,7 @@ void testCreateProxyWithContentSupplierOverride() throws Exception { } @Test - void testCreateProxyWithSymlinkDestinationSupplierOverride() throws Exception { + void createProxyWithSymlinkDestinationSupplierOverride() throws Exception { SymlinkDestinationSupplier ns = () -> "mordor"; final PlexusIoResource proxy = ProxyFactory.createProxy(getDummySymlinkResource(), ns); assertEquals("mordor", ((SymlinkDestinationSupplier) proxy).getSymlinkDestination());