Handle user disabled exception as its own result code

Signed-off-by: tobiasKaminsky <[email protected]>

Author: tobiasKaminsky

library/src/androidTest/java/com/owncloud/android/lib/resources/files/UploadFileRemoteOperationIT.kt

@@ -9,9 +9,13 @@ package com.owncloud.android.lib.resources.files
 
 import android.os.Build
 import com.owncloud.android.AbstractIT
+import com.owncloud.android.lib.common.OwnCloudBasicCredentials
+import com.owncloud.android.lib.common.OwnCloudClientFactory
+import com.owncloud.android.lib.common.operations.RemoteOperationResult
 import com.owncloud.android.lib.common.utils.Log_OC
 import com.owncloud.android.lib.resources.files.model.RemoteFile
 import junit.framework.TestCase.assertEquals
+import org.junit.Assert.assertFalse
 import org.junit.Assert.assertNotNull
 import org.junit.Assert.assertTrue
 import org.junit.Test
@@ -22,6 +26,7 @@ import java.nio.file.attribute.BasicFileAttributes
 import java.util.concurrent.TimeUnit
 
 class UploadFileRemoteOperationIT : AbstractIT() {
+    @Throws(IOException::class)
     @Test
     fun creationTime() {
         val imageFile = getFile("imageFile.png")
@@ -30,6 +35,7 @@ class UploadFileRemoteOperationIT : AbstractIT() {
         assertTrue(creationDate!! > (System.currentTimeMillis() / MILLI_TO_SECOND) - TIME_OFFSET)
     }
 
+    @Throws(IOException::class)
     @Test
     fun upload() {
         // create file
@@ -80,6 +86,34 @@ class UploadFileRemoteOperationIT : AbstractIT() {
         )
     }
 
+    @Throws(IOException::class)
+    @Test
+    fun uploadWithDisabledUser() {
+        // use disabled user
+        val client3 = OwnCloudClientFactory.createOwnCloudClient(url, context, true)
+        client3.credentials = OwnCloudBasicCredentials("disabled", "disabled")
+
+        // create file
+        val filePath = createFile("text")
+        val remotePath = "/test.md"
+
+        val creationTimestamp = getCreationTimestamp(File(filePath))
+        val sut =
+            UploadFileRemoteOperation(
+                filePath,
+                remotePath,
+                "text/markdown",
+                "",
+                RANDOM_MTIME,
+                creationTimestamp,
+                true
+            )
+
+        val uploadResult = sut.execute(client3)
+        assertFalse(uploadResult.isSuccess)
+        assertEquals(RemoteOperationResult.ResultCode.USER_DISABLED, uploadResult.code)
+    }
+
     private fun getCreationTimestamp(file: File): Long? {
         return if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O) {
             return null

library/src/androidTest/java/com/owncloud/android/lib/resources/files/webdav/ChunkedFileUploadRemoteOperationIT.kt

@@ -8,12 +8,16 @@
 package com.owncloud.android.lib.resources.files.webdav
 
 import com.owncloud.android.AbstractIT
+import com.owncloud.android.lib.common.OwnCloudBasicCredentials
+import com.owncloud.android.lib.common.OwnCloudClientFactory
 import com.owncloud.android.lib.common.network.WebdavEntry
 import com.owncloud.android.lib.common.network.WebdavUtils
 import com.owncloud.android.lib.common.operations.RemoteOperationResult
 import com.owncloud.android.lib.common.operations.RemoteOperationResult.ResultCode
 import com.owncloud.android.lib.resources.files.ChunkedFileUploadRemoteOperation
 import junit.framework.TestCase
+import junit.framework.TestCase.assertEquals
+import junit.framework.TestCase.assertFalse
 import junit.framework.TestCase.assertNotNull
 import org.apache.jackrabbit.webdav.client.methods.PropFindMethod
 import org.junit.Test
@@ -59,6 +63,19 @@ class ChunkedFileUploadRemoteOperationIT : AbstractIT() {
         TestCase.assertSame(ResultCode.CANCELLED, uploadResult?.code)
     }
 
+    @Test
+    fun uploadWithDisabledUser() {
+        // use disabled user
+        val client3 = OwnCloudClientFactory.createOwnCloudClient(url, context, true)
+        client3.credentials = OwnCloudBasicCredentials("disabled", "disabled")
+
+        val sut = genLargeUpload(true)
+        val uploadResult = sut.execute(client3)
+
+        assertFalse(uploadResult.isSuccess)
+        assertEquals(ResultCode.USER_DISABLED, uploadResult.code)
+    }
+
     @Test
     fun resume() {
         val filePath = createFile("chunkedFile.txt", BIG_FILE_ITERATION * 2)

library/src/main/java/com/owncloud/android/lib/common/operations/ExceptionParser.java

@@ -30,6 +30,8 @@ public class ExceptionParser {
     private static final String INVALID_PATH_EXCEPTION_STRING = "OC\\Connector\\Sabre\\Exception\\InvalidPath";
     private static final String INVALID_PATH_EXCEPTION_UPLOAD_STRING = "OCP\\Files\\InvalidPathException";
     private static final String VIRUS_EXCEPTION_STRING = "OCA\\DAV\\Connector\\Sabre\\Exception\\UnsupportedMediaType";
+    private static final String USER_DISABLED_MESSAGE = "OC\\User\\LoginException: Account disabled";
+    private static final String SERVICE_UNAVAILABLE_STRING = "Sabre\\DAV\\Exception\\ServiceUnavailable";
 
     // No namespaces
     private static final String ns = null;
@@ -76,6 +78,10 @@ public boolean isVirusException() {
         return VIRUS_EXCEPTION_STRING.equalsIgnoreCase(exception) && message.startsWith("Virus");
     }
 
+    public boolean isUserDisabledException() {
+        return SERVICE_UNAVAILABLE_STRING.equals(exception) && USER_DISABLED_MESSAGE.equalsIgnoreCase(message);
+    }
+
     /**
      * Parse OCS node
      *

library/src/main/java/com/owncloud/android/lib/common/operations/RemoteOperationResult.java

@@ -130,7 +130,8 @@ public enum ResultCode {
         VIRUS_DETECTED,
         FOLDER_ALREADY_EXISTS,
         CANNOT_CREATE_FILE,
-        LOCKED
+        LOCKED,
+        USER_DISABLED
     }
 
     private boolean mSuccess = false;
@@ -324,7 +325,10 @@ public RemoteOperationResult(boolean success, OkHttpMethodBase httpMethod) {
     public RemoteOperationResult(boolean success, HttpMethod httpMethod) {
         this(success, httpMethod.getStatusCode(), httpMethod.getStatusText(), httpMethod.getResponseHeaders());
 
-        if (mHttpCode == HttpStatus.SC_BAD_REQUEST || mHttpCode == HttpStatus.SC_UNSUPPORTED_MEDIA_TYPE) {
+        if (mHttpCode == HttpStatus.SC_BAD_REQUEST ||
+                mHttpCode == HttpStatus.SC_UNSUPPORTED_MEDIA_TYPE ||
+                mHttpCode == HttpStatus.SC_SERVICE_UNAVAILABLE ||
+                mHttpCode  == HttpStatus.SC_UNAUTHORIZED) {
             try {
                 String bodyResponse = httpMethod.getResponseBodyAsString();
 
@@ -339,6 +343,10 @@ public RemoteOperationResult(boolean success, HttpMethod httpMethod) {
                         mCode = ResultCode.VIRUS_DETECTED;
                     }
 
+                    if (xmlParser.isUserDisabledException()) {
+                        mCode = ResultCode.USER_DISABLED;
+                    }
+
                     mHttpPhrase = xmlParser.getMessage();
                 }
             } catch (Exception e) {