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

Verified Commit e52d9df4 authored by Fahim M. Choudhury's avatar Fahim M. Choudhury
Browse files

refactor: rename AppUpdateCompletionHandler to InstallationCompletionHandler

parent ccf4d93a

app/src/main/java/foundation/e/apps/data/install/core/helper/AppUpdateCompletionHandler.kt→app/src/main/java/foundation/e/apps/data/install/core/helper/InstallationCompletionHandler.kt

+1 −1
Original line number Diff line number Diff line
@@ -34,7 +34,7 @@ import java.util.Date 
import java.util.Locale

import javax.inject.Inject



class AppUpdateCompletionHandler @Inject constructor(

class InstallationCompletionHandler @Inject constructor(

    @ApplicationContext private val context: Context,

    private val appInstallRepository: AppInstallRepository,

    private val appManagerWrapper: AppManagerWrapper,

app/src/main/java/foundation/e/apps/data/install/core/helper/InstallationProcessor.kt

+2 −2
Original line number Diff line number Diff line
@@ -34,7 +34,7 @@ class InstallationProcessor @Inject constructor( 
    private val appInstallRepository: AppInstallRepository,

    private val appManagerWrapper: AppManagerWrapper,

    private val downloadManager: DownloadManagerUtils,

    private val appUpdateCompletionHandler: AppUpdateCompletionHandler,

    private val installationCompletionHandler: InstallationCompletionHandler,

) {

    @Suppress("ReturnCount")

    @OptIn(DelicateCoroutinesApi::class)
@@ -185,6 +185,6 @@ class InstallationProcessor @Inject constructor( 
    }



    private suspend fun finishInstallation(appInstall: AppInstall, isUpdateWork: Boolean) {

        appUpdateCompletionHandler.onInstallFinished(appInstall, isUpdateWork)

        installationCompletionHandler.onInstallFinished(appInstall, isUpdateWork)

    }

}

app/src/test/java/foundation/e/apps/installProcessor/AppUpdateCompletionHandlerTest.kt→app/src/test/java/foundation/e/apps/installProcessor/InstallationCompletionHandlerTest.kt

+4 −4
Original line number Diff line number Diff line
@@ -24,7 +24,7 @@ import foundation.e.apps.R 
import foundation.e.apps.data.install.AppInstallRepository

import foundation.e.apps.data.install.AppManagerWrapper

import foundation.e.apps.data.install.models.AppInstall

import foundation.e.apps.data.install.core.helper.AppUpdateCompletionHandler

import foundation.e.apps.data.install.core.helper.InstallationCompletionHandler

import foundation.e.apps.data.install.wrapper.UpdatesNotificationSender

import foundation.e.apps.data.install.wrapper.UpdatesTracker

import foundation.e.apps.data.preference.PlayStoreAuthStore
@@ -42,7 +42,7 @@ import org.junit.Test 
import java.util.Locale



@OptIn(ExperimentalCoroutinesApi::class)

class AppUpdateCompletionHandlerTest {

class InstallationCompletionHandlerTest {



    @get:Rule

    var mainCoroutineRule = MainCoroutineRule()
@@ -53,7 +53,7 @@ class AppUpdateCompletionHandlerTest { 
    private lateinit var updatesTracker: UpdatesTracker

    private lateinit var updatesNotificationSender: UpdatesNotificationSender

    private lateinit var context: Context

    private lateinit var handler: AppUpdateCompletionHandler

    private lateinit var handler: InstallationCompletionHandler



    @Before

    fun setup() {
@@ -64,7 +64,7 @@ class AppUpdateCompletionHandlerTest { 
        updatesTracker = mockk(relaxed = true)

        updatesNotificationSender = mockk(relaxed = true)

        coEvery { playStoreAuthStore.awaitAuthData() } returns null

        handler = AppUpdateCompletionHandler(

        handler = InstallationCompletionHandler(

            context,

            appInstallRepository,

            appManagerWrapper,

app/src/test/java/foundation/e/apps/installProcessor/InstallationProcessorTest.kt

+4 −4
Original line number Diff line number Diff line
@@ -26,7 +26,7 @@ import foundation.e.apps.data.install.AppManager 
import foundation.e.apps.data.install.download.DownloadManagerUtils

import foundation.e.apps.data.install.models.AppInstall

import foundation.e.apps.data.install.core.helper.InstallationProcessor

import foundation.e.apps.data.install.core.helper.AppUpdateCompletionHandler

import foundation.e.apps.data.install.core.helper.InstallationCompletionHandler

import foundation.e.apps.domain.model.install.Status

import foundation.e.apps.util.MainCoroutineRule

import io.mockk.mockk
@@ -54,7 +54,7 @@ class InstallationProcessorTest { 
    private lateinit var appInstallRepository: AppInstallRepository

    private lateinit var fakeFusedManagerRepository: FakeAppManagerWrapper

    private lateinit var downloadManagerUtils: DownloadManagerUtils

    private lateinit var appUpdateCompletionHandler: AppUpdateCompletionHandler

    private lateinit var installationCompletionHandler: InstallationCompletionHandler

    private lateinit var workRunner: InstallationProcessor

    private lateinit var context: Context
@@ -73,12 +73,12 @@ class InstallationProcessorTest { 
        fakeFusedManagerRepository =

            FakeAppManagerWrapper(fakeFusedDownloadDAO, context, fakeFusedManager, fakeFDroidRepository)

        downloadManagerUtils = mockk(relaxed = true)

        appUpdateCompletionHandler = mockk(relaxed = true)

        installationCompletionHandler = mockk(relaxed = true)

        workRunner = InstallationProcessor(

            appInstallRepository,

            fakeFusedManagerRepository,

            downloadManagerUtils,

            appUpdateCompletionHandler

            installationCompletionHandler

        )

    }