Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d1f0b1bc authored by Vincent Bourgmayer's avatar Vincent Bourgmayer
Browse files

Use new UploadFileOperation constructor in unitTest

parent 7bc76f7e
Loading
Loading
Loading
Loading
Loading
+7 −6
Original line number Diff line number Diff line
@@ -161,7 +161,7 @@ public class UploadFileOperationTest {


        boolean checkEtag = false;
        UploadFileOperation testOperation = new UploadFileOperation(syncedFileStates.get(0), checkEtag);
        UploadFileOperation testOperation = new UploadFileOperation(syncedFileStates.get(0));
        testOperation.setContext(RuntimeEnvironment.application); //Without it, it won't update database

        RemoteOperationResult result = testOperation.execute(client);
@@ -189,9 +189,10 @@ public class UploadFileOperationTest {
    @Test
    public void nullSyncedFileState_shouldFail(){

        SyncedFileState syncedFileState = null;
        //Test fails at the moment because of UploadFileOperation's constructor not checking for syncedFileState is null)
        // check https://gitlab.e.foundation/e/apps/eDrive/-/issues/120
        UploadFileOperation testOperation = new UploadFileOperation(null, false);
        UploadFileOperation testOperation = new UploadFileOperation(syncedFileState);
        testOperation.setContext(RuntimeEnvironment.application);

        RemoteOperationResult result = testOperation.execute(client);
@@ -212,7 +213,7 @@ public class UploadFileOperationTest {
        Assert.assertTrue("SyncedFileState loaded from DB must have an empty Etag", sfs_fromDB.getLastETAG().isEmpty());

        boolean checkEtag = false;
        UploadFileOperation testOperation = new UploadFileOperation(syncedFileStates.get(0), checkEtag);
        UploadFileOperation testOperation = new UploadFileOperation(syncedFileStates.get(0));

        File smallFile = new File(sfs_fromDB.getLocalPath());
        Assert.assertTrue("Local file deletion return false instead of true", smallFile.delete());
@@ -230,7 +231,7 @@ public class UploadFileOperationTest {
        //long freeQuota = getUserRemoteFreeQuota();
        Assert.assertFalse("Reading remote free quota fails"+userFreeQuota, -1 == userFreeQuota);
        //We don't care of parameter of UploadFileOperation's constructor
        RemoteOperationResult actualResult = new UploadFileOperation(Mockito.mock(SyncedFileState.class), false)
        RemoteOperationResult actualResult = new UploadFileOperation(Mockito.mock(SyncedFileState.class))
                .checkAvailableSpace(client, (userFreeQuota+1));
        Assert.assertEquals("Quota check ("+ userFreeQuota+"vs"+(userFreeQuota+1)+") failed", RemoteOperationResult.ResultCode.QUOTA_EXCEEDED, actualResult.getCode());
    }
@@ -245,7 +246,7 @@ public class UploadFileOperationTest {
        //long freeQuota = getUserRemoteFreeQuota();
        Assert.assertFalse("Reading remote free quota fails"+userFreeQuota, -1 == userFreeQuota);

        RemoteOperationResult actualResult = new UploadFileOperation(Mockito.mock(SyncedFileState.class), false)
        RemoteOperationResult actualResult = new UploadFileOperation(Mockito.mock(SyncedFileState.class))
                .checkAvailableSpace(client, userFreeQuota);
        Assert.assertEquals("Quota check  ("+ userFreeQuota+" vs "+userFreeQuota+") failed",
                RemoteOperationResult.ResultCode.QUOTA_EXCEEDED,
@@ -262,7 +263,7 @@ public class UploadFileOperationTest {
        //long freeQuota = getUserRemoteFreeQuota();
        Assert.assertFalse("Reading remote free quota fails "+userFreeQuota, -1 == userFreeQuota);

        RemoteOperationResult actualResult = new UploadFileOperation(Mockito.mock(SyncedFileState.class), false)
        RemoteOperationResult actualResult = new UploadFileOperation(Mockito.mock(SyncedFileState.class))
                .checkAvailableSpace(client, (userFreeQuota-1));
        Assert.assertEquals("Quota check ("+ userFreeQuota+" vs "+(userFreeQuota-1)+") failed",
                RemoteOperationResult.ResultCode.OK,