NetCopyClientConnectionPool refactorings

- Use LruCache
- Remove use of CountdownLatch
- Made NetCopyClientConnectionPool lifecycle-aware
- PasswordUtil remove Context parameter

Author: TranceLove

.github/workflows/android-feature.yml

@@ -22,10 +22,10 @@ jobs:
         with:
           distribution: "temurin"
           java-version: 17
+      - name: Set up Gradle
+        uses: gradle/actions/setup-gradle@v4
       - name: Check formatting using spotless
-        uses: gradle/actions/setup-gradle@v3
-        with:
-          arguments: spotlessCheck --stacktrace
+        run: ./gradlew spotlessCheck --stacktrace
 
   build:
     name: Build debug and run Jacoco tests
@@ -37,11 +37,26 @@ jobs:
         with:
           distribution: "temurin"
           java-version: 17
+      - name: Set up Gradle
+        uses: gradle/actions/setup-gradle@v4
       - name: Build with Gradle
-        uses: gradle/actions/setup-gradle@v3
-        with:
-          arguments: assembledebug --stacktrace
+        run: ./gradlew assembledebug --stacktrace
       - name: Run test cases
-        uses: gradle/actions/setup-gradle@v3
+        run: ./gradlew jacocoTestPlayDebugUnitTestReport --stacktrace --info
+      - name: Upload
+        uses: actions/upload-artifact@v4
+        with:
+          name: Event File
+          path: ${{ github.event_path }}
+      - name: Upload Unit Test Results
+        if: (!cancelled())
+        uses: actions/upload-artifact@v4
         with:
-          arguments: jacocoTestPlayDebugUnitTestReport --stacktrace --info
+          name: unit-test-results
+          path: app/build/test-results/test*UnitTest/
+#      - name: Upload Jacoco Coverage Report
+#        if: (!cancelled())
+#        uses: actions/upload-artifact@v4
+#        with:
+#          name: jacoco-coverage-report
+#          path: "app/build/jacoco/jacoco.xml"

.github/workflows/publish-test-results.yaml

@@ -0,0 +1,31 @@
+name: Publish Test Results
+
+on:
+  workflow_run:
+    workflows: ["Android Build CI", "Android Feature CI"]
+    types:
+      - completed
+permissions: {}
+
+jobs:
+  publish_test_results:
+    runs-on: ubuntu-latest
+    if: github.event.workflow_run.conclusion == 'success' || github.event.workflow_run.conclusion == 'failure'
+
+    permissions:
+      checks: write
+      pull-requests: write
+      actions: read
+    steps:
+      - name: Download and Extract Artifacts
+        uses: dawidd6/action-download-artifact@e7466d1a7587ed14867642c2ca74b5bcc1e19a2d
+        with:
+         run_id: ${{ github.event.workflow_run.id }}
+         path: artifacts
+      - name: Publish Test Results
+        uses: EnricoMi/publish-unit-test-result-action@v2
+        with:
+          commit: ${{ github.event.workflow_run.head_sha }}
+          event_file: artifacts/Event File/event.json
+          event_name: ${{ github.event.workflow_run.event }}
+          files: "artifacts/**/*.xml"

app/build.gradle

@@ -145,6 +145,8 @@ dependencies {
     implementation libs.androidX.constraintLayout
     implementation libs.androidX.multidex //Multiple dex files
     implementation libs.androidX.biometric
+    implementation libs.androidX.lifecycle
+    implementation libs.androidX.lifecycle.process
     implementation libs.room.runtime
     implementation libs.room.rxjava2
 

app/src/androidTest/java/com/amaze/filemanager/asynchronous/services/ftp/FtpServiceEspressoTest.kt

@@ -146,7 +146,6 @@ class FtpServiceEspressoTest {
             .putString(
                 FtpService.KEY_PREFERENCE_PASSWORD,
                 PasswordUtil.encryptPassword(
-                    ApplicationProvider.getApplicationContext(),
                     "passw0rD",
                 ),
             )

app/src/androidTest/java/com/amaze/filemanager/filesystem/files/CryptUtilTest.java

@@ -22,29 +22,19 @@
 
 import static org.junit.Assert.assertEquals;
 
-import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
 import com.amaze.filemanager.BuildConfig;
 import com.amaze.filemanager.utils.PasswordUtil;
 
-import android.content.Context;
 import android.util.Base64;
 
 import androidx.test.ext.junit.runners.AndroidJUnit4;
-import androidx.test.platform.app.InstrumentationRegistry;
 
 @RunWith(AndroidJUnit4.class)
 public class CryptUtilTest {
 
-  private Context context;
-
-  @Before
-  public void setUp() {
-    context = InstrumentationRegistry.getInstrumentation().getTargetContext();
-  }
-
   @Test
   public void testIvValueIsCorrect() {
     assertEquals("LxbHiJhhUXcj", BuildConfig.CRYPTO_IV);
@@ -53,8 +43,7 @@ public void testIvValueIsCorrect() {
   @Test
   public void testEncryptDecrypt() throws Exception {
     String password = "hackme";
-    String encrypted = PasswordUtil.INSTANCE.encryptPassword(context, password, Base64.URL_SAFE);
-    assertEquals(
-        password, PasswordUtil.INSTANCE.decryptPassword(context, encrypted, Base64.URL_SAFE));
+    String encrypted = PasswordUtil.INSTANCE.encryptPassword(password, Base64.URL_SAFE);
+    assertEquals(password, PasswordUtil.INSTANCE.decryptPassword(encrypted, Base64.URL_SAFE));
   }
 }

app/src/main/java/com/amaze/filemanager/application/AppConfig.java

@@ -41,6 +41,7 @@
 import com.amaze.filemanager.fileoperations.exceptions.ShellNotRunningException;
 import com.amaze.filemanager.fileoperations.filesystem.OpenMode;
 import com.amaze.filemanager.filesystem.HybridFile;
+import com.amaze.filemanager.filesystem.ftp.NetCopyClientConnectionPool;
 import com.amaze.filemanager.filesystem.ssh.CustomSshJConfig;
 import com.amaze.filemanager.ui.fragments.preferencefragments.PreferencesConstants;
 import com.amaze.filemanager.ui.provider.UtilitiesProvider;
@@ -108,6 +109,8 @@ public void onCreate() {
     utilsHandler = new UtilsHandler(this, utilitiesDatabase);
 
     runInBackground(Config::registerSmbURLHandler);
+    // Force Kotlin object NetCopyClientConnectionPool to initialize
+    NetCopyClientConnectionPool.INSTANCE.toString();
 
     // disabling file exposure method check for api n+
     StrictMode.VmPolicy.Builder builder = new StrictMode.VmPolicy.Builder();

app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/ftp/AbstractGetHostInfoTask.kt

@@ -43,7 +43,7 @@ abstract class AbstractGetHostInfoTask<V, T : Callable<V>>(
         AppConfig.getInstance().run {
             progressDialog =
                 ProgressDialog.show(
-                    this.mainActivityContext,
+                    requireNotNull(this.mainActivityContext),
                     "",
                     this.resources.getString(R.string.processing),
                 )

app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/ftp/auth/FtpAuthenticationTask.kt

@@ -27,7 +27,6 @@ import com.amaze.filemanager.asynchronous.asynctasks.Task
 import com.amaze.filemanager.filesystem.ftp.NetCopyClientConnectionPool.FTP_URI_PREFIX
 import org.apache.commons.net.ftp.FTPClient
 import org.json.JSONObject
-import java.net.ConnectException
 import java.net.SocketException
 import java.net.SocketTimeoutException
 
@@ -62,8 +61,7 @@ class FtpAuthenticationTask(
 
     @MainThread
     override fun onError(error: Throwable) {
-        if (error is SocketException || error is SocketTimeoutException || error is ConnectException
-        ) {
+        if (error is SocketException || error is SocketTimeoutException) {
             AppConfig.toast(
                 AppConfig.getInstance(),
                 AppConfig.getInstance()
@@ -79,7 +77,5 @@ class FtpAuthenticationTask(
     }
 
     @MainThread
-    override fun onFinish(value: FTPClient) {
-        android.util.Log.d("TEST", value.toString())
-    }
+    override fun onFinish(value: FTPClient) = Unit
 }

app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/ftp/auth/FtpAuthenticationTaskCallable.kt

@@ -21,7 +21,6 @@
 package com.amaze.filemanager.asynchronous.asynctasks.ftp.auth
 
 import androidx.annotation.WorkerThread
-import com.amaze.filemanager.application.AppConfig
 import com.amaze.filemanager.filesystem.ftp.FTPClientImpl
 import com.amaze.filemanager.filesystem.ftp.NetCopyClientConnectionPool
 import com.amaze.filemanager.filesystem.ftp.NetCopyClientConnectionPool.CONNECT_TIMEOUT
@@ -55,7 +54,7 @@ open class FtpAuthenticationTaskCallable(
                 ftpClient.login(
                     decode(username, UTF_8.name()),
                     decode(
-                        PasswordUtil.decryptPassword(AppConfig.getInstance(), password),
+                        PasswordUtil.decryptPassword(password),
                         UTF_8.name(),
                     ),
                 )

app/src/main/java/com/amaze/filemanager/asynchronous/asynctasks/ftp/auth/FtpsAuthenticationTaskCallable.kt

@@ -20,7 +20,6 @@
 
 package com.amaze.filemanager.asynchronous.asynctasks.ftp.auth
 
-import com.amaze.filemanager.application.AppConfig
 import com.amaze.filemanager.filesystem.ftp.FTPClientImpl
 import com.amaze.filemanager.filesystem.ftp.FTPClientImpl.Companion.ARG_TLS
 import com.amaze.filemanager.filesystem.ftp.FTPClientImpl.Companion.TLS_EXPLICIT
@@ -57,7 +56,7 @@ class FtpsAuthenticationTaskCallable(
             } else {
                 ftpClient.login(
                     username,
-                    PasswordUtil.decryptPassword(AppConfig.getInstance(), password),
+                    PasswordUtil.decryptPassword(password),
                 )
             }
         return if (loginSuccess) {