Browse Source

Fix createMockTask test

Signed-off-by: alperozturk <alper_ozturk@proton.me>
alperozturk 1 year ago
parent
commit
d601998858

+ 1 - 1
app/src/androidTest/java/com/nextcloud/client/files/downloader/TransferManagerTest.kt

@@ -107,7 +107,7 @@ class TransferManagerTest {
 
         private fun createMockTask(): DownloadTask {
             val task = mockk<DownloadTask>()
-            every { task.download(any()) } answers {
+            every { task.download(any(), any(), any()) } answers {
                 taskProgress.forEach {
                     arg<OnProgressCallback<Int>>(1).invoke(it)
                 }

+ 3 - 1
app/src/main/java/com/nextcloud/client/files/downloader/DownloadTask.kt

@@ -21,6 +21,7 @@ package com.nextcloud.client.files.downloader
 
 import android.content.ContentResolver
 import android.content.Context
+import com.nextcloud.client.core.IsCancelled
 import com.nextcloud.client.files.DownloadRequest
 import com.owncloud.android.datamodel.FileDataStorageManager
 import com.owncloud.android.datamodel.OCFile
@@ -62,7 +63,8 @@ class DownloadTask(
         }
     }
 
-    fun download(request: DownloadRequest): Result {
+    // Unused progress, isCancelled arguments needed for TransferManagerTest
+    fun download(request: DownloadRequest, progress: (Int) -> Unit, isCancelled: IsCancelled): Result {
         val op = DownloadFileOperation(request.user, request.file, context)
         val client = clientProvider.invoke()
         val result = op.execute(client)

+ 2 - 2
app/src/main/java/com/nextcloud/client/files/transfer/TransferManagerImpl.kt

@@ -138,8 +138,8 @@ class TransferManagerImpl(
             }
         } else {
             val downloadTask = downloadTaskFactory.create()
-            val wrapper: TaskFunction<DownloadTask.Result, Int> = { _: ((Int) -> Unit), _ ->
-                downloadTask.download(request)
+            val wrapper: TaskFunction<DownloadTask.Result, Int> = { progress: ((Int) -> Unit), isCancelled ->
+                downloadTask.download(request, progress, isCancelled)
             }
             wrapper
         }