From 300dd009781148eeb57f448313fb6cbaa987de4e Mon Sep 17 00:00:00 2001
From: alperozturk <alper_ozturk@proton.me>
Date: Thu, 8 May 2025 10:30:03 +0200
Subject: [PATCH 1/3] fixes

Signed-off-by: alperozturk <alper_ozturk@proton.me>
---
 .../java/com/owncloud/android/AbstractIT.java | 100 ++++++++++++------
 1 file changed, 65 insertions(+), 35 deletions(-)

diff --git a/library/src/androidTest/java/com/owncloud/android/AbstractIT.java b/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
index 8e91093a2b..4b7ebbf80d 100644
--- a/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
+++ b/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
@@ -10,7 +10,6 @@
 
 import static junit.framework.TestCase.assertTrue;
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.fail;
 import static org.junit.Assume.assumeTrue;
 
 import android.content.Context;
@@ -19,6 +18,7 @@
 
 import androidx.test.platform.app.InstrumentationRegistry;
 
+import com.nextcloud.android.lib.resources.files.ToggleFileLockRemoteOperation;
 import com.nextcloud.common.NextcloudClient;
 import com.owncloud.android.lib.common.OwnCloudBasicCredentials;
 import com.owncloud.android.lib.common.OwnCloudClient;
@@ -54,6 +54,7 @@
 import java.security.NoSuchAlgorithmException;
 import java.security.cert.CertificateException;
 import java.security.cert.X509Certificate;
+import java.util.concurrent.TimeUnit;
 
 import okhttp3.Credentials;
 
@@ -77,7 +78,8 @@ public abstract class AbstractIT {
     protected String baseFolderPath = "/test_for_build/";
 
     public static final String ASSETS__TEXT_FILE_NAME = "textFile.txt";
-    private static String LOCAL_TRUSTSTORE_FILENAME = "knownServers.bks";
+    private static final String LOCAL_TRUSTSTORE_FILENAME = "knownServers.bks";
+    private static final String TAG = "AbstractIT";
 
     @BeforeClass
     public static void beforeAll() throws InterruptedException,
@@ -115,26 +117,26 @@ public static void beforeAll() throws InterruptedException,
     }
 
     private static void waitForServer(OwnCloudClient client, Uri baseUrl) {
-        // use http 
-        Uri httpUrl = Uri.parse(baseUrl.toString().replaceFirst("https", "http"));
-        GetMethod get = new GetMethod(httpUrl + "/status.php");
+        String statusUrl = baseUrl + "/status.php";
+        GetMethod get;
+        int maxRetries = 3;
 
-        try {
-            int i = 0;
-            while (client.executeMethod(get) != HttpStatus.SC_OK && i < 3) {
-                System.out.println("wait…");
-                Thread.sleep(60 * 1000);
-                i++;
-            }
+        for (int i=0;i<maxRetries;i++) {
+            get = new GetMethod(statusUrl);
 
-            if (i == 3) {
-                fail("Server not ready!");
-            }
+            try {
+                if (client.executeMethod(get) == HttpStatus.SC_OK) {
+                    Log_OC.d(TAG, "Server is ready");
+                    return;
+                }
 
-        } catch (IOException e) {
-            e.printStackTrace();
-        } catch (InterruptedException e) {
-            e.printStackTrace();
+                Log_OC.d(TAG, "Server not ready, retrying in 60 seconds...");
+                TimeUnit.MINUTES.sleep(1);
+            } catch (Exception e) {
+                Log_OC.d(TAG, "Server not ready, failed: " + e);
+            } finally {
+                get.releaseConnection();
+            }
         }
     }
 
@@ -265,30 +267,58 @@ public void after() {
     }
 
     private void removeOnClient(OwnCloudClient client) {
-        RemoteOperationResult result = new ReadFolderRemoteOperation("/").execute(client);
+        final var result = new ReadFolderRemoteOperation("/").execute(client);
         assertTrue(result.getLogMessage(context), result.isSuccess());
 
         for (Object object : result.getData()) {
-            RemoteFile remoteFile = (RemoteFile) object;
-
-            if (!"/".equals(remoteFile.getRemotePath()) &&
-                remoteFile.getMountType() != WebdavEntry.MountType.GROUP) {
-                if (remoteFile.isEncrypted()) {
-                    assertTrue(new ToggleEncryptionRemoteOperation(
-                        remoteFile.getLocalId(),
-                        remoteFile.getRemotePath(),
-                        false)
-                        .execute(client)
-                        .isSuccess());
-                }
+            if (!(object instanceof RemoteFile remoteFile)) {
+                Log_OC.d(TAG, "removeOnClient skipping not instance of RemoteFile");
+                continue;
+            }
+
+            String remotePath = remoteFile.getRemotePath();
+
+            if ("/".equals(remotePath) || remoteFile.getMountType() == WebdavEntry.MountType.GROUP) {
+                Log_OC.d(TAG, "removeOnClient skipping root path or mount type is group");
+                continue;
+            }
+
+            if (remoteFile.isEncrypted()) {
+                assertTrue(toggleEncryptionRemoteFile(remoteFile));
+            }
 
-                assertTrue("Failed to remove " + remoteFile.getRemotePath(),
-                    new RemoveFileRemoteOperation(remoteFile.getRemotePath()).execute(client).isSuccess());
+            if (remoteFile.isLocked() && remotePath != null) {
+                unlockRemoteFile(remotePath);
             }
+
+            boolean isRemoteFileRemoved = removeRemoteFile(remotePath);
+            final var removeFileOperationErrorMessage = ("Failed to remove " + remotePath);
+            assertTrue(removeFileOperationErrorMessage, isRemoteFileRemoved);
         }
 
         // delete keystore
-        new File(context.getFilesDir(), LOCAL_TRUSTSTORE_FILENAME).delete();
+        boolean isKeyStoreDeleted = new File(context.getFilesDir(), LOCAL_TRUSTSTORE_FILENAME).delete();
+        Log_OC.d(TAG,"KeyStore file deletion result: " + isKeyStoreDeleted);
+    }
+
+    private boolean toggleEncryptionRemoteFile(RemoteFile remoteFile) {
+        final var operation = new ToggleEncryptionRemoteOperation(remoteFile.getLocalId(), remoteFile.getRemotePath(), false);
+        final var result = operation.execute(client);
+        return result.isSuccess();
+    }
+
+    private void unlockRemoteFile(String path) {
+        final var operation = new ToggleFileLockRemoteOperation(false, path);
+        final var result = operation.execute(nextcloudClient);
+        if (result.isSuccess()) {
+            Log_OC.d(TAG,"Locked file: " + path + " unlocked");
+        }
+    }
+
+    private boolean removeRemoteFile(String path) {
+        final var operation = new RemoveFileRemoteOperation(path);
+        final var result = operation.execute(client);
+        return result.isSuccess();
     }
 
     public static File getFile(String filename) throws IOException {

From 70120b09a9d61fb84de1afda4258bd1064eb022f Mon Sep 17 00:00:00 2001
From: alperozturk <alper_ozturk@proton.me>
Date: Thu, 8 May 2025 10:37:45 +0200
Subject: [PATCH 2/3] better log

Signed-off-by: alperozturk <alper_ozturk@proton.me>
---
 .../java/com/owncloud/android/AbstractIT.java         | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/library/src/androidTest/java/com/owncloud/android/AbstractIT.java b/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
index 4b7ebbf80d..d88f1ff419 100644
--- a/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
+++ b/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
@@ -121,7 +121,7 @@ private static void waitForServer(OwnCloudClient client, Uri baseUrl) {
         GetMethod get;
         int maxRetries = 3;
 
-        for (int i=0;i<maxRetries;i++) {
+        for (int i = 0; i < maxRetries; i++) {
             get = new GetMethod(statusUrl);
 
             try {
@@ -272,14 +272,14 @@ private void removeOnClient(OwnCloudClient client) {
 
         for (Object object : result.getData()) {
             if (!(object instanceof RemoteFile remoteFile)) {
-                Log_OC.d(TAG, "removeOnClient skipping not instance of RemoteFile");
+                Log_OC.d(TAG, "Skipping removeOnClient: not instance of RemoteFile");
                 continue;
             }
 
             String remotePath = remoteFile.getRemotePath();
 
             if ("/".equals(remotePath) || remoteFile.getMountType() == WebdavEntry.MountType.GROUP) {
-                Log_OC.d(TAG, "removeOnClient skipping root path or mount type is group");
+                Log_OC.d(TAG, "Skipping removeOnClient: remote path is root path or mount type is group");
                 continue;
             }
 
@@ -296,9 +296,8 @@ private void removeOnClient(OwnCloudClient client) {
             assertTrue(removeFileOperationErrorMessage, isRemoteFileRemoved);
         }
 
-        // delete keystore
         boolean isKeyStoreDeleted = new File(context.getFilesDir(), LOCAL_TRUSTSTORE_FILENAME).delete();
-        Log_OC.d(TAG,"KeyStore file deletion result: " + isKeyStoreDeleted);
+        Log_OC.d(TAG, "KeyStore file deletion result: " + isKeyStoreDeleted);
     }
 
     private boolean toggleEncryptionRemoteFile(RemoteFile remoteFile) {
@@ -311,7 +310,7 @@ private void unlockRemoteFile(String path) {
         final var operation = new ToggleFileLockRemoteOperation(false, path);
         final var result = operation.execute(nextcloudClient);
         if (result.isSuccess()) {
-            Log_OC.d(TAG,"Locked file: " + path + " unlocked");
+            Log_OC.d(TAG, "Locked file: " + path + " unlocked");
         }
     }
 

From 58bd46aec334d158b171161bc53ebc4dcb0d9d39 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alper=20=C3=96zt=C3=BCrk?=
 <67455295+alperozturk96@users.noreply.github.com>
Date: Wed, 25 Jun 2025 10:33:34 +0200
Subject: [PATCH 3/3] Update
 library/src/androidTest/java/com/owncloud/android/AbstractIT.java
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: Tom <70907959+ZetaTom@users.noreply.github.com>
Signed-off-by: Alper Öztürk <67455295+alperozturk96@users.noreply.github.com>
---
 .../src/androidTest/java/com/owncloud/android/AbstractIT.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/library/src/androidTest/java/com/owncloud/android/AbstractIT.java b/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
index d88f1ff419..d36d360d1f 100644
--- a/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
+++ b/library/src/androidTest/java/com/owncloud/android/AbstractIT.java
@@ -292,8 +292,7 @@ private void removeOnClient(OwnCloudClient client) {
             }
 
             boolean isRemoteFileRemoved = removeRemoteFile(remotePath);
-            final var removeFileOperationErrorMessage = ("Failed to remove " + remotePath);
-            assertTrue(removeFileOperationErrorMessage, isRemoteFileRemoved);
+            assertTrue("Failed to remove " + remotePath, isRemoteFileRemoved);
         }
 
         boolean isKeyStoreDeleted = new File(context.getFilesDir(), LOCAL_TRUSTSTORE_FILENAME).delete();