You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2021/03/13 16:20:05 UTC

[commons-vfs] branch master updated (8a1c637 -> 7638a9e)

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/commons-vfs.git.


    from 8a1c637  Let a slow provider do its work, in an embedded server, perhaps (FTPS for example).
     new 42ce5db  Better local variable name.
     new 7638a9e  Close VFS at the end of each test suite to stop any VFS background threads.

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../src/test/java/org/apache/commons/vfs2/AbstractTestSuite.java      | 3 ++-
 .../apache/commons/vfs2/provider/sftp/SftpMultiThreadWriteTests.java  | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)


[commons-vfs] 02/02: Close VFS at the end of each test suite to stop any VFS background threads.

Posted by gg...@apache.org.
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-vfs.git

commit 7638a9e096aec3ddd1b8dea324a7272740327b27
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sat Mar 13 11:20:01 2021 -0500

    Close VFS at the end of each test suite to stop any VFS background
    threads.
---
 .../src/test/java/org/apache/commons/vfs2/AbstractTestSuite.java       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/commons-vfs2/src/test/java/org/apache/commons/vfs2/AbstractTestSuite.java b/commons-vfs2/src/test/java/org/apache/commons/vfs2/AbstractTestSuite.java
index 049ef75..5b73084 100644
--- a/commons-vfs2/src/test/java/org/apache/commons/vfs2/AbstractTestSuite.java
+++ b/commons-vfs2/src/test/java/org/apache/commons/vfs2/AbstractTestSuite.java
@@ -191,7 +191,7 @@ public abstract class AbstractTestSuite extends TestSetup {
         baseFolder = null;
         testSuite = null;
 
-        // Suggest to threads (SoftRefFilesChache) to free all files.
+        // Suggest to threads (SoftRefFilesCache) to free all files.
         System.gc();
         Thread.sleep(1000);
         System.gc();
@@ -208,6 +208,7 @@ public abstract class AbstractTestSuite extends TestSetup {
 
         // Make sure temp directory is empty or gone
         checkTempDir("Temp dir not empty after test");
+        VFS.close();
     }
 
     private void validateThreadSnapshot() {


[commons-vfs] 01/02: Better local variable name.

Posted by gg...@apache.org.
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-vfs.git

commit 42ce5db10e09b4867d3fecd6198d256aa0d8e805
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sat Mar 13 11:12:05 2021 -0500

    Better local variable name.
---
 .../apache/commons/vfs2/provider/sftp/SftpMultiThreadWriteTests.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/sftp/SftpMultiThreadWriteTests.java b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/sftp/SftpMultiThreadWriteTests.java
index 57d6c31..09d5bbe 100644
--- a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/sftp/SftpMultiThreadWriteTests.java
+++ b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/sftp/SftpMultiThreadWriteTests.java
@@ -89,9 +89,9 @@ public class SftpMultiThreadWriteTests extends AbstractProviderTestCase {
         final ExecutorService service = Executors.newFixedThreadPool(10);
         try {
             final List<Future<Boolean>> futures = service.invokeAll(tasks);
-            assertTrue(futures.stream().allMatch(fut -> {
+            assertTrue(futures.stream().allMatch(future -> {
                 try {
-                    return fut.get(5, TimeUnit.SECONDS);
+                    return future.get(5, TimeUnit.SECONDS);
                 } catch (final Exception e) {
                     return false;
                 }