This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-net.git
commit f50f43145f40a2c0f86c1c7000f0e6b5427cdbee Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Sat Feb 24 12:21:42 2024 -0500 Use final --- .../apache/commons/net/ftp/FTPClientModeZTest.java | 40 +++++++++++----------- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/src/test/java/org/apache/commons/net/ftp/FTPClientModeZTest.java b/src/test/java/org/apache/commons/net/ftp/FTPClientModeZTest.java index b37aa1a9..b8d0475d 100644 --- a/src/test/java/org/apache/commons/net/ftp/FTPClientModeZTest.java +++ b/src/test/java/org/apache/commons/net/ftp/FTPClientModeZTest.java @@ -35,7 +35,7 @@ public class FTPClientModeZTest extends TestCase { private final int port; private final FtpServer ftpServer; - FtpServerAndPort(FtpServer ftpServer, int port) { + FtpServerAndPort(final FtpServer ftpServer, final int port) { this.port = port; this.ftpServer = ftpServer; } @@ -52,11 +52,11 @@ public class FTPClientModeZTest extends TestCase { final PropertiesUserManagerFactory propertiesUserManagerFactory = new PropertiesUserManagerFactory(); final UserManager userManager = propertiesUserManagerFactory.createUserManager(); - BaseUser user = new BaseUser(); + final BaseUser user = new BaseUser(); user.setName(username); user.setPassword(password); - List<Authority> authorities = new ArrayList<Authority>(); + final List<Authority> authorities = new ArrayList<>(); authorities.add(new WritePermission()); user.setAuthorities(authorities); @@ -66,11 +66,11 @@ public class FTPClientModeZTest extends TestCase { return userManager; } - private static void runWithFTPserver(Runner runner) throws Exception { + private static void runWithFTPserver(final Runner runner) throws Exception { - String username = "test"; - String password = "test"; - FtpServerAndPort ftpServerAndPort = setupPlainFTPserver(username, password); + final String username = "test"; + final String password = "test"; + final FtpServerAndPort ftpServerAndPort = setupPlainFTPserver(username, password); try { runner.run(ftpServerAndPort.port, username, password); } finally { @@ -90,11 +90,11 @@ public class FTPClientModeZTest extends TestCase { factory.setPort(0); // replace the default listener - Listener listener = factory.createListener(); + final Listener listener = factory.createListener(); serverFactory.addListener("default", listener); // start the server - FtpServer server = serverFactory.createServer(); + final FtpServer server = serverFactory.createServer(); server.start(); return new FtpServerAndPort(server, listener.getPort()); @@ -113,21 +113,21 @@ public class FTPClientModeZTest extends TestCase { public void testRetrievingFiles() throws Exception { new File(DEFAULT_HOME).mkdirs(); - String filename = "test_download.txt"; - String fileContent = "Created at " + Instant.now(); + final String filename = "test_download.txt"; + final String fileContent = "Created at " + Instant.now(); Files.write(Paths.get(DEFAULT_HOME).resolve(filename), fileContent.getBytes(UTF_8)); runWithFTPserver((port, user, password) -> { - FTPClient client = new FTPClient(); + final FTPClient client = new FTPClient(); try { client.connect("localhost", port); client.login(user, password); assertTrue("Mode Z successfully activated", client.setFileTransferMode(COMPRESSED_MODE_Z_TRANSFER_MODE)); - FTPFile[] files = client.listFiles(); + final FTPFile[] files = client.listFiles(); assertEquals("Only single file in home directory", 1, files.length); - ByteArrayOutputStream bos = new ByteArrayOutputStream(); + final ByteArrayOutputStream bos = new ByteArrayOutputStream(); assertTrue("File successfully transferred", client.retrieveFile(files[0].getName(), bos)); assertEquals("File content is not corrupted", fileContent, new String(bos.toByteArray(), UTF_8)); } finally { @@ -139,23 +139,23 @@ public class FTPClientModeZTest extends TestCase { public void testStoringFiles() throws Exception { runWithFTPserver((port, user, password) -> { - FTPClient client = new FTPClient(); + final FTPClient client = new FTPClient(); try { client.connect("localhost", port); client.login(user, password); assertTrue("Mode Z successfully activated", client.setFileTransferMode(COMPRESSED_MODE_Z_TRANSFER_MODE)); - FTPFile[] filesBeforeUpload = client.listFiles(); + final FTPFile[] filesBeforeUpload = client.listFiles(); assertEquals("No files in home directory before upload", 0, filesBeforeUpload.length); - String filename = "test_upload.txt"; - String fileContent = "Created at " + Instant.now(); + final String filename = "test_upload.txt"; + final String fileContent = "Created at " + Instant.now(); assertTrue("File successfully transferred", client.storeFile(filename, new ByteArrayInputStream(fileContent.getBytes(UTF_8)))); - FTPFile[] filesAfterUpload = client.listFiles(); + final FTPFile[] filesAfterUpload = client.listFiles(); assertEquals("Single file in home directory after upload", 1, filesAfterUpload.length); - Path p = Paths.get(DEFAULT_HOME, filename); + final Path p = Paths.get(DEFAULT_HOME, filename); assertEquals("File content is not corrupted", fileContent, new String(readAllBytes(p), UTF_8)); } finally { client.logout();